Skip to content

Conversation

@notriddle
Copy link
Contributor

@notriddle notriddle commented Oct 27, 2025

This option hasn't done anything for a long time, and can be removed. I've kept a shim in place to avoid breaking docs.rs, but the option no longer does anything.

Using git-blame, I tracked this option down to
f77ebd4, the commit that introduced EmitType in the first place. It was used with SharedResource::Unversioned, which no longer exists since f9e1f6f removed them.

CC #146220
Part of #83784

@rustbot rustbot added A-run-make Area: port run-make Makefiles to rmake.rs S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. labels Oct 27, 2025
@rustbot
Copy link
Collaborator

rustbot commented Oct 27, 2025

r? @GuillaumeGomez

rustbot has assigned @GuillaumeGomez.
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

Copy link
Member

@weihanglo weihanglo left a comment

Choose a reason for hiding this comment

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

Not on t-rustdoc but I did pretty much the same investigation in #146220, and got the same result as Michael's.

Thanks for this!

View changes since this review

This option hasn't done anything for a long time, and can be removed. I've kept a shim in place to avoid breaking docs.rs, but the option no longer does anything. Using git-blame, I tracked this option down to f77ebd4, the commit that introduced EmitType in the first place. It was used with SharedResource::Unversioned, which no longer exists since f9e1f6f removed them. CC rust-lang#146220 Part of rust-lang#83784
@notriddle notriddle force-pushed the unversioned-shared-resources branch from 87c8068 to fe3490c Compare October 28, 2025 00:29
notriddle added a commit to notriddle/docs.rs that referenced this pull request Oct 28, 2025
syphar pushed a commit to rust-lang/docs.rs that referenced this pull request Oct 28, 2025
@GuillaumeGomez
Copy link
Member

Thanks!

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Oct 28, 2025

📌 Commit fe3490c has been approved by GuillaumeGomez

It is now in the queue for this repository.

@bors bors 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 Oct 28, 2025
bors added a commit that referenced this pull request Oct 28, 2025
Rollup of 3 pull requests Successful merges: - #148173 (Emit delayed bug during wfck for stranded opaque) - #148177 (Allow codegen backends to indicate which crate types they support) - #148180 (rustdoc: remove `--emit=unversioned-shared-resources`) r? `@ghost` `@rustbot` modify labels: rollup
@bors bors merged commit 71c4c89 into rust-lang:master Oct 28, 2025
11 checks passed
rust-timer added a commit that referenced this pull request Oct 28, 2025
Rollup merge of #148180 - notriddle:unversioned-shared-resources, r=GuillaumeGomez rustdoc: remove `--emit=unversioned-shared-resources` This option hasn't done anything for a long time, and can be removed. I've kept a shim in place to avoid breaking docs.rs, but the option no longer does anything. Using git-blame, I tracked this option down to f77ebd4, the commit that introduced EmitType in the first place. It was used with SharedResource::Unversioned, which no longer exists since f9e1f6f removed them. CC #146220 Part of #83784
@rustbot rustbot added this to the 1.93.0 milestone Oct 28, 2025
@notriddle notriddle deleted the unversioned-shared-resources branch October 29, 2025 04:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-run-make Area: port run-make Makefiles to rmake.rs S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.

5 participants