travis: Update to job matrix
diff mbox series

Message ID ef0b9472-81a0-a210-28e3-f870fb637243@infradead.org
State Accepted
Headers show
Series
  • travis: Update to job matrix
Related show

Commit Message

Geoff Levand Aug. 17, 2018, 5:59 p.m. UTC
o Set language to bash (same as generic), so just the
   scrip steps run.
 o Move build-builder to 'before_script'.
 o Switch to a job matrix of 'os' entries with names.

Signed-off-by: Geoff Levand <geoff@infradead.org>
---
Hi Sam,

This updates the travis file to use a job matrix.  The jobs run should
be unchanged, but this gives the jobs a name, which is displayed in the
travis UI.  A job matrix allows for more advanced logic that we can take
advantage of if we start adding more jobs. 

Please consider.

-Geoff

 .travis.yml | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

Comments

Samuel Mendoza-Jonas Aug. 28, 2018, 7:01 a.m. UTC | #1
On Fri, 2018-08-17 at 10:59 -0700, Geoff Levand wrote:
>  o Set language to bash (same as generic), so just the
>    scrip steps run.
>  o Move build-builder to 'before_script'.
>  o Switch to a job matrix of 'os' entries with names.
> 
> Signed-off-by: Geoff Levand <geoff@infradead.org>
> ---
> Hi Sam,
> 
> This updates the travis file to use a job matrix.  The jobs run should
> be unchanged, but this gives the jobs a name, which is displayed in the
> travis UI.  A job matrix allows for more advanced logic that we can take
> advantage of if we start adding more jobs. 
> 
> Please consider.
> 
> -Geoff

Thanks, merged as 27c55969.

> 
>  .travis.yml | 14 +++++++++-----
>  1 file changed, 9 insertions(+), 5 deletions(-)
> 
> diff --git a/.travis.yml b/.travis.yml
> index f458213..cff60dc 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -1,13 +1,17 @@
> -language: c
> +language: bash
>  
>  sudo: required
>  dist: trusty
>  services: docker
> -os:
> - - linux
> - - linux-ppc64le
>  
> -before_install:
> +matrix:
> +  include:
> +    - os: linux
> +      name: "linux"
> +    - os: linux-ppc64le
> +      name: "linux-ppc64le"
> +
> +before_script:
>   - ./docker/build-builder
>  
>  script:

Patch
diff mbox series

diff --git a/.travis.yml b/.travis.yml
index f458213..cff60dc 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -1,13 +1,17 @@ 
-language: c
+language: bash
 
 sudo: required
 dist: trusty
 services: docker
-os:
- - linux
- - linux-ppc64le
 
-before_install:
+matrix:
+  include:
+    - os: linux
+      name: "linux"
+    - os: linux-ppc64le
+      name: "linux-ppc64le"
+
+before_script:
  - ./docker/build-builder
 
 script: