* [Qemu-devel] [PATCH] .travis.yml: separate tools and docs into another entry
@ 2019-02-01 17:12 Alex Bennée
2019-02-03 15:34 ` no-reply
2019-02-04 12:04 ` Philippe Mathieu-Daudé
0 siblings, 2 replies; 6+ messages in thread
From: Alex Bennée @ 2019-02-01 17:12 UTC (permalink / raw)
To: qemu-devel; +Cc: Alex Bennée
Re-building the tools and documents by default is a little wasteful as
they are not really affected by the main build options. Split tools
and documents into their own task with a minimal softmmu and
linux-user target list just to check they don't interact badly.
Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
---
.travis.yml | 14 ++++++++++++--
1 file changed, 12 insertions(+), 2 deletions(-)
diff --git a/.travis.yml b/.travis.yml
index 87d9fa971c..1c8effcf63 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -61,6 +61,7 @@ env:
global:
- SRC_DIR="."
- BUILD_DIR="."
+ - BASE_CONFIG="--disable-docs --disable-tools"
- TEST_CMD="make check -j3 V=1"
@@ -71,7 +72,7 @@ git:
before_script:
- mkdir -p ${BUILD_DIR} && cd ${BUILD_DIR}
- - ${SRC_DIR}/configure ${CONFIG} || { cat config.log && exit 1; }
+ - ${SRC_DIR}/configure ${BASE_CONFIG} ${CONFIG} || { cat config.log && exit 1; }
script:
- make -j3 && ${TEST_CMD}
@@ -106,6 +107,15 @@ matrix:
- CONFIG="--with-coroutine=sigaltstack --disable-linux-user"
+ # Check we can build docs and tools
+ - env:
+ - BASE_CONFIG=""
+ - CONFIG="--target-list=x86_64-softmmu,aarch64-linux-user"
+ addons:
+ apt:
+ packages:
+ - python-sphinx
+
# Test out-of-tree builds
- env:
- CONFIG="--enable-debug --enable-debug-tcg"
@@ -243,5 +253,5 @@ matrix:
- env:
- - CONFIG="--disable-system --disable-docs"
+ - CONFIG="--disable-system"
- TEST_CMD="make -j3 check-tcg V=1"
--
2.20.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH] .travis.yml: separate tools and docs into another entry
2019-02-01 17:12 [Qemu-devel] [PATCH] .travis.yml: separate tools and docs into another entry Alex Bennée
@ 2019-02-03 15:34 ` no-reply
2019-02-04 8:28 ` Alex Bennée
2019-02-04 12:04 ` Philippe Mathieu-Daudé
1 sibling, 1 reply; 6+ messages in thread
From: no-reply @ 2019-02-03 15:34 UTC (permalink / raw)
To: alex.bennee; +Cc: fam, qemu-devel
Patchew URL: https://patchew.org/QEMU/20190201171229.3361-1-alex.bennee@linaro.org/
Hi,
This series failed the docker-mingw@fedora build test. Please find the testing commands and
their output below. If you have Docker installed, you can probably reproduce it
locally.
=== TEST SCRIPT BEGIN ===
#!/bin/bash
time make docker-test-mingw@fedora SHOW_ENV=1 J=14
=== TEST SCRIPT END ===
Configure options:
--enable-werror --target-list=x86_64-softmmu,aarch64-softmmu --prefix=/tmp/qemu-test/install --python=/usr/bin/python3 --cross-prefix=x86_64-w64-mingw32- --enable-trace-backends=simple --enable-gnutls --enable-nettle --enable-curl --enable-vnc --enable-bzip2 --enable-guest-agent --with-sdlabi=2.0
ERROR: unknown option --with-sdlabi=2.0
Try '/tmp/qemu-test/src/configure --help' for more information
# QEMU configure log Sun Feb 3 15:33:56 UTC 2019
# Configured with: '/tmp/qemu-test/src/configure' '--enable-werror' '--target-list=x86_64-softmmu,aarch64-softmmu' '--prefix=/tmp/qemu-test/install' '--python=/usr/bin/python3' '--cross-prefix=x86_64-w64-mingw32-' '--enable-trace-backends=simple' '--enable-gnutls' '--enable-nettle' '--enable-curl' '--enable-vnc' '--enable-bzip2' '--enable-guest-agent' '--with-sdlabi=2.0'
---
funcs: do_compiler do_cc compile_object check_define main
lines: 92 122 617 634 0
x86_64-w64-mingw32-gcc -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -Wstrict-prototypes -Wredundant-decls -Wall -Wundef -Wwrite-strings -Wmissing-prototypes -fno-strict-aliasing -fno-common -fwrapv -std=gnu99 -c -o config-temp/qemu-conf.o config-temp/qemu-conf.c
config-temp/qemu-conf.c:2:2: error: #error __linux__ not defined
#error __linux__ not defined
^~~~~
---
funcs: do_compiler do_cc compile_object check_define main
lines: 92 122 617 686 0
x86_64-w64-mingw32-gcc -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -Wstrict-prototypes -Wredundant-decls -Wall -Wundef -Wwrite-strings -Wmissing-prototypes -fno-strict-aliasing -fno-common -fwrapv -std=gnu99 -c -o config-temp/qemu-conf.o config-temp/qemu-conf.c
config-temp/qemu-conf.c:2:2: error: #error __i386__ not defined
#error __i386__ not defined
^~~~~
---
funcs: do_compiler do_cc compile_object check_define main
lines: 92 122 617 689 0
x86_64-w64-mingw32-gcc -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -Wstrict-prototypes -Wredundant-decls -Wall -Wundef -Wwrite-strings -Wmissing-prototypes -fno-strict-aliasing -fno-common -fwrapv -std=gnu99 -c -o config-temp/qemu-conf.o config-temp/qemu-conf.c
config-temp/qemu-conf.c:2:2: error: #error __ILP32__ not defined
#error __ILP32__ not defined
^~~~~
---
lines: 92 128 920 0
x86_64-w64-mingw32-gcc -mthreads -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -Wstrict-prototypes -Wredundant-decls -Wall -Wundef -Wwrite-strings -Wmissing-prototypes -fno-strict-aliasing -fno-common -fwrapv -std=gnu99 -o config-temp/qemu-conf.exe config-temp/qemu-conf.c -g -liberty
/usr/lib/gcc/x86_64-w64-mingw32/8.2.0/../../../../x86_64-w64-mingw32/bin/ld: cannot find -liberty
collect2: error: ld returned 1 exit status
Failed to run 'configure'
Traceback (most recent call last):
File "./tests/docker/docker.py", line 563, in <module>
The full log is available at
http://patchew.org/logs/20190201171229.3361-1-alex.bennee@linaro.org/testing.docker-mingw@fedora/?type=message.
---
Email generated automatically by Patchew [http://patchew.org/].
Please send your feedback to patchew-devel@redhat.com
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH] .travis.yml: separate tools and docs into another entry
2019-02-03 15:34 ` no-reply
@ 2019-02-04 8:28 ` Alex Bennée
0 siblings, 0 replies; 6+ messages in thread
From: Alex Bennée @ 2019-02-04 8:28 UTC (permalink / raw)
To: qemu-devel; +Cc: fam
no-reply@patchew.org writes:
> Patchew URL: https://patchew.org/QEMU/20190201171229.3361-1-alex.bennee@linaro.org/
>
>
>
> Hi,
>
> This series failed the docker-mingw@fedora build test. Please find the testing commands and
> their output below. If you have Docker installed, you can probably reproduce it
> locally.
>
> === TEST SCRIPT BEGIN ===
> #!/bin/bash
> time make docker-test-mingw@fedora SHOW_ENV=1 J=14
> === TEST SCRIPT END ===
>
>
> Configure options:
> --enable-werror --target-list=x86_64-softmmu,aarch64-softmmu --prefix=/tmp/qemu-test/install --python=/usr/bin/python3 --cross-prefix=x86_64-w64-mingw32- --enable-trace-backends=simple --enable-gnutls --enable-nettle --enable-curl --enable-vnc --enable-bzip2 --enable-guest-agent --with-sdlabi=2.0
> ERROR: unknown option --with-sdlabi=2.0
> Try '/tmp/qemu-test/src/configure --help' for more information
That looks like patchew is confused.
--
Alex Bennée
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH] .travis.yml: separate tools and docs into another entry
2019-02-01 17:12 [Qemu-devel] [PATCH] .travis.yml: separate tools and docs into another entry Alex Bennée
2019-02-03 15:34 ` no-reply
@ 2019-02-04 12:04 ` Philippe Mathieu-Daudé
2019-02-04 12:59 ` Alex Bennée
1 sibling, 1 reply; 6+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-02-04 12:04 UTC (permalink / raw)
To: Alex Bennée, qemu-devel; +Cc: Fam Zheng
Hi Alex,
On 2/1/19 6:12 PM, Alex Bennée wrote:
> Re-building the tools and documents by default is a little wasteful as
> they are not really affected by the main build options. Split tools
> and documents into their own task with a minimal softmmu and
> linux-user target list just to check they don't interact badly.
>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
> .travis.yml | 14 ++++++++++++--
> 1 file changed, 12 insertions(+), 2 deletions(-)
>
> diff --git a/.travis.yml b/.travis.yml
> index 87d9fa971c..1c8effcf63 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -61,6 +61,7 @@ env:
> global:
> - SRC_DIR="."
> - BUILD_DIR="."
> + - BASE_CONFIG="--disable-docs --disable-tools"
> - TEST_CMD="make check -j3 V=1"
>
>
> @@ -71,7 +72,7 @@ git:
>
> before_script:
> - mkdir -p ${BUILD_DIR} && cd ${BUILD_DIR}
> - - ${SRC_DIR}/configure ${CONFIG} || { cat config.log && exit 1; }
> + - ${SRC_DIR}/configure ${BASE_CONFIG} ${CONFIG} || { cat config.log && exit 1; }
> script:
> - make -j3 && ${TEST_CMD}
>
> @@ -106,6 +107,15 @@ matrix:
> - CONFIG="--with-coroutine=sigaltstack --disable-linux-user"
>
>
> + # Check we can build docs and tools
> + - env:
> + - BASE_CONFIG=""
What about using explicit configure options?
BASE_CONFIG="--enable-docs --enable-tools"
Regardless:
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> + - CONFIG="--target-list=x86_64-softmmu,aarch64-linux-user"
> + addons:
> + apt:
> + packages:
> + - python-sphinx
> +
> # Test out-of-tree builds
> - env:
> - CONFIG="--enable-debug --enable-debug-tcg"
> @@ -243,5 +253,5 @@ matrix:
>
>
> - env:
> - - CONFIG="--disable-system --disable-docs"
> + - CONFIG="--disable-system"
> - TEST_CMD="make -j3 check-tcg V=1"
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH] .travis.yml: separate tools and docs into another entry
2019-02-04 12:04 ` Philippe Mathieu-Daudé
@ 2019-02-04 12:59 ` Alex Bennée
2019-02-04 13:01 ` Philippe Mathieu-Daudé
0 siblings, 1 reply; 6+ messages in thread
From: Alex Bennée @ 2019-02-04 12:59 UTC (permalink / raw)
To: Philippe Mathieu-Daudé; +Cc: qemu-devel, Fam Zheng
Philippe Mathieu-Daudé <philmd@redhat.com> writes:
> Hi Alex,
>
> On 2/1/19 6:12 PM, Alex Bennée wrote:
>> Re-building the tools and documents by default is a little wasteful as
>> they are not really affected by the main build options. Split tools
>> and documents into their own task with a minimal softmmu and
>> linux-user target list just to check they don't interact badly.
>>
>> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
>> ---
>> .travis.yml | 14 ++++++++++++--
>> 1 file changed, 12 insertions(+), 2 deletions(-)
>>
>> diff --git a/.travis.yml b/.travis.yml
>> index 87d9fa971c..1c8effcf63 100644
>> --- a/.travis.yml
>> +++ b/.travis.yml
>> @@ -61,6 +61,7 @@ env:
>> global:
>> - SRC_DIR="."
>> - BUILD_DIR="."
>> + - BASE_CONFIG="--disable-docs --disable-tools"
>> - TEST_CMD="make check -j3 V=1"
>>
>>
>> @@ -71,7 +72,7 @@ git:
>>
>> before_script:
>> - mkdir -p ${BUILD_DIR} && cd ${BUILD_DIR}
>> - - ${SRC_DIR}/configure ${CONFIG} || { cat config.log && exit 1; }
>> + - ${SRC_DIR}/configure ${BASE_CONFIG} ${CONFIG} || { cat config.log && exit 1; }
>> script:
>> - make -j3 && ${TEST_CMD}
>>
>> @@ -106,6 +107,15 @@ matrix:
>> - CONFIG="--with-coroutine=sigaltstack --disable-linux-user"
>>
>>
>> + # Check we can build docs and tools
>> + - env:
>> + - BASE_CONFIG=""
>
> What about using explicit configure options?
>
> BASE_CONFIG="--enable-docs --enable-tools"
Will add for clarity (although not effect ;-)
>
> Regardless:
> Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
>
>> + - CONFIG="--target-list=x86_64-softmmu,aarch64-linux-user"
>> + addons:
>> + apt:
>> + packages:
>> + - python-sphinx
>> +
>> # Test out-of-tree builds
>> - env:
>> - CONFIG="--enable-debug --enable-debug-tcg"
>> @@ -243,5 +253,5 @@ matrix:
>>
>>
>> - env:
>> - - CONFIG="--disable-system --disable-docs"
>> + - CONFIG="--disable-system"
>> - TEST_CMD="make -j3 check-tcg V=1"
>>
--
Alex Bennée
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [Qemu-devel] [PATCH] .travis.yml: separate tools and docs into another entry
2019-02-04 12:59 ` Alex Bennée
@ 2019-02-04 13:01 ` Philippe Mathieu-Daudé
0 siblings, 0 replies; 6+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-02-04 13:01 UTC (permalink / raw)
To: Alex Bennée; +Cc: qemu-devel, Fam Zheng
On 2/4/19 1:59 PM, Alex Bennée wrote:
> Philippe Mathieu-Daudé <philmd@redhat.com> writes:
>> On 2/1/19 6:12 PM, Alex Bennée wrote:
>>> Re-building the tools and documents by default is a little wasteful as
>>> they are not really affected by the main build options. Split tools
>>> and documents into their own task with a minimal softmmu and
>>> linux-user target list just to check they don't interact badly.
>>>
>>> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
>>> ---
>>> .travis.yml | 14 ++++++++++++--
>>> 1 file changed, 12 insertions(+), 2 deletions(-)
>>>
>>> diff --git a/.travis.yml b/.travis.yml
>>> index 87d9fa971c..1c8effcf63 100644
>>> --- a/.travis.yml
>>> +++ b/.travis.yml
>>> @@ -61,6 +61,7 @@ env:
>>> global:
>>> - SRC_DIR="."
>>> - BUILD_DIR="."
>>> + - BASE_CONFIG="--disable-docs --disable-tools"
>>> - TEST_CMD="make check -j3 V=1"
>>>
>>>
>>> @@ -71,7 +72,7 @@ git:
>>>
>>> before_script:
>>> - mkdir -p ${BUILD_DIR} && cd ${BUILD_DIR}
>>> - - ${SRC_DIR}/configure ${CONFIG} || { cat config.log && exit 1; }
>>> + - ${SRC_DIR}/configure ${BASE_CONFIG} ${CONFIG} || { cat config.log && exit 1; }
>>> script:
>>> - make -j3 && ${TEST_CMD}
>>>
>>> @@ -106,6 +107,15 @@ matrix:
>>> - CONFIG="--with-coroutine=sigaltstack --disable-linux-user"
>>>
>>>
>>> + # Check we can build docs and tools
>>> + - env:
>>> + - BASE_CONFIG=""
>>
>> What about using explicit configure options?
>>
>> BASE_CONFIG="--enable-docs --enable-tools"
>
> Will add for clarity (although not effect ;-)
The effect will be in 10 months when I'll look at this again :P
Thanks!
>>
>> Regardless:
>> Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
>>
>>> + - CONFIG="--target-list=x86_64-softmmu,aarch64-linux-user"
>>> + addons:
>>> + apt:
>>> + packages:
>>> + - python-sphinx
>>> +
>>> # Test out-of-tree builds
>>> - env:
>>> - CONFIG="--enable-debug --enable-debug-tcg"
>>> @@ -243,5 +253,5 @@ matrix:
>>>
>>>
>>> - env:
>>> - - CONFIG="--disable-system --disable-docs"
>>> + - CONFIG="--disable-system"
>>> - TEST_CMD="make -j3 check-tcg V=1"
>>>
>
>
> --
> Alex Bennée
>
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-02-04 13:01 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-02-01 17:12 [Qemu-devel] [PATCH] .travis.yml: separate tools and docs into another entry Alex Bennée
2019-02-03 15:34 ` no-reply
2019-02-04 8:28 ` Alex Bennée
2019-02-04 12:04 ` Philippe Mathieu-Daudé
2019-02-04 12:59 ` Alex Bennée
2019-02-04 13:01 ` Philippe Mathieu-Daudé
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).