Skip to content

Fix cross-platform Bazel Test failures#26

Merged
dzbarsky merged 1 commit into
mainfrom
codex/fix-main-ci
Jun 13, 2026
Merged

Fix cross-platform Bazel Test failures#26
dzbarsky merged 1 commit into
mainfrom
codex/fix-main-ci

Conversation

@dzbarsky

Copy link
Copy Markdown
Member

What

Restore CARGO_PKG_* variables for Rust compilation and Cargo build-script execution, forward OutputGroupInfo.pdb_file beside the executable produced by dbg_rust_binary, and use the hermetic LLVM C++ toolchain in the macOS Bazel Test job.

Why

The main Bazel Test run failed because serde_core and serde_derive require CARGO_PKG_VERSION_PATCH, the Windows remap_path_prefix test could not find panic_bin.pdb, and the macOS C++ toolchain exported libtool as Cargo's AR.

Checks

  • buildifier -mode=check on modified Bazel files
  • rustfmt --check on modified Rust files
  • @rrtd__serde-1.0.228//:serde with the macOS LLVM platform
  • //cargo/tests/build_env:cargo_env-vars_test
  • //cargo/tests/build_env:cargo-env-vars-custom-crate-name-test
  • //cargo/tests/dep_env:build_read_b
  • //test/unit/remap_path_prefix:integration_test

@dzbarsky dzbarsky force-pushed the codex/fix-main-ci branch 6 times, most recently from 89c69b3 to 5d72f6a Compare June 13, 2026 22:25
@dzbarsky dzbarsky force-pushed the codex/fix-main-ci branch from 5d72f6a to 8424603 Compare June 13, 2026 23:20
@dzbarsky dzbarsky changed the title [codex] Fix cross-platform Bazel Test failures Fix cross-platform Bazel Test failures Jun 13, 2026
@dzbarsky dzbarsky marked this pull request as ready for review June 13, 2026 23:32
@dzbarsky dzbarsky merged commit 64fcfaa into main Jun 13, 2026
3 checks passed
dzbarsky added a commit that referenced this pull request Jun 14, 2026
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