aboutsummaryrefslogtreecommitdiff
path: root/hack
diff options
context:
space:
mode:
authorChris Evich <cevich@redhat.com>2022-04-21 14:01:42 -0400
committerChris Evich <cevich@redhat.com>2022-04-21 14:07:20 -0400
commitdc02e99d8f3b7698f8003d45b420c4c200ec5833 (patch)
treeb9fb8840a1dc38773579e0009ea1185ec0efb41a /hack
parentcb09c26c6519c2fff8be7285d585672a9066f2da (diff)
downloadpodman-dc02e99d8f3b7698f8003d45b420c4c200ec5833.tar.gz
podman-dc02e99d8f3b7698f8003d45b420c4c200ec5833.tar.bz2
podman-dc02e99d8f3b7698f8003d45b420c4c200ec5833.zip
Fix size-check to display more context
When going through the rebase+build loop, the repository state won't match the exact branch or PR history. This results in the `Building: XYZSHA` indications being entirely useless. Fix this by at least including the title line of the commit being built. This will allow a human to make sense of any size-check failure WRT their view of history. Signed-off-by: Chris Evich <cevich@redhat.com>
Diffstat (limited to 'hack')
-rwxr-xr-xhack/make-and-check-size5
1 files changed, 3 insertions, 2 deletions
diff --git a/hack/make-and-check-size b/hack/make-and-check-size
index f2345b815..5b0021d12 100755
--- a/hack/make-and-check-size
+++ b/hack/make-and-check-size
@@ -92,9 +92,10 @@ if [[ ! -d $context_dir ]]; then
fi
# This is the original (and primary) purpose of this check: if 'make' fails,
-# there is no point in continuing
+# there is no point in continuing. Show at least the commit title since
+# the ID may not match anything human recognisable.
echo
-echo "Building: $(git rev-parse HEAD)"
+echo "Building: $(git log -n 1 --no-show-signature --oneline)"
make
# Determine size of each built file.