Skip to content

Commit

Permalink
ci: Streamline the old MacOS-12 CI test (AcademySoftwareFoundation#4465)
Browse files Browse the repository at this point in the history
It's an old flavor not well supported by homebrew any longer, so what's
happening is that some of the dependency packages we ask to install have
to build from source, which means it takes a long long time.

By disabling OpenCV and Qt homebrew installation for it, we can make it
fast again. Neither the minimal OpenCV interop, nor osltoy (the only
thing that needs Qt) are exercised by the automated testsuite anyway, so
I don't think there's any loss of testing functionality in practice.

Signed-off-by: Larry Gritz <lg@larrygritz.com>
  • Loading branch information
lgritz authored and zachlewis committed Oct 1, 2024
1 parent b13a055 commit b4b1269
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -512,7 +512,7 @@ jobs:
cc_compiler: clang
cxx_compiler: clang++
cxx_std: 17
python_ver: "3.10"
python_ver: "3.11"
aclang: 13
setenvs: export INSTALL_QT=0 INSTALL_OPENCV=0
- desc: MacOS-13
Expand Down
2 changes: 1 addition & 1 deletion src/build-scripts/install_homebrew_deps.bash
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ brew install --display-times -q freetype libraw dcmtk pybind11 numpy || true
brew install --display-times -q ffmpeg libheif ptex || true
brew install --display-times -q tbb || true
brew install --display-times -q openvdb || true
if [[ "${USE_OPENCV}" != "0" ]] && [[ "${USE_OPENCV:=1}" != "0" ]] ; then
if [[ "${USE_OPENCV}" != "0" ]] && [[ "${INSTALL_OPENCV:=1}" != "0" ]] ; then
brew install --display-times -q opencv || true
fi
if [[ "${USE_QT:=1}" != "0" ]] && [[ "${INSTALL_QT:=1}" != "0" ]] ; then
Expand Down

0 comments on commit b4b1269

Please sign in to comment.