From dc02e99d8f3b7698f8003d45b420c4c200ec5833 Mon Sep 17 00:00:00 2001 From: Chris Evich Date: Thu, 21 Apr 2022 14:01:42 -0400 Subject: 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 --- hack/make-and-check-size | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) (limited to 'hack') 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. -- cgit v1.2.3-54-g00ecf