Re: [Qemu-devel] [PATCH] docker: fedora: test more components

2018-03-16 Thread Paolo Bonzini
On 16/03/2018 10:43, Daniel P. Berrangé wrote: >> I wonder if we should simply do "dnf build-dep qemu" and only list additional >> (mingw, for example) packages explicitly, like the debian dockerfiles. > From an upstream POV it feels wrong to have our builds depend on the state > of stuff listed

Re: [Qemu-devel] [PATCH] docker: fedora: test more components

2018-03-16 Thread Daniel P . Berrangé
On Fri, Mar 16, 2018 at 10:40:23AM +0800, Fam Zheng wrote: > On Thu, 03/15 14:47, Daniel P. Berrangé wrote: > > On Tue, Mar 13, 2018 at 01:05:52PM +0100, Paolo Bonzini wrote: > > > Install optional dependencies of QEMU to get better coverage. > > > > > > Signed-off-by: Paolo Bonzini

Re: [Qemu-devel] [PATCH] docker: fedora: test more components

2018-03-15 Thread Fam Zheng
On Thu, 03/15 14:47, Daniel P. Berrangé wrote: > On Tue, Mar 13, 2018 at 01:05:52PM +0100, Paolo Bonzini wrote: > > Install optional dependencies of QEMU to get better coverage. > > > > Signed-off-by: Paolo Bonzini > > --- > > tests/docker/dockerfiles/fedora.docker | 13

Re: [Qemu-devel] [PATCH] docker: fedora: test more components

2018-03-15 Thread Daniel P . Berrangé
On Tue, Mar 13, 2018 at 01:05:52PM +0100, Paolo Bonzini wrote: > Install optional dependencies of QEMU to get better coverage. > > Signed-off-by: Paolo Bonzini > --- > tests/docker/dockerfiles/fedora.docker | 13 ++--- > 1 file changed, 10 insertions(+), 3

Re: [Qemu-devel] [PATCH] docker: fedora: test more components

2018-03-15 Thread Philippe Mathieu-Daudé
On 03/13/2018 01:33 PM, Fam Zheng wrote: > On Tue, 03/13 13:05, Paolo Bonzini wrote: >> Install optional dependencies of QEMU to get better coverage. >> >> Signed-off-by: Paolo Bonzini >> --- >> tests/docker/dockerfiles/fedora.docker | 13 ++--- >> 1 file changed, 10

Re: [Qemu-devel] [PATCH] docker: fedora: test more components

2018-03-13 Thread Fam Zheng
On Tue, 03/13 13:05, Paolo Bonzini wrote: > Install optional dependencies of QEMU to get better coverage. > > Signed-off-by: Paolo Bonzini > --- > tests/docker/dockerfiles/fedora.docker | 13 ++--- > 1 file changed, 10 insertions(+), 3 deletions(-) > > diff --git

[Qemu-devel] [PATCH] docker: fedora: test more components

2018-03-13 Thread Paolo Bonzini
Install optional dependencies of QEMU to get better coverage. Signed-off-by: Paolo Bonzini --- tests/docker/dockerfiles/fedora.docker | 13 ++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/tests/docker/dockerfiles/fedora.docker