Skip to content

Fix macOS conda-forge OpenMP builds: install clang_osx-* activation packages - #236

Draft
lesteve with Copilot wants to merge 2 commits into
masterfrom
copilot/debug-conda-forge-ci-failure
Draft

Fix macOS conda-forge OpenMP builds: install clang_osx-* activation packages#236
lesteve with Copilot wants to merge 2 commits into
masterfrom
copilot/debug-conda-forge-ci-failure

Conversation

Copilot AI commented Sep 4, 2026

Copy link
Copy Markdown

macOS conda-forge CI jobs fail with clang: error: unsupported option '-fopenmp' (and, for the flexiblas job, 'omp.h' file not found). Root cause: conda-forge's "minimally activated compilers" change means the compilers metapackage now depends on the bare clang package, which provides only a versioned compiler binary and never sets CC/CFLAGS/CPPFLAGS/LDFLAGS on conda activate. With no unversioned clang on PATH inside the conda env and CC unset, the build tools fall back to the system Apple clang, which doesn't support -fopenmp and can't find conda-installed llvm-openmp headers.

Changes

  • continuous_integration/install.sh: when INSTALL_LIBOMP=conda-forge on Darwin, additionally install clang_osx-<arch>/clangxx_osx-<arch> (arch detected via uname -m). These packages ship the activation scripts that set CC/CXX/CFLAGS/CPPFLAGS/LDFLAGS pointing at $CONDA_PREFIX, restoring the pre-"minimal activation" behavior.
  • .github/workflows/test.yml: set INSTALL_LIBOMP: "conda-forge" on the pylatest_flexiblas macOS job so it goes through the same fixed activation path (it hit the same root cause via a missing omp.h).
  • Debugging: print which clang, clang --version, and CC/CPPFLAGS/LDFLAGS right after conda activate on macOS, to make it obvious in CI logs which compiler is actually picked up.

Co-authored-by: lesteve <1680079+lesteve@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix macOS conda-forge CI job for clang compatibility Fix macOS conda-forge OpenMP builds: install clang_osx-* activation packages Sep 4, 2026
Copilot AI requested a review from lesteve September 4, 2026 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants