Skip to content

fix(ostool): use toolchain llvm-objcopy for bin artifacts#140

Merged
ZR233 merged 1 commit into
mainfrom
codex/internal-llvm-objcopy
Jun 18, 2026
Merged

fix(ostool): use toolchain llvm-objcopy for bin artifacts#140
ZR233 merged 1 commit into
mainfrom
codex/internal-llvm-objcopy

Conversation

@ZR233

@ZR233 ZR233 commented Jun 18, 2026

Copy link
Copy Markdown
Member

Summary

  • resolve llvm-objcopy from the active Rust toolchain instead of invoking PATH rust-objcopy in ostool runtime artifact generation
  • remove the injectable ObjectTools/fake objcopy path from production and tests
  • add cargo-binutils rust-objcopy comparison tests for ostool to_bin output in debug and non-debug modes
  • install llvm-tools and cargo-binutils in CI for the new production path and comparison tests

Tests

  • cargo fmt --all -- --check
  • cargo test -p ostool
  • cargo clippy --target x86_64-unknown-linux-gnu --all-features
  • cargo build --target x86_64-unknown-linux-gnu --all-features

@ZR233 ZR233 merged commit 8f524ce into main Jun 18, 2026
2 checks passed
@ZR233 ZR233 deleted the codex/internal-llvm-objcopy branch June 18, 2026 06: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