From 1a5cddfc1f9d0355e2c2e0912d794203c8d95e39 Mon Sep 17 00:00:00 2001
From: Antoine du Hamel <duhamelantoine1995@gmail.com>
Date: Mon, 29 May 2023 12:59:52 +0200
Subject: [PATCH] tools: harmonize `dep_updaters` scripts

PR-URL: https://github.com/nodejs/node/pull/48201
Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Mestery <mestery@protonmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
---
 tools/dep_updaters/update-acorn.sh            | 21 ++++++++-------
 tools/dep_updaters/update-base64.sh           |  2 +-
 tools/dep_updaters/update-cjs-module-lexer.sh |  2 +-
 tools/dep_updaters/update-undici.sh           | 26 +++++++++++++------
 4 files changed, 31 insertions(+), 20 deletions(-)

diff --git a/tools/dep_updaters/update-acorn.sh b/tools/dep_updaters/update-acorn.sh
index 4f6bf6b0dd7a43..21b2e018c0221d 100755
--- a/tools/dep_updaters/update-acorn.sh
+++ b/tools/dep_updaters/update-acorn.sh
@@ -34,17 +34,18 @@ rm -rf deps/acorn/acorn
     "$NODE" "$NPM" init --yes
 
     "$NODE" "$NPM" install --global-style --no-bin-links --ignore-scripts "acorn@$NEW_VERSION"
-    cd node_modules/acorn
-    # update this version information in src/acorn_version.h
-    FILE_PATH="$ROOT/src/acorn_version.h"
-    echo "// This is an auto generated file, please do not edit." > "$FILE_PATH"
-    echo "// Refer to tools/update-acorn.sh" >> "$FILE_PATH"
-    echo "#ifndef SRC_ACORN_VERSION_H_" >> "$FILE_PATH"
-    echo "#define SRC_ACORN_VERSION_H_" >> "$FILE_PATH"
-    echo "#define ACORN_VERSION \"$NEW_VERSION\"" >> "$FILE_PATH"
-    echo "#endif  // SRC_ACORN_VERSION_H_" >> "$FILE_PATH"
 )
 
+# update version information in src/acorn_version.h
+cat > "$ROOT/src/acorn_version.h" <<EOF
+// This is an auto generated file, please do not edit.
+// Refer to tools/update-acorn.sh
+#ifndef SRC_ACORN_VERSION_H_
+#define SRC_ACORN_VERSION_H_
+#define ACORN_VERSION "$NEW_VERSION"
+#endif  // SRC_ACORN_VERSION_H_
+EOF
+
 mv acorn-tmp/node_modules/acorn deps/acorn
 
 rm -rf acorn-tmp/
@@ -53,7 +54,7 @@ echo "All done!"
 echo ""
 echo "Please git add acorn, commit the new version:"
 echo ""
-echo "$ git add -A deps/acorn"
+echo "$ git add -A deps/acorn src/acorn_version.h"
 echo "$ git commit -m \"deps: update acorn to $NEW_VERSION\""
 echo ""
 
diff --git a/tools/dep_updaters/update-base64.sh b/tools/dep_updaters/update-base64.sh
index 539c75713374ef..954d0abb7bf2d7 100755
--- a/tools/dep_updaters/update-base64.sh
+++ b/tools/dep_updaters/update-base64.sh
@@ -63,7 +63,7 @@ echo "All done!"
 echo ""
 echo "Please git add base64/base64, commit the new version:"
 echo ""
-echo "$ git add -A deps/base64/base64"
+echo "$ git add -A deps/base64/base64 src/base64_version.h"
 echo "$ git commit -m \"deps: update base64 to $NEW_VERSION\""
 echo ""
 
diff --git a/tools/dep_updaters/update-cjs-module-lexer.sh b/tools/dep_updaters/update-cjs-module-lexer.sh
index 878624ba03f994..c9f5b037431822 100755
--- a/tools/dep_updaters/update-cjs-module-lexer.sh
+++ b/tools/dep_updaters/update-cjs-module-lexer.sh
@@ -54,7 +54,7 @@ echo "All done!"
 echo ""
 echo "Please git add cjs-module-lexer, commit the new version:"
 echo ""
-echo "$ git add -A deps/cjs-module-lexer"
+echo "$ git add -A deps/cjs-module-lexer src/cjs_module_lexer_version.h"
 echo "$ git commit -m \"deps: update cjs-module-lexer to $NEW_VERSION\""
 echo ""
 
diff --git a/tools/dep_updaters/update-undici.sh b/tools/dep_updaters/update-undici.sh
index f9d40fd3fa1e9e..7879e373267bd8 100755
--- a/tools/dep_updaters/update-undici.sh
+++ b/tools/dep_updaters/update-undici.sh
@@ -36,22 +36,32 @@ rm -f deps/undici/undici.js
     "$NODE" "$NPM" install --global-style --no-bin-links --ignore-scripts "undici@$NEW_VERSION"
     cd node_modules/undici
     "$NODE" "$NPM" run build:node
-    # update this version information in src/undici_version.h
-    FILE_PATH="$ROOT/src/undici_version.h"
-    echo "// This is an auto generated file, please do not edit." > "$FILE_PATH"
-    echo "// Refer to tools/update-undici.sh" >> "$FILE_PATH"
-    echo "#ifndef SRC_UNDICI_VERSION_H_" >> "$FILE_PATH"
-    echo "#define SRC_UNDICI_VERSION_H_" >> "$FILE_PATH"
-    echo "#define UNDICI_VERSION \"$NEW_VERSION\"" >> "$FILE_PATH"
-    echo "#endif  // SRC_UNDICI_VERSION_H_" >> "$FILE_PATH"
 )
 
+# update version information in src/undici_version.h
+cat > "$ROOT/src/undici_version.h" <<EOF
+// This is an auto generated file, please do not edit.
+// Refer to tools/update-undici.sh
+#ifndef SRC_UNDICI_VERSION_H_
+#define SRC_UNDICI_VERSION_H_
+#define UNDICI_VERSION "$NEW_VERSION"
+#endif  // SRC_UNDICI_VERSION_H_
+EOF
+
 mv undici-tmp/node_modules/undici deps/undici/src
 mv deps/undici/src/undici-fetch.js deps/undici/undici.js
 cp deps/undici/src/LICENSE deps/undici/LICENSE
 
 rm -rf undici-tmp/
 
+echo "All done!"
+echo ""
+echo "Please git add and commit the new version:"
+echo ""
+echo "$ git add -A deps/undici src/undici_version.h"
+echo "$ git commit -m \"deps: update Undici to $NEW_VERSION\""
+echo ""
+
 # The last line of the script should always print the new version,
 # as we need to add it to $GITHUB_ENV variable.
 echo "NEW_VERSION=$NEW_VERSION"