Hi, John Labenski ha scritto: >>> Fixed now, hopefully, please try again. >> Sorry, but I don't understand this last change to configure.ac... >> >> I've modified the test and tested it under Ubuntu 7.10 with system's >> lua5.1 installed and it seems to work... >> > > The Ubuntu package liblua5.1-0-dev puts the includes in > "/usr/include/lua5.1". My last commit for the bakefiles and > configure.ac was to remove the "/include" from $(LUA_DIR)/include so > you could exactly specify where the includes are. decoupling the include folder from the lib folder is ok :)
> I'm not sure we want to force "$lua_prefix/include/lua5.1" because > we're going to find someone with some system who has it elsewhere and > then we'll have more problems. typically this problem in the configure script I've found in other packages are "resolved" putting a list of paths to check; e.g.: for d in /usr/local/include/lua5.1 /usr/local/include ... ; do # check if $d is a valid folder done > I would have kept this code, the includes can be anywhere, but the > system Lua libs should be in usr/lib or usr/local/lib or somewhere in > the standard lib search path Typically, if the includes are in e.g. xxx/include/lua5.1, then the relative library should be in xxx/lib, i.e. if headers are installed in a certain prefix, then the library should be in that same prefix. > so I set our Lua lib path to the local > one which is a harmless duplication. this is in particular what I do not understand: why creating a LUA_LIB_DIR and then set it to our local lib folder, which is not going to ever contain the lua library if USE_SYSTEM_LUA=1? If we allow for an option called --with-lua-prefix then we must respect the prefix given by the user using it as base for both the include and the lib search folders. Anyway if the current code works, there's no need to further loose time on it ;) Francesco ------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ _______________________________________________ wxlua-users mailing list wxlua-users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/wxlua-users