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

aes: remove use of aarch64_target_feature #325

Merged
merged 1 commit into from
Aug 25, 2022

Conversation

tarcieri
Copy link
Member

@tarcieri tarcieri commented Aug 25, 2022

It's been stabilized: rust-lang/rust#90620

Because of that, it's breaking the build on recent nightlies:

https://github.com/RustCrypto/block-ciphers/runs/7968517726?check_suite_focus=true#step:7:103

error: the feature aarch64_target_feature has been stable since 1.61.0 and no longer requires an attribute to enable

@tarcieri tarcieri force-pushed the aes/remove-aarch64_target_feature branch 3 times, most recently from fd02193 to a2dd64f Compare August 25, 2022 20:32
It's been stabilized: rust-lang/rust#90620

Because of that, it's breaking the build on recent nightlies:

https://github.com/RustCrypto/block-ciphers/runs/7968517726?check_suite_focus=true#step:7:103

> error: the feature `aarch64_target_feature` has been stable since
> 1.61.0 and no longer requires an attribute to enable    aes: remove use of `aarch64_target_feature`

It's been stabilized: rust-lang/rust#90620

Because of that, it's breaking the build on recent nightlies:

https://github.com/RustCrypto/block-ciphers/runs/7968517726?check_suite_focus=true#step:7:103

> error: the feature `aarch64_target_feature` has been stable since
> 1.61.0 and no longer requires an attribute to enable
@tarcieri tarcieri force-pushed the aes/remove-aarch64_target_feature branch from a2dd64f to 28e3d89 Compare August 25, 2022 20:33
@tarcieri tarcieri merged commit a81c990 into master Aug 25, 2022
@tarcieri tarcieri deleted the aes/remove-aarch64_target_feature branch August 25, 2022 20:42
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