Skip to content

Commit 2b7b6e8

Browse files
committed
Remove setting of ninja path in CMake options now that we already assume it exists in $PATH
1 parent 3627b74 commit 2b7b6e8

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

azure-pipelines.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,8 @@ jobs:
2121
# Download various tools such as autoconf, automake and libtool
2222
wget https://github.com/light-tech/LLVM-On-iOS/releases/download/llvm12.0.0/tools.tar.xz
2323
tar xzf tools.tar.xz
24+
mv ninja tools/bin
25+
export PATH=$PATH:`pwd`/tools/bin
2426
./build-llvm-framework.sh maccatalyst iphoneos iphonesimulator
2527
displayName: 'Checkout and build llvm framework'
2628
enabled: true

build-llvm-framework.sh

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ PLATFORMS=( "$@" )
99

1010
# Constants
1111
export REPO_ROOT=`pwd`
12-
export PATH=$PATH:$REPO_ROOT/tools/bin
1312

1413
# Build libffi for a given platform
1514
function build_libffi() {
@@ -88,8 +87,7 @@ function build_llvm() {
8887
-DFFI_LIBRARY_DIR=$LIBFFI_INSTALL_DIR \
8988
-DCMAKE_BUILD_TYPE=Release \
9089
-DCMAKE_INSTALL_PREFIX=$LLVM_INSTALL_DIR \
91-
-DCMAKE_TOOLCHAIN_FILE=../llvm/cmake/platforms/iOS.cmake \
92-
-DCMAKE_MAKE_PROGRAM=$REPO_ROOT/ninja)
90+
-DCMAKE_TOOLCHAIN_FILE=../llvm/cmake/platforms/iOS.cmake)
9391

9492
case $PLATFORM in
9593
"iphoneos")

0 commit comments

Comments
 (0)