Re: [libvirt] [dbus PATCH] build: convert to Meson/Ninja build system

On Thu, 2019-09-19 at 09:36 +0100, Daniel P. Berrangé wrote: > On Thu, Sep 19, 2019 at 10:20:04AM +0200, Pavel Hrdina wrote: > > Another argument from my side was that we require syntax-check and check > > to be executed before posting patches to mailing list, having it under > > single target would simplify things for new contributors and would align > > more with meson community and projects. This is pretty nice. I've been guilty of posting patches that passed check but not syntax-check in the past O:-) and I know for a fact I'm not the only one who's done that ;) > > Dan pointed out an issue with our CI that we would loose the separation > > in our CI results, which can be solved by using 'suite' labels for tests > > that we run, so for syntax-check we can use 'syntax' label and for unit > > tests we can use 'unit' label, there can be multiple labels assigned to > > each test and to run only a set of tests with a specific label we just > > need to run these commands: > > > > meson test --suite syntax > > meson test --suite unit > > > > This way we would still have the separation in our CI and contributors > > could easily run `meson test` or `ninja test` to run everything. > > Yes, that's a usable approach. Jumping in a bit late, but why do we care about this distinction in CI at all? In our existing setup, for all projects 'make check' is executed if and only if 'make syntax-check' has succeeded, so the separation is fairly arbitrary and doesn't really buy us anything as far as I can tell.