-
-
Notifications
You must be signed in to change notification settings - Fork 14.4k
feat(rustdoc): stabilize --emit flag
#146220
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
base: main
Are you sure you want to change the base?
Conversation
|
Questions:
|
|
cc @aDotInTheVoid since you've involved in the PR removing the last blocker :) |
|
We discussed on today's rustdoc meeting. Before going any further, we will investigate exactly what each |
Thanks for it! This PR contains a doc based on my understanding and the history of these options, but I am not 100% sure if that is correct |
e1a5014 to
cc82db2
Compare
This comment has been minimized.
This comment has been minimized.
cc82db2 to
bfb63c3
Compare
|
I haven't read this yet but here are probably some very important and relevant considerations: #83784. |
|
For the use case in Cargo, we actually don't need anything other than
I guess one way forward is that rustdoc provides an extra emit type, say,
|
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. Part of rust-lang#146220
|
I've opened #148180, a follow-up that removes the no-op unversioned-shared-resources, which hasn't done anything ever since we switched to using hashes for cache busting. |
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
bfb63c3 to
b7f8908
Compare
This comment has been minimized.
This comment has been minimized.
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
…rces, 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 rust-lang#146220 Part of rust-lang#83784
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
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR has updated and removed the unversioned-shared-resources option, which was removed in #148180
…=GuillaumeGomez rustdoc: fix `--emit=dep-info` on scraped examples Makes sure both stages (the scraping process itself, and the doc build) emit complete dependency lists. CC rust-lang#146220 Part of rust-lang#83784
…=GuillaumeGomez rustdoc: fix `--emit=dep-info` on scraped examples Makes sure both stages (the scraping process itself, and the doc build) emit complete dependency lists. CC rust-lang#146220 Part of rust-lang#83784
…=GuillaumeGomez rustdoc: fix `--emit=dep-info` on scraped examples Makes sure both stages (the scraping process itself, and the doc build) emit complete dependency lists. CC rust-lang#146220 Part of rust-lang#83784
…eGomez rustdoc: fix `--emit=dep-info` on scraped examples Makes sure both stages (the scraping process itself, and the doc build) emit complete dependency lists. CC rust-lang/rust#146220 Part of rust-lang/rust#83784
|
I haven’t found any more problems with this feature (that don’t involve its interaction with other unstable features). I think it can be stabilized in its current form. |
|
@rfcbot fcp merge |
|
Error encounted: |
|
@rfcbot fcp merge rustdoc |
|
Team member @GuillaumeGomez has proposed to merge this. The next step is review by the rest of the tagged team members: No concerns currently listed. Once a majority of reviewers approve (and at most 2 approvals are outstanding), this will enter its final comment period. If you spot a major issue that hasn't been raised at any point in this process, please speak up! See this document for info about what commands tagged team members can give me. |
| This flag controls the types of output by rustdoc. It accepts a comma-separated | ||
| list of values, and may be specified multiple times. The valid emit kinds are: | ||
|
|
||
| - `toolchain-shared-resources` --- Generates shared static files that their |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| - `toolchain-shared-resources` --- Generates shared static files that their | |
| - `toolchain-shared-resources` --- Generates shared static files whose |
| change if the toolchain version or their contents change, so it is safe to | ||
| cache them wit the `Cache-Control: immutable` directive. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| change if the toolchain version or their contents change, so it is safe to | |
| cache them wit the `Cache-Control: immutable` directive. | |
| change if the toolchain version or their contents change, or with the `Cache-Control: immutable` directive. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated. Thanks!
| This flag controls the types of output by rustdoc. It accepts a comma-separated | ||
| list of values, and may be specified multiple times. The valid emit kinds are: | ||
|
|
||
| - `toolchain-shared-resources` --- Generates shared static files that their |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure whether these triple-dashes are considered standard :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's a mdbook stuff. Not sure about rustdoc but Cargo book uses that everywhere
https://rust-lang.github.io/mdBook/format/markdown.html#smart-punctuation
b7f8908 to
d5d59c7
Compare
|
This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed. Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers. |
TODO: stabilization report?
r? rustdoc