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

polyval: pin nightly version used for ARMv8 testing #151

Merged
merged 1 commit into from
Mar 21, 2022

Conversation

tarcieri
Copy link
Member

The build is currently failing:

https://github.com/RustCrypto/universal-hashes/runs/5629586918?check_suite_focus=true

It's for two reasons:

  1. aarch64_target_feature was recently stabilized in Stabilise aarch64_target_feature rust-lang/rust#90621 but we still include it
  2. There's a bug relating to neon/fp activation. See Fold aarch64 feature +fp into +neon rust-lang/rust#91608 and Fix unnecessary error when using neon target feature rust-lang/rust#95044

Until one of the fixes in the second issue is merged, we can't really make progress. So this commit pins to nightly-2022-03-01 so we can continue to have a clean build.

Once either of the above solutions lands we can remove aarch64_target_feature and unpin nightly again.

The build is currently failing:

https://github.com/RustCrypto/universal-hashes/runs/5629586918?check_suite_focus=true

It's for two reasons:

1. `aarch64_target_feature` was recently stabilized in
   rust-lang/rust#90621 but we still include it
2. There's a bug releating to `neon`/`fp` activation. See
   rust-lang/rust#91608 and rust-lang/rust#95044

Until one of the fixes in the second issue is merged, we can't really
make progress. So this commit pins to `nightly-2022-03-01` so we can
continue to have a clean build.

Once either of the above solutions lands we can remove
`aarch64_target_feature` and unpin nightly again.
@tarcieri tarcieri merged commit c4f030f into master Mar 21, 2022
@tarcieri tarcieri deleted the polyval/pin-nightly-rust-for-armv8-testing branch March 21, 2022 15:38
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.

1 participant