testing/next queue (iotests, docker, tests/vm)

[Qemu-devel] [PATCH v1 07/26] .travis.yml: bump gcc sanitiser job to gcc-9

Posted by Alex Bennée 6 years, 8 months ago

The toolchain PPA has it so we might as well use it.

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

diff --git a/.travis.yml b/.travis.yml
index b053a836a32..f0aa37f2d12 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -240,8 +240,8 @@ matrix:
             - ubuntu-toolchain-r-test
           packages:
             # Extra toolchains
-            - gcc-7
-            - g++-7
+            - gcc-9
+            - g++-9
             # Build dependencies
             - libaio-dev
             - libattr1-dev
@@ -270,11 +270,11 @@ matrix:
       language: generic
       compiler: none
       env:
-        - COMPILER_NAME=gcc CXX=g++-7 CC=gcc-7
-        - CONFIG="--cc=gcc-7 --cxx=g++-7 --disable-pie --disable-linux-user"
+        - COMPILER_NAME=gcc CXX=g++-9 CC=gcc-9
+        - CONFIG="--cc=gcc-9 --cxx=g++-9 --disable-pie --disable-linux-user"
         - TEST_CMD=""
       before_script:
-        - ./configure ${CONFIG} --extra-cflags="-g3 -O0 -fsanitize=thread -fuse-ld=gold" || { cat config.log && exit 1; }
+        - ./configure ${CONFIG} --extra-cflags="-g3 -O0 -Wno-error=stringop-truncation -fsanitize=thread -fuse-ld=gold" || { cat config.log && exit 1; }
 
 
     # Run check-tcg against linux-user
-- 
2.20.1

Re: [Qemu-devel] [PATCH v1 07/26] .travis.yml: bump gcc sanitiser job to gcc-9

Posted by Stefano Garzarella 6 years, 8 months ago

On Thu, May 30, 2019 at 11:15:44AM +0100, Alex Bennée wrote:
> The toolchain PPA has it so we might as well use it.
> 
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
>  .travis.yml | 10 +++++-----
>  1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/.travis.yml b/.travis.yml
> index b053a836a32..f0aa37f2d12 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -240,8 +240,8 @@ matrix:
>              - ubuntu-toolchain-r-test
>            packages:
>              # Extra toolchains
> -            - gcc-7
> -            - g++-7
> +            - gcc-9
> +            - g++-9
>              # Build dependencies
>              - libaio-dev
>              - libattr1-dev
> @@ -270,11 +270,11 @@ matrix:
>        language: generic
>        compiler: none
>        env:
> -        - COMPILER_NAME=gcc CXX=g++-7 CC=gcc-7
> -        - CONFIG="--cc=gcc-7 --cxx=g++-7 --disable-pie --disable-linux-user"
> +        - COMPILER_NAME=gcc CXX=g++-9 CC=gcc-9
> +        - CONFIG="--cc=gcc-9 --cxx=g++-9 --disable-pie --disable-linux-user"
>          - TEST_CMD=""
>        before_script:
> -        - ./configure ${CONFIG} --extra-cflags="-g3 -O0 -fsanitize=thread -fuse-ld=gold" || { cat config.log && exit 1; }
> +        - ./configure ${CONFIG} --extra-cflags="-g3 -O0 -Wno-error=stringop-truncation -fsanitize=thread -fuse-ld=gold" || { cat config.log && exit 1; }
>  

What about describing in the commit message that we are adding
'-Wno-error=stringop-truncation' in the cflags?

Thanks,
Stefano

Re: [Qemu-devel] [PATCH v1 07/26] .travis.yml: bump gcc sanitiser job to gcc-9

Posted by Alex Bennée 6 years, 8 months ago

Stefano Garzarella <sgarzare@redhat.com> writes:

> On Thu, May 30, 2019 at 11:15:44AM +0100, Alex Bennée wrote:
>> The toolchain PPA has it so we might as well use it.
>>
>> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
>> ---
>>  .travis.yml | 10 +++++-----
>>  1 file changed, 5 insertions(+), 5 deletions(-)
>>
>> diff --git a/.travis.yml b/.travis.yml
>> index b053a836a32..f0aa37f2d12 100644
>> --- a/.travis.yml
>> +++ b/.travis.yml
>> @@ -240,8 +240,8 @@ matrix:
>>              - ubuntu-toolchain-r-test
>>            packages:
>>              # Extra toolchains
>> -            - gcc-7
>> -            - g++-7
>> +            - gcc-9
>> +            - g++-9
>>              # Build dependencies
>>              - libaio-dev
>>              - libattr1-dev
>> @@ -270,11 +270,11 @@ matrix:
>>        language: generic
>>        compiler: none
>>        env:
>> -        - COMPILER_NAME=gcc CXX=g++-7 CC=gcc-7
>> -        - CONFIG="--cc=gcc-7 --cxx=g++-7 --disable-pie --disable-linux-user"
>> +        - COMPILER_NAME=gcc CXX=g++-9 CC=gcc-9
>> +        - CONFIG="--cc=gcc-9 --cxx=g++-9 --disable-pie --disable-linux-user"
>>          - TEST_CMD=""
>>        before_script:
>> -        - ./configure ${CONFIG} --extra-cflags="-g3 -O0 -fsanitize=thread -fuse-ld=gold" || { cat config.log && exit 1; }
>> +        - ./configure ${CONFIG} --extra-cflags="-g3 -O0 -Wno-error=stringop-truncation -fsanitize=thread -fuse-ld=gold" || { cat config.log && exit 1; }
>>
>
> What about describing in the commit message that we are adding
> '-Wno-error=stringop-truncation' in the cflags?

I was sure I'd written that commit message, but obviously only in my
head. Will update.

>
> Thanks,
> Stefano


--
Alex Bennée