From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: [PATCH v8 1/2] OSSTEST: introduce a raisin build test Date: Tue, 4 Aug 2015 12:04:31 +0100 Message-ID: <1438686271.31129.96.camel@citrix.com> References: <1437670366-29034-1-git-send-email-stefano.stabellini@eu.citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1437670366-29034-1-git-send-email-stefano.stabellini@eu.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: Stefano Stabellini , xen-devel@lists.xen.org Cc: wei.liu2@citrix.com, ian.jackson@eu.citrix.com List-Id: xen-devel@lists.xenproject.org On Thu, 2015-07-23 at 17:52 +0100, Stefano Stabellini wrote: > +sub build () { > + target_cmd_build($ho, 9000, $builddir, < + cd $raisindir > + ./raise -n build > +END > +#/; Running this is pretty quiet it seems, which is normally good but I think for automated test we likely want to crank up the verbosity somewhat. Is that possible? Or else if this is being captured into some logfile somewhere we should capture that. It also said: [raisin] Missing packages: pciutils-dev which I guess is something which needs to be added to ts-xen-build-prep? It hasn't actually failed though, is that normal? It also said [raisin] Raisin requires sudo to install build dependencies for you. [raisin] You can only build without it. but I think that is desirable/expected. Not installing sudo on osstest build hosts (as we do) seems like a good safety catch to maintain to me. Ian.