itsscb/cargo
1
0
Fork 0
mirror of https://github.com/rust-lang/cargo.git synced 2025-09-28 11:20:36 +00:00
Code Issues Packages Projects Releases Wiki Activity

Auto merge of #12660 - epage:dry, r=weihanglo

Browse Source 
feat(cli): Add '-n' to dry-run

This came from #12638 and my many small frustrations from wanting to use `-n` and not being able to.

We do not have any existing `-n` flags for this to be confused with.

I would wager that `-n` is such an entrenched short flag in build tools that it would not make sense for us to use it with any other flag.

For a survey of where `-n` is used as a short, see https://www.gnu.org/prep/standards/html_node/Option-Table.html#Option-Table
This commit is contained in:
bors 2023-09-18 02:53:19 +00:00
commit 695416e8d8
5 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/cargo/util/command_prelude.rs
View File

@ -302,7 +302,7 @@ pub trait CommandExt: Sized {
} }
fn arg_dry_run(self, dry_run: &'static str) -> Self { fn arg_dry_run(self, dry_run: &'static str) -> Self {
self._arg(flag("dry-run", dry_run)) self._arg(flag("dry-run", dry_run).short('n'))
} }
fn arg_ignore_rust_version(self) -> Self { fn arg_ignore_rust_version(self) -> Self {

2
tests/testsuite/cargo_add/help/stdout.log
View File

@ -42,7 +42,7 @@ Options:
--ignore-rust-version --ignore-rust-version
Ignore `rust-version` specification in packages (unstable) Ignore `rust-version` specification in packages (unstable)
--dry-run -n, --dry-run
Don't actually write the manifest Don't actually write the manifest
-q, --quiet -q, --quiet

2
tests/testsuite/cargo_publish/help/stdout.log
View File

@ -3,7 +3,7 @@ Upload a package to the registry
Usage: cargo[EXE] publish [OPTIONS] Usage: cargo[EXE] publish [OPTIONS]
Options: Options:
--dry-run Perform all checks without uploading -n, --dry-run Perform all checks without uploading
--index <INDEX> Registry index URL to upload the package to --index <INDEX> Registry index URL to upload the package to
--registry <REGISTRY> Registry to upload the package to --registry <REGISTRY> Registry to upload the package to
--token <TOKEN> Token to use when uploading --token <TOKEN> Token to use when uploading

2
tests/testsuite/cargo_remove/help/stdout.log
View File

@ -6,7 +6,7 @@ Arguments:
<DEP_ID>... Dependencies to be removed <DEP_ID>... Dependencies to be removed
Options: Options:
--dry-run Don't actually write the manifest -n, --dry-run Don't actually write the manifest
-q, --quiet Do not print cargo log messages -q, --quiet Do not print cargo log messages
-v, --verbose... Use verbose output (-vv very verbose/build.rs output) -v, --verbose... Use verbose output (-vv very verbose/build.rs output)
--color <WHEN> Coloring: auto, always, never --color <WHEN> Coloring: auto, always, never

2
tests/testsuite/cargo_update/help/stdout.log
View File

@ -3,7 +3,7 @@ Update dependencies as recorded in the local lock file
Usage: cargo[EXE] update [OPTIONS] [SPEC]... Usage: cargo[EXE] update [OPTIONS] [SPEC]...
Options: Options:
--dry-run Don't actually write the lockfile -n, --dry-run Don't actually write the lockfile
--recursive Force updating all dependencies of [SPEC]... as well --recursive Force updating all dependencies of [SPEC]... as well
--precise <PRECISE> Update [SPEC] to exactly PRECISE --precise <PRECISE> Update [SPEC] to exactly PRECISE
-q, --quiet Do not print cargo log messages -q, --quiet Do not print cargo log messages