remove pybind

pytorch · Apr 9, 2024 · ac4af0b · ac4af0b
1 parent 1e49784
commit ac4af0b
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/.github/workflows/runner_et.yml b/.github/workflows/runner_et.yml
@@ -35,9 +35,9 @@ jobs:
           pip install ninja
           pip install zstd
           pip install -r requirements.txt
-          softwareupdate --install xcode--select
 
           export LLAMA_FAST_ROOT=${PWD}
+          export ET_NO_PYBIND=1
           ./scripts/install_et.sh
           cmake -S ./runner-et -B build/cmake-out -G Ninja
           cmake --build ./build/cmake-out

diff --git a/scripts/install_et.sh b/scripts/install_et.sh
@@ -16,7 +16,13 @@ cp ${LLAMA_FAST_ROOT}/scripts/fixes_et/module.cpp ${LLAMA_FAST_ROOT}/build/src/e
 cp ${LLAMA_FAST_ROOT}/scripts/fixes_et/managed_tensor.h ${LLAMA_FAST_ROOT}/build/src/executorch/extension/runner_util/managed_tensor.h # ET is missing headers for vector/memory.  This causes downstream issues when building runner-et.
 
 echo "Building and installing python libraries"
-./install_requirements.sh --pybind xnnpack
+if [ z "${ET_NO_PYBIND}" ]; then
+    echo "Installing pybind"
+    ./install_requirements.sh --pybind xnnpack
+else
+    echo "Not installing pybind"
+    ./install_requirements.sh
+fi
 
 echo "Building and installing C++ libraries"
 echo "Inside: ${PWD}"