Enhanced configuration discovery for cargo install
and cargo uninstall
commands
#11775
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR orthogonalizes the behaviour concerning configuration resolution of
cargo uninstall
withcargo install
. Namely, both should ignore local configuration and only consider$CARGO_HOME/config.toml
as configuration.This PR is draft with the following still open:
since this PR already changes the configuration discovery of
cargo uninstall
, we could also make the configuration discovery ofcargo install
more intuitive by changing it from "start configuration discovery at$CARGO_HOME/config.toml
" to "only consider global configuration in$CARGO_HOME/config.toml
."See here:
cargo/src/bin/cargo/commands/install.rs
Lines 93 to 94 in efd3733
The same should then be done for the
cargo uninstall
commandrebase so the changes to the man-page of
cargo install
made in Added documentation for the configuration discovery ofcargo install
to the man pages #11763 can be enhancedupdate man-page documentation of the configuration discovery behaviour of
cargo install
andcargo uninstall
commandssince this PR is a breaking change, a different PR with warning messages for future breaking changes is needed to land first: Warnings for breaking changes to the configuration discovery of
cargo install
andcargo uninstall
proposed in #11775 #11786once Warnings for breaking changes to the configuration discovery of
cargo install
andcargo uninstall
proposed in #11775 #11786 lands, rebase this branch and revert the changes made in that PRFor more information see discussions in:
cargo install
does not respect.cargo/config.toml
#11660cargo install
to the man pages #11763