Re: [PATCH 5/6] opensuse: Rename autoinst files according to their id

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

 



On Thu, Jun 02, 2016 at 05:55:42PM +0200, Fabiano Fidêncio wrote:
> On Thu, Jun 2, 2016 at 2:01 PM, Christophe Fergeau <cfergeau@xxxxxxxxxx> wrote:
> > This causes warnings when running ./test/test-isodetect if they are not
> > named as expected:
> >
> > ** (process:28952): WARNING **: Entity http://opensuse.org/opensuse/autoyast/desktop
> > should be in file install-script/opensuse.org/opensuse-autoyast-desktop.xml not
> > install-script/opensuse.org/autoinst-desktop.xml
> 
> Hmmm. I don't remember seeing these warnings before pushing these patches. :-\
> Wouldn't be the case to, somehow, make the test fail in case of these warnings?

The test could set warnings to be fatal through g_log_set_fatal_mask().
However, iirc there were somewhat valid warnings when the database
layout changed. We can try to make warnings fatals for a while, and
disable it if it proves to be too much of a pain.

Christophe

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Libosinfo mailing list
Libosinfo@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libosinfo

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

  Powered by Linux