Rollup merge of #113127 - jieyouxu:fix-error-color-summary, r=davidtwco

Set error handler output format as soon as possible

Should fix #112692 (comment).

Author: matthiaskrgr

compiler/rustc_session/src/config.rs

@@ -2405,6 +2405,8 @@ pub fn build_session_options(
 
     let error_format = parse_error_format(handler, matches, color, json_rendered);
 
+    handler.abort_if_error_and_set_error_format(error_format);
+
     let diagnostic_width = matches.opt_get("diagnostic-width").unwrap_or_else(|_| {
         handler.early_error("`--diagnostic-width` must be an positive integer");
     });

tests/ui/conditional-compilation/cfg-arg-invalid-1.rs

@@ -1,3 +1,3 @@
-// compile-flags: --cfg a(b=c)
+// compile-flags: --error-format=human --cfg a(b=c)
 // error-pattern: invalid `--cfg` argument: `a(b=c)` (expected `key` or `key="value"`, ensure escaping is appropriate for your shell, try 'key="value"' or key=\"value\")
 fn main() {}

tests/ui/conditional-compilation/cfg-arg-invalid-2.rs

@@ -1,3 +1,3 @@
-// compile-flags: --cfg a{b}
+// compile-flags: --error-format=human --cfg a{b}
 // error-pattern: invalid `--cfg` argument: `a{b}` (expected `key` or `key="value"`)
 fn main() {}

tests/ui/conditional-compilation/cfg-arg-invalid-4.rs

@@ -1,3 +1,3 @@
-// compile-flags: --cfg a(b)
+// compile-flags: --error-format=human --cfg a(b)
 // error-pattern: invalid `--cfg` argument: `a(b)` (expected `key` or `key="value"`)
 fn main() {}

tests/ui/conditional-compilation/cfg-arg-invalid-6.rs

@@ -1,3 +1,3 @@
-// compile-flags: --cfg a{
+// compile-flags: --error-format=human --cfg a{
 // error-pattern: invalid `--cfg` argument: `a{` (expected `key` or `key="value"`)
 fn main() {}

tests/ui/conditional-compilation/cfg-arg-invalid-8.rs

@@ -1,3 +1,3 @@
-// compile-flags: --cfg )
+// compile-flags: --error-format=human --cfg )
 // error-pattern: invalid `--cfg` argument: `)` (expected `key` or `key="value"`)
 fn main() {}

tests/ui/conditional-compilation/cfg-arg-invalid-9.rs

@@ -1,4 +1,4 @@
 // Test for missing quotes around value, issue #66450.
-// compile-flags: --cfg key=value
+// compile-flags: --error-format=human --cfg key=value
 // error-pattern: invalid `--cfg` argument: `key=value` (expected `key` or `key="value"`, ensure escaping is appropriate for your shell, try 'key="value"' or key=\"value\")
 fn main() {}

tests/ui/conditional-compilation/cfg-empty-codemap.rs

@@ -1,6 +1,6 @@
 // Tests that empty source_maps don't ICE (#23301)
 
-// compile-flags: --cfg ""
+// compile-flags: --error-format=human --cfg ""
 
 // error-pattern: invalid `--cfg` argument: `""` (expected `key` or `key="value"`)
 

tests/ui/diagnostic-flags/colored-session-opt-error.rs

@@ -0,0 +1,3 @@
+// check-pass
+// compile-flags: -Cremark=foo --error-format=human --color always
+fn main() {}

tests/ui/diagnostic-flags/colored-session-opt-error.stderr

@@ -0,0 +1,2 @@
+warning: -C remark requires "-C debuginfo=n" to show source locations
+