Re: [libvirt PATCH] rpm: Don't require qemu-img at build time

On Wed, May 10, 2023 at 02:40:38PM +0200, Peter Krempa wrote: > On Wed, May 10, 2023 at 05:28:56 -0700, Andrea Bolognani wrote: > > Peter, is there a reason why we're still calling 'qemu-img create' > > for those specific images? Or could we replace those calls with > > preformatted images as well, and no longer use qemu-img from the test > > suite at all? > > The idea why I kept using freshly formatted images is to validate our > qcow2 header parser (we have our own, which is intentionally kept very > simple to only parse the format and backing image location) against the > current state of qemu, to catch potential regressions when qemu's format > would change. It's a form of integration testing albeit a bit weird in > how it's executed (notably it depeds on the system version of qemu-img > rather than doing anything specific to test against current upstream). Alright, so it makes sense to keep running qemu-img as part of the test suite and thus to have it as a build dependency. I'm just going to update the comment in the spec file and point out this usage explicitly, because it's clearly non-obvious right now.

-- Andrea Bolognani / Red Hat / Virtualization