unixPB: add checksums for ant and ant-contrib everywhere#2280
unixPB: add checksums for ant and ant-contrib everywhere#2280sxa merged 3 commits intoadoptium:masterfrom jeffalder:add-checksums-for-all
Conversation
|
Oh yeah - I built all the docker images locally. |
sxa
left a comment
There was a problem hiding this comment.
As per comment in the other PR the --check on sha512sum doesn't appear to work on Alpine (checked Alpine 313 in this PR)
Step 8/20 : RUN sha512sum --check --strict /tmp/ant.sha512
---> Running in 5e073c0b5769
sha512sum: unrecognized option: check
BusyBox v1.32.1 () multi-call binary.
|
@sxa Thanks for looking; I'll fix this up. It's disappointing that the busy-box version is so limited. |
|
Re-running VPC checks at https://ci.adoptopenjdk.net/view/Tooling/job/VagrantPlaybookCheck/1241/console |
|
@sxa looks like the check failed on some platforms, not sure where to go from here. |
|
I've lost interest in continuing this, I'm going to close it. |
|
None of the failures in my VPC run is related to this PR (although they need to be resolved separately), so I'm personally ocmfortable with merging it at this point. Sorry you've found this a bit disheartening and I hope it won't put you off contributing in the future :-) |
Fixes #1954
SHA-512 for ant-1.10.5 was downloaded from the same site; I was not immediately able to identify a secondary trusted source.
changed
-cto--checkbecause longer options serve as better self-documentation.added
--strictbecause otherwise sha512sum will silently ignore malformatted lines (seems suboptimal, but ok)Checklist
FAQ.md updated if appropriateother documentation is changed or added (if applicable)playbook changes run through VPC or QPC (if you have access)for inventory.yml changes, bastillion/nagios/jenkins updated accordingly