Fix rsync permission issue for macOS 15.4 or higher (#3167)

Signed-off-by: Lucas Marçal <[email protected]>

Author: 0xLucasMarcal

xcodeproj/internal/bazel_integration_files/copy_dsyms.sh

@@ -49,10 +49,10 @@ EOF
 if [[ -n "${BAZEL_OUTPUTS_DSYM:-}" ]]; then
   cd "${BAZEL_OUT%/*}"
 
-  if [[ $(sw_vers -productVersion | cut -d '.' -f 1-2) == "15.4" ]]; then
+  if (( $(echo "$(sw_vers -productVersion | cut -d '.' -f 1-2)" | sed 's/\.//g') >= 154 )); then
     # 15.4's `rsync` has a bug that requires the src to have write permissions.
     # We normally shouldn't do this as it modifies the bazel output base, so we
-    # limit this to only macOS 15.4.
+    # limit this to only macOS 15.4 or higher.
     # shellcheck disable=SC2046
     chmod -R +w $(xargs -n1 <<< "$BAZEL_OUTPUTS_DSYM")
   fi

xcodeproj/internal/bazel_integration_files/copy_outputs.sh

@@ -30,10 +30,10 @@ if [[ "$ACTION" != indexbuild ]]; then
       # rpaths to work
       ln -sfh "$PWD/$BAZEL_OUTPUTS_PRODUCT_BASENAME" "$TARGET_BUILD_DIR/$PRODUCT_NAME"
     else
-      if [[ $(sw_vers -productVersion | cut -d '.' -f 1-2) == "15.4" ]]; then
+      if (( $(echo "$(sw_vers -productVersion | cut -d '.' -f 1-2)" | sed 's/\.//g') >= 154 )); then
         # 15.4's `rsync` has a bug that requires the src to have write
         # permissions. We normally shouldn't do this as it modifies the bazel
-        # output base, so we limit this to only macOS 15.4.
+        # output base, so we limit this to only macOS 15.4 or higher.
         chmod -R +w "$BAZEL_OUTPUTS_PRODUCT_BASENAME"
       fi
 

xcodeproj/internal/templates/incremental_installer.sh

@@ -74,10 +74,10 @@ dest_dir="$(dirname "${dest}")"
 # Copy over `xcschemes`
 readonly dest_xcschemes="$dest/xcshareddata/xcschemes"
 
-if [[ $(sw_vers -productVersion | cut -d '.' -f 1-2) == "15.4" ]]; then
+if (( $(echo "$(sw_vers -productVersion | cut -d '.' -f 1-2)" | sed 's/\.//g') >= 154 )); then
   # 15.4's `rsync` has a bug that requires the src to have write permissions.
   # We normally shouldn't do this as it modifies the bazel output base, so we
-  # limit this to only macOS 15.4.
+  # limit this to only macOS 15.4 or higher.
   chmod -R +w "$src_xcschemes"
 fi
 

xcodeproj/internal/templates/legacy_installer.sh

@@ -136,10 +136,10 @@ fi
 
 # Sync over the project, changing the permissions to be writable
 
-if [[ $(sw_vers -productVersion | cut -d '.' -f 1-2) == "15.4" ]]; then
+if (( $(echo "$(sw_vers -productVersion | cut -d '.' -f 1-2)" | sed 's/\.//g') >= 154 )); then
   # 15.4's `rsync` has a bug that requires the src to have write permissions.
   # We normally shouldn't do this as it modifies the bazel output base, so we
-  # limit this to only macOS 15.4.
+  # limit this to only macOS 15.4 or higher.
   chmod -R +w "$src"
 fi