Message ID | 1457102767-3012-1-git-send-email-cardoe@cardoe.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
On 04/03/16 14:46, Doug Goldstein wrote: > Skip building of the coverity, smoke and master branches since they just > fast forward from staging. > > Suggested-by: Andrew Cooper <andrew.cooper3@citrix.com> > Signed-off-by: Doug Goldstein <cardoe@cardoe.com> > --- > CC: Ian Jackson <ian.jackson@eu.citrix.com> > CC: Jan Beulich <jbeulich@suse.com> > CC: Keir Fraser <keir@xen.org> > CC: Tim Deegan <tim@xen.org> > CC: Andrew Cooper <andrew.cooper3@citrix.com> > --- > .travis.yml | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/.travis.yml b/.travis.yml > index 6693af2..74fa14e 100644 > --- a/.travis.yml > +++ b/.travis.yml > @@ -1,6 +1,12 @@ > language: c > dist: trusty > sudo: required > +# don't test master, smoke and coverity branches > +branches: > + except: > + - master > + - smoke > + - coverity-tested/master I would extend this to stable-* as well, as each stable-$X.$Y also fast-forwards from staging-$X.$Y Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com> > matrix: > include: > - compiler: gcc
>>> On 04.03.16 at 15:46, <cardoe@cardoe.com> wrote: > --- a/.travis.yml > +++ b/.travis.yml > @@ -1,6 +1,12 @@ > language: c > dist: trusty > sudo: required > +# don't test master, smoke and coverity branches > +branches: > + except: > + - master > + - smoke > + - coverity-tested/master With the intention to perhaps make this (iirc) coverity-tested/smoke, perhaps use a wildcard: coverity-tested/* ? Jan
On 3/4/16 9:48 AM, Jan Beulich wrote: >>>> On 04.03.16 at 15:46, <cardoe@cardoe.com> wrote: >> --- a/.travis.yml >> +++ b/.travis.yml >> @@ -1,6 +1,12 @@ >> language: c >> dist: trusty >> sudo: required >> +# don't test master, smoke and coverity branches >> +branches: >> + except: >> + - master >> + - smoke >> + - coverity-tested/master > > With the intention to perhaps make this (iirc) coverity-tested/smoke, > perhaps use a wildcard: coverity-tested/* ? > > Jan > Yes. I can fix that up. Let me post a v2 and actually test it. They don't use globbing or wildcards but instead "Ruby regular expression syntax". I can't say I know Ruby so it might take one or two tries to get the pattern right. The other two patches in this series can be merged independently (3/3 does depend on 2/3).
On 3/4/16 9:30 AM, Andrew Cooper wrote: > On 04/03/16 14:46, Doug Goldstein wrote: >> Skip building of the coverity, smoke and master branches since they just >> fast forward from staging. >> >> Suggested-by: Andrew Cooper <andrew.cooper3@citrix.com> >> Signed-off-by: Doug Goldstein <cardoe@cardoe.com> >> --- >> CC: Ian Jackson <ian.jackson@eu.citrix.com> >> CC: Jan Beulich <jbeulich@suse.com> >> CC: Keir Fraser <keir@xen.org> >> CC: Tim Deegan <tim@xen.org> >> CC: Andrew Cooper <andrew.cooper3@citrix.com> >> --- >> .travis.yml | 6 ++++++ >> 1 file changed, 6 insertions(+) >> >> diff --git a/.travis.yml b/.travis.yml >> index 6693af2..74fa14e 100644 >> --- a/.travis.yml >> +++ b/.travis.yml >> @@ -1,6 +1,12 @@ >> language: c >> dist: trusty >> sudo: required >> +# don't test master, smoke and coverity branches >> +branches: >> + except: >> + - master >> + - smoke >> + - coverity-tested/master > > I would extend this to stable-* as well, as each stable-$X.$Y also > fast-forwards from staging-$X.$Y > > Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com> > Will do. I'll post a v2 of this one patch for the same reasons I gave Jan.
diff --git a/.travis.yml b/.travis.yml index 6693af2..74fa14e 100644 --- a/.travis.yml +++ b/.travis.yml @@ -1,6 +1,12 @@ language: c dist: trusty sudo: required +# don't test master, smoke and coverity branches +branches: + except: + - master + - smoke + - coverity-tested/master matrix: include: - compiler: gcc
Skip building of the coverity, smoke and master branches since they just fast forward from staging. Suggested-by: Andrew Cooper <andrew.cooper3@citrix.com> Signed-off-by: Doug Goldstein <cardoe@cardoe.com> --- CC: Ian Jackson <ian.jackson@eu.citrix.com> CC: Jan Beulich <jbeulich@suse.com> CC: Keir Fraser <keir@xen.org> CC: Tim Deegan <tim@xen.org> CC: Andrew Cooper <andrew.cooper3@citrix.com> --- .travis.yml | 6 ++++++ 1 file changed, 6 insertions(+)