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

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

 



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.

>From my point of view, while offering the option above for users can
be nice, once a project switches to Meson and its syntax-check rules
are moved to the test suite, there might as well be a single CI job
which runs 'ninja test' and that's it.

-- 
Andrea Bolognani / Red Hat / Virtualization

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[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