fix:Using --release/debug and --profile together becomes an error

heisen-li · heisen-li · commit 5276f6e0b0b1 · 2024-05-28T16:57:17.000+08:00
diff --git a/src/bin/cargo/commands/bench.rs b/src/bin/cargo/commands/bench.rs
@@ -63,7 +63,7 @@ pub fn exec(gctx: &mut GlobalContext, args: &ArgMatches) -> CliResult {
         args.compile_options(gctx, CompileMode::Bench, Some(&ws), ProfileChecking::Custom)?;
 
     compile_opts.build_config.requested_profile =
-        args.get_profile_name(gctx, "bench", ProfileChecking::Custom)?;
+        args.get_profile_name("bench", ProfileChecking::Custom)?;
 
     let ops = TestOptions {
         no_run: args.flag("no-run"),
diff --git a/src/bin/cargo/commands/clean.rs b/src/bin/cargo/commands/clean.rs
@@ -146,7 +146,7 @@ pub fn exec(gctx: &mut GlobalContext, args: &ArgMatches) -> CliResult {
         gctx,
         spec: values(args, "package"),
         targets: args.targets()?,
-        requested_profile: args.get_profile_name(gctx, "dev", ProfileChecking::Custom)?,
+        requested_profile: args.get_profile_name("dev", ProfileChecking::Custom)?,
         profile_specified: args.contains_id("profile") || args.flag("release"),
         doc: args.flag("doc"),
         dry_run: args.dry_run(),
diff --git a/src/bin/cargo/commands/install.rs b/src/bin/cargo/commands/install.rs
@@ -196,7 +196,7 @@ pub fn exec(gctx: &mut GlobalContext, args: &ArgMatches) -> CliResult {
     )?;
 
     compile_opts.build_config.requested_profile =
-        args.get_profile_name(gctx, "release", ProfileChecking::Custom)?;
+        args.get_profile_name("release", ProfileChecking::Custom)?;
 
     if args.flag("list") {
         ops::install_list(root, gctx)?;
diff --git a/src/bin/cargo/commands/test.rs b/src/bin/cargo/commands/test.rs
@@ -74,7 +74,7 @@ pub fn exec(gctx: &mut GlobalContext, args: &ArgMatches) -> CliResult {
         args.compile_options(gctx, CompileMode::Test, Some(&ws), ProfileChecking::Custom)?;
 
     compile_opts.build_config.requested_profile =
-        args.get_profile_name(gctx, "test", ProfileChecking::Custom)?;
+        args.get_profile_name("test", ProfileChecking::Custom)?;
 
     // `TESTNAME` is actually an argument of the test binary, but it's
     // important, so we explicitly mention it and reconfigure.
diff --git a/src/cargo/util/command_prelude.rs b/src/cargo/util/command_prelude.rs
@@ -558,7 +558,6 @@ Run `{cmd}` to see possible targets."
 
     fn get_profile_name(
         &self,
-        gctx: &GlobalContext,
         default: &str,
         profile_checking: ProfileChecking,
     ) -> CargoResult<InternedString> {
@@ -572,47 +571,36 @@ Run `{cmd}` to see possible targets."
             // `cargo fix` and `cargo check` has legacy handling of this profile name
             | (Some(name @ "test"), ProfileChecking::LegacyTestOnly) => {
                 if self.maybe_flag("release") {
-                    gctx.shell().warn(
-                        "the `--release` flag should not be specified with the `--profile` flag\n\
-                         The `--release` flag will be ignored.\n\
-                         This was historically accepted, but will become an error \
-                         in a future release."
-                    )?;
+                    bail!("\
+the `--release` flag can not be specified with the `--profile` flag
+Please remove one of the flags.");
                 }
                 return Ok(InternedString::new(name));
             }
             _ => {}
         }
 
-        let conflict = |flag: &str, equiv: &str, specified: &str| -> anyhow::Error {
-            anyhow::format_err!(
-                "conflicting usage of --profile={} and --{flag}\n\
-                 The `--{flag}` flag is the same as `--profile={equiv}`.\n\
-                 Remove one flag or the other to continue.",
-                specified,
-                flag = flag,
-                equiv = equiv
-            )
-        };
+        let err_message = |flag: &str| format!(
+            "\
+the `--{flag}` flag can not be specified with the `--profile` flag
+Please remove one of the flags.");
 
         let name = match (
             self.maybe_flag("release"),
             self.maybe_flag("debug"),
             specified_profile,
         ) {
             (false, false, None) => default,
-            (true, _, None | Some("release")) => "release",
-            (true, _, Some(name)) => return Err(conflict("release", "release", name)),
-            (_, true, None | Some("dev")) => "dev",
-            (_, true, Some(name)) => return Err(conflict("debug", "dev", name)),
+            (true, _, None) => "release",
+            (true, _, Some(_)) => bail!(err_message("release")),
+            (_, true, None) => "dev",
+            (_, true, Some(_)) => bail!(err_message("debug")),
             // `doc` is separate from all the other reservations because
             // [profile.doc] was historically allowed, but is deprecated and
             // has no effect. To avoid potentially breaking projects, it is a
             // warning in Cargo.toml, but since `--profile` is new, we can
             // reject it completely here.
-            (_, _, Some("doc")) => {
-                bail!("profile `doc` is reserved and not allowed to be explicitly specified")
-            }
+            (_, _, Some("doc")) => bail!("profile `doc` is reserved and not allowed to be explicitly specified"),
             (_, _, Some(name)) => {
                 ProfileName::new(name)?;
                 name
@@ -710,7 +698,7 @@ Run `{cmd}` to see possible targets."
             mode,
         )?;
         build_config.message_format = message_format.unwrap_or(MessageFormat::Human);
-        build_config.requested_profile = self.get_profile_name(gctx, "dev", profile_checking)?;
+        build_config.requested_profile = self.get_profile_name("dev", profile_checking)?;
         build_config.build_plan = self.flag("build-plan");
         build_config.unit_graph = self.flag("unit-graph");
         build_config.future_incompat_report = self.flag("future-incompat-report");
diff --git a/tests/testsuite/check.rs b/tests/testsuite/check.rs
@@ -289,8 +289,21 @@ fn rustc_check() {
 
     // Verify compatible usage of --profile with --release, issue #7488
     foo.cargo("rustc --profile check --release -- --emit=metadata")
+        .with_status(101)
+        .with_stderr(
+            "\
+error: the `--release` flag can not be specified with the `--profile` flag
+Please remove one of the flags.",
+        )
         .run();
+
     foo.cargo("rustc --profile test --release -- --emit=metadata")
+        .with_status(101)
+        .with_stderr(
+            "\
+error: the `--release` flag can not be specified with the `--profile` flag
+Please remove one of the flags.",
+        )
         .run();
 }
 
diff --git a/tests/testsuite/profile_custom.rs b/tests/testsuite/profile_custom.rs
@@ -383,32 +383,26 @@ fn conflicting_usage() {
         .with_status(101)
         .with_stderr(
             "\
-error: conflicting usage of --profile=dev and --release
-The `--release` flag is the same as `--profile=release`.
-Remove one flag or the other to continue.
-",
+error: the `--release` flag can not be specified with the `--profile` flag
+Please remove one of the flags.",
         )
         .run();
 
     p.cargo("install --profile=release --debug")
         .with_status(101)
         .with_stderr(
             "\
-error: conflicting usage of --profile=release and --debug
-The `--debug` flag is the same as `--profile=dev`.
-Remove one flag or the other to continue.
-",
+error: the `--debug` flag can not be specified with the `--profile` flag
+Please remove one of the flags.",
         )
         .run();
 
     p.cargo("rustc --profile=dev --release")
+    .with_status(101)
         .with_stderr(
             "\
-warning: the `--release` flag should not be specified with the `--profile` flag
-The `--release` flag will be ignored.
-This was historically accepted, but will become an error in a future release.
-[COMPILING] foo [..]
-[FINISHED] `dev` profile [..]
+error: the `--release` flag can not be specified with the `--profile` flag
+Please remove one of the flags.
 ",
         )
         .run();
@@ -417,63 +411,54 @@ This was historically accepted, but will become an error in a future release.
         .with_status(101)
         .with_stderr(
             "\
-error: conflicting usage of --profile=dev and --release
-The `--release` flag is the same as `--profile=release`.
-Remove one flag or the other to continue.
-",
+error: the `--release` flag can not be specified with the `--profile` flag
+Please remove one of the flags.",
         )
         .run();
 
     p.cargo("check --profile=test --release")
+    .with_status(101)
         .with_stderr(
             "\
-warning: the `--release` flag should not be specified with the `--profile` flag
-The `--release` flag will be ignored.
-This was historically accepted, but will become an error in a future release.
-[CHECKING] foo [..]
-[FINISHED] `test` profile [..]
-",
+error: the `--release` flag can not be specified with the `--profile` flag
+Please remove one of the flags.",
         )
         .run();
 
     // This is OK since the two are the same.
     p.cargo("rustc --profile=release --release")
+    .with_status(101)
         .with_stderr(
             "\
-[COMPILING] foo [..]
-[FINISHED] `release` profile [..]
-",
+error: the `--release` flag can not be specified with the `--profile` flag
+Please remove one of the flags.",
         )
         .run();
 
     p.cargo("build --profile=release --release")
+    .with_status(101)
         .with_stderr(
             "\
-[FINISHED] `release` profile [..]
-",
+error: the `--release` flag can not be specified with the `--profile` flag
+Please remove one of the flags.",
         )
         .run();
 
     p.cargo("install --path . --profile=dev --debug")
+    .with_status(101)
         .with_stderr(
             "\
-[INSTALLING] foo [..]
-[FINISHED] `dev` profile [..]
-[INSTALLING] [..]
-[INSTALLED] [..]
-[WARNING] be sure to add [..]
-",
+error: the `--debug` flag can not be specified with the `--profile` flag
+Please remove one of the flags.",
         )
         .run();
 
     p.cargo("install --path . --profile=release --debug")
         .with_status(101)
         .with_stderr(
             "\
-error: conflicting usage of --profile=release and --debug
-The `--debug` flag is the same as `--profile=dev`.
-Remove one flag or the other to continue.
-",
+error: the `--debug` flag can not be specified with the `--profile` flag
+Please remove one of the flags.",
         )
         .run();
 }
