[lxc-devel] [PATCH] lua: fix logic to enable lua support in configure

[Natanael Copa]

When there is no --enable-lua or --with-lua-pc, Lua should not be
enabled.

This fixes a bug introduced with 12e93188 (configure/makefile:
Allow specify Lua pkg-config file with --with-lua-pc) that caused
configure script to fail if lua headers was missing.

Signed-off-by: Natanael Copa <ncopa at alpinelinux.org>
---
PS.

I wonder it it would be an idea to replace the 2 switches:

  --enable-lua
  --with-lua-pc[=PKG]

with a single switch:

  --with-lua[=PKG]

As it works now, if you specify either --enable-lua or --with-lua-pc,
lua will be enabled and configure will fail if Lua headers is missing.
Even '--disable-lua --with-lua-pc' will actually enable Lua.

In any case, this patch fixes the current problem.

 configure.ac | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/configure.ac b/configure.ac
index 0909da3..481bf06 100644
--- a/configure.ac
+++ b/configure.ac
@@ -182,13 +182,13 @@ AC_ARG_WITH([lua-pc],
 		[Specify pkg-config package name for lua]
 	)],
 	[LUAPKGCONFIG=$with_lua_pc
-	enable_lua=yes], [with_lua_pc=auto])
+	enable_lua=yes])
 
 AM_CONDITIONAL([ENABLE_LUA],
-	[test "x$enable_lua" = "xyes" -o "xwith_lua_pc != xno"])
+	[test "x$enable_lua" = "xyes"])
 
 AM_COND_IF([ENABLE_LUA],
-	[AS_IF([test "x$with_lua_pc" = "xauto"],
+	[AS_IF([test "x$with_lua_pc" = "xyes"],
 		[PKG_CHECK_MODULES([LUA], [lua],[LUAPKGCONFIG=lua],
 			[PKG_CHECK_MODULES([LUA], [lua5.2],[LUAPKGCONFIG=lua5.2],
 				[PKG_CHECK_MODULES([LUA], [lua5.1],
-- 
1.8.4