Re: [Xen-devel] [PATCH] bug fix in configure file

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Mon, 2013-02-11 at 13:22 +0000, Shakeel Butt wrote:
> > Please can you describe what goes wrong with it where it is?
> Executing "./configure --enable-stubdom" doesn't call
> stubdom/configure and I think the following patch will fix it
> correctly.

I rewrote your commit message as followed and checked this in. Please
try and supply a usable commit message with a meaningful subject in the
future.

>From f478258a4d3b5b9f27b265af8014617c53b83368 Mon Sep 17 00:00:00 2001
From: Shakeel Butt <shakeel.butt@xxxxxxxxx>
Date: Mon, 11 Feb 2013 08:22:48 -0500
Subject: [PATCH] configure: configure a subsystem when explicitly requested.

Executing "./configure --enable-stubdom" doesn't call stubdom/configure, fix
this.

Signed-off-by: Shakeel Butt <shakeel.butt@xxxxxxxxx>
Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
---
 configure       |    6 +++---
 m4/subsystem.m4 |    2 +-
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/configure b/configure
index 69803a7..06c0e99 100755
--- a/configure
+++ b/configure
@@ -2000,6 +2000,9 @@ stubdom=n
 
 fi
 
+fi
+
+
 if test -e "stubdom/configure"; then :
 
 if test "x$stubdom" = "xy" || test "x$stubdom" = "x" ; then
@@ -2011,9 +2014,6 @@ fi
 
 
 
-fi
-
-
 
 # Check whether --enable-docs was given.
 if test "${enable_docs+set}" = set; then :
diff --git a/m4/subsystem.m4 b/m4/subsystem.m4
index 9e1e61f..5e2eb58 100644
--- a/m4/subsystem.m4
+++ b/m4/subsystem.m4
@@ -39,10 +39,10 @@ AX_ENABLE_SUBSYSTEM([$1])
 ],[
 AX_DISABLE_SUBSYSTEM([$1])
 ])
+])
 AX_SUBSYSTEM_CONFIGURE([$1])
 AC_SUBST($1)
 ])
-])
 
 AC_DEFUN([AX_SUBSYSTEM_FINISH], [
 AC_SUBST(SUBSYSTEMS)
-- 
1.7.2.5




_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel