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