Skip to content

Remove crates.io compiler-builtins from the library workspace #142265

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
tgross35 opened this issue Jun 9, 2025 · 0 comments
Open

Remove crates.io compiler-builtins from the library workspace #142265

tgross35 opened this issue Jun 9, 2025 · 0 comments
Assignees
Labels
A-compiler-builtins Area: compiler-builtins (https://github.com/rust-lang/compiler-builtins) C-cleanup Category: PRs that clean code up or issues documenting cleanup. T-libs Relevant to the library team, which will review and decide on the PR/issue.

Comments

@tgross35
Copy link
Contributor

tgross35 commented Jun 9, 2025

After #141993 we should eliminate compiler-builtins from any crates that are depending on it via crates.io to remove the duplicate dependency so the patch isn't needed. Todo list, taken from the PR:

Updates in rust-lang/rust:

@rustbot rustbot added the needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. label Jun 9, 2025
@tgross35 tgross35 added C-cleanup Category: PRs that clean code up or issues documenting cleanup. T-libs Relevant to the library team, which will review and decide on the PR/issue. A-compiler-builtins Area: compiler-builtins (https://github.com/rust-lang/compiler-builtins) and removed needs-triage This issue may need triage. Remove it if it has been sufficiently triaged. labels Jun 9, 2025
@tgross35 tgross35 self-assigned this Jun 9, 2025
tgross35 added a commit to tgross35/rust that referenced this issue Jun 10, 2025
This removes the `compiler_builtins` dependency from a handful of
library dependencies, which is progress toward [1].

[1]: rust-lang#142265
tgross35 added a commit to tgross35/rust that referenced this issue Jun 10, 2025
This removes the `compiler_builtins` dependency from a handful of
library dependencies, which is progress toward [1].

[1]: rust-lang#142265
workingjubilee added a commit to workingjubilee/rustc that referenced this issue Jun 11, 2025
…r=Mark-Simulacrum

Update dependencies in `library/Cargo.lock`

This removes the `compiler_builtins` dependency from a handful of library dependencies, which is progress toward [1].

[1]: rust-lang#142265
tgross35 added a commit to tgross35/rust that referenced this issue Jun 11, 2025
This removes the `compiler_builtins` dependency from a handful of
library dependencies, which is progress toward [1].

[1]: rust-lang#142265
tgross35 added a commit to tgross35/rust that referenced this issue Jun 11, 2025
0.37.0 is a semver-breaking release but the only breakage is in
`elf::R_RISCV_GNU_*` and `pe::IMAGE_WEAK_EXTERN_*` constants, as well as
Mach-O dyld. This API is not used by `std`, so we should be fine to
upgrade.

This new version also includes functionality for parsing Wasm object
files that we may eventually like to make use of.

Also includes the minor bump from 0.37.0 to 0.37.1 to help [1].

Changelog: https://github.com/gimli-rs/object/blob/master/CHANGELOG.md#0370

[1]: rust-lang#142265
tgross35 added a commit to tgross35/rust that referenced this issue Jun 11, 2025
0.25.0 is a breaking change only because it upgrades the `gimli`
version. It also includes a change to the `compiler-builtins` dependency
that helps with [1].

Changelog: https://github.com/gimli-rs/addr2line/blob/master/CHANGELOG.md#0250-20250611

[1]: rust-lang#142265
tgross35 added a commit to tgross35/unwinding that referenced this issue Jun 11, 2025
The breaking change here involves API for `read::Dwarf`,
`read::CallFrameInstruction`, and `write::LineProgram`, none of which
are used in this crate. Making this change will allow `std` to upgrade
to a version of `gimli` that drops the `compiler_builtins` dependency,
which is needed for [1].

Changelog: https://github.com/gimli-rs/gimli/blob/master/CHANGELOG.md#0320

[1]: rust-lang/rust#142265
nbdd0121 pushed a commit to nbdd0121/unwinding that referenced this issue Jun 11, 2025
The breaking change here involves API for `read::Dwarf`,
`read::CallFrameInstruction`, and `write::LineProgram`, none of which
are used in this crate. Making this change will allow `std` to upgrade
to a version of `gimli` that drops the `compiler_builtins` dependency,
which is needed for [1].

Changelog: https://github.com/gimli-rs/gimli/blob/master/CHANGELOG.md#0320

[1]: rust-lang/rust#142265
workingjubilee added a commit to workingjubilee/rustc that referenced this issue Jun 12, 2025
…r=Mark-Simulacrum

Update dependencies in `library/Cargo.lock`

This removes the `compiler_builtins` dependency from a handful of library dependencies, which is progress toward [1].

[1]: rust-lang#142265
workingjubilee added a commit to workingjubilee/rustc that referenced this issue Jun 12, 2025
…r=Mark-Simulacrum

Update dependencies in `library/Cargo.lock`

This removes the `compiler_builtins` dependency from a handful of library dependencies, which is progress toward [1].

[1]: rust-lang#142265
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this issue Jun 13, 2025
…r=Mark-Simulacrum

Update dependencies in `library/Cargo.lock`

This removes the `compiler_builtins` dependency from a handful of library dependencies, which is progress toward [1].

[1]: rust-lang#142265
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this issue Jun 13, 2025
…=Mark-Simulacrum

Upgrade `object`, `addr2line`, and `unwinding` in the standard library

Object:

0.37.0 is a semver-breaking release but the only breakage is in `elf::R_RISCV_GNU_*` and `pe::IMAGE_WEAK_EXTERN_*` constants, as well as Mach-O dyld. This API is not used by `std`, so we should be fine to upgrade.

This new version also includes functionality for parsing Wasm object files that we may eventually like to make use of.

Changelog: https://github.com/gimli-rs/object/blob/master/CHANGELOG.md#0370

Addr2line:

0.25.0 is a breaking change only because it upgrades the `gimli` version. It also includes a change to the `compiler-builtins` dependency that helps with [1].

Changelog: https://github.com/gimli-rs/addr2line/blob/master/CHANGELOG.md#0250-20250611

[1]: rust-lang#142265
rust-timer added a commit that referenced this issue Jun 13, 2025
Rollup merge of #142308 - tgross35:upgrade-library-object, r=Mark-Simulacrum

Upgrade `object`, `addr2line`, and `unwinding` in the standard library

Object:

0.37.0 is a semver-breaking release but the only breakage is in `elf::R_RISCV_GNU_*` and `pe::IMAGE_WEAK_EXTERN_*` constants, as well as Mach-O dyld. This API is not used by `std`, so we should be fine to upgrade.

This new version also includes functionality for parsing Wasm object files that we may eventually like to make use of.

Changelog: https://github.com/gimli-rs/object/blob/master/CHANGELOG.md#0370

Addr2line:

0.25.0 is a breaking change only because it upgrades the `gimli` version. It also includes a change to the `compiler-builtins` dependency that helps with [1].

Changelog: https://github.com/gimli-rs/addr2line/blob/master/CHANGELOG.md#0250-20250611

[1]: #142265
rust-timer added a commit that referenced this issue Jun 13, 2025
Rollup merge of #142276 - tgross35:update-library-lockfile, r=Mark-Simulacrum

Update dependencies in `library/Cargo.lock`

This removes the `compiler_builtins` dependency from a handful of library dependencies, which is progress toward [1].

[1]: #142265
tgross35 added a commit to tgross35/unwinding that referenced this issue Jun 13, 2025
Since [1] the crates.io version of `compiler-builtins` is no longer
needed as part of `rustc-dep-of-std`, so remove it here.

[1]: rust-lang/rust#142265
github-actions bot pushed a commit to rust-lang/miri that referenced this issue Jun 14, 2025
…mulacrum

Update dependencies in `library/Cargo.lock`

This removes the `compiler_builtins` dependency from a handful of library dependencies, which is progress toward [1].

[1]: rust-lang/rust#142265
github-actions bot pushed a commit to rust-lang/miri that referenced this issue Jun 14, 2025
…ulacrum

Upgrade `object`, `addr2line`, and `unwinding` in the standard library

Object:

0.37.0 is a semver-breaking release but the only breakage is in `elf::R_RISCV_GNU_*` and `pe::IMAGE_WEAK_EXTERN_*` constants, as well as Mach-O dyld. This API is not used by `std`, so we should be fine to upgrade.

This new version also includes functionality for parsing Wasm object files that we may eventually like to make use of.

Changelog: https://github.com/gimli-rs/object/blob/master/CHANGELOG.md#0370

Addr2line:

0.25.0 is a breaking change only because it upgrades the `gimli` version. It also includes a change to the `compiler-builtins` dependency that helps with [1].

Changelog: https://github.com/gimli-rs/addr2line/blob/master/CHANGELOG.md#0250-20250611

[1]: rust-lang/rust#142265
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-compiler-builtins Area: compiler-builtins (https://github.com/rust-lang/compiler-builtins) C-cleanup Category: PRs that clean code up or issues documenting cleanup. T-libs Relevant to the library team, which will review and decide on the PR/issue.
Projects
None yet
Development

No branches or pull requests

2 participants