[CRIU] [PATCH] zdtm: Make arch specific tests to have \Space at the end

Andrew Vagin avagin at gmail.com
Wed Jun 25 04:03:15 PDT 2014


On Wed, Jun 25, 2014 at 02:56:01PM +0400, Cyrill Gorcunov wrote:
> On Wed, Jun 25, 2014 at 02:52:28PM +0400, Andrew Vagin wrote:
> > On Wed, Jun 25, 2014 at 02:39:04PM +0400, Cyrill Gorcunov wrote:
> > > Otherwise we might have a clash
> > > 
> > > | Execute zdtm/live/static/vdso01ns/static/pipe00
> > > 
> > > Reported-by: Andrey Vagin <avagin at openvz.org>
> > > Signed-off-by: Cyrill Gorcunov <gorcunov at openvz.org>
> > > ---
> > >  test/zdtm.sh | 4 +++-
> > >  1 file changed, 3 insertions(+), 1 deletion(-)
> > > 
> > > diff --git a/test/zdtm.sh b/test/zdtm.sh
> > > index 7e581924d476..6c5f19dab958 100755
> > > --- a/test/zdtm.sh
> > > +++ b/test/zdtm.sh
> > > @@ -123,7 +123,9 @@ static/dumpable02
> > >  #
> > >  # Arch specific tests
> > >  if [ $ARCH = "x86_64" ]; then
> > > -	TEST_LIST_ARCH="static/vdso01"
> > > +	TEST_LIST_ARCH="
> > > +static/vdso01
> > > +"
> > >  fi
> > >  
> > >  TEST_LIST=$TEST_LIST$TEST_LIST_ARCH
> > 
> > 
> > TEST_LIST="$TEST_LIST
> > $TEST_LIST_ARCH"
> 
> I like my variant rather because
> 
>  - it's same fashion as TEST_LIST
>  - adding new arch tests will be easier

Ok, thanks.

Acked-by: Andrew Vagin <avagin at gmail.com>


More information about the CRIU mailing list