Skip to content

Commit 4ce2b61

Browse files
epageepage
committed
Revert "fix(context): Configure Shell before emitting messages"
This reverts commit f525e1f. This removes color control from warnings for unstable features. For some reason this removed color support from `cargo -Zhelp` in the tests but I can't reproduce it locally. The most important thing was getting the config fix in. There are two follow ups - Can we have the config working *and* color? - Why did this fail for this field and not the others we already had tests for? I ran out my immediate time box for looking into these. Fixes #13991
1 parent c0a7957 commit 4ce2b61

File tree

3 files changed

+60
-63
lines changed

3 files changed

+60
-63
lines changed

‎src/cargo/util/context/mod.rs

+25-25
Original file line numberDiff line numberDiff line change
@@ -1020,6 +1020,31 @@ impl GlobalContext {
10201020
unstable_flags: &[String],
10211021
cli_config: &[String],
10221022
) -> CargoResult<()> {
1023+
for warning in self
1024+
.unstable_flags
1025+
.parse(unstable_flags, self.nightly_features_allowed)?
1026+
{
1027+
self.shell().warn(warning)?;
1028+
}
1029+
if !unstable_flags.is_empty() {
1030+
// store a copy of the cli flags separately for `load_unstable_flags_from_config`
1031+
// (we might also need it again for `reload_rooted_at`)
1032+
self.unstable_flags_cli = Some(unstable_flags.to_vec());
1033+
}
1034+
if !cli_config.is_empty() {
1035+
self.cli_config = Some(cli_config.iter().map(|s| s.to_string()).collect());
1036+
self.merge_cli_args()?;
1037+
}
1038+
if self.unstable_flags.config_include {
1039+
// If the config was already loaded (like when fetching the
1040+
// `[alias]` table), it was loaded with includes disabled because
1041+
// the `unstable_flags` hadn't been set up, yet. Any values
1042+
// fetched before this step will not process includes, but that
1043+
// should be fine (`[alias]` is one of the only things loaded
1044+
// before configure). This can be removed when stabilized.
1045+
self.reload_rooted_at(self.cwd.clone())?;
1046+
}
1047+
10231048
// Ignore errors in the configuration files. We don't want basic
10241049
// commands like `cargo version` to error out due to config file
10251050
// problems.
@@ -1066,31 +1091,6 @@ impl GlobalContext {
10661091
let cli_target_dir = target_dir.as_ref().map(|dir| Filesystem::new(dir.clone()));
10671092
self.target_dir = cli_target_dir;
10681093

1069-
for warning in self
1070-
.unstable_flags
1071-
.parse(unstable_flags, self.nightly_features_allowed)?
1072-
{
1073-
self.shell().warn(warning)?;
1074-
}
1075-
if !unstable_flags.is_empty() {
1076-
// store a copy of the cli flags separately for `load_unstable_flags_from_config`
1077-
// (we might also need it again for `reload_rooted_at`)
1078-
self.unstable_flags_cli = Some(unstable_flags.to_vec());
1079-
}
1080-
if !cli_config.is_empty() {
1081-
self.cli_config = Some(cli_config.iter().map(|s| s.to_string()).collect());
1082-
self.merge_cli_args()?;
1083-
}
1084-
if self.unstable_flags.config_include {
1085-
// If the config was already loaded (like when fetching the
1086-
// `[alias]` table), it was loaded with includes disabled because
1087-
// the `unstable_flags` hadn't been set up, yet. Any values
1088-
// fetched before this step will not process includes, but that
1089-
// should be fine (`[alias]` is one of the only things loaded
1090-
// before configure). This can be removed when stabilized.
1091-
self.reload_rooted_at(self.cwd.clone())?;
1092-
}
1093-
10941094
self.load_unstable_flags_from_config()?;
10951095

10961096
Ok(())

‎tests/testsuite/cargo/z_help/stdout.term.svg

+33-36
Loading

‎tests/testsuite/config_cli.rs

+2-2
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ fn basic() {
1616
.config_arg("net.git-fetch-with-cli=true")
1717
.build();
1818
assert_eq!(gctx.get::<String>("foo").unwrap(), "bar");
19-
assert_eq!(gctx.net_config().unwrap().git_fetch_with_cli, None);
19+
assert_eq!(gctx.net_config().unwrap().git_fetch_with_cli, Some(true));
2020
}
2121

2222
#[cargo_test]
@@ -52,7 +52,7 @@ fn cli_priority() {
5252
assert_eq!(gctx.get::<i32>("build.jobs").unwrap(), 1);
5353
assert_eq!(gctx.get::<String>("build.rustc").unwrap(), "cli");
5454
assert_eq!(gctx.get::<bool>("term.verbose").unwrap(), true);
55-
assert_eq!(gctx.net_config().unwrap().git_fetch_with_cli, Some(false));
55+
assert_eq!(gctx.net_config().unwrap().git_fetch_with_cli, Some(true));
5656

5757
// Setting both term.verbose and term.quiet is invalid and is tested
5858
// in the run test suite.

0 commit comments

Comments
 (0)