[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] .travis.yml: reduce scope of the --enable-debug build
From: |
Alex Bennée |
Subject: |
Re: [PATCH] .travis.yml: reduce scope of the --enable-debug build |
Date: |
Mon, 07 Oct 2019 20:05:10 +0100 |
User-agent: |
mu4e 1.3.5; emacs 27.0.50 |
Peter Maydell <address@hidden> writes:
> On Mon, 7 Oct 2019 at 17:22, Alex Bennée <address@hidden> wrote:
>>
>> Adding debug makes things run a bit slower so lets not hammer all the
>> targets.
>>
>> Signed-off-by: Alex Bennée <address@hidden>
>> ---
>> .travis.yml | 5 +++--
>> 1 file changed, 3 insertions(+), 2 deletions(-)
>>
>> diff --git a/.travis.yml b/.travis.yml
>> index d0b9e099b9..fc0888aff5 100644
>> --- a/.travis.yml
>> +++ b/.travis.yml
>> @@ -124,12 +124,13 @@ matrix:
>> - CACHE_NAME="${TRAVIS_BRANCH}-linux-gcc-default"
>>
>>
>> + # --enable-debug implies --enable-debug-tcg, also runs quite a bit
>> slower
>> - env:
>> - - CONFIG="--enable-debug --enable-debug-tcg --disable-user"
>> + - CONFIG="--enable-debug --enable-debug-tcg
>> --target-list=${MAIN_SOFTMMU_TARGETS}"
>> - CACHE_NAME="${TRAVIS_BRANCH}-linux-gcc-debug"
>
> If (as noted in the comment) --enable-debug implies
> --enable-debug-tcg, why do we need to give both options ?
We don't really - I was just being verbose. I'll drop it.
>
> thanks
> -- PMM
--
Alex Bennée