Libguestfs - Apr 2013 - [PATCH] add --with-supermin-extra-options configure option

Useful to pass --use-installed and other options to supermin.

Signed-off-by: Olaf Hering <olaf at aepfle.de>
---

Not even compile tested!!
Just copy&paste from SUPERMIN_PACKAGER_CONFIG


 appliance/make.sh.in |  3 +++
 configure.ac         | 13 +++++++++++++
 2 files changed, 16 insertions(+)

diff --git a/appliance/make.sh.in b/appliance/make.sh.in
index 42d6b23..8ba1fa4 100755
--- a/appliance/make.sh.in
+++ b/appliance/make.sh.in
@@ -33,6 +33,9 @@ exec 5<&-
 if [ "x at SUPERMIN_PACKAGER_CONFIG@" != "xno" ]; then
     extra="--yum-config @SUPERMIN_PACKAGER_CONFIG@"
 fi
+if [ "x at SUPERMIN_EXTRA_OPTIONS@" != "xno" ]; then
+    extra="$extra @SUPERMIN_EXTRA_OPTIONS@"
+fi
 
 echo @SUPERMIN@ -v -o supermin.d --names $(< packagelist ) $excludes $extra
 @SUPERMIN@ -v -o supermin.d --names $(< packagelist ) $excludes $extra
diff --git a/configure.ac b/configure.ac
index f94a411..87d517b 100644
--- a/configure.ac
+++ b/configure.ac
@@ -473,6 +473,19 @@ AC_ARG_WITH([febootstrap-packager-config],
 
 AC_SUBST([SUPERMIN_PACKAGER_CONFIG])
 
+dnl Pass additional supermin options.
+dnl
+SUPERMIN_EXTRA_OPTIONS=no
+AC_MSG_CHECKING([for --with-supermin-extra-options option])
+AC_ARG_WITH([supermin-extra-options],
+    [AS_HELP_STRING([--with-supermin-extra-options=STRING],
+        [Pass additional supermin options. @<:@default=no@:>@])],
+    [SUPERMIN_EXTRA_OPTIONS="$withval"
+     AC_MSG_RESULT([$SUPERMIN_EXTRA_OPTIONS"])],
+    [AC_MSG_RESULT([not set])])
+
+AC_SUBST([SUPERMIN_EXTRA_OPTIONS])
+
 if test "x$enable_appliance" = "xyes"; then
     test "x$SUPERMIN" = "xno" &&
         AC_MSG_ERROR([supermin (formerly called febootstrap) must be
installed])

On Thu, Apr 11, 2013 at 08:41:44PM +0200, Olaf Hering
wrote:
> Useful to pass --use-installed and other options to supermin.
> 
> Signed-off-by: Olaf Hering <olaf at aepfle.de>
> ---
> 
> Not even compile tested!!
> Just copy&paste from SUPERMIN_PACKAGER_CONFIG
I did test it, and it works.

Pushed, thanks.

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#)