From 43aeaff1a873612079095ef22936ce27865ed089 Mon Sep 17 00:00:00 2001 From: Peter Neuroth Date: Fri, 25 Aug 2023 12:13:25 +0200 Subject: [PATCH] tools: Add sha256sum check for detailed hint Removing the Fedora fake as it seems to be not necessary. We later copy the checksum from the release captains checksums anyway. Also adding the sum check as it gives more details about which file did not match if so. Signed-off-by: Peter Neuroth --- tools/build-release.sh | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/tools/build-release.sh b/tools/build-release.sh index 7d6aefe1a..6a9c6a427 100755 --- a/tools/build-release.sh +++ b/tools/build-release.sh @@ -323,10 +323,13 @@ if [ "$VERIFY_RELEASE" = "true" ]; then echo "Place it under the project root as \"$sumfile\"." exit 1 fi + sumfile="$(pwd)/${sumfile}" cd release/ - # Creating fake Fedora tar for SHA256SUMS match - # It is important for zipfile checksum match - touch clightning-v23.05-Fedora-28-amd64.tar.gz + # Check that the release captains sum matches. Ignore missing entries as we + # do not have a repro build for Fedora. Strictly this is not necessary here + # as we compare our checksums with the release captains checksums later, but + # it gives a direct hint which specific checksums don't match if so. + sha256sum --check --ignore-missing "${sumfile}" # Creating SHA256SUMS sha256sum clightning-"$VERSION"* > SHA256SUMS # Replacing Fedora checksums from root file to release/SHA256SUMS