From c85e7bb4eda26a225bcf1289a68c90e6873db9b5 Mon Sep 17 00:00:00 2001 From: Kir Kolyshkin Date: Thu, 1 May 2025 18:00:56 -0700 Subject: [PATCH] Remove hack/tree_status.sh The sole reason of having this was to produce a nice informative error message in case of failure (and because the message was not explicitly specified, it was entirely wrong in 1 of 2 use cases). Let's drop it and just use git to fail the job if a tree is dirty. Signed-off-by: Kir Kolyshkin --- .cirrus.yml | 4 ++-- hack/tree_status.sh | 17 ----------------- 2 files changed, 2 insertions(+), 19 deletions(-) delete mode 100755 hack/tree_status.sh diff --git a/.cirrus.yml b/.cirrus.yml index 40916f3..4aadf46 100644 --- a/.cirrus.yml +++ b/.cirrus.yml @@ -87,7 +87,7 @@ config_task: - dnf install -y make glib2-devel git gcc golang - cd $CIRRUS_WORKING_DIR - make config - - ./hack/tree_status.sh + - git diff --exit-code # Verify code was fmt'ed properly @@ -102,7 +102,7 @@ fmt_task: - dnf install -y clang clang-tools-extra golang - cd $CIRRUS_WORKING_DIR - make fmt - - ./hack/tree_status.sh + - git diff --exit-code # Build the static binary diff --git a/hack/tree_status.sh b/hack/tree_status.sh deleted file mode 100755 index 270aaf6..0000000 --- a/hack/tree_status.sh +++ /dev/null @@ -1,17 +0,0 @@ -#!/bin/bash -# this script is based off of the similarly named in github.com/containers/libpod/hack/tree_status.sh - -set -e - -SUGGESTION="${SUGGESTION:-call 'make config' and commit all changes.}" - -STATUS=$(git status --porcelain) -if [[ -z $STATUS ]] -then - echo "tree is clean" -else - echo "tree is dirty, please $SUGGESTION" - echo "" - echo "$STATUS" - exit 1 -fi