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

[Philippe Mathieu-Daudé]

On 10/7/19 9:05 PM, Alex Bennée wrote:
> 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.

Reviewed-by: Philippe Mathieu-Daudé <address@hidden>