Skip to content

Commit 69eca07

Browse files
davidpichardiemeta-codesync[bot]
authored andcommitted
[release] skip opam switch setup in release script when already configured
Summary: The release script unconditionally ran `build-infer.sh --only-setup-opam` which created a fresh opam switch, losing the LLVM OCaml bindings installed during the prior build step. This caused `ctypes-foreign` to fail during `make install-with-libs` on macOS with a path error. Only set up the opam switch when one is not already active. Reviewed By: ngorogiannis Differential Revision: D103987909 fbshipit-source-id: ba642637ba3f9432cbc67087d1e88ebb1c955787
1 parent ebb6cb5 commit 69eca07

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

scripts/create_binary_release.sh

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,9 @@ JOBS=${JOBS:-$NCPUS}
3838
pushd "$ROOT_DIR"
3939
rm -fr "$RELEASE_NAME"
4040

41-
./build-infer.sh --only-setup-opam
41+
if ! opam switch show >/dev/null 2>&1; then
42+
./build-infer.sh --only-setup-opam
43+
fi
4244
eval $(opam env)
4345
touch .release
4446
./autogen.sh

0 commit comments

Comments
 (0)