Wrap get_gitHash in try-catch to prevent failures in tarball builds.

Bug: chromium:1321370
Change-Id: If39d2236d1b4d965f7bd189f6bd1cdc70436c41d
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/88582
Commit-Queue: Loko Kung <lokokung@google.com>
Reviewed-by: Austin Eng <enga@chromium.org>
Kokoro: Kokoro <noreply+kokoro@google.com>
This commit is contained in:
Loko Kung 2022-05-03 00:28:53 +00:00 committed by Dawn LUCI CQ
parent f05cba72c3
commit 03ddfbb81f
1 changed files with 8 additions and 5 deletions

View File

@ -27,11 +27,14 @@ def get_git():
def get_gitHash(dawnDir): def get_gitHash(dawnDir):
result = subprocess.run([get_git(), "rev-parse", "HEAD"], try:
stdout=subprocess.PIPE, result = subprocess.run([get_git(), "rev-parse", "HEAD"],
cwd=dawnDir) stdout=subprocess.PIPE,
if result.returncode == 0: cwd=dawnDir)
return result.stdout.decode("utf-8").strip() if result.returncode == 0:
return result.stdout.decode("utf-8").strip()
except Exception:
return ""
# No hash was available (possibly) because the directory was not a git checkout. Dawn should # No hash was available (possibly) because the directory was not a git checkout. Dawn should
# explicitly handle its absenece and disable features relying on the hash, i.e. caching. # explicitly handle its absenece and disable features relying on the hash, i.e. caching.
return "" return ""