From 58701128e80d02a154bb055c79cb44d7e7fd4864 Mon Sep 17 00:00:00 2001 From: Jouni Malinen Date: Thu, 14 Apr 2022 11:50:41 +0300 Subject: [PATCH] tests: Handle git rev-parse failures more robustly Do not add the --commit argument if the current git commitid cannot be determined. This prevents complete failure to run the tests if the git command cannot be used for some reason (like a recent change that stopped allowing root user within the VM from running the git operation for the case where the host system uses non-root account). Signed-off-by: Jouni Malinen --- tests/hwsim/run-all.sh | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/tests/hwsim/run-all.sh b/tests/hwsim/run-all.sh index ee48cd058..7c50edd79 100755 --- a/tests/hwsim/run-all.sh +++ b/tests/hwsim/run-all.sh @@ -15,7 +15,11 @@ export LOGDIR if [ -z "$DBFILE" ]; then DB="" else - DB="-S $DBFILE --commit $(git rev-parse HEAD)" + DB="-S $DBFILE" + COMMITID="$(git rev-parse HEAD)" + if [ -n "$COMMITID" ]; then + DB="$DB --commit $COMMITID" + fi if [ -n "$BUILD" ]; then DB="$DB -b $BUILD" fi