Skip to content

Fix bitrotted local builds of the support library#578

Merged
maleadt merged 1 commit into
mainfrom
tb/fix_build_ci
Jun 10, 2026
Merged

Fix bitrotted local builds of the support library#578
maleadt merged 1 commit into
mainfrom
tb/fix_build_ci

Conversation

@maleadt

@maleadt maleadt commented Jun 10, 2026

Copy link
Copy Markdown
Member

The local build of liboneapi_support (as triggered by deps/build_ci.jl on CI when the wrappers are newer than the latest oneAPI_Support_jll build) had bitrotted in two ways:

  • Conda.jl precompilation failure: when the Conda root environment directory has been removed (e.g., by depot clean-up on CI agents) while Conda.jl's deps.jl still points to it, precompilation fails with ArgumentError: Path to conda environment is not valid, because Conda.prefix(ROOTENV) runs at module top level. We now pre-create the directory before instantiating; Conda lazily re-installs itself on first use anyway.

  • SSL certificate verification failure: old Conda root environments ship a certifi CA bundle (e.g., 2022.06.15) that cannot validate the Sectigo certificate chain currently served by software.repos.intel.com, so every conda install from the Intel channel fails with CERTIFICATE_VERIFY_FAILED. We now update ca-certificates/certifi in the root environment before installing the toolchain.

With these changes, deps/build_ci.jl again builds the support library end-to-end locally (dpcpp/MKL 2025.2.0, only deprecation warnings during compilation).

🤖 Generated with Claude Code

Two issues prevented deps/build_ci.jl from building liboneapi_support:

- Conda.jl fails to precompile when its root environment directory has
  been removed (e.g., by depot clean-up) while its deps.jl still points
  to it. Pre-create the directory before instantiating so that
  precompilation succeeds.

- The CA roots bundled with old Conda root environments (certifi from
  2022) do not validate the certificate chain currently used by Intel's
  package repository, making every package installation fail with
  CERTIFICATE_VERIFY_FAILED. Update ca-certificates/certifi first.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@codecov

codecov Bot commented Jun 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.20%. Comparing base (8f44c47) to head (5a0c973).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #578   +/-   ##
=======================================
  Coverage   79.20%   79.20%           
=======================================
  Files          48       48           
  Lines        3232     3232           
=======================================
  Hits         2560     2560           
  Misses        672      672           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@maleadt maleadt merged commit 6e2e2d8 into main Jun 10, 2026
5 checks passed
@maleadt maleadt deleted the tb/fix_build_ci branch June 10, 2026 20:11
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.

1 participant