[Lldb-commits] [lldb] r328020 - [lldb-dotest] Wrap arguments in single quotes

[Jonas Devlieghere via lldb-commits]

Author: jdevlieghere
Date: Tue Mar 20 12:18:11 2018
New Revision: 328020

URL: http://llvm.org/viewvc/llvm-project?rev=328020&view=rev
Log:
[lldb-dotest] Wrap arguments in single quotes

If we don't wrap arguments to the wrapper in single quotes, combined
arguments, for example for -E, don't reach dotest.py as a unit but as
separate arguments, causing the latter to fail.

Modified:
    lldb/trunk/test/lldb-dotest.in

Modified: lldb/trunk/test/lldb-dotest.in
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/test/lldb-dotest.in?rev=328020&r1=328019&r2=328020&view=diff
==============================================================================
--- lldb/trunk/test/lldb-dotest.in (original)
+++ lldb/trunk/test/lldb-dotest.in Tue Mar 20 12:18:11 2018
@@ -6,9 +6,13 @@ dotest_path = '@LLDB_SOURCE_DIR@/test/do
 dotest_args = '@LLDB_DOTEST_ARGS_STR@'
 
 if __name__ == '__main__':
+    # Wrap arguments in single quotes. This is necessary because we want to
+    # forward the arguments and otherwise we might split up arguments that were
+    # originally wrapped in single quotes.
+    wrapper_args = list("'" + i + "'" for i in sys.argv[1:])
     # FIXME: It would be nice if we can mimic the approach taken by llvm-lit
     # and pass a python configuration straight to dotest, rather than going
     # through the operating system.
-    command = '{} -q {} {}'.format(dotest_path, dotest_args, ' '.join(
-        sys.argv[1:]))
+    command = '{} -q {} {}'.format(dotest_path, dotest_args,
+                                   ' '.join(wrapper_args))
     os.system(command)