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

Re: [Xen-devel] [xen-unstable test] 12007: regressions - FAIL



On Wed, 2012-02-22 at 10:04 +0000, Roger Pau Monnà wrote:
> 2012/2/22 Ian Campbell <Ian.Campbell@xxxxxxxxxx>:
> > On Wed, 2012-02-22 at 09:20 +0000, Roger Pau Monnà wrote:
> >> 2012/2/22 xen.org <ian.jackson@xxxxxxxxxxxxx>:
> >> > flight 12007 xen-unstable real [real]
> >> > http://www.chiark.greenend.org.uk/~xensrcts/logs/12007/
> >> >
> >> > Regressions :-(
> >> >
> >> > Tests which did not succeed and are blocking,
> >> > including tests which could not be run:
> >> >  build-i386-oldkern            4 xen-build                 fail REGR. 
> >> > vs. 12003
> >> >  build-i386                    4 xen-build                 fail REGR. 
> >> > vs. 12003
> >> >  build-amd64                   4 xen-build                 fail REGR. 
> >> > vs. 12003
> >> >  build-amd64-oldkern           4 xen-build                 fail REGR. 
> >> > vs. 12003
> >>
> >>
> >> brctl is not installed, or the user that is trying to build Xen
> >> doesn't have permissions to run it. I've found this test quite
> >> useless, because you should be able to build Xen as a regular user,
> >> that doesn't have access to brctl, do you want me to send a patch that
> >> removes this check?
> >
> > I build Xen on a machine which doesn't actually run Xen and therefore
> > doesn't have brctl installed, I don't think this is unreasonable since
> > brctl is a runtime not compile-time dependency.
> >
> > I think we should remove the test from configure -- although it does
> > make me wonder (perhaps too late) if we shouldn't have kept the existing
> > install time checks.
> 
> I've submitted a patch that should fix that. I'm afraid that removing
> the brctl/brconfig check will bring trouble, because this is used in
> hotplug scripts and errors on hotplug scripts are hard to spot right
> now for regular users.

This is why I am concerned that we have removed the install time as well
as the development time checks...

Perhaps we should move these new checks to a second configure script
which can be run at install time and produces as its output just a pass
or fail?

Ian.



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.