diff mbox

[v2,5/5] .travis.yml: reduce target list on core configure tweaks

Message ID 1458753657-21664-6-git-send-email-alex.bennee@linaro.org (mailing list archive)
State New, archived
Headers show

Commit Message

Alex Bennée March 23, 2016, 5:20 p.m. UTC
A number of configure options only really affect the core code and any
arch specific stuff should be flushed out by other builds:

  - trace-backends, log build is all targets, others can be less
  - --disable-build, ensuring disabling stuff doesn't break host
  - co-routine, default already built with all, common API
  - tcg interpreter is front end agnostic

I've used the ./configure --target-list stem support to do all of one
arch type in each reduced build.

Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
 .travis.yml | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

Comments

David Gibson March 24, 2016, 12:22 a.m. UTC | #1
On Wed, Mar 23, 2016 at 05:20:57PM +0000, Alex Bennée wrote:
> A number of configure options only really affect the core code and any
> arch specific stuff should be flushed out by other builds:
> 
>   - trace-backends, log build is all targets, others can be less
>   - --disable-build, ensuring disabling stuff doesn't break host
>   - co-routine, default already built with all, common API
>   - tcg interpreter is front end agnostic
> 
> I've used the ./configure --target-list stem support to do all of one
> arch type in each reduced build.
> 
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>

Reviewed-by: David Gibson <david@gibson.dropbear.id.au>

AFAICT compiling extra targets is very unlikely to detect extra
problems with those config options.

> ---
>  .travis.yml | 18 +++++++++---------
>  1 file changed, 9 insertions(+), 9 deletions(-)
> 
> diff --git a/.travis.yml b/.travis.yml
> index 50ac17f..d13bffc 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -45,10 +45,10 @@ env:
>    matrix:
>      - CONFIG=""
>      - CONFIG="--enable-debug --enable-debug-tcg --enable-trace-backends=log"
> -    - CONFIG="--disable-linux-aio --disable-cap-ng --disable-attr --disable-brlapi --disable-uuid --disable-libusb"
> -    - CONFIG="--enable-modules"
> -    - CONFIG="--with-coroutine=ucontext"
> -    - CONFIG="--with-coroutine=sigaltstack"
> +    - CONFIG="--disable-linux-aio --disable-cap-ng --disable-attr --disable-brlapi --disable-uuid --disable-libusb --target-list=86"
> +    - CONFIG="--enable-modules --target-list=86"
> +    - CONFIG="--with-coroutine=ucontext --target-list=arm"
> +    - CONFIG="--with-coroutine=sigaltstack --target-list=ppc"
>  git:
>    # we want to do this ourselves
>    submodules: false
> @@ -70,19 +70,19 @@ matrix:
>      - env: CONFIG="--enable-gprof --enable-gcov --disable-pie"
>        compiler: gcc
>      # We manually include builds which we disable "make check" for
> -    - env: CONFIG="--enable-debug --enable-tcg-interpreter"
> +    - env: CONFIG="--enable-debug --enable-tcg-interpreter --target-list=mips"
>             TEST_CMD=""
>        compiler: gcc
> -    - env: CONFIG="--enable-trace-backends=simple"
> +    - env: CONFIG="--enable-trace-backends=simple --target-list=alpha"
>             TEST_CMD=""
>        compiler: gcc
> -    - env: CONFIG="--enable-trace-backends=ftrace"
> +    - env: CONFIG="--enable-trace-backends=ftrace --target-list=s390"
>             TEST_CMD=""
>        compiler: gcc
> -    - env: CONFIG="--enable-trace-backends=ust"
> +    - env: CONFIG="--enable-trace-backends=ust --target-list=sparc"
>             TEST_CMD=""
>        compiler: gcc
> -    - env: CONFIG="--with-coroutine=gthread"
> +    - env: CONFIG="--with-coroutine=gthread --target-list=sh4"
>             TEST_CMD=""
>        compiler: gcc
>      - env: CONFIG=""
diff mbox

Patch

diff --git a/.travis.yml b/.travis.yml
index 50ac17f..d13bffc 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -45,10 +45,10 @@  env:
   matrix:
     - CONFIG=""
     - CONFIG="--enable-debug --enable-debug-tcg --enable-trace-backends=log"
-    - CONFIG="--disable-linux-aio --disable-cap-ng --disable-attr --disable-brlapi --disable-uuid --disable-libusb"
-    - CONFIG="--enable-modules"
-    - CONFIG="--with-coroutine=ucontext"
-    - CONFIG="--with-coroutine=sigaltstack"
+    - CONFIG="--disable-linux-aio --disable-cap-ng --disable-attr --disable-brlapi --disable-uuid --disable-libusb --target-list=86"
+    - CONFIG="--enable-modules --target-list=86"
+    - CONFIG="--with-coroutine=ucontext --target-list=arm"
+    - CONFIG="--with-coroutine=sigaltstack --target-list=ppc"
 git:
   # we want to do this ourselves
   submodules: false
@@ -70,19 +70,19 @@  matrix:
     - env: CONFIG="--enable-gprof --enable-gcov --disable-pie"
       compiler: gcc
     # We manually include builds which we disable "make check" for
-    - env: CONFIG="--enable-debug --enable-tcg-interpreter"
+    - env: CONFIG="--enable-debug --enable-tcg-interpreter --target-list=mips"
            TEST_CMD=""
       compiler: gcc
-    - env: CONFIG="--enable-trace-backends=simple"
+    - env: CONFIG="--enable-trace-backends=simple --target-list=alpha"
            TEST_CMD=""
       compiler: gcc
-    - env: CONFIG="--enable-trace-backends=ftrace"
+    - env: CONFIG="--enable-trace-backends=ftrace --target-list=s390"
            TEST_CMD=""
       compiler: gcc
-    - env: CONFIG="--enable-trace-backends=ust"
+    - env: CONFIG="--enable-trace-backends=ust --target-list=sparc"
            TEST_CMD=""
       compiler: gcc
-    - env: CONFIG="--with-coroutine=gthread"
+    - env: CONFIG="--with-coroutine=gthread --target-list=sh4"
            TEST_CMD=""
       compiler: gcc
     - env: CONFIG=""