On 14 June 2017 at 17:49, Paolo Bonzini <pbonz...@redhat.com> 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