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

[Roger Pau Monnà <roger.pau@xxxxxxxxxxxxx>]

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.

> Ian.
>
>

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