Re: [PATCH] .travis.yml: reduce scope of the --enable-debug build

["Philippe Mathieu-Daudé" <philmd@redhat.com>]

On 10/7/19 9:05 PM, Alex Bennée wrote:
> 
> Peter Maydell <peter.maydell@linaro.org> writes:
> 
>> On Mon, 7 Oct 2019 at 17:22, Alex Bennée <alex.bennee@linaro.org> wrote:
>>>
>>> Adding debug makes things run a bit slower so lets not hammer all the
>>> targets.
>>>
>>> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
>>> ---
>>>   .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.

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>