Skip to content

Bump thorin-dwp (0.10), gimli (0.33), and object (0.38.1) - #160318

Open
Zalathar wants to merge 1 commit into
rust-lang:mainfrom
Zalathar:thorin
Open

Bump thorin-dwp (0.10), gimli (0.33), and object (0.38.1)#160318
Zalathar wants to merge 1 commit into
rust-lang:mainfrom
Zalathar:thorin

Conversation

@Zalathar

@Zalathar Zalathar commented Aug 1, 2026

Copy link
Copy Markdown
Member

Originally, the hope was that this would reduce duplication of object within the compiler.

In practice, it turns out that this merely bumps the object major-version-set from {0.36, 0.37} to {0.37, 0.38}.

(The holdout for object@0.37 is ar_archive_writer, which does have a newer release available. But that would end up pulling in object@0.39 instead.)

There is still an object@0.36 entry in Cargo.lock, but that comes via ui_test which is a dev-dependency for clippy and miri.

@rustbot

rustbot commented Aug 1, 2026

Copy link
Copy Markdown
Collaborator

The list of allowed third-party dependencies may have been modified! You must ensure that any new dependencies have compatible licenses before merging.

cc @davidtwco, @BoxyUwU

These commits modify the Cargo.lock file. Unintentional changes to Cargo.lock can be introduced when switching branches and rebasing PRs.

If this was unintentional then you should revert the changes before this PR is merged.
Otherwise, you can ignore this comment.

The run-make-support library was changed

cc @jieyouxu

@rustbot rustbot added A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. A-run-make Area: port run-make Makefiles to rmake.rs A-tidy Area: The tidy tool S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) labels Aug 1, 2026
@rustbot

rustbot commented Aug 1, 2026

Copy link
Copy Markdown
Collaborator

r? @Mark-Simulacrum

rustbot has assigned @Mark-Simulacrum.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: @Mark-Simulacrum

@rustbot rustbot added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label Aug 1, 2026
@Zalathar

Zalathar commented Aug 1, 2026

Copy link
Copy Markdown
Member Author

cc @mejrs who reminded me of this.

@rust-log-analyzer

This comment has been minimized.

@Zalathar

Zalathar commented Aug 1, 2026

Copy link
Copy Markdown
Member Author

Ah, looks like bumping the deps of run-make-support (which is not strictly required) broke some individual run-make tests.

Comment on lines +83 to +87
let entry_depth = entry.depth;
let Some(child_entry) = cursor.next_dfs().unwrap() else {
panic!("Missing child of DW_TAG_variant");
};
assert_eq!(child_entry.depth, entry_depth + 1);

@Zalathar Zalathar Aug 1, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Per the changelog for 0.33.0:

Changed read::EntriesCursor::next_dfs to no longer return the depth delta. Use read::DebuggingInformationEntry::depth instead.

View changes since the review

@jieyouxu jieyouxu assigned jieyouxu and unassigned jieyouxu Aug 1, 2026
@Mark-Simulacrum

Copy link
Copy Markdown
Member

(The holdout for object@0.37 is ar_archive_writer, which does have a newer release available. But that would end up pulling in object@0.39 instead.)

This looks fixed - thanks @bjorn3 - but I guess it can be separately bumped anyway since it's not going to help dedupe.

@bors r+

@rust-bors

rust-bors Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

📌 Commit 83d6b96 has been approved by Mark-Simulacrum

It is now in the queue for this repository.

@rust-bors rust-bors Bot added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 2, 2026
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Aug 2, 2026
Bump thorin-dwp (0.10), gimli (0.33), and object (0.38.1)

Originally, the hope was that this would reduce duplication of `object` within the compiler.

In practice, it turns out that this merely bumps the object major-version-set from {0.36, 0.37} to {0.37, 0.38}.

(The holdout for object@0.37 is `ar_archive_writer`, which does have a newer release available. But that would end up pulling in object@0.39 instead.)

There is still an object@0.36 entry in Cargo.lock, but that comes via `ui_test` which is a dev-dependency for clippy and miri.
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Aug 2, 2026
Bump thorin-dwp (0.10), gimli (0.33), and object (0.38.1)

Originally, the hope was that this would reduce duplication of `object` within the compiler.

In practice, it turns out that this merely bumps the object major-version-set from {0.36, 0.37} to {0.37, 0.38}.

(The holdout for object@0.37 is `ar_archive_writer`, which does have a newer release available. But that would end up pulling in object@0.39 instead.)

There is still an object@0.36 entry in Cargo.lock, but that comes via `ui_test` which is a dev-dependency for clippy and miri.
rust-bors Bot pushed a commit that referenced this pull request Aug 2, 2026
…uwer

Rollup of 5 pull requests

Successful merges:

 - #159195 (Drop elaboration: Only create a reset block if there are flags to reset.)
 - #160318 (Bump thorin-dwp (0.10), gimli (0.33), and object (0.38.1))
 - #159756 (bootstrap: Move some shared source files into a new `shim_utils` crate)
 - #160385 (Assign ownership for Renovate configuration file)
 - #160397 (Box::new: fix safety comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. A-run-make Area: port run-make Makefiles to rmake.rs A-tidy Area: The tidy tool S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants