updated bash formatting

Author: hinriksnaer

scripts/build_apple_frameworks.sh

@@ -87,8 +87,8 @@ usage() {
   echo "  --mps                Only build the Metal Performance Shaders backend."
   echo "  --optimized          Only build the Optimized kernels."
   echo "  --quantized          Only build the Quantized kernels."
+  echo "  --torchao            Only build the TorchAO kernels."
   echo "  --xnnpack            Only build the XNNPACK backend."
-  echo "  --torchao            Build the TorchAO kernels."
   echo
   exit 0
 }
@@ -105,8 +105,8 @@ set_cmake_options_override() {
       "-DEXECUTORCH_BUILD_MPS=OFF"
       "-DEXECUTORCH_BUILD_KERNELS_OPTIMIZED=OFF"
       "-DEXECUTORCH_BUILD_KERNELS_QUANTIZED=OFF"
-      "-DEXECUTORCH_BUILD_XNNPACK=OFF"
       "-DEXECUTORCH_BUILD_KERNELS_TORCHAO=OFF"
+      "-DEXECUTORCH_BUILD_XNNPACK=OFF"
     )
   fi
 
@@ -120,28 +120,28 @@ set_cmake_options_override() {
 
 for arg in "$@"; do
   case $arg in
-  -h | --help) usage ;;
-  --Release)
-    if [[ ! " ${MODES[*]:-} " =~ \bRelease\b ]]; then
-      MODES+=("Release")
-    fi
-    ;;
-  --Debug)
-    if [[ ! " ${MODES[*]:-} " =~ \bDebug\b ]]; then
-      MODES+=("Debug")
-    fi
-    ;;
-  --coreml) set_cmake_options_override "EXECUTORCH_BUILD_COREML" ;;
-  --custom) set_cmake_options_override "EXECUTORCH_BUILD_KERNELS_CUSTOM" ;;
-  --mps) set_cmake_options_override "EXECUTORCH_BUILD_MPS" ;;
-  --optimized) set_cmake_options_override "EXECUTORCH_BUILD_KERNELS_OPTIMIZED" ;;
-  --quantized) set_cmake_options_override "EXECUTORCH_BUILD_KERNELS_QUANTIZED" ;;
-  --xnnpack) set_cmake_options_override "EXECUTORCH_BUILD_XNNPACK" ;;
-  --torchao) set_cmake_options_override "EXECUTORCH_BUILD_KERNELS_TORCHAO" ;;
-  *)
-    echo -e "\033[31m[error] unknown option: ${arg}\033[0m"
-    exit 1
-    ;;
+      -h|--help) usage ;;
+      --Release)
+        if [[ ! " ${MODES[*]:-} " =~ \bRelease\b ]]; then
+          MODES+=("Release")
+        fi
+        ;;
+      --Debug)
+        if [[ ! " ${MODES[*]:-} " =~ \bDebug\b ]]; then
+          MODES+=("Debug")
+        fi
+        ;;
+      --coreml) set_cmake_options_override "EXECUTORCH_BUILD_COREML";;
+      --custom) set_cmake_options_override "EXECUTORCH_BUILD_KERNELS_CUSTOM" ;;
+      --mps) set_cmake_options_override "EXECUTORCH_BUILD_MPS" ;;
+      --optimized) set_cmake_options_override "EXECUTORCH_BUILD_KERNELS_OPTIMIZED" ;;
+      --quantized) set_cmake_options_override "EXECUTORCH_BUILD_KERNELS_QUANTIZED" ;;
+      --xnnpack) set_cmake_options_override "EXECUTORCH_BUILD_XNNPACK" ;;
+      --torchao) set_cmake_options_override "EXECUTORCH_BUILD_KERNELS_TORCHAO" ;;
+      *)
+        echo -e "\033[31m[error] unknown option: ${arg}\033[0m"
+        exit 1
+      ;;
   esac
 done
 
@@ -157,15 +157,15 @@ for preset_index in "${!PRESETS[@]}"; do
 
     # Do NOT add options here. Update the respective presets instead.
     cmake -S "${SOURCE_ROOT_DIR}" \
