[Qemu-devel] [PATCH] travis: trim out most clang builds

[Qemu-devel] [PATCH] travis: trim out most clang builds

[Daniel P. Berrange]

We test with both gcc and clang in order to detect cases
where clang issues warnings that gcc misses. To achieve
this though we don't need to build QEMU in multiple
different configurations. Just a single clang-on-linux
build will be sufficient, if we have an "all enabled"
config.

This cuts the number of build jobs from 21 to 16,
reducing the load imposed on shared Travis CI infra.
This will make it practical to enable jobs for other
interesting & useful configurations without DOS'ing
Travis to much.

Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
---
 .travis.yml | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/.travis.yml b/.travis.yml
index 9916178..0706b9a 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -4,7 +4,6 @@ python:
   - "2.4"
 compiler:
   - gcc
-  - clang
 cache: ccache
 addons:
   apt:
@@ -68,6 +67,9 @@ script:
   - make -j3 && ${TEST_CMD}
 matrix:
   include:
+    # Test with CLang for compile portability
+    - env: CONFIG=""
+      compiler: clang
     # gprof/gcov are GCC features
     - env: CONFIG="--enable-gprof --enable-gcov --disable-pie"
       compiler: gcc
-- 
2.9.3

Re: [Qemu-devel] [PATCH] travis: trim out most clang builds

[Stefan Hajnoczi]

[-- Attachment #1: Type: text/plain, Size: 812 bytes --]

On Thu, Oct 27, 2016 at 03:23:45PM +0200, Daniel P. Berrange wrote:
> We test with both gcc and clang in order to detect cases
> where clang issues warnings that gcc misses. To achieve
> this though we don't need to build QEMU in multiple
> different configurations. Just a single clang-on-linux
> build will be sufficient, if we have an "all enabled"
> config.
> 
> This cuts the number of build jobs from 21 to 16,
> reducing the load imposed on shared Travis CI infra.
> This will make it practical to enable jobs for other
> interesting & useful configurations without DOS'ing
> Travis to much.
> 
> Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
> ---
>  .travis.yml | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)

Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 455 bytes --]

Re: [Qemu-devel] [PATCH] travis: trim out most clang builds

[Alex Bennée]

Daniel P. Berrange <berrange@redhat.com> writes:

> We test with both gcc and clang in order to detect cases
> where clang issues warnings that gcc misses. To achieve
> this though we don't need to build QEMU in multiple
> different configurations. Just a single clang-on-linux
> build will be sufficient, if we have an "all enabled"
> config.
>
> This cuts the number of build jobs from 21 to 16,
> reducing the load imposed on shared Travis CI infra.
> This will make it practical to enable jobs for other
> interesting & useful configurations without DOS'ing
> Travis to much.
>
> Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
> ---
>  .travis.yml | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/.travis.yml b/.travis.yml
> index 9916178..0706b9a 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -4,7 +4,6 @@ python:
>    - "2.4"
>  compiler:
>    - gcc
> -  - clang
>  cache: ccache
>  addons:
>    apt:
> @@ -68,6 +67,9 @@ script:
>    - make -j3 && ${TEST_CMD}
>  matrix:
>    include:
> +    # Test with CLang for compile portability
> +    - env: CONFIG=""
> +      compiler: clang
>      # gprof/gcov are GCC features
>      - env: CONFIG="--enable-gprof --enable-gcov --disable-pie"
>        compiler: gcc

Good idea. I'll add it to my tree.

Thanks.

--
Alex Bennée