Re: [Wxlua-users] Bakefiles and MSVS dsp files

[John L. <jla...@gm...>]

On 11/26/05, Francesco Montorsi <f18...@ya...> 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 t=
he environment
> variable WXSTEDIT should not be used to find the headers.
> On unix it's common to install the dependencies in the default system pat=
hs; so I think
> that wxLua should expect to find wxStEdit headers & lib into system paths=
, too.
> Thus I would remove the lines
>
>          <if cond=3D"FORMAT=3D=3D'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=3D"FORMAT!=3D'autoconf'">
>
> to
>
> <if cond=3D"TARGETING_WIN32=3D=3D'1'">
>
> as there is not only the autoconf format for linux (also cbx_unix, gnu, e=
tc).

Ok, thanks.

> > 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 bakefi=
les (and
> mod_lua_lib depends from mod_lua in bakefiles); thus I added the dependen=
cy manually.
> Now also MSVC6prj will know that mod_lua_lib needs to be compiled before =
mod_lua.

I didn't realize that was by hand, then it should be easy. I'll do it.

> BTW, I see that luasetup.h.in has not been removed yet and that wxLua use=
s 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 t=
hey're currently
> unused.

Tomorrow night, I'm off for today...

> PS: I've updated linux configure script so that now all wxLua options are=
 present there,
> too (--enable-wxlua-app, --enable-wxluaedit-app, etc)

Great.

Thanks alot,
    John