[Libguestfs] [PATCH] Change fallback name for external supermin helper
Richard W.M. Jones
rjones at redhat.com
Fri May 31 09:49:50 UTC 2013
On Fri, May 31, 2013 at 10:41:47AM +0200, Olaf Hering wrote:
> Use basename of external helper instead of 'no' for external supermin
> helpers. This gives a clear error messages what binary is actually
> missing, and it is now possible to install the missing package without
> recompiling libguestfs. In addition its now also possible to use private
> builds of supermin (if they are in PATH) with a given libguestfs binary
> package.
>
> Signed-off-by: Olaf Hering <olaf at aepfle.de>
> ---
> configure.ac | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index 30af871..8f025d0 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -442,9 +442,9 @@ AC_SUBST([ENABLE_APPLIANCE])
>
> dnl Check for supermin >= 4.1.0 or febootstrap >= 3.20.
> AC_CHECK_PROGS([SUPERMIN],
> - [supermin febootstrap],[no])
> + [supermin febootstrap],[supermin])
Currently later code in configure.ac checks if "x$SUPERMIN" = "xno"
and gives an error. That error would be broken/ineffective after this
change. Also it does a version check on $SUPERMIN to make sure it's
not the ancient febootstrap 2.x which doesn't work.
> AC_CHECK_PROGS([SUPERMIN_HELPER],
> - [supermin-helper febootstrap-supermin-helper],[no])
> + [supermin-helper febootstrap-supermin-helper],[supermin-helper])
This part of the change OTOH seems OK.
Rich.
--
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Read my programming blog: http://rwmj.wordpress.com
Fedora now supports 80 OCaml packages (the OPEN alternative to F#)
More information about the Libguestfs
mailing list