Re: [Xen-devel] [PATCH] gitlab-ci: use git clean -ffdx in build each commit test

2019-04-08 Thread Wei Liu
On Mon, Apr 08, 2019 at 01:45:15PM +0100, Wei Liu wrote:
> On Mon, Apr 08, 2019 at 01:43:57PM +0100, Anthony PERARD wrote:
> > On Mon, Apr 08, 2019 at 12:50:31PM +0100, Wei Liu wrote:
> > > The build script invoked is designed to run in a pristine checkout.
> > > 
> > > Signed-off-by: Wei Liu 
> > > ---
> > >  automation/gitlab-ci/build-each-commit.sh | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > > 
> > > diff --git a/automation/gitlab-ci/build-each-commit.sh 
> > > b/automation/gitlab-ci/build-each-commit.sh
> > > index 275bc71902..879028b5a7 100755
> > > --- a/automation/gitlab-ci/build-each-commit.sh
> > > +++ b/automation/gitlab-ci/build-each-commit.sh
> > > @@ -15,4 +15,4 @@ fi
> > >  echo "Building ${CI_COMMIT_BEFORE_SHA}..${CI_COMMIT_SHA}"
> > >  
> > >  NON_SYMBOLIC_REF=1 ./automation/scripts/build-test.sh 
> > > ${CI_COMMIT_BEFORE_SHA} ${CI_COMMIT_SHA} \
> > > -bash -c "make -j4 distclean && ./automation/scripts/build"
> > > +bash -c "git clean -ffdx && ./automation/scripts/build"
> > 
> > Is this mean that QEMU, OVMF, SeaBIOS, ... are going to be clonned again
> > and again? But maybe it doesn't matter.
> 
> Yes. But I don't think that matters much.

BTW, distclean deletes those repositories as well. The nature of this
specific test means it is best to start in a pristine environment,
because we could potentially move the HEAD of all those repositories.

Wei.

___
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel

Re: [Xen-devel] [PATCH] gitlab-ci: use git clean -ffdx in build each commit test

2019-04-08 Thread Wei Liu
On Mon, Apr 08, 2019 at 01:43:57PM +0100, Anthony PERARD wrote:
> On Mon, Apr 08, 2019 at 12:50:31PM +0100, Wei Liu wrote:
> > The build script invoked is designed to run in a pristine checkout.
> > 
> > Signed-off-by: Wei Liu 
> > ---
> >  automation/gitlab-ci/build-each-commit.sh | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/automation/gitlab-ci/build-each-commit.sh 
> > b/automation/gitlab-ci/build-each-commit.sh
> > index 275bc71902..879028b5a7 100755
> > --- a/automation/gitlab-ci/build-each-commit.sh
> > +++ b/automation/gitlab-ci/build-each-commit.sh
> > @@ -15,4 +15,4 @@ fi
> >  echo "Building ${CI_COMMIT_BEFORE_SHA}..${CI_COMMIT_SHA}"
> >  
> >  NON_SYMBOLIC_REF=1 ./automation/scripts/build-test.sh 
> > ${CI_COMMIT_BEFORE_SHA} ${CI_COMMIT_SHA} \
> > -bash -c "make -j4 distclean && ./automation/scripts/build"
> > +bash -c "git clean -ffdx && ./automation/scripts/build"
> 
> Is this mean that QEMU, OVMF, SeaBIOS, ... are going to be clonned again
> and again? But maybe it doesn't matter.

Yes. But I don't think that matters much.

Wei.

> 
> -- 
> Anthony PERARD

___
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel

[Xen-devel] [PATCH] gitlab-ci: use git clean -ffdx in build each commit test

2019-04-08 Thread Wei Liu
The build script invoked is designed to run in a pristine checkout.

Signed-off-by: Wei Liu 
---
 automation/gitlab-ci/build-each-commit.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/automation/gitlab-ci/build-each-commit.sh 
b/automation/gitlab-ci/build-each-commit.sh
index 275bc71902..879028b5a7 100755
--- a/automation/gitlab-ci/build-each-commit.sh
+++ b/automation/gitlab-ci/build-each-commit.sh
@@ -15,4 +15,4 @@ fi
 echo "Building ${CI_COMMIT_BEFORE_SHA}..${CI_COMMIT_SHA}"
 
 NON_SYMBOLIC_REF=1 ./automation/scripts/build-test.sh ${CI_COMMIT_BEFORE_SHA} 
${CI_COMMIT_SHA} \
-bash -c "make -j4 distclean && ./automation/scripts/build"
+bash -c "git clean -ffdx && ./automation/scripts/build"
-- 
2.20.1


___
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel