| Message ID | 20180313153210.28264-5-alex.bennee@linaro.org |
|---|---|
| State | New |
| Headers | show |
| Series | Travis Build Speed-ups | expand |
On Tue, Mar 13, 2018 at 03:32:10PM +0000, Alex Bennée wrote: > The tools are already built elsewhere so we might as well not build it > here as we just need to test the modules build of QEMU itself. > > Signed-off-by: Alex Bennée <alex.bennee@linaro.org> > --- > .travis.yml | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/.travis.yml b/.travis.yml > index c1e99237b2..373181c109 100644 > --- a/.travis.yml > +++ b/.travis.yml > @@ -53,7 +53,7 @@ env: > - CONFIG="--disable-user" > - CONFIG="--enable-debug --enable-debug-tcg" > - CONFIG="--disable-linux-aio --disable-cap-ng --disable-attr --disable-brlapi --disable-uuid --disable-libusb --disable-user" > - - CONFIG="--enable-modules --disable-linux-user" > + - CONFIG="--enable-modules --disable-tools --disable-linux-user" > - CONFIG="--with-coroutine=ucontext --disable-linux-user" > - CONFIG="--with-coroutine=sigaltstack --disable-linux-user" > git: Modules affects the block drivers in particular, which could in theory affect the qemu-img, qemu-io and qemu-nbd tools, so possibly worth keeping that enabled.... but only if we actually run those tools during the job otherwise we're not likely to catch any problems ? Regards, Daniel
Daniel P. Berrangé <berrange@redhat.com> writes: > On Tue, Mar 13, 2018 at 03:32:10PM +0000, Alex Bennée wrote: >> The tools are already built elsewhere so we might as well not build it >> here as we just need to test the modules build of QEMU itself. >> >> Signed-off-by: Alex Bennée <alex.bennee@linaro.org> >> --- >> .travis.yml | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/.travis.yml b/.travis.yml >> index c1e99237b2..373181c109 100644 >> --- a/.travis.yml >> +++ b/.travis.yml >> @@ -53,7 +53,7 @@ env: >> - CONFIG="--disable-user" >> - CONFIG="--enable-debug --enable-debug-tcg" >> - CONFIG="--disable-linux-aio --disable-cap-ng --disable-attr --disable-brlapi --disable-uuid --disable-libusb --disable-user" >> - - CONFIG="--enable-modules --disable-linux-user" >> + - CONFIG="--enable-modules --disable-tools --disable-linux-user" >> - CONFIG="--with-coroutine=ucontext --disable-linux-user" >> - CONFIG="--with-coroutine=sigaltstack --disable-linux-user" >> git: > > Modules affects the block drivers in particular, which could in theory > affect the qemu-img, qemu-io and qemu-nbd tools, so possibly worth > keeping that enabled.... but only if we actually run those tools > during the job otherwise we're not likely to catch any problems ? OK, I've dropped this for the pull-req as it shouldn't affect our timeouts anyway. > > > Regards, > Daniel -- Alex Bennée
diff --git a/.travis.yml b/.travis.yml index c1e99237b2..373181c109 100644 --- a/.travis.yml +++ b/.travis.yml @@ -53,7 +53,7 @@ env: - CONFIG="--disable-user" - CONFIG="--enable-debug --enable-debug-tcg" - CONFIG="--disable-linux-aio --disable-cap-ng --disable-attr --disable-brlapi --disable-uuid --disable-libusb --disable-user" - - CONFIG="--enable-modules --disable-linux-user" + - CONFIG="--enable-modules --disable-tools --disable-linux-user" - CONFIG="--with-coroutine=ucontext --disable-linux-user" - CONFIG="--with-coroutine=sigaltstack --disable-linux-user" git:
The tools are already built elsewhere so we might as well not build it here as we just need to test the modules build of QEMU itself. Signed-off-by: Alex Bennée <alex.bennee@linaro.org> --- .travis.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)