Skip to content
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

1.9.0 compilation fails on old libc (centos) due to atomics missing? #531

Closed
aviramha opened this issue Sep 22, 2024 · 3 comments
Closed
Assignees

Comments

@aviramha
Copy link

Problem:

Latest (1.9.0) causes build failure on old libc machines.
We're using cross + custom base image https://github.com/metalbear-co/ci/blob/main/layer-build/Dockerfile to build and it started failing on the version

I created a simple repro repository here
https://github.com/aviramha/aws-issue-repro

github action that fails:
https://github.com/aviramha/aws-issue-repro/actions/runs/10982117213/job/30489990053

I couldn't reproduce it locally (i'm on M1 macbook so probably Cross doesn't like that..)

@justinwsmith
Copy link
Contributor

Hello!

This occurs with older (<= v4.9) versions of the GCC compiler. (Perhaps this happens with older Clang compilers as well?) There was a change recently made upstream that should fix this. We plan for this upstream change to be included in our next patch release (aws-lc-rs v1.9.1).

One workaround for this would be to set AWS_LC_SYS_C_STD=99 in the build environment. (Note that with cross-rs, its build environment does not automatically pick up variables from the host environment. More information about this is here.)

Let us know if this workaround, or the next patch release, doesn't fix this issue. Thanks!

@aviramha
Copy link
Author

Hey @justinwsmith
Kudos for the swift reply! I am happy to hear this is already known and will be fixed next patch release.

We'd wait for that instead of going with the workaround as there's no urgency from our side (and I appreciate you going into the details and quirks of cross who I unfortunately already know 🤣 )

@justsmth
Copy link
Contributor

justsmth commented Oct 3, 2024

I believe this issue is resolved with the release of aws-lc-sys v1.10.0. Please feel free to reopen if you're still experiencing this issue after upgrading. Thanks for helping us improve aws-lc-rs!

@justsmth justsmth closed this as completed Oct 3, 2024
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

No branches or pull requests

3 participants