Skip to content

fix: rename gnu→gnu_omp variant and add stdlib(c) for Intel glibc#13

Merged
lxknll merged 2 commits into
conda-forge:mainfrom
gunnar1987:fix/gnu-omp-variant-and-intel-glibc
Jun 9, 2026
Merged

fix: rename gnu→gnu_omp variant and add stdlib(c) for Intel glibc#13
lxknll merged 2 commits into
conda-forge:mainfrom
gunnar1987:fix/gnu-omp-variant-and-intel-glibc

Conversation

@gunnar1987

Copy link
Copy Markdown
Contributor

Summary

Fixes two reported build issues:

1. GNU/OMP variant resolves to MPI at install time

The gnu variant's build string (gnu_h…) is a prefix of gnu_mpi_h…, so conda install "runner=*=gnu_*" matches both variants. Renamed gnugnu_omp to make the globs unambiguous.

2. Intel variant linked to wrong glibc

${{ stdlib('c') }} was only in the GNU branch of the compiler conditional. The Intel build:

  • Used system glibc 2.34 (Alma 9 Docker image) instead of the conda-forge sysroot
  • Produced a package with no __glibc constraint in the metadata

Fix: moved stdlib('c') outside the conditional and set c_stdlib_version=2.28 for Intel (required by ifx_linux-64).

Changes

  • recipe/recipe.yaml — moved stdlib('c') before the compiler conditional; updated variant description
  • recipe/conda_build_config.yamlgnugnu_omp; added per-variant c_stdlib_version with zip_keys
  • .ci_support/ — updated runner_variant and c_stdlib_version in affected configs

Verification

All three Linux variants rebuilt and tested successfully with rattler-build:

Variant Build string __glibc Tests
gnu_omp gnu_omp_h6d0f835_1 >=2.17
gnu_mpi gnu_mpi_h1733be7_1 >=2.17
intel intel_h6facef5_1 >=2.28

Note: A conda smithy rerender is needed after merge to regenerate CI support file names.

- Rename 'gnu' variant to 'gnu_omp' to prevent build-string glob
  overlap with 'gnu_mpi' (conda install "runner=*=gnu_*" matched both)
- Move stdlib('c') outside compiler conditional so Intel variant gets
  proper sysroot linkage and __glibc run_export constraint
- Set c_stdlib_version=2.28 for Intel variant (Intel compilers require
  sysroot>=2.28, incompatible with default 2.17)
@conda-forge-admin

Copy link
Copy Markdown
Contributor

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe/recipe.yaml) and found it was in an excellent condition.

@lxknll lxknll left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, thank you a lot @gunnar1987.

Comment thread recipe/conda_build_config.yaml Outdated
Comment thread recipe/recipe.yaml
@lxknll

lxknll commented Jun 9, 2026 via email

Copy link
Copy Markdown
Contributor

@lxknll lxknll merged commit 2062c86 into conda-forge:main Jun 9, 2026
8 checks passed
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.

3 participants