Skip to content

ci: enable aarch64 Linux build in CI matrix#28

Open
lh-sag wants to merge 6 commits intoinc/liebherrfrom
ci/enable-aarch64
Open

ci: enable aarch64 Linux build in CI matrix#28
lh-sag wants to merge 6 commits intoinc/liebherrfrom
ci/enable-aarch64

Conversation

@lh-sag
Copy link

@lh-sag lh-sag commented Mar 18, 2026

Summary

Add aarch64 Linux build in CI matrix

Checklist

  • I have tested my changes locally
  • I have added or updated documentation
  • I have linked related issues or discussions
  • I have added or updated tests

Related

None

Notes for Reviewers

Testing CI.

@lh-sag lh-sag force-pushed the ci/enable-aarch64 branch from 0c96d01 to 2029529 Compare March 18, 2026 13:04
@lh-sag lh-sag marked this pull request as ready for review March 18, 2026 13:14
@lh-sag lh-sag requested review from iliescuioana and rama492 March 18, 2026 13:14
@lh-sag
Copy link
Author

lh-sag commented Mar 18, 2026

This should fix some issues with outdated crates e.g. CI now passes and aarch64 is added for Linux too.

@rama492
Copy link
Contributor

rama492 commented Mar 19, 2026

In general we can write PR message as:

1.Migrates from ring to aws-lc-rs for better support and security
2.Enables aarch64 Linux builds in CI

Rest all looks good to me.

tokio = { workspace = true, features = ["macros", "rt"] }
http = { workspace = true }
rsa = { workspace = true }
aws-lc-rs = { workspace = true }
Copy link
Contributor

Choose a reason for hiding this comment

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

do we also need version pinning like below? just that all crates are using same version.

Copy link
Author

Choose a reason for hiding this comment

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

We do, see Cargo.lock/toml in the toplevel directory.

@lh-sag lh-sag requested a review from rama492 March 23, 2026 06:49
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