From: Francesco M. <f18...@ya...> - 2005-11-26 14:45:02
|
Hi, John Labenski wrote: >Maybe you saw that I went through the bakefiles > and tried to add the wxluaedit app. I stuck in some more pronounced > comments to help me find my way. I saw, nice work. However for app_wxluaedit target I think that on unix the environment variable WXSTEDIT should not be used to find the headers. On unix it's common to install the dependencies in the default system paths; so I think that wxLua should expect to find wxStEdit headers & lib into system paths, too. Thus I would remove the lines <if cond="FORMAT=='autoconf'"> <include>$(DOLLAR)(WXSTEDIT)/include</include> </if> and put instead a simple check for a wxStEdit include in the configure.ac script. Also I suggest to change line: <if cond="FORMAT!='autoconf'"> to <if cond="TARGETING_WIN32=='1'"> as there is not only the autoconf format for linux (also cbx_unix, gnu, etc). > > Compiles using MSVC6 project files and linux out of the box! > > I will see if I can reorder mod_lua_lib to get compiled before > mod_lua. Take in count that wxLua\build\msw\wxLua.dsw was hand-written (by me) and not generated by bakefile (since msvc6prj doesn't support <subproject> targets). So, it doesn't contain all dependency "links" which are coded into bakefiles (and mod_lua_lib depends from mod_lua in bakefiles); thus I added the dependency manually. Now also MSVC6prj will know that mod_lua_lib needs to be compiled before mod_lua. BTW, I see that luasetup.h.in has not been removed yet and that wxLua uses it instead of the new wxluasetup.h... let me know when you've done with this so that I can make the WXLUASETUP_DIR and WXLUABINDLIB_DIR options work. I have already declared them in modules\build\bakefiles\options.bkl but they're currently unused. Francesco PS: I've updated linux configure script so that now all wxLua options are present there, too (--enable-wxlua-app, --enable-wxluaedit-app, etc) |