[clang-tools-extra] eac5672 - Fix target_info.test on Windows with a hack

Reid Kleckner via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 13 13:14:13 PDT 2020


Author: Reid Kleckner
Date: 2020-04-13T13:14:06-07:00
New Revision: eac56724fd955af0f8521557cacc57a83f371649

URL: https://github.com/llvm/llvm-project/commit/eac56724fd955af0f8521557cacc57a83f371649
DIFF: https://github.com/llvm/llvm-project/commit/eac56724fd955af0f8521557cacc57a83f371649.diff

LOG: Fix target_info.test on Windows with a hack

Add a quote character to the sed command to cause it to be quoted.
Hopefully, which will help both gnuwin and MSys versions of sed tokenize
the command line the same way.

Added: 
    

Modified: 
    clang-tools-extra/clangd/test/target_info.test

Removed: 
    


################################################################################
diff  --git a/clang-tools-extra/clangd/test/target_info.test b/clang-tools-extra/clangd/test/target_info.test
index 6017dbd321fa..6db8ab891aa8 100644
--- a/clang-tools-extra/clangd/test/target_info.test
+++ b/clang-tools-extra/clangd/test/target_info.test
@@ -9,7 +9,7 @@
 # RUN: sed -e "s|INPUT_DIR|%/t.dir|g" %s > %t.test.1
 # On Windows, we need the URI in didOpen to look like "uri":"file:///C:/..."
 # (with the extra slash in the front), so we add it here.
-# RUN: sed -E -e "s|file://([A-Z]):/|file:///\1:/|g" %t.test.1 > %t.test
+# RUN: sed -E -e 's|"file://([A-Z]):/|"file:///\1:/|g' %t.test.1 > %t.test
 
 # RUN: clangd -lit-test < %t.test 2>&1 | FileCheck -strict-whitespace %t.test
 {"jsonrpc":"2.0","id":0,"method":"initialize","params":{}}


        


More information about the cfe-commits mailing list