Disable `)` escaping in buildspec, command line variable overrides

Similar to line continuations, that would make directory paths on Windows
unusable, for example:

b info(C:\myproj\)

Note that while this is less of a problem in command line variable overrides,
we disable it there for consistency.

1 files changed, 5 insertions, 6 deletions

diff --git a/libbuild2/parser.cxx b/libbuild2/parser.cxx
index 2c8c8a6..0d51b49 100644
--- a/libbuild2/parser.cxx
+++ b/libbuild2/parser.cxx
@@ -8163,14 +8163,13 @@ namespace build2
   buildspec parser::
   parse_buildspec (istream& is, const path_name& in)
   {
-    // We do "effective escaping" of the special `'"\$(` characters plus `)`
-    // for symmetry (basically what's escapable inside a double-quoted literal
-    // plus the single quote; note, however, that we exclude line
-    // continuations since they would make directory paths on Windows
-    // unusable).
+    // We do "effective escaping" of the special `'"\$(` characters (basically
+    // what's escapable inside a double-quoted literal plus the single quote;
+    // note, however, that we exclude line continuations and `)` since they
+    // would make directory paths on Windows unusable).
     //
     path_ = ∈
-    lexer l (is, *path_, 1 /* line */, "\'\"\\$()");
+    lexer l (is, *path_, 1 /* line */, "\'\"\\$(");
     lexer_ = &l;
 
     root_ = &ctx->global_scope.rw ();