Skip to content

Conversation

@fmease
Copy link
Member

@fmease fmease commented Oct 25, 2025

Context: #133073, #139224 (TL;DR: libtest has soft-deprecated --nocapture in favor a new & stable --no-capture; we should follow suit).

Since the rustdoc flag is unstable (tracking issue: #148116), we're allowed to remove the old flag immediately. However since the flag has existed for 4 years we could hard-deprecate the flag first or at least be considerate and provide a diagnostic referring users to the new flag. This PR does neither. Let me know what you would think would be best.

Cargo doesn't use this flag, not yet at least (rust-lang/cargo#9705), so we really are free to sunset this flag without bigger consequences.

@rustbot rustbot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. labels Oct 25, 2025
@fmease fmease force-pushed the rustdoc-no-capture branch from 43b53fb to 7442a43 Compare October 25, 2025 16:59
@rust-log-analyzer

This comment has been minimized.

@fmease fmease force-pushed the rustdoc-no-capture branch from 7442a43 to 11b82e1 Compare October 25, 2025 17:46
@rustbot rustbot added the A-run-make Area: port run-make Makefiles to rmake.rs label Oct 25, 2025
@fmease fmease changed the title rustdoc: Rename unstable option --nocapture to --no-capture rustdoc: Rename unstable option --nocapture to --no-capture in accordance with libtest Oct 25, 2025
@fmease fmease marked this pull request as ready for review October 25, 2025 21:15
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 25, 2025
@rustbot rustbot removed the S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. label Oct 25, 2025
@rustbot
Copy link
Collaborator

rustbot commented Oct 25, 2025

r? @notriddle

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

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-review Status: Awaiting review from the assignee but also interested parties. T-rustdoc Relevant to the rustdoc 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