Update macOS build script to use Ninja Multi-Config generator (#10377)

* Update macOS build script to use Ninja Multi-Config generator

* fix
This commit is contained in:
SoftFever
2025-08-11 22:35:17 +08:00
committed by GitHub
parent 05181779d1
commit 05adf5f9c9
2 changed files with 6 additions and 19 deletions

View File

@@ -1,19 +1,10 @@
name: Claude Code Review name: Claude Code Review
on: on:
pull_request: workflow_dispatch:
types: [opened, synchronize]
# Optional: Only run on specific file changes
# paths:
# - "src/**/*.ts"
# - "src/**/*.tsx"
# - "src/**/*.js"
# - "src/**/*.jsx"
jobs: jobs:
claude-review: claude-review:
# Only run for PRs from the same repository (skip forks)
if: ${{ github.event.pull_request.head.repo.full_name == github.repository }}
# Optional: Filter by PR author # Optional: Filter by PR author
# if: | # if: |
# github.event.pull_request.user.login == 'external-contributor' || # github.event.pull_request.user.login == 'external-contributor' ||

View File

@@ -24,9 +24,9 @@ while getopts ":dpa:snt:xbc:h" opt; do
export OSX_DEPLOYMENT_TARGET="$OPTARG" export OSX_DEPLOYMENT_TARGET="$OPTARG"
;; ;;
x ) x )
export SLICER_CMAKE_GENERATOR="Ninja" export SLICER_CMAKE_GENERATOR="Ninja Multi-Config"
export SLICER_BUILD_TARGET="all" export SLICER_BUILD_TARGET="all"
export DEPS_CMAKE_GENERATOR="Ninja" export DEPS_CMAKE_GENERATOR="Ninja Multi-Config"
;; ;;
b ) b )
export BUILD_ONLY="1" export BUILD_ONLY="1"
@@ -43,7 +43,7 @@ while getopts ":dpa:snt:xbc:h" opt; do
echo " -s: Build slicer only" echo " -s: Build slicer only"
echo " -n: Nightly build" echo " -n: Nightly build"
echo " -t: Specify minimum version of the target platform, default is 11.3" echo " -t: Specify minimum version of the target platform, default is 11.3"
echo " -x: Use Ninja CMake generator, default is Xcode" echo " -x: Use Ninja Multi-Config CMake generator, default is Xcode"
echo " -b: Build without reconfiguring CMake" echo " -b: Build without reconfiguring CMake"
echo " -c: Set CMake build configuration, default is Release" echo " -c: Set CMake build configuration, default is Release"
echo " -1: Use single job for building" echo " -1: Use single job for building"
@@ -112,12 +112,8 @@ DEPS_DIR="$PROJECT_DIR/deps"
DEPS_BUILD_DIR="$DEPS_DIR/build/$ARCH" DEPS_BUILD_DIR="$DEPS_DIR/build/$ARCH"
DEPS="$DEPS_BUILD_DIR/OrcaSlicer_deps" DEPS="$DEPS_BUILD_DIR/OrcaSlicer_deps"
# Fix for Multi-config generators # For Multi-config generators like Ninja and Xcode
if [ "$SLICER_CMAKE_GENERATOR" == "Xcode" ]; then export BUILD_DIR_CONFIG_SUBDIR="/$BUILD_CONFIG"
export BUILD_DIR_CONFIG_SUBDIR="/$BUILD_CONFIG"
else
export BUILD_DIR_CONFIG_SUBDIR=""
fi
function build_deps() { function build_deps() {
# iterate over two architectures: x86_64 and arm64 # iterate over two architectures: x86_64 and arm64