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

source release hashes: only sha512. #7

Merged
merged 1 commit into from
Mar 5, 2022
Merged

Conversation

bmarwell
Copy link
Collaborator

@bmarwell bmarwell commented Mar 4, 2022

After talking to @hboutemy, only sha512 is really important for source release zip files.
After apache/shiro#349 and apache/shiro-site#152, let's simplify this script again.

@bmarwell bmarwell requested a review from bdemers March 4, 2022 21:10
@hboutemy
Copy link

hboutemy commented Mar 5, 2022

@bmarwell you don't even need old md5 and sha1 on the Apache distribution area https://dlcdn.apache.org/shiro/ (which will be archived to https://archive.apache.org/dist/shiro/ )

(md5 and sha1 are the checksums on Maven Central, which has completely different rules/prerequisites)

@hboutemy
Copy link

hboutemy commented Mar 5, 2022

and you'll need to add sha512 for old releases, like in https://archive.apache.org/dist/shiro/1.1.0/ for example

@bmarwell bmarwell changed the title source release hashes: only md5, sha1 and sha512. source release hashes: only sha512. Mar 5, 2022
@bmarwell
Copy link
Collaborator Author

bmarwell commented Mar 5, 2022

Done! Thanks for the hints!

@bmarwell bmarwell merged commit f169e30 into master Mar 5, 2022
@bmarwell bmarwell deleted the source-release-hashes branch March 5, 2022 12:21
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.

3 participants