[libvirt] [rust PATCH] travis: fix syntax for script commands

Daniel P. Berrangé berrange at redhat.com
Tue Apr 24 12:51:14 UTC 2018


On Tue, Apr 24, 2018 at 02:48:29PM +0200, Sahid Orentino Ferdjaoui wrote:
> On Tue, Apr 24, 2018 at 01:33:20PM +0100, Daniel P. Berrangé wrote:
> > The script: commands were listed without a leading '-' which caused
> > travis to concatenate them into a single command. This meant the second
> > command became a set of arguments to the first command. Historically
> > cargo ignored these extra args so the mistake was not noticed, but it
> > now generates a fatal error.
> >
> 
> Thanks for this Daniel. Do you want me to merge it or are you going to
> take care of that too?

I'll push it - was just posting for review

> 
> Reviewed-by: Sahid Orentino Ferdjaoui <sahid.ferdjaoui at redhat.com>
> 
> > Signed-off-by: Daniel P. Berrangé <berrange at redhat.com>
> > ---
> >  .travis.yml | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/.travis.yml b/.travis.yml
> > index f9844bb..c52f745 100644
> > --- a/.travis.yml
> > +++ b/.travis.yml
> > @@ -40,5 +40,5 @@ install:
> >    - echo "pass" | sudo saslpasswd2 -p -a libvirt user
> >  
> >  script:
> > -  cargo test --verbose
> > -  cargo test --verbose -- --ignored
> > +  - cargo test --verbose
> > +  - cargo test --verbose -- --ignored
> > -- 
> > 2.14.3

Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|




More information about the libvir-list mailing list