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 <nc...@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


------------------------------------------------------------------------------
Learn the latest--Visual Studio 2012, SharePoint 2013, SQL 2012, more!
Discover the easy way to master current and previous Microsoft technologies
and advance your career. Get an incredible 1,500+ hours of step-by-step
tutorial videos with LearnDevNow. Subscribe today and save!
http://pubads.g.doubleclick.net/gampad/clk?id=58041391&iu=/4140/ostg.clktrk
_______________________________________________
Lxc-devel mailing list
Lxc-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/lxc-devel

Reply via email to