[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/3] travis: install more library dependencies
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH 1/3] travis: install more library dependencies |
Date: |
Thu, 29 Jun 2017 15:37:08 +0100 |
On 14 June 2017 at 17:49, Paolo Bonzini <address@hidden> wrote:
> This patch and patch 2 are
> useful, but I think I'd rather get full coverage, either with Shippable
> or by keeping on doing manual builds, than to rush things and switch to
> CI when it's not ready.
I agree with this, so is there a path forwards where we
get patch 2 into a shape where we can commit it and you
switch to using it for the manual uploads, (and maybe
eventually we have Shippable do it, but at least we have
a reproducible way for somebody other than you to do it
if needed) ?
I think for that we should update the comments at the top
to delete the bit that says "primary purpose is for travis".
I think we should also make the configure line include
--enable-foo for all the features we currently test, so that
if somebody tries to run it on a machine with missing
dependencies they get a configure error rather than silently
uploading with reduced coverage.
Can you suggest a set of --enable-foo flags which give us
the coverage we're getting currently with the machine you
do the manual runs on?
thanks
-- PMM
- [Qemu-devel] [PATCH 0/3] Automate coverity scan uploads via Travis, Peter Maydell, 2017/06/13
- [Qemu-devel] [PATCH 1/3] travis: install more library dependencies, Peter Maydell, 2017/06/13
- Re: [Qemu-devel] [PATCH 1/3] travis: install more library dependencies, Philippe Mathieu-Daudé, 2017/06/13
- Re: [Qemu-devel] [PATCH 1/3] travis: install more library dependencies, Paolo Bonzini, 2017/06/14
- Re: [Qemu-devel] [PATCH 1/3] travis: install more library dependencies, Philippe Mathieu-Daudé, 2017/06/14
- Re: [Qemu-devel] [PATCH 1/3] travis: install more library dependencies, Paolo Bonzini, 2017/06/14
- Re: [Qemu-devel] [PATCH 1/3] travis: install more library dependencies, Peter Maydell, 2017/06/14
- Re: [Qemu-devel] [PATCH 1/3] travis: install more library dependencies, Paolo Bonzini, 2017/06/14
- Re: [Qemu-devel] [PATCH 1/3] travis: install more library dependencies,
Peter Maydell <=
Re: [Qemu-devel] [PATCH 1/3] travis: install more library dependencies, Alex Bennée, 2017/06/14
Re: [Qemu-devel] [PATCH 1/3] travis: install more library dependencies, Daniel P. Berrange, 2017/06/14
[Qemu-devel] [PATCH 3/3] travis: Add config to do a Coverity Scan upload, Peter Maydell, 2017/06/13
[Qemu-devel] [PATCH 2/3] scripts/run-coverity-scan: Script to run Coverity Scan build, Peter Maydell, 2017/06/13