On Fri, Nov 20, 2020 at 04:29:18PM +0100, Andrea Bolognani wrote:
On Mon, 2020-11-09 at 12:20 +0100, Erik Skultety wrote:
First add the meson required bits to be able to run the build. NOTE: inspired by our gitlab-ci.yml
This note seems unnecessary.
+++ b/ci/Makefile @@ -227,6 +230,8 @@ ci-run-command@%: ci-prepare-tree CI_CONFIGURE="$(CI_CONFIGURE)" \ CI_CONFIGURE_ARGS="$(CI_CONFIGURE_ARGS)" \ CI_MAKE_ARGS="$(CI_MAKE_ARGS)" \ + MESON_OPTS="$$MESON_OPTS" \
Please keep this right after CONFIGURE_OPTS and before all the CI_* variables.
+++ b/ci/build.sh -mkdir -p "$CI_CONT_BUILDDIR" || exit 1 -cd "$CI_CONT_BUILDDIR" +mkdir -p "$CI_CONT_SRCDIR" || exit 1 +cd "$CI_CONT_SRCDIR"
$CI_CONT_SRCDIR is the source directory, which is guaranteed to exist because we mount it inside the container as a volume. So you can drop the first line altogether.
+meson build --werror $MESON_OPTS || (cat build/meson-logs/meson-log.txt && exit 1) +ninja -C build $CI_NINJA_ARGS
We enable -Werror automatically when building from a git clone, which is always going to be the case when using this scaffoling, so I think you can leave that option out. I see it's used in the GitLab CI configuration, so you can maybe keep it in right now and then consider removing it from both places at the same time.
find -name test-suite.log -delete
This should be updated to look for testlog.txt instead, but actually you might be able to leave it out completely since meson seems to do a good job at displaying the relevant part of the log if a test fails? We don't have anything like this in the GitLab CI configuration, so either it's not needed here either or we should add it there as well.
Oh, you're right, the testlog dump is quite useless except for e.g. these two lines: "Some tests failed. Run them using: VIR_TEST_DEBUG=1 VIR_TEST_RANGE=849 /home/eskultety/libvirt/build/tests/qemuxml2argvtest" So I'll drop the suggested hunks. Erik