maketgz: fix RELEASE-TOOLS.md for daily tarballs

The daily snapshots have no associated git tag, so provide a commit hash
instead in these cases. Fix the dpkg detection since the shell would
exit immediately without showing an error message if it weren't found.

Closes #14820

diff --git a/scripts/maketgz b/scripts/maketgz
index 996fd06fede4df2ab50606e500e6ccb594bc8d0a..af62708bc1c92570fd5c74b13cdffee1cef75040 100755 (executable)
--- a/scripts/maketgz
+++ b/scripts/maketgz
@@ -156,14 +156,14 @@ else
   automake --include-deps Makefile >/dev/null
 fi
 
-echo "produce RELEASE-TOOLS.md"
-./scripts/release-tools.sh "$timestamp" "$version" > docs/RELEASE-TOOLS.md.dist
-
 if test -n "$commit"; then
   echo "produce docs/tarball-commit.txt"
   git rev-parse HEAD >docs/tarball-commit.txt.dist
 fi
 
+echo "produce RELEASE-TOOLS.md"
+./scripts/release-tools.sh "$timestamp" "$version" "$commit" > docs/RELEASE-TOOLS.md.dist
+
 ############################################################################
 #
 # Now run make dist to generate a tar.gz archive

diff --git a/scripts/release-tools.sh b/scripts/release-tools.sh
index faf5d32603c9ed0ccfe3eac2630a1cb64c62f47d..4c4ed7d8097d561e5d394997aa3918df679ae702 100755 (executable)
--- a/scripts/release-tools.sh
+++ b/scripts/release-tools.sh
@@ -29,6 +29,11 @@ set -eu
 timestamp=${1:-unknown}
 version=${2:-unknown}
 tag=$(echo "curl-$version" | tr '.' '_')
+commit=${3}
+if [ -n "$commit" ] && [ -r "docs/tarball-commit.txt.dist" ]; then
+  # If commit is given, then the tag likely doesn't actually exist
+  tag="$(cat docs/tarball-commit.txt.dist)"
+fi
 
 cat <<MOO
 # Release tools used for curl $version
@@ -38,10 +43,9 @@ produce this release tarball.
 
 MOO
 
-exists=$(command -v dpkg 2>/dev/null)
-if test ! -e "$exists"; then
-  echo "(unknown, could not find dpkg)"
-  exit
+if ! command -v dpkg >/dev/null; then
+  echo "Error: could not find dpkg" >&2
+  exit 1
 fi
 
 debian() {
@@ -58,7 +62,7 @@ cat <<MOO
 
 # Reproduce the tarball
 
-- Clone the repo and checkout the tag: $tag
+- Clone the repo and checkout the tag/commit: $tag
 - Install the same set of tools + versions as listed above
 
 ## Do a standard build