[PATCH] D35852: Quote '?' in llvm-rc test

Jacob Gravelle via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 25 13:16:28 PDT 2017


jgravelle-google created this revision.

Bash interperets the '?' character as matching an arbitrary character.
On systems that have a file or directory with exactly one character in
their root directory, '/?' gets reinterpreted into that pathname, which
fails to match the expected Help text for llvm-rc.
This patch quotes the '/?' to avoid that edge case.


https://reviews.llvm.org/D35852

Files:
  test/tools/llvm-rc/helpmsg.test


Index: test/tools/llvm-rc/helpmsg.test
===================================================================
--- test/tools/llvm-rc/helpmsg.test
+++ test/tools/llvm-rc/helpmsg.test
@@ -1,5 +1,5 @@
-; RUN: llvm-rc /h > %t1
-; RUN: llvm-rc /? > %t2
+; RUN: llvm-rc '/h' > %t1
+; RUN: llvm-rc '/?' > %t2
 ; RUN: diff %t1 %t2
 ; RUN: FileCheck -input-file=%t1 %s
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D35852.108144.patch
Type: text/x-patch
Size: 359 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170725/e6080de6/attachment.bin>


More information about the llvm-commits mailing list