Re: [Xen-devel] [PATCH for-4.10] scripts: add a script for build testing

2017-10-23 Thread Wei Liu
On Mon, Oct 23, 2017 at 04:09:58PM +0100, Ian Jackson wrote: > Wei Liu writes ("Re: [Xen-devel] [PATCH for-4.10] scripts: add a script for > build testing"): > > On Mon, Oct 23, 2017 at 03:50:31PM +0100, Anthony PERARD wrote: > > > FIY, I do like to put script and o

Re: [Xen-devel] [PATCH for-4.10] scripts: add a script for build testing

2017-10-23 Thread Ian Jackson
Wei Liu writes ("Re: [Xen-devel] [PATCH for-4.10] scripts: add a script for build testing"): > On Mon, Oct 23, 2017 at 03:50:31PM +0100, Anthony PERARD wrote: > > FIY, I do like to put script and other files in my checkouts, the git > > clean will remove them. &g

Re: [Xen-devel] [PATCH for-4.10] scripts: add a script for build testing

2017-10-23 Thread Wei Liu
On Mon, Oct 23, 2017 at 03:50:31PM +0100, Anthony PERARD wrote: > On Mon, Oct 23, 2017 at 02:02:53PM +0100, George Dunlap wrote: > > On 10/20/2017 06:32 PM, Wei Liu wrote: > > > Signed-off-by: Wei Liu > > > --- > > > Cc: Andrew Cooper > > > Cc: George Dunlap > > > Cc: Ian Jackson > > > Cc: Jan

Re: [Xen-devel] [PATCH for-4.10] scripts: add a script for build testing

2017-10-23 Thread Anthony PERARD
On Mon, Oct 23, 2017 at 02:02:53PM +0100, George Dunlap wrote: > On 10/20/2017 06:32 PM, Wei Liu wrote: > > Signed-off-by: Wei Liu > > --- > > Cc: Andrew Cooper > > Cc: George Dunlap > > Cc: Ian Jackson > > Cc: Jan Beulich > > Cc: Konrad Rzeszutek Wilk > > Cc: Stefano Stabellini > > Cc: Tim

Re: [Xen-devel] [PATCH for-4.10] scripts: add a script for build testing

2017-10-23 Thread George Dunlap
On 10/20/2017 06:32 PM, Wei Liu wrote: > Signed-off-by: Wei Liu > --- > Cc: Andrew Cooper > Cc: George Dunlap > Cc: Ian Jackson > Cc: Jan Beulich > Cc: Konrad Rzeszutek Wilk > Cc: Stefano Stabellini > Cc: Tim Deegan > Cc: Wei Liu > Cc: Julien Grall > > The risk for this is zero, hence th

Re: [Xen-devel] [PATCH for-4.10] scripts: add a script for build testing

2017-10-23 Thread Wei Liu
On Mon, Oct 23, 2017 at 01:07:36PM +0100, Ian Jackson wrote: > Wei Liu writes ("Re: [PATCH for-4.10] scripts: add a script for build > testing"): > > On Mon, Oct 23, 2017 at 01:02:00PM +0100, Ian Jackson wrote: > > > In particular, if you: > > > * check that the tree is not dirty > > > * detach

Re: [Xen-devel] [PATCH for-4.10] scripts: add a script for build testing

2017-10-23 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH for-4.10] scripts: add a script for build testing"): > On Mon, Oct 23, 2017 at 02:24:40AM -0600, Jan Beulich wrote: > > On 20.10.17 at 19:32, wrote: > > > +git rebase $BASE $TIP -x "$CMD" > > > > Is this quoting on $CMD really going to work right no matter what > > the

Re: [Xen-devel] [PATCH for-4.10] scripts: add a script for build testing

2017-10-23 Thread Jan Beulich
>>> On 23.10.17 at 13:41, wrote: > On Mon, Oct 23, 2017 at 02:24:40AM -0600, Jan Beulich wrote: >> >>> On 20.10.17 at 19:32, wrote: >> > --- /dev/null >> > +++ b/scripts/build-test.sh >> > @@ -0,0 +1,40 @@ >> > +#!/bin/sh >> > + >> > +# WARNING: Always backup the branch by creating another refere

Re: [Xen-devel] [PATCH for-4.10] scripts: add a script for build testing

2017-10-23 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH for-4.10] scripts: add a script for build testing"): > On Mon, Oct 23, 2017 at 01:02:00PM +0100, Ian Jackson wrote: > > In particular, if you: > > * check that the tree is not dirty > > * detach HEAD > > I think these two checks are good. > > > * reattach HEAD after

Re: [Xen-devel] [PATCH for-4.10] scripts: add a script for build testing

2017-10-23 Thread Wei Liu
On Mon, Oct 23, 2017 at 01:02:00PM +0100, Ian Jackson wrote: > Wei Liu writes ("Re: [PATCH for-4.10] scripts: add a script for build > testing"): > > On Mon, Oct 23, 2017 at 02:24:40AM -0600, Jan Beulich wrote: > > > What is this startup delay intended for? > > > > To give user a chance to check

Re: [Xen-devel] [PATCH for-4.10] scripts: add a script for build testing

2017-10-23 Thread Ian Jackson
Wei Liu writes ("Re: [PATCH for-4.10] scripts: add a script for build testing"): > On Mon, Oct 23, 2017 at 02:24:40AM -0600, Jan Beulich wrote: > > What is this startup delay intended for? > > To give user a chance to check the command -- git-rebase can be > destructive after all. I can't resist

Re: [Xen-devel] [PATCH for-4.10] scripts: add a script for build testing

2017-10-23 Thread Wei Liu
On Mon, Oct 23, 2017 at 12:30:33PM +0100, Anthony PERARD wrote: > On Fri, Oct 20, 2017 at 06:32:55PM +0100, Wei Liu wrote: > > +CMD=${3:-git clean -fdx && ./configure && make -j4} > > + > > +echo "Running command \"$CMD\" on every commit from $BASE to $TIP" > > +echo -n "Starting in " > > + > > +fo

Re: [Xen-devel] [PATCH for-4.10] scripts: add a script for build testing

2017-10-23 Thread Wei Liu
On Mon, Oct 23, 2017 at 02:24:40AM -0600, Jan Beulich wrote: > >>> On 20.10.17 at 19:32, wrote: > > --- /dev/null > > +++ b/scripts/build-test.sh > > @@ -0,0 +1,40 @@ > > +#!/bin/sh > > + > > +# WARNING: Always backup the branch by creating another reference to it if > > +# you're not familiar wit

Re: [Xen-devel] [PATCH for-4.10] scripts: add a script for build testing

2017-10-23 Thread Anthony PERARD
On Mon, Oct 23, 2017 at 12:30:33PM +0100, Anthony PERARD wrote: > On Fri, Oct 20, 2017 at 06:32:55PM +0100, Wei Liu wrote: > > +CMD=${3:-git clean -fdx && ./configure && make -j4} > > + > > +echo "Running command \"$CMD\" on every commit from $BASE to $TIP" > > +echo -n "Starting in " > > + > > +fo

Re: [Xen-devel] [PATCH for-4.10] scripts: add a script for build testing

2017-10-23 Thread Anthony PERARD
On Fri, Oct 20, 2017 at 06:32:55PM +0100, Wei Liu wrote: > +CMD=${3:-git clean -fdx && ./configure && make -j4} > + > +echo "Running command \"$CMD\" on every commit from $BASE to $TIP" > +echo -n "Starting in " > + > +for i in `seq 5 -1 1`; do > +echo -n "$i ... " > +sleep 1 > +done > + I

Re: [Xen-devel] [PATCH for-4.10] scripts: add a script for build testing

2017-10-23 Thread Julien Grall
Hi Wei, On 20/10/17 18:32, Wei Liu wrote: Signed-off-by: Wei Liu --- Cc: Andrew Cooper Cc: George Dunlap Cc: Ian Jackson Cc: Jan Beulich Cc: Konrad Rzeszutek Wilk Cc: Stefano Stabellini Cc: Tim Deegan Cc: Wei Liu Cc: Julien Grall The risk for this is zero, hence the for-4.10 tag. Ag

Re: [Xen-devel] [PATCH for-4.10] scripts: add a script for build testing

2017-10-23 Thread Jan Beulich
>>> On 20.10.17 at 19:32, wrote: > --- /dev/null > +++ b/scripts/build-test.sh > @@ -0,0 +1,40 @@ > +#!/bin/sh > + > +# WARNING: Always backup the branch by creating another reference to it if > +# you're not familiar with git-rebase(1). > +# > +# Use `git rebase` to run command or script on every

[Xen-devel] [PATCH for-4.10] scripts: add a script for build testing

2017-10-20 Thread Wei Liu
Signed-off-by: Wei Liu --- Cc: Andrew Cooper Cc: George Dunlap Cc: Ian Jackson Cc: Jan Beulich Cc: Konrad Rzeszutek Wilk Cc: Stefano Stabellini Cc: Tim Deegan Cc: Wei Liu Cc: Julien Grall The risk for this is zero, hence the for-4.10 tag. --- scripts/build-test.sh | 40 +