diff options
author | Nirbheek Chauhan <nirbheek@centricular.com> | 2017-02-23 04:47:55 +0530 |
---|---|---|
committer | Nirbheek Chauhan <nirbheek@centricular.com> | 2017-02-23 11:56:46 +0530 |
commit | 9fffcef290a84e398dc2b9cdbac0bdfcdd60961a (patch) | |
tree | ed069051d2b24dc3a63a6e243455879a0cbefff8 /mesontest.py | |
parent | 62c7dcf32d1043fa939a1fade0610fec7b6b1b3d (diff) | |
download | meson-9fffcef290a84e398dc2b9cdbac0bdfcdd60961a.zip meson-9fffcef290a84e398dc2b9cdbac0bdfcdd60961a.tar.gz meson-9fffcef290a84e398dc2b9cdbac0bdfcdd60961a.tar.bz2 |
mesontest: Fix --repeat with --gdb
It would add --args to `wrap` repeatedly for each re-run, resulting in
gdb erroring out with `--args: No such file or directory.`
Also don't make --gdb and --wrapper mutually exclusive. Sometimes people
want to run under a wrapper *and* run it under gdb.
Diffstat (limited to 'mesontest.py')
-rwxr-xr-x | mesontest.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/mesontest.py b/mesontest.py index ef5cff6..0cccfe2 100755 --- a/mesontest.py +++ b/mesontest.py @@ -421,7 +421,9 @@ TIMEOUT: %4d wrap = ['gdb', '--quiet', '--nh'] if self.options.repeat > 1: wrap += ['-ex', 'run', '-ex', 'quit'] - elif self.options.wrapper: + # Signal the end of arguments to gdb + wrap += ['--args'] + if self.options.wrapper: wrap += self.options.wrapper assert(isinstance(wrap, list)) return wrap @@ -452,8 +454,6 @@ TIMEOUT: %4d if self.options.gdb: test.timeout = None - if len(test.cmd_args): - wrap.append('--args') if not test.is_parallel or self.options.gdb: self.drain_futures(futures, logfile, jsonlogfile) |