Skip to content

[beta 1.97] fix(publish): avoid false deadlock when to_confirm is non-empty#17094

Merged
epage merged 1 commit into
rust-lang:rust-1.97.0from
weihanglo:beta-backport-17071
Jun 11, 2026
Merged

[beta 1.97] fix(publish): avoid false deadlock when to_confirm is non-empty#17094
epage merged 1 commit into
rust-lang:rust-1.97.0from
weihanglo:beta-backport-17071

Conversation

@weihanglo

Copy link
Copy Markdown
Member

Beta backports

In order to make CI pass, the following PRs are also cherry-picked:

…-empty (rust-lang#17071)

Backport of rust-lang#17071.

Fixes rust-lang#17028

PR rust-lang#16722 introduced a check that fires an internal error when `ready`
is empty inside the publish loop. It fired even when packages were still
legitimately waiting for registry confirmation in `to_confirm`, causing
valid workspace publishes to fail with a false deadlock error. A true
deadlock only exists when both `ready` and `to_confirm` are empty.
@rustbot rustbot added A-interacts-with-crates.io Area: interaction with registries Command-publish S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jun 11, 2026
@rustbot

rustbot commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator

r? @epage

rustbot has assigned @epage.
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: @ehuss, @epage, @weihanglo
  • @ehuss, @epage, @weihanglo expanded to ehuss, epage, weihanglo
  • Random selection from ehuss, epage

@rustbot

rustbot commented Jun 11, 2026

Copy link
Copy Markdown
Collaborator

⚠️ Warning ⚠️

  • Pull requests are usually filed against the master branch for this repo, but this one is against rust-1.97.0. Please double check that you specified the right target!

@epage epage merged commit 910306f into rust-lang:rust-1.97.0 Jun 11, 2026
29 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jun 11, 2026
@weihanglo weihanglo deleted the beta-backport-17071 branch June 11, 2026 19:39
@rustbot rustbot added this to the 1.97.0 milestone Jun 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-interacts-with-crates.io Area: interaction with registries Command-publish

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants