Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/11217 )
Change subject: [build] Switch to sha512 checksum for releases ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/11217/1/build-support/build_source_release.py File build-support/build_source_release.py: http://gerrit.cloudera.org:8080/#/c/11217/1/build-support/build_source_release.py@134 PS1, Line 134: > why switch away from a tab here? I think the tabs made things nicely line u Sorry, should've clarified in the commit message... On MacOS shasum the tab version doesn't work. I verified the double white space on Linux as well and it works on both OSes. The Linux version of sha512sum also uses double-space to separate the checksum from the filename: $ sha512sum apache-kudu-1.8.0-SNAPSHOT.tar.gz | xxd # ... 0000070: 3065 3132 3666 3463 3664 6332 3065 6338 0e126f4c6dc20ec8 0000080: 2020 6170 6163 6865 2d6b 7564 752d 312e apache-kudu-1. # ... -- To view, visit http://gerrit.cloudera.org:8080/11217 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I7646b4559bf39d2415b32a1dcdd4cd7ecde41531 Gerrit-Change-Number: 11217 Gerrit-PatchSet: 1 Gerrit-Owner: Attila Bukor <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Comment-Date: Tue, 14 Aug 2018 20:50:06 +0000 Gerrit-HasComments: Yes