-      -B "${preset_output_dir}" \
-      -DCMAKE_ARCHIVE_OUTPUT_DIRECTORY="${preset_output_dir}" \
-      -DCMAKE_BUILD_TYPE="${mode}" \
-      ${CMAKE_OPTIONS_OVERRIDE[@]:-} \
-      --preset "${preset}"
+          -B "${preset_output_dir}" \
+          -DCMAKE_ARCHIVE_OUTPUT_DIRECTORY="${preset_output_dir}" \
+          -DCMAKE_BUILD_TYPE="${mode}" \
+          ${CMAKE_OPTIONS_OVERRIDE[@]:-} \
+          --preset "${preset}"
 
     cmake --build "${preset_output_dir}" \
-      --config "${mode}" \
-      -j$(sysctl -n hw.ncpu)
+          --config "${mode}" \
+          -j$(sysctl -n hw.ncpu)
   done
 done
 
@@ -175,7 +175,7 @@ mkdir -p "$HEADERS_ABSOLUTE_PATH"
 
 "$SOURCE_ROOT_DIR"/scripts/print_exported_headers.py --buck2=$(realpath "$BUCK2") --targets \
   //extension/module: \
-  //extension/tensor: |
+  //extension/tensor: \
   rsync -av --files-from=- "$SOURCE_ROOT_DIR" "$HEADERS_ABSOLUTE_PATH/executorch"
 
 # HACK: XCFrameworks don't appear to support exporting any build
@@ -186,16 +186,16 @@ mkdir -p "$HEADERS_ABSOLUTE_PATH"
 sed -i '' '1i\
 #define C10_USING_CUSTOM_GENERATED_MACROS
 ' \
-"$HEADERS_ABSOLUTE_PATH/executorch/runtime/core/portable_type/c10/c10/macros/Macros.h" \
-"$HEADERS_ABSOLUTE_PATH/executorch/runtime/core/portable_type/c10/c10/macros/Export.h" \
-"$HEADERS_ABSOLUTE_PATH/executorch/runtime/core/portable_type/c10/torch/standalone/macros/Export.h"
+  "$HEADERS_ABSOLUTE_PATH/executorch/runtime/core/portable_type/c10/c10/macros/Macros.h" \
+  "$HEADERS_ABSOLUTE_PATH/executorch/runtime/core/portable_type/c10/c10/macros/Export.h" \
+  "$HEADERS_ABSOLUTE_PATH/executorch/runtime/core/portable_type/c10/torch/standalone/macros/Export.h"
 
 cp -r $HEADERS_ABSOLUTE_PATH/executorch/runtime/core/portable_type/c10/c10 "$HEADERS_ABSOLUTE_PATH/"
 cp -r $HEADERS_ABSOLUTE_PATH/executorch/runtime/core/portable_type/c10/torch "$HEADERS_ABSOLUTE_PATH/"
 
 cp "$SOURCE_ROOT_DIR/extension/apple/ExecuTorch/Exported/"*.h "$HEADERS_ABSOLUTE_PATH/executorch"
 
-cat >"$HEADERS_ABSOLUTE_PATH/module.modulemap" <<'EOF'
+cat > "$HEADERS_ABSOLUTE_PATH/module.modulemap" << 'EOF'
 module ExecuTorch {
   umbrella header "ExecuTorch/ExecuTorch.h"
   export *
@@ -219,10 +219,10 @@ append_framework_flag() {
   fi
 
   if [[ -n "$mode" && "$mode" != "Release" ]]; then
-    local name spec
-    name=$(echo "$framework" | cut -d: -f1)
-    spec=$(echo "$framework" | cut -d: -f2-)
-    framework="${name}_$(echo "$mode" | tr '[:upper:]' '[:lower:]'):${spec}"
+      local name spec
+      name=$(echo "$framework" | cut -d: -f1)
+      spec=$(echo "$framework" | cut -d: -f2-)
+      framework="${name}_$(echo "$mode" | tr '[:upper:]' '[:lower:]'):${spec}"
   fi
   echo "Adding framework: ${framework}"
   FRAMEWORK_FLAGS+=("--framework=$framework")