[Libguestfs] [PATCH] lib: Pick up qemu-img path at build time.

[Pino Toscano]

On Friday, 6 October 2017 15:59:42 CEST Richard W.M. Jones wrote:
> The main purpose of this change is two-fold:
> 
> (1) Ensure that we run the same version of qemu-img that we are
> built against.
> 
> (2) Allow the qemu-img path to be overridden at build time in case
> it's on a nonstandard path or (like RHV) has a nonstandard name.
> ---

As I mentioned in other series, I am not a fan of hardcoding at build
time paths used at runtime.  This patch has different issues:
a) the hardcoding itself (which IMHO is a step backward)
b) what the patch does just cover a very minimal set of all the
   qemu-img invocations; all the tools rely on $PATH
c) it is not overridable at all: since qemu can be changed in different
   ways (API entry, envvar), it is not logic that qemu-img (built from
   the same sources of qemu) cannot be changed at all; this situation
   e.g. makes it not possible to use a custom qemu build (that says
   enables some feature) in libguestfs
d) the point (2) above implies supporting RHV would mean rebuilding
   libguestfs just for that, which makes testing harder than needed

qemu is the "special flower" in this situation, since it can be outside
of $PATH (hi RHEL), with a custom name, etc.  qemu-img does not have any
of the qemu peculiarities.

-- 
Pino Toscano
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: This is a digitally signed message part.
URL: <http://listman.redhat.com/archives/libguestfs/attachments/20171006/7cdf4b56/attachment.sig>