Skip to content

ci-wheel: use GCC toolset when building CPython and its dependencies#421

Merged
rapids-bot[bot] merged 1 commit into
mainfrom
fix/sysconfig
Jun 12, 2026
Merged

ci-wheel: use GCC toolset when building CPython and its dependencies#421
rapids-bot[bot] merged 1 commit into
mainfrom
fix/sysconfig

Conversation

@jameslamb

@jameslamb jameslamb commented Jun 11, 2026

Copy link
Copy Markdown
Member

Contributes to #420 (hopefully "fixes", but we'll see 😅)

As described there, we observed wheel builds using scikit-build-core falling back to the GCC 8.5 at /usr/bin/g++.

  -- The CXX compiler identification is GNU 8.5.0
  -- The CUDA compiler identification is NVIDIA 12.9.86 with host compiler GNU 14.2.1
  -- Detecting CXX compiler ABI info
  -- Detecting CXX compiler ABI info - done
  -- Check for working CXX compiler: /usr/bin/g++ - skipped

Despite enabling the GCC 14 toolset:

ci-imgs/ci-wheel.Dockerfile

Lines 143 to 144 in 2bbcab7

gcc-toolset-14-gcc
gcc-toolset-14-gcc-c++

It looks like the root cause is something like:

This attempts to fix it by activating the GCC toolset before building CPython.

Notes for Reviewers

I haven't tested this locally, getting the PR up to start CI before I have to step away for a bit. Will try to return to it tonight.

I also think I have an idea of which CPython change caused this, will report an issue upstream tonight.

@jameslamb jameslamb requested a review from a team as a code owner June 11, 2026 22:35
@jameslamb jameslamb requested review from msarahan and removed request for a team June 11, 2026 22:35
@jameslamb jameslamb added breaking Introduces a breaking change bug Something isn't working labels Jun 11, 2026

@vyasr vyasr 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.

Good sleuthing! I'll trust you to go forth and test/verify, but the reasoning sounds plausible and the links you posted pass the smell test.

@bdice bdice 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.

Validated with:

docker run --rm rapidsai/staging:ci-wheel-421-26.08-cuda12.9.2-rockylinux8-py3.14 \
  python -c 'import sysconfig; print("CC =", sysconfig.get_config_var("CC")); print("CXX =", sysconfig.get_config_var("CXX"))'

Output:

CC = gcc -pthread
CXX = /opt/rh/gcc-toolset-14/root/usr/bin/g++ -pthread

@bdice

bdice commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

/merge

@rapids-bot rapids-bot Bot merged commit 44cc6aa into main Jun 12, 2026
399 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking Introduces a breaking change bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants