From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?B?Um9nZXIgUGF1IE1vbm7DqQ==?= Subject: Re: [PATCH v8 1/2] OSSTEST: introduce a raisin build test Date: Tue, 4 Aug 2015 15:47:09 +0200 Message-ID: <55C0C25D.1070709@citrix.com> References: <1437670366-29034-1-git-send-email-stefano.stabellini@eu.citrix.com> <1438686271.31129.96.camel@citrix.com> <1438688274.31129.103.camel@citrix.com> <1438695690.31129.114.camel@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1438695690.31129.114.camel@citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Ian Campbell , Stefano Stabellini Cc: wei.liu2@citrix.com, ian.jackson@eu.citrix.com, xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org El 04/08/15 a les 15.41, Ian Campbell ha escrit: > On Tue, 2015-08-04 at 14:18 +0100, Stefano Stabellini wrote: >> On Tue, 4 Aug 2015, Ian Campbell wrote: >>> On Tue, 2015-08-04 at 12:18 +0100, Stefano Stabellini wrote: >>>>> It hasn't actually failed though, is that normal? >>>> >>>> It is an "optional" dependency, without it passthrough is disabled in >>>> qemu-traditional. Passing -n tell raisin to "shut up, do nothing and >>>> continue", so at the end the build still succeed. >>> >>> What other sorts of things are optional? >> >> There isn't really a distinction between mandatory dependencies and >> optional dependencies in raisin. This is what raisin is currently >> complaining about: >> >> bridge-utils pciutils-dev libnl-3-dev libnl-route-3-dev >> >> bridge-utils is actually a run time dependency for Xen, while >> libnl-3-dev and libnl-route-3-dev are build time dependencies for >> libvirt. I think they are needed for vlan support. Should I add them >> too? > > I think so, I have a feeling Remus might use them too? cf "[PATCH] osstest: > install libnl3 packages" from Roger. Yes, Remus needs libnl-3-dev and libnl-route-3-dev. They are available since Wheezy, but only the ones on Squeeze are new enough for the Remus code. Feel free to pick that patch, it only needs a comment in the description to mention that support for anything older than Wheezy is dropped. Roger.