Changes

Author: Luca Prete

tools/change_fabric_ref.sh

@@ -6,7 +6,7 @@
 # you may not use this file except in compliance with the License.
 # You may obtain a copy of the License at
 #
-#      https://www.apache.org/licenses/LICENSE-2.0
+#     https://www.apache.org/licenses/LICENSE-2.0
 #
 # Unless required by applicable law or agreed to in writing, software
 # distributed under the License is distributed on an "AS IS" BASIS,
@@ -26,19 +26,15 @@ fi
 TARGET_DIR="$1"
 NEW_VERSION="$2"
 
-# Define the pattern to search for and the new suffix
-PATTERN="github.com/GoogleCloudPlatform/cloud-foundation-fabric//modules/"
+PREFIX="github.com/GoogleCloudPlatform/cloud-foundation-fabric//modules/"
 NEW_SUFFIX="?ref=$NEW_VERSION"
 
 # Set locale to C to avoid character encoding issues with sed
 export LC_CTYPE=C
 
 # Find and replace strings in .tf files, excluding .terraform directories
 find "$TARGET_DIR" -type f -name "*.tf" -not -path "*/.terraform/*" -print0 | while IFS= read -r -d $'\0' file; do
-    # Corrected sed command
-    # Replaces the entire '?ref=v...' with the new suffix.
-    # The backslashes `\\` before the first `?` are removed because it's no longer a capture group.
-    sed -i "s|${PATTERN}[^\"[:space:]]*\\?ref=v[0-9.]*\"|&${NEW_SUFFIX}\"|g" "$file"
+    sed -i "s|\\(\"$PREFIX[^\"?[:space:]]*\\)?ref=v[0-9.]*\"|\\1$NEW_SUFFIX\"|g" "$file"
 done
 
 # Unset the locale variable