-
-
Notifications
You must be signed in to change notification settings - Fork 30.9k
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
gh-106123: Modules/_sha3 appears to no longer be necessary #106124
Conversation
Thanks, Skip! Can you do a |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like it was indeed removed in #103597
Thanks @smontanaro for the PR, and @hauntsaninja for merging it 🌮🎉.. I'm working now to backport this PR to: 3.12. |
…honGH-106124) (cherry picked from commit 0345b0c) Co-authored-by: Skip Montanaro <[email protected]>
GH-106127 is a backport of this pull request to the 3.12 branch. |
…-106124) (#106127) Co-authored-by: Skip Montanaro <[email protected]>
This is a pretty trivial PR. I'm guessing
Modules/_sha3
was a holdover from beforeModules/_hacl
existed. Perhaps there's a valid reason for it to still exist (used on another platform?), but if so, maybe it should be described in a comment inconfigure.ac