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

New release for aws-lc-sys, please #591

Closed
asomers opened this issue Nov 15, 2024 · 6 comments
Closed

New release for aws-lc-sys, please #591

asomers opened this issue Nov 15, 2024 · 6 comments
Assignees
Labels
aws-lc-sys-v0.23.0 question Further information is requested

Comments

@asomers
Copy link

asomers commented Nov 15, 2024

Problem:

The builds of current aws-lc-sys releases are broken on FreeBSD. The build got broken by 8510fd1 and that breakage made it into the 0.21.0 and 0.22.0 releases. But it's already fixed, by 5f19cae . However, that change hasn't yet made it into a new release.

The error is:

cargo build --features=bindgen
...
  bindgen-STDERR: error: Found argument '--prefix-link-name' which wasn't expected, or isn't valid in this context

This is an awkward problem, because Cargo doesn't have a good way to limit transitive dependencies.

Solution:

Publishing a new 0.22.1 patch release would neatly solve this problem.

@merklefruit
Copy link

I am also encountering this issue and second that a patch release would be really neat.

@darylmartin100 darylmartin100 added the question Further information is requested label Nov 15, 2024
@justsmth
Copy link
Contributor

Hello! I'm sorry about this build failure.

My first thought was that you might workaround this by setting AWS_LC_SYS_EXTERNAL_BINDGEN=0:

AWS_LC_SYS_EXTERNAL_BINDGEN=0 cargo build --features=bindgen

However, I think the current release of aws-lc-sys essentially ignores the flag on FreeBSD. 😔 (Also, specifying the bindgen feature shouldn't be required, but let me know if I'm wrong.)

I'm curious about the change that broke the build. The option it's complaining about (--prefix-link-name) has been used since v0.16.0? 🤔

Which version of bindgen are you using? Perhaps that option is not supported on older versions?

❯ bindgen --version
bindgen 0.70.1

Anyways... we plan to have a new release very soon! I'll update here when it's available.

@asomers
Copy link
Author

asomers commented Nov 15, 2024

Locally, I had an old version of bindgen installed: 0.55.1. Removing that and installing bindgen-cli 0.70.1 fixed the problem. However, nothing in my CI configuration explicitly installs either bindgen or bindgen-cli. Maybe that's the problem? Did aws-lc-sys 0.20.1 switch from using the bindgen library to the bindgen CLI?

@justsmth
Copy link
Contributor

Did aws-lc-sys 0.20.1 switch from using the bindgen library to the bindgen CLI?

Yeah, we initially thought that there was an issue with our internal bindings generation when using FreeBSD. We found that external bindgen was working so we forced external bindgen on FreeBSD. But after reports of similar issues on other platforms, we realized that the problem was actually with the version of bindgen we were on and Clang 19. Once that got resolved, we restored the use of internal bindgen. It's been a roller coaster! 🎢 😀

@darylmartin100
Copy link

@justsmth just finished releasing 1.11.0. Please give that a shot and let us know if you have any issues!

@asomers
Copy link
Author

asomers commented Nov 21, 2024

Works for me. Thanks!

MrTarantoga added a commit to MrTarantoga/meta-clang that referenced this issue Dec 27, 2024
Related to some issues in my project with compiling aws-lc-rs, issue
aws/aws-lc-rs#591, I updated to the current
version.

Tested on my project
MrTarantoga/SpotyPee@f83b2a0
and with the poky test
MrTarantoga added a commit to MrTarantoga/meta-clang that referenced this issue Dec 27, 2024
Related to some issues in my project with compiling aws-lc-rs, issue
aws/aws-lc-rs#591, I updated to the current
version.

Tested on my project
MrTarantoga/SpotyPee@f83b2a0
and with the poky standard arm environment.

Changelog https://github.com/rust-lang/rust-bindgen/releases/tag/v0.71.1

Signed-off-by: Götz Grimmer <[email protected]>
kraj pushed a commit to kraj/meta-clang that referenced this issue Jan 7, 2025
Related to some issues in my project with compiling aws-lc-rs, issue
aws/aws-lc-rs#591, I updated to the current
version.

Tested on my project
MrTarantoga/SpotyPee@f83b2a0
and with the poky standard arm environment.

Changelog https://github.com/rust-lang/rust-bindgen/releases/tag/v0.71.1

Signed-off-by: Götz Grimmer <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
aws-lc-sys-v0.23.0 question Further information is requested
Projects
None yet
Development

No branches or pull requests

4 participants