diff options
author | Nicholas Hutchinson <nshutchinson@gmail.com> | 2013-11-30 22:33:02 (GMT) |
---|---|---|
committer | Nicholas Hutchinson <nshutchinson@gmail.com> | 2014-01-06 19:30:28 (GMT) |
commit | 664edba804ddd5363299f2af5183109bdc9715b1 (patch) | |
tree | c50e267ade56530904676f103905d4a8a2ba3ba0 /src/graph_test.cc | |
parent | 2517b75781decd49bc82f05f8336c509f7b2c62e (diff) | |
download | Ninja-664edba804ddd5363299f2af5183109bdc9715b1.zip Ninja-664edba804ddd5363299f2af5183109bdc9715b1.tar.gz Ninja-664edba804ddd5363299f2af5183109bdc9715b1.tar.bz2 |
More robust escaping of $in, $out paths
In summary: don’t escape if the path doesn’t contain problematic characters, otherwise:
- Shell: enclose string in single quotes, escape embedded single quotes with the magic quote-backslash-quote sequence
- Win32: Escape double quotes by doubling the number of consecutive backslashes that precede them (if any) and adding one more. Finally, double the number of trailing backslashes, and enclose the whole thing in double quotes.
Diffstat (limited to 'src/graph_test.cc')
-rw-r--r-- | src/graph_test.cc | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/src/graph_test.cc b/src/graph_test.cc index 8521216..14dc678 100644 --- a/src/graph_test.cc +++ b/src/graph_test.cc @@ -139,13 +139,18 @@ TEST_F(GraphTest, RootNodes) { } } -TEST_F(GraphTest, VarInOutQuoteSpaces) { +TEST_F(GraphTest, VarInOutPathEscaping) { ASSERT_NO_FATAL_FAILURE(AssertParse(&state_, -"build a$ b: cat nospace with$ space nospace2\n")); +"build a$ b: cat no'space with$ space$$ no\"space2\n")); Edge* edge = GetNode("a b")->in_edge(); - EXPECT_EQ("cat nospace \"with space\" nospace2 > \"a b\"", +#if _WIN32 + EXPECT_EQ("cat no'space \"with space$\" \"no\\\"space2\" > \"a b\"", edge->EvaluateCommand()); +#else + EXPECT_EQ("cat 'no'\\''space' 'with space$' 'no\"space2' > 'a b'", + edge->EvaluateCommand()); +#endif } // Regression test for https://github.com/martine/ninja/issues/380 |