Commit 1a453b5
tools: improve error handling in test426 update script
If the current SHA is not found in the README for some reason,
exit with an error.
PR-URL: #62121
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>1 parent 4d62e95 commit 1a453b5
1 file changed
Lines changed: 8 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -6,11 +6,17 @@ BASE_DIR=$(cd "$(dirname "$0")/../.." && pwd) | |||
| 6 | 6 | ||
| 7 | 7 | TARGET_DIR="$BASE_DIR/test/fixtures/test426" | |
| 8 | 8 | README="$BASE_DIR/test/test426/README.md" | |
| 9 | - TARBALL_URL=$(curl -fsIo /dev/null -w '%header{Location}' https://github.com/tc39/source-map-tests/archive/HEAD.tar.gz) | ||
| 10 | - SHA=$(basename "$TARBALL_URL") | ||
| 11 | 9 | ||
| 12 | 10 | CURRENT_SHA=$(sed -n 's#^.*https://github.com/tc39/source-map-tests/commit/\([0-9a-f]*\).*$#\1#p' "$README") | |
| 13 | 11 | ||
| 12 | + if [ -z "$CURRENT_SHA" ]; then | ||
| 13 | + echo "Could not find source-map-tests commit marker in $README" >&2 | ||
| 14 | + exit 1 | ||
| 15 | + fi | ||
| 16 | + | ||
| 17 | + TARBALL_URL=$(curl -fsIo /dev/null -w '%header{Location}' https://github.com/tc39/source-map-tests/archive/HEAD.tar.gz) | ||
| 18 | + SHA=$(basename "$TARBALL_URL") | ||
| 19 | + | ||
| 14 | 20 | if [ "$CURRENT_SHA" = "$SHA" ]; then | |
| 15 | 21 | echo "Already up-to-date" | |
| 16 | 22 | exit 0 | |
0 commit comments