From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:37821)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <alex.bennee@linaro.org>) id 1c2hXM-0007h1-Nv
	for qemu-devel@nongnu.org; Fri, 04 Nov 2016 12:40:17 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <alex.bennee@linaro.org>) id 1c2hXJ-0000vQ-Iu
	for qemu-devel@nongnu.org; Fri, 04 Nov 2016 12:40:16 -0400
Received: from mail-wm0-x22b.google.com ([2a00:1450:400c:c09::22b]:37825)
	by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16)
	(Exim 4.71) (envelope-from <alex.bennee@linaro.org>)
	id 1c2hXJ-0000sz-Bs
	for qemu-devel@nongnu.org; Fri, 04 Nov 2016 12:40:13 -0400
Received: by mail-wm0-x22b.google.com with SMTP id t79so60914668wmt.0
	for <qemu-devel@nongnu.org>; Fri, 04 Nov 2016 09:40:12 -0700 (PDT)
References: <20161027132345.21072-1-berrange@redhat.com>
From: Alex =?utf-8?Q?Benn=C3=A9e?= <alex.bennee@linaro.org>
In-reply-to: <20161027132345.21072-1-berrange@redhat.com>
Date: Fri, 04 Nov 2016 16:40:09 +0000
Message-ID: <87k2cjtd06.fsf@linaro.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 8bit
Subject: Re: [Qemu-devel] [PATCH] travis: trim out most clang builds
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel/>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: "Daniel P. Berrange" <berrange@redhat.com>
Cc: qemu-devel@nongnu.org


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