On Wed, Aug 01, 2012 at 06:45:29AM -0600, Eric Blake wrote:
> On 08/01/2012 02:17 AM, Daniel Veillard wrote:
> >
> > The 'make check' was rebuilding the binaries just overrided,
> > so for more safety also override the C program
> > Also daemon-conf isn't built anymore so remove it from the list
> >
> > Pushed under the build breaker rule
> >
> > Daniel
> >
> > diff --git a/libvirt.spec.in b/libvirt.spec.in
> > index c642f80..2d86e3c 100644
> > --- a/libvirt.spec.in
> > +++ b/libvirt.spec.in
> > @@ -1323,11 +1323,13 @@ rm -fr %{buildroot}
> >
> > %check
> > cd tests
> > -# These 3 tests don't current work in a mock build root
> > -for i in nodeinfotest daemon-conf seclabeltest
> > +make
> > +# These tests don't current work in a mock build root
> > +for i in nodeinfotest seclabeltest
>
> I recently fixed nodeinfotest so it wouldn't read host files. Is it
> still broken under a mock build, or did commit 2b366b4 fix the need for
> this hack?
>
> Also, what is the failure with seclabeltest? It has also had recent
> fixes (commit a22a36e), although I'm not sure that they would be of the
> nature to fix a host-dependency.
Honnestly I didn't tried, I removed daemon-conf as a cleanup since
it didn't exist anymore. The two others were actually removed from that
list in the RHEL builds, so I'm inclined to revisit that part.
What I actually used that patch for was to work around the moduletest
error you reported and isn't fixed yet:
https://www.redhat.com/archives/libvir-list/2012-July/msg01762.html
Daniel
--
Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/
daniel@xxxxxxxxxxxx | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library http://libvirt.org/
--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list