Skip to content

Commit a05004f

Browse files
committed
tools: Write current_ver.dep after file change
1 parent 0919440 commit a05004f

File tree

1 file changed

+5
-6
lines changed

1 file changed

+5
-6
lines changed

tools/dep_updaters/update-icu.sh

+5-6
Original file line numberDiff line numberDiff line change
@@ -47,17 +47,14 @@ KEY_URL="https://raw.githubusercontent.com/unicode-org/icu/release-$(echo $NEW_V
4747

4848
CHECKSUM=$(curl -sL "$NEW_VERSION_MD5" | grep "$NEW_VERSION_TGZ" | grep -v "\.asc$" | awk '{print $1}')
4949

50-
51-
echo "Comparing checksums: deposited '$CHECKSUM' with '$GENERATED_CHECKSUM'"
52-
5350
if [ -n "$CHECKSUM" ]; then
5451
GENERATED_CHECKSUM=$( curl -sL "$NEW_VERSION_TGZ_URL" | md5sum | cut -d ' ' -f1)
55-
echo "Comparing checksums: deposited $CHECKSUM with $GENERATED_CHECKSUM"
52+
echo "Comparing checksums: deposited '$CHECKSUM' with '$GENERATED_CHECKSUM'"
5653
if [ "$CHECKSUM" != "$GENERATED_CHECKSUM" ]; then
5754
echo "Skipped because checksums do not match."
5855
exit 0
5956
fi
60-
perl -i -pe "s|\"(md5\|gpg)\": .*|\"md5\": \"$CHECKSUM\"|" "$TOOLS_DIR/icu/current_ver.dep"
57+
REGEX="s|\"(md5\|gpg)\": .*|\"md5\": \"$CHECKSUM\"|"
6158
else
6259
echo "Checksum not found"
6360
echo "check with gpg"
@@ -68,7 +65,8 @@ else
6865
if gpg --verify signature.asc data.tgz; then
6966
echo "Signature verified"
7067
rm data.tgz signature.asc KEYS
71-
perl -i -pe "s|\"(gpg\|md5)\": .*|\"gpg\": { \"key\": \"$KEY_URL\", \"asc\": \"$NEW_VERSION_TGZ_ASC_URL\" }|" "$TOOLS_DIR/icu/current_ver.dep"
68+
69+
REGEX="s|\"(gpg\|md5)\": .*|\"gpg\": { \"key\": \"$KEY_URL\", \"asc\": \"$NEW_VERSION_TGZ_ASC_URL\" }|"
7270
else
7371
echo "Skipped because signature verification failed."
7472
rm data.tgz signature.asc KEYS
@@ -84,6 +82,7 @@ rm -rf "$DEPS_DIR/icu"
8482

8583
perl -i -pe "s|\"url\": .*|\"url\": \"$NEW_VERSION_TGZ_URL\",|" "$TOOLS_DIR/icu/current_ver.dep"
8684

85+
perl -i -pe "$REGEX" "$TOOLS_DIR/icu/current_ver.dep"
8786

8887
rm -rf out "$DEPS_DIR/icu" "$DEPS_DIR/icu4c*"
8988

0 commit comments

Comments
 (0)