summaryrefslogtreecommitdiff
path: root/scripts/mtest2make.py
diff options
context:
space:
mode:
authorPaolo Bonzini <pbonzini@redhat.com>2020-08-13 09:28:11 -0400
committerPaolo Bonzini <pbonzini@redhat.com>2020-10-17 10:45:51 -0400
commit09e93326e448ab43fa26a9e2d9cc20ecf951f32b (patch)
tree6b923a531aae0b5c4a06879170561adf07948e84 /scripts/mtest2make.py
parent2b8575bd5fbc8a8880e9ecfb1c7e7990feb1fea6 (diff)
downloadqemu-09e93326e448ab43fa26a9e2d9cc20ecf951f32b.zip
build: replace ninjatool with ninja
Now that the build is done entirely by Meson, there is no need to keep the Makefile conversion. Instead, we can ask Ninja about the targets it exposes and forward them. The main advantages are, from smallest to largest: - reducing the possible namespace pollution within the Makefile - removal of a relatively large Python program - faster build because parsing Makefile.ninja is slower than parsing build.ninja; and faster build after Meson runs because we do not have to generate Makefile.ninja. - tracking of command lines, which provides more accurate rebuilds In addition the change removes the requirement for GNU make 3.82, which was annoying on Mac, and avoids bugs on Windows due to ninjatool not knowing how to convert Windows escapes to POSIX escapes. Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'scripts/mtest2make.py')
-rw-r--r--scripts/mtest2make.py3
1 files changed, 2 insertions, 1 deletions
diff --git a/scripts/mtest2make.py b/scripts/mtest2make.py
index c3489a4605..25ee6887cf 100644
--- a/scripts/mtest2make.py
+++ b/scripts/mtest2make.py
@@ -70,8 +70,9 @@ def process_tests(test, targets, suites):
print('.test.driver.%d := %s' % (i, driver))
print('.test.env.%d := $(.test.env) %s' % (i, env))
print('.test.cmd.%d := %s' % (i, cmd))
+ print('.test.deps.%d := %s' % (i, ' '.join(deps)))
print('.PHONY: run-test-%d' % (i,))
- print('run-test-%d: %s' % (i, ' '.join(deps)))
+ print('run-test-%d: $(.test.deps.%d)' % (i,i))
print('\t@$(call .test.run,%d,$(.test.output-format))' % (i,))
test_suites = test['suite'] or ['default']