1. Patchew
  2. QEMU
  3. [Qemu-devel] [PATCH v1 0/8] Travis stability and a few docker patches
  4. View patch

[Qemu-devel] [PATCH v1 5/8] .travis.yml: rationalise clang testing

Alex Bennée posted 8 patches 7 years, 5 months ago
[Qemu-devel] [PATCH v1 5/8] .travis.yml: rationalise clang testing
Posted by Alex Bennée 7 years, 5 months ago 
As Travis includes Clang 5.0 in its own build environment there is no
point manually building with older Clangs. We still need to test with
the two pythons though so we leave them as system only builds. We also
split the clang build into two as it often exceeds the 40 minute build
time limit.

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

diff --git a/.travis.yml b/.travis.yml
index aa83e9aed7..85ee2a1edb 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -70,8 +70,10 @@ script:
   - make ${MAKEFLAGS} && ${TEST_CMD}
 matrix:
   include:
-    # Test with CLang for compile portability
-    - env: CONFIG=""
+    # Test with Clang for compile portability (Travis uses clang-5.0)
+    - env: CONFIG="--disable-system"
+      compiler: clang
+    - env: CONFIG="--disable-user"
       compiler: clang
     # gprof/gcov are GCC features
     - env: CONFIG="--enable-gprof --enable-gcov --disable-pie --disable-linux-user"
@@ -95,70 +97,13 @@ matrix:
     - env: CONFIG=""
       os: osx
       compiler: clang
-    # Plain Trusty System Build
+    # Python builds
     - env: CONFIG="--disable-linux-user"
-      sudo: required
-      addons:
-      dist: trusty
-      compiler: gcc
-      before_install:
-        - sudo apt-get update -qq
-        - sudo apt-get build-dep -qq qemu
-        - wget -O - http://people.linaro.org/~alex.bennee/qemu-submodule-git-seed.tar.xz | tar -xvJ
-        - git submodule update --init --recursive
-    # Plain Trusty Linux User Build
-    - env: CONFIG="--disable-system"
-      sudo: required
-      addons:
-      dist: trusty
-      compiler: gcc
-      before_install:
-        - sudo apt-get update -qq
-        - sudo apt-get build-dep -qq qemu
-        - wget -O - http://people.linaro.org/~alex.bennee/qemu-submodule-git-seed.tar.xz | tar -xvJ
-        - git submodule update --init --recursive
-    # Trusty System build with latest stable clang & python 3.0
-    - sudo: required
-      addons:
-      dist: trusty
-      language: generic
-      compiler: none
       python:
         - "3.0"
-      env:
-        - COMPILER_NAME=clang CXX=clang++-3.9 CC=clang-3.9
-        - CONFIG="--disable-linux-user --cc=clang-3.9 --cxx=clang++-3.9 --python=/usr/bin/python3"
-      before_install:
-        - wget -nv -O - http://llvm.org/apt/llvm-snapshot.gpg.key | sudo apt-key add -
-        - sudo apt-add-repository -y 'deb http://llvm.org/apt/trusty llvm-toolchain-trusty-3.9 main'
-        - sudo apt-get update -qq
-        - sudo apt-get install -qq -y clang-3.9
-        - sudo apt-get build-dep -qq qemu
-        - wget -O - http://people.linaro.org/~alex.bennee/qemu-submodule-git-seed.tar.xz | tar -xvJ
-        - git submodule update --init --recursive
-      before_script:
-        - ./configure ${CONFIG} || cat config.log
-    # Trusty Linux User build with latest stable clang & python 3.6
-    - sudo: required
-      addons:
-      dist: trusty
-      language: generic
-      compiler: none
+    - env: CONFIG="--disable-linux-user"
       python:
         - "3.6"
-      env:
-        - COMPILER_NAME=clang CXX=clang++-3.9 CC=clang-3.9
-        - CONFIG="--disable-system --cc=clang-3.9 --cxx=clang++-3.9 --python=/usr/bin/python3"
-      before_install:
-        - wget -nv -O - http://llvm.org/apt/llvm-snapshot.gpg.key | sudo apt-key add -
-        - sudo apt-add-repository -y 'deb http://llvm.org/apt/trusty llvm-toolchain-trusty-3.9 main'
-        - sudo apt-get update -qq
-        - sudo apt-get install -qq -y clang-3.9
-        - sudo apt-get build-dep -qq qemu
-        - wget -O - http://people.linaro.org/~alex.bennee/qemu-submodule-git-seed.tar.xz | tar -xvJ
-        - git submodule update --init --recursive
-      before_script:
-        - ./configure ${CONFIG} || cat config.log
     # Using newer GCC with sanitizers
     - addons:
         apt:
-- 
2.17.0
Re: [Qemu-devel] [PATCH v1 5/8] .travis.yml: rationalise clang testing
Posted by Philippe Mathieu-Daudé 7 years, 5 months ago 
On 05/30/2018 08:06 AM, Alex Bennée wrote:
> As Travis includes Clang 5.0 in its own build environment there is no
> point manually building with older Clangs. We still need to test with
> the two pythons though so we leave them as system only builds. We also
> split the clang build into two as it often exceeds the 40 minute build
> time limit.
> 
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>

Nice cut.

Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Tested-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

> ---
>  .travis.yml | 67 +++++------------------------------------------------
>  1 file changed, 6 insertions(+), 61 deletions(-)
> 
> diff --git a/.travis.yml b/.travis.yml
> index aa83e9aed7..85ee2a1edb 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -70,8 +70,10 @@ script:
>    - make ${MAKEFLAGS} && ${TEST_CMD}
>  matrix:
>    include:
> -    # Test with CLang for compile portability
> -    - env: CONFIG=""
> +    # Test with Clang for compile portability (Travis uses clang-5.0)
> +    - env: CONFIG="--disable-system"
> +      compiler: clang
> +    - env: CONFIG="--disable-user"
>        compiler: clang
>      # gprof/gcov are GCC features
>      - env: CONFIG="--enable-gprof --enable-gcov --disable-pie --disable-linux-user"
> @@ -95,70 +97,13 @@ matrix:
>      - env: CONFIG=""
>        os: osx
>        compiler: clang
> -    # Plain Trusty System Build
> +    # Python builds
>      - env: CONFIG="--disable-linux-user"
> -      sudo: required
> -      addons:
> -      dist: trusty
> -      compiler: gcc
> -      before_install:
> -        - sudo apt-get update -qq
> -        - sudo apt-get build-dep -qq qemu
> -        - wget -O - http://people.linaro.org/~alex.bennee/qemu-submodule-git-seed.tar.xz | tar -xvJ
> -        - git submodule update --init --recursive
> -    # Plain Trusty Linux User Build
> -    - env: CONFIG="--disable-system"
> -      sudo: required
> -      addons:
> -      dist: trusty
> -      compiler: gcc
> -      before_install:
> -        - sudo apt-get update -qq
> -        - sudo apt-get build-dep -qq qemu
> -        - wget -O - http://people.linaro.org/~alex.bennee/qemu-submodule-git-seed.tar.xz | tar -xvJ
> -        - git submodule update --init --recursive
> -    # Trusty System build with latest stable clang & python 3.0
> -    - sudo: required
> -      addons:
> -      dist: trusty
> -      language: generic
> -      compiler: none
>        python:
>          - "3.0"
> -      env:
> -        - COMPILER_NAME=clang CXX=clang++-3.9 CC=clang-3.9
> -        - CONFIG="--disable-linux-user --cc=clang-3.9 --cxx=clang++-3.9 --python=/usr/bin/python3"
> -      before_install:
> -        - wget -nv -O - http://llvm.org/apt/llvm-snapshot.gpg.key | sudo apt-key add -
> -        - sudo apt-add-repository -y 'deb http://llvm.org/apt/trusty llvm-toolchain-trusty-3.9 main'
> -        - sudo apt-get update -qq
> -        - sudo apt-get install -qq -y clang-3.9
> -        - sudo apt-get build-dep -qq qemu
> -        - wget -O - http://people.linaro.org/~alex.bennee/qemu-submodule-git-seed.tar.xz | tar -xvJ
> -        - git submodule update --init --recursive
> -      before_script:
> -        - ./configure ${CONFIG} || cat config.log
> -    # Trusty Linux User build with latest stable clang & python 3.6
> -    - sudo: required
> -      addons:
> -      dist: trusty
> -      language: generic
> -      compiler: none
> +    - env: CONFIG="--disable-linux-user"
>        python:
>          - "3.6"
> -      env:
> -        - COMPILER_NAME=clang CXX=clang++-3.9 CC=clang-3.9
> -        - CONFIG="--disable-system --cc=clang-3.9 --cxx=clang++-3.9 --python=/usr/bin/python3"
> -      before_install:
> -        - wget -nv -O - http://llvm.org/apt/llvm-snapshot.gpg.key | sudo apt-key add -
> -        - sudo apt-add-repository -y 'deb http://llvm.org/apt/trusty llvm-toolchain-trusty-3.9 main'
> -        - sudo apt-get update -qq
> -        - sudo apt-get install -qq -y clang-3.9
> -        - sudo apt-get build-dep -qq qemu
> -        - wget -O - http://people.linaro.org/~alex.bennee/qemu-submodule-git-seed.tar.xz | tar -xvJ
> -        - git submodule update --init --recursive
> -      before_script:
> -        - ./configure ${CONFIG} || cat config.log
>      # Using newer GCC with sanitizers
>      - addons:
>          apt:
>

Patchew 2.1-devel-b67e4c2

© 2016 - 2025 Red Hat, Inc.