[libvirt PATCH] ci: jobs.sh: Add back '--no-suite syntax-check --print-errorlogs'

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Commit f688a53a converted .gitlab-ci.yml to the usage of ci/jobs.sh
functions, but in doing that our test options
    '--no-suite syntax-check --print-errorlogs'
got lost in the process and since commit 8e660c52 didn't introduce them
in the first place, it caused a behavioral regression. This patch adds
them back.

Fixes: 8e660c5286d7e2d07dd61681074bf1555955552d

Signed-off-by: Erik Skultety <eskultet@xxxxxxxxxx>
---

Technically this is a build breaker fix, but sending for review anyway to see
if there's an agreement on this approach.

 ci/jobs.sh | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/ci/jobs.sh b/ci/jobs.sh
index abd695e231..eb4a4e29cf 100644
--- a/ci/jobs.sh
+++ b/ci/jobs.sh
@@ -39,7 +39,10 @@ run_dist() {
 }
 
 run_test() {
+    TEST_ARGS="${TEST_ARGS:=--no-suite syntax-check --print-errorlogs}"
+
     test -f $GIT_ROOT/build/build.ninja || run_meson_setup
+
     run_cmd meson test -C build $TEST_ARGS
 }
 
-- 
2.41.0




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux