util/release: Improve git worktree checks

The bash script `genrelnotes` checks for `.git` to be present to
determine whether the current directory is the top directory of a git
worktree. This check is rather weak and doesn't handle many edge cases
like that of a broken gitfile.

Add a proper `git rev-parse` call to check the condition.

Change-Id: I32b06ca982d55fd8e88e55651b6bc53014905823
Signed-off-by: Alex Thiessen <alex.thiessen.de+coreboot@gmail.com>
Reviewed-on: https://review.coreboot.org/23252
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
This commit is contained in:
Alex Thiessen 2018-01-13 16:31:28 +00:00 committed by Martin Roth
parent 7d3f23ac48
commit da384dd7a4
1 changed files with 2 additions and 1 deletions

View File

@ -32,7 +32,8 @@ then
exit 1 exit 1
fi fi
if [ ! -e ".git" ];then if ! { cdup="$(git rev-parse --show-cdup 2>/dev/null)" && [ -z "${cdup}" ]; }
then
echo "ERROR: This is not the top directory of a git repo. Exiting." echo "ERROR: This is not the top directory of a git repo. Exiting."
exit 1 exit 1
fi fi