aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPaolo Bonzini <pbonzini@redhat.com>2021-01-14 12:05:23 +0100
committerPaolo Bonzini <pbonzini@redhat.com>2021-01-22 12:18:59 +0100
commita0da710172c4743abccaef774c99c0cef4b1d9c3 (patch)
treebb6394f896770858d2b6c3f34515669834ab17e0
parent04bab9d595aea063bfe0cd832bbafb230184d7c3 (diff)
downloadmeson-a0da710172c4743abccaef774c99c0cef4b1d9c3.zip
meson-a0da710172c4743abccaef774c99c0cef4b1d9c3.tar.gz
meson-a0da710172c4743abccaef774c99c0cef4b1d9c3.tar.bz2
mtest: quote environment variable values and command line arguments
-rw-r--r--mesonbuild/mtest.py11
1 files changed, 9 insertions, 2 deletions
diff --git a/mesonbuild/mtest.py b/mesonbuild/mtest.py
index 8bfacf3..e996931 100644
--- a/mesonbuild/mtest.py
+++ b/mesonbuild/mtest.py
@@ -30,6 +30,7 @@ import random
import re
import signal
import subprocess
+import shlex
import sys
import textwrap
import time
@@ -188,8 +189,13 @@ def returncode_to_status(retcode: int) -> str:
signame = 'SIGinvalid'
return '(exit status {} or signal {} {})'.format(retcode, signum, signame)
+# TODO for Windows
+sh_quote: T.Callable[[str], str] = lambda x: x
+if not is_windows():
+ sh_quote = shlex.quote
+
def env_tuple_to_str(env: T.Iterable[T.Tuple[str, str]]) -> str:
- return ''.join(["{}='{}' ".format(k, v) for k, v in env])
+ return ''.join(["{}={} ".format(k, sh_quote(v)) for k, v in env])
class TestException(MesonException):
@@ -871,7 +877,8 @@ class TestRun:
if not self.cmd:
return None
test_only_env = set(self.env.items()) - set(os.environ.items())
- return env_tuple_to_str(test_only_env) + ' '.join(self.cmd)
+ return env_tuple_to_str(test_only_env) + \
+ ' '.join((sh_quote(x) for x in self.cmd))
def complete_skip(self, message: str) -> None:
self.starttime = time.time()