Re: [Qemu-devel] [PATCH] .travis.yml: reduce number of targets built while disabling things

[Stefano Garzarella]

On Thu, Mar 21, 2019 at 03:57:43PM +0000, Alex Bennée wrote:
> 
> Stefano Garzarella <address@hidden> writes:
> 
> > Hi Alex,
> > thanks also for the other patch series that I'm testing right now to speed
> > up Travis :)
> >
> > Just a comment below:
> >
> > On Thu, Mar 21, 2019 at 12:48:57PM +0000, Alex Bennée wrote:
> >> This build keeps timing out on Travis and it's unlikely including the
> >> additional guest front-ends will catch any failures in the fallback
> >> code.
> >>
> >> Signed-off-by: Alex Bennée <address@hidden>
> >> ---
> >>  .travis.yml | 2 +-
> >>  1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/.travis.yml b/.travis.yml
> >> index 3fb3dab46e..2e06aee9d0 100644
> >> --- a/.travis.yml
> >> +++ b/.travis.yml
> >> @@ -107,7 +107,7 @@ matrix:
> >>
> >>
> >>      - env:
> >> -        - CONFIG="--disable-linux-aio --disable-cap-ng --disable-attr 
> >> --disable-brlapi --disable-libusb --disable-user --disable-replication"
> >> +        - CONFIG="--disable-linux-aio --disable-cap-ng --disable-attr 
> >> --disable-brlapi --disable-libusb --disable-replication 
> >> --target-list=${MAIN_SOFTMMU_TARGETS}"
> >>
> >
> > Is removing --disable-user intentional here?
> 
> Yes - --disable-user just removes all user mode targets from the default
> target list. However as we are now explicitly listing the softmmu
> targets it is redundant.
> 

Clear! Sorry for the stupid question!

Acked-by: Stefano Garzarella <address@hidden>

Thanks,
Stefano