* [PATCH] travis.yml: Remove the redundant clang-with-MAIN_SOFTMMU_TARGETS entry
@ 2019-11-19 9:21 Thomas Huth
2019-11-19 13:37 ` Wainer dos Santos Moschetta
2019-12-18 19:55 ` Alex Bennée
0 siblings, 2 replies; 3+ messages in thread
From: Thomas Huth @ 2019-11-19 9:21 UTC (permalink / raw)
To: qemu-devel, Alex Bennée; +Cc: Fam Zheng, Philippe Mathieu-Daudé
We test clang with the MAIN_SOFTMMU_TARGETS twice, once without
sanitizers and once with sanitizers enabled. That's somewhat redundant
since if compilation and tests succeeded with sanitizers enabled, it
should also work fine without sanitizers. Thus remove the clang entry
without sanitizers to speed up the CI testing a little bit.
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
.travis.yml | 6 ------
1 file changed, 6 deletions(-)
diff --git a/.travis.yml b/.travis.yml
index b9a026c8ee..47875bdafe 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -181,12 +181,6 @@ matrix:
compiler: clang
- - env:
- - CONFIG="--disable-user --target-list=${MAIN_SOFTMMU_TARGETS}"
- - CACHE_NAME="${TRAVIS_BRANCH}-linux-clang-default"
- compiler: clang
-
-
- env:
- CONFIG="--target-list=${MAIN_SOFTMMU_TARGETS} "
- CACHE_NAME="${TRAVIS_BRANCH}-linux-clang-sanitize"
--
2.23.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] travis.yml: Remove the redundant clang-with-MAIN_SOFTMMU_TARGETS entry
2019-11-19 9:21 [PATCH] travis.yml: Remove the redundant clang-with-MAIN_SOFTMMU_TARGETS entry Thomas Huth
@ 2019-11-19 13:37 ` Wainer dos Santos Moschetta
2019-12-18 19:55 ` Alex Bennée
1 sibling, 0 replies; 3+ messages in thread
From: Wainer dos Santos Moschetta @ 2019-11-19 13:37 UTC (permalink / raw)
To: Thomas Huth, qemu-devel, Alex Bennée
Cc: Fam Zheng, Philippe Mathieu-Daudé
On 11/19/19 7:21 AM, Thomas Huth wrote:
> We test clang with the MAIN_SOFTMMU_TARGETS twice, once without
> sanitizers and once with sanitizers enabled. That's somewhat redundant
> since if compilation and tests succeeded with sanitizers enabled, it
> should also work fine without sanitizers. Thus remove the clang entry
> without sanitizers to speed up the CI testing a little bit.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
> .travis.yml | 6 ------
> 1 file changed, 6 deletions(-)
Reviewed-by: Wainer dos Santos Moschetta <wainersm@redhat.com>
>
> diff --git a/.travis.yml b/.travis.yml
> index b9a026c8ee..47875bdafe 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -181,12 +181,6 @@ matrix:
> compiler: clang
>
>
> - - env:
> - - CONFIG="--disable-user --target-list=${MAIN_SOFTMMU_TARGETS}"
> - - CACHE_NAME="${TRAVIS_BRANCH}-linux-clang-default"
> - compiler: clang
> -
> -
> - env:
> - CONFIG="--target-list=${MAIN_SOFTMMU_TARGETS} "
> - CACHE_NAME="${TRAVIS_BRANCH}-linux-clang-sanitize"
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] travis.yml: Remove the redundant clang-with-MAIN_SOFTMMU_TARGETS entry
2019-11-19 9:21 [PATCH] travis.yml: Remove the redundant clang-with-MAIN_SOFTMMU_TARGETS entry Thomas Huth
2019-11-19 13:37 ` Wainer dos Santos Moschetta
@ 2019-12-18 19:55 ` Alex Bennée
1 sibling, 0 replies; 3+ messages in thread
From: Alex Bennée @ 2019-12-18 19:55 UTC (permalink / raw)
To: Thomas Huth; +Cc: Fam Zheng, Philippe Mathieu-Daudé, qemu-devel
Thomas Huth <thuth@redhat.com> writes:
> We test clang with the MAIN_SOFTMMU_TARGETS twice, once without
> sanitizers and once with sanitizers enabled. That's somewhat redundant
> since if compilation and tests succeeded with sanitizers enabled, it
> should also work fine without sanitizers. Thus remove the clang entry
> without sanitizers to speed up the CI testing a little bit.
Queued to pullreq/testing-logging-misc, thanks.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
> .travis.yml | 6 ------
> 1 file changed, 6 deletions(-)
>
> diff --git a/.travis.yml b/.travis.yml
> index b9a026c8ee..47875bdafe 100644
> --- a/.travis.yml
> +++ b/.travis.yml
> @@ -181,12 +181,6 @@ matrix:
> compiler: clang
>
>
> - - env:
> - - CONFIG="--disable-user --target-list=${MAIN_SOFTMMU_TARGETS}"
> - - CACHE_NAME="${TRAVIS_BRANCH}-linux-clang-default"
> - compiler: clang
> -
> -
> - env:
> - CONFIG="--target-list=${MAIN_SOFTMMU_TARGETS} "
> - CACHE_NAME="${TRAVIS_BRANCH}-linux-clang-sanitize"
--
Alex Bennée
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-12-18 19:56 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-11-19 9:21 [PATCH] travis.yml: Remove the redundant clang-with-MAIN_SOFTMMU_TARGETS entry Thomas Huth
2019-11-19 13:37 ` Wainer dos Santos Moschetta
2019-12-18 19:55 ` Alex Bennée
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).