Auto merge of #14006 - epage:add-quotes, r=weihanglo

fix(add): Avoid escaping double-quotes by using string literals

### What does this PR try to resolve?

Fixes #14002

### How should we test and review this PR?

### Additional information

Author: bors

Cargo.lock

@@ -96,8 +96,8 @@ tar = { version = "0.4.40", default-features = false }
 tempfile = "3.10.1"
 thiserror = "1.0.59"
 time = { version = "0.3.36", features = ["parsing", "formatting", "serde"] }
-toml = "0.8.12"
-toml_edit = { version = "0.22.12", features = ["serde"] }
+toml = "0.8.14"
+toml_edit = { version = "0.22.14", features = ["serde"] }
 tracing = "0.1.40" # be compatible with rustc_log: https://github.com/rust-lang/rust/blob/e51e98dde6a/compiler/rustc_log/Cargo.toml#L9
 tracing-chrome = "0.7.2"
 tracing-subscriber = { version = "0.3.18", features = ["env-filter"] }

Cargo.toml

@@ -2922,16 +2922,16 @@ fn bad_target_cfg() {
         .with_stderr(
             "\
 [ERROR] error in [..]/foo/.cargo/config.toml: \
-could not load config key `target.\"cfg(not(target_os = \\\"none\\\"))\".runner`
+could not load config key `target.'cfg(not(target_os = \"none\"))'.runner`
 
 Caused by:
   error in [..]/foo/.cargo/config.toml: \
-  could not load config key `target.\"cfg(not(target_os = \\\"none\\\"))\".runner`
+  could not load config key `target.'cfg(not(target_os = \"none\"))'.runner`
 
 Caused by:
-  invalid configuration for key `target.\"cfg(not(target_os = \\\"none\\\"))\".runner`
+  invalid configuration for key `target.'cfg(not(target_os = \"none\"))'.runner`
   expected a string or array of strings, but found a boolean for \
-  `target.\"cfg(not(target_os = \\\"none\\\"))\".runner` in [..]/foo/.cargo/config.toml
+  `target.'cfg(not(target_os = \"none\"))'.runner` in [..]/foo/.cargo/config.toml
 ",
         )
         .run();

tests/testsuite/bad_config.rs

@@ -28,7 +28,7 @@ fn case() {
 
     snapbox::cmd::Command::cargo_ui()
         .arg("add")
-        .arg_line("my-package1 my-package2 --target cfg(unix)")
+        .arg_line("my-package1 my-package2 --target 'cfg(target_os=\"linux\")'")
         .current_dir(cwd)
         .assert()
         .success()

tests/testsuite/cargo_add/target_cfg/mod.rs

@@ -5,6 +5,6 @@ name = "cargo-list-test-fixture"
 version = "0.0.0"
 edition = "2015"
 
-[target."cfg(unix)".dependencies]
+[target.'cfg(target_os="linux")'.dependencies]
 my-package1 = "99999.0.0"
 my-package2 = "99999.0.0"

tests/testsuite/cargo_add/target_cfg/out/Cargo.toml

@@ -88,7 +88,7 @@ build.rustflags = [\"--flag-global\", \"--flag-directory\"]
 extra-table.somekey = \"somevalue\"
 profile.dev.opt-level = 3
 profile.dev.package.foo.opt-level = 1
-target.\"cfg(target_os = \\\"linux\\\")\".runner = \"runme\"
+target.'cfg(target_os = \"linux\")'.runner = \"runme\"
 # The following environment variables may affect the loaded values.
 # CARGO_ALIAS_BAR=[..]cat dog[..]
 # CARGO_BUILD_JOBS=100
@@ -265,7 +265,7 @@ build.rustflags = [
 extra-table.somekey = \"somevalue\" # [ROOT]/home/.cargo/config.toml
 profile.dev.opt-level = 3 # [ROOT]/home/.cargo/config.toml
 profile.dev.package.foo.opt-level = 1 # [ROOT]/home/.cargo/config.toml
-target.\"cfg(target_os = \\\"linux\\\")\".runner = \"runme\" # [ROOT]/home/.cargo/config.toml
+target.'cfg(target_os = \"linux\")'.runner = \"runme\" # [ROOT]/home/.cargo/config.toml
 # The following environment variables may affect the loaded values.
 # CARGO_HOME=[ROOT]/home/.cargo
 ",
@@ -361,7 +361,7 @@ build.rustflags = [\"--flag-global\"]
 extra-table.somekey = \"somevalue\"
 profile.dev.opt-level = 3
 profile.dev.package.foo.opt-level = 1
-target.\"cfg(target_os = \\\"linux\\\")\".runner = \"runme\"
+target.'cfg(target_os = \"linux\")'.runner = \"runme\"
 
 ",
         )
@@ -513,7 +513,7 @@ build.rustflags = [\"--flag-global\"]
 extra-table.somekey = \"somevalue\"
 profile.dev.opt-level = 3
 profile.dev.package.foo.opt-level = 1
-target.\"cfg(target_os = \\\"linux\\\")\".runner = \"runme\"
+target.'cfg(target_os = \"linux\")'.runner = \"runme\"
 
 ",
         )