Re: [jenkins-ci PATCH] jenkins: Remove meson-syntax-check job

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

 



On Mon, Sep 23, 2019 at 1:16 PM Andrea Bolognani <abologna@xxxxxxxxxx> wrote:
>
> On Mon, 2019-09-23 at 11:47 +0100, Daniel P. Berrangé wrote:
> > On Mon, Sep 23, 2019 at 12:43:18PM +0200, Andrea Bolognani wrote:
> > > On Mon, 2019-09-23 at 12:10 +0200, Fabiano Fidêncio wrote:
> > > > On Thu, Sep 19, 2019 at 11:11 AM Pavel Hrdina <phrdina@xxxxxxxxxx> wrote:
> > > > > Instead of removing the syntax-check job we can keep it if we use
> > > > > 'suite' labels for our tests in a way that we would use 'syntax' label
> > > > > for syntax-check tests and 'unit' label for unit tests.
> > > > >
> > > > > That way the syntax-check job will call
> > > > >
> > > > >     `meson test --suite syntax`
> > > > >
> > > > > and check job will call
> > > > >
> > > > >     `meson test --suite unit`
> > > >
> > > > Personally, I don't see a valid point on keeping the job.
> > >
> > > Same here. Dan was arguing for keeping it, however. I wonder if he
> > > changed his mind in the meantime...
> >
> > I'd prefer separate, but I'm not going to block it if everyone else things
> > it is better to have a single job.
>
> Since me and Fabiano are for having a single job, Pavel is okay with
> either solution and you would prefer separate jobs but don't feel
> strongly enough about that to NACK the other solution, let's slap a
>
>   Reviewed-by: Andrea Bolognani <abologna@xxxxxxxxxx>
>
> on this bad boy and move on :)
>
> Fabiano, let me know when it's a good time to push the patch and
> apply the changes.

Feel free to merge it whenever you have time. It's not going to cause
any breakage on osinfo-db-tools. :-)

Best Regards,
-- 
Fabiano Fidêncio

--
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