Skip to content

Add CI run on Linux arm64 #1656

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Mar 5, 2025
Merged

Add CI run on Linux arm64 #1656

merged 1 commit into from
Mar 5, 2025

Conversation

dbwiddis
Copy link
Contributor

@dbwiddis dbwiddis commented Mar 4, 2025

GitHub now has Linux ARM64 runners in public preview, see https://github.blog/changelog/2025-01-16-linux-arm64-hosted-runners-now-available-for-free-in-public-repositories-public-preview/

Adding one run to the build matrix.'

Note there are also Windows 2025 runners, but since we use AppVeyor for those builds, we can't take advantage.

@dbwiddis
Copy link
Contributor Author

dbwiddis commented Mar 4, 2025

Seems to be failing on fetching jdk21 artifacts even though jdk17 is specified....

Probably the default-jdk included here:

run: sudo apt-get install -yq --force-yes zip unzip libtool automake libltdl-dev texinfo ant ant-optional debhelper-compat default-jdk javahelper libasm-java libffi-dev libx11-dev libxt-dev maven-repo-helper pkg-config

@matthiasblaesing
Copy link
Member

JDK21 might be pulled in by the build-deps of JNA. What I see here though is an inconsistent package information. Please try to add apt-get update to Linux requirements. That ensures, that the package DB is up-to-date.

@dbwiddis
Copy link
Contributor Author

dbwiddis commented Mar 4, 2025

That seems to have done the trick.

Will make one more change, though, because of:

--force-yes is deprecated, use one of the options starting with --allow instead.

Signed-off-by: Daniel Widdis <[email protected]>
Copy link
Member

@matthiasblaesing matthiasblaesing left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks sane to me. Thank you.

@dbwiddis dbwiddis merged commit 4b091e6 into master Mar 5, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants