wxlua-users

[wxlua-users] building 2.8.0.0 with wxstedit-1.2.4

[Doug C. <dou...@gm...>]

In the midst of building 2.8.0.0 with wxstedit-1.2.4 with mingw and
msys on WinXP using configure...

When making wxstedit-1.2.4 I ran into the old problem of
wxstedit-1.2.4\samples\stedit\wxstedit.rc  having paths such as
"../../art/pencil.ico" that I must modify to "../art/pencil.ico" to
make the build proceed.

Also, in setup.h if I use:

#define STE_USE_LANG_VB          0 // 8

I get:

./bk-deps g++ -c -o stedit_lib_stelangs.o  -I../include -I/contrib/include -I/usr/local/lib/wx/include/msw-unicode-release-static-2.8 -I/usr/local/include/wx-2.8 -D__WXMSW__ -mthreads  -g0 -O2 .././src/stelangs.cpp
.././src/stelangs.cpp:4475: error: `STE_WordList1_STE_LANG_VB' was not declared in this scope
make: *** [stedit_lib_stelangs.o] Error 1

Also, the Makefile built by configure doesn't have any include path
for windres for wx, so I had to add:

--include-dir /usr/local/include/wx-2.8

at the end of the WINDRES line in the wxstedit_wxstedit_rc.o target

Regards,

e

-- 
Doug Currie
Londonderry, NH, USA

Re: [wxlua-users] building 2.8.0.0 with wxstedit-1.2.4

[Francesco M. <f18...@ya...>]

Doug Currie ha scritto:
> In the midst of building 2.8.0.0 with wxstedit-1.2.4 with mingw and
> msys on WinXP using configure...
> 
> When making wxstedit-1.2.4 I ran into the old problem of
> wxstedit-1.2.4\samples\stedit\wxstedit.rc  having paths such as
> "../../art/pencil.ico" that I must modify to "../art/pencil.ico" to
> make the build proceed.
I don't understand why that's required: wxstedit.rc indeed needs to use 
../.. to reference to the "art" directory... I don't understand why with 
msys you'd need to use only a single ".."


> Also, in setup.h if I use:
> 
> #define STE_USE_LANG_VB          0 // 8
> 
> I get:
> 
> ./bk-deps g++ -c -o stedit_lib_stelangs.o  -I../include -I/contrib/include -I/usr/local/lib/wx/include/msw-unicode-release-static-2.8 -I/usr/local/include/wx-2.8 -D__WXMSW__ -mthreads  -g0 -O2 .././src/stelangs.cpp
> .././src/stelangs.cpp:4475: error: `STE_WordList1_STE_LANG_VB' was not declared in this scope
> make: *** [stedit_lib_stelangs.o] Error 1
I'm not sure that those STE_USE_LANG* settings are meant to be changed 
by the user to customize the build; John surely knows it better.


> Also, the Makefile built by configure doesn't have any include path
> for windres for wx, so I had to add:
> 
> --include-dir /usr/local/include/wx-2.8
> 
> at the end of the WINDRES line in the wxstedit_wxstedit_rc.o target
that's a known "bug" of wxWidgets 2.6.x wxpresets - it has been fixed in 
wxCVS and the fix should be in wxWidgets 2.8.x wxpresets.

Thus, just rebaking with newer presets will erase this bug ;)

Thanks for your report,
Francesco

Re: [wxlua-users] building 2.8.0.0 with wxstedit-1.2.4

[Doug C. <dou...@gm...>]

On Tuesday, December 26, 2006 Francesco Montorsi wrote: 

> Doug Currie ha scritto:
>> In the midst of building 2.8.0.0 with wxstedit-1.2.4 with mingw and
>> msys on WinXP using configure...
>> 
>> When making wxstedit-1.2.4 I ran into the old problem of
>> wxstedit-1.2.4\samples\stedit\wxstedit.rc  having paths such as
>> "../../art/pencil.ico" that I must modify to "../art/pencil.ico" to
>> make the build proceed.
> I don't understand why that's required: wxstedit.rc indeed needs to use
> ../.. to reference to the "art" directory... I don't understand why with
> msys you'd need to use only a single ".."

The makefile is in a directory at the top level of the distribution;
i.e., ../configure produces the Makefile line

srcdir = ..

and the make target is:

wxstedit_wxstedit_rc.o: $(srcdir)/./samples/stedit/wxstedit.rc

So, we are compiling

.././samples/stedit/wxstedit.rc

Now, pencil.ico is in

.././art/pencil.ico

which is

../art/pencil.ico

e

-- 
Doug Currie
Londonderry, NH, USA

Re: [wxlua-users] building 2.8.0.0 with wxstedit-1.2.4

[Francesco M. <f18...@ya...>]

Doug Currie ha scritto:
> On Tuesday, December 26, 2006 Francesco Montorsi wrote: 
> 
>> Doug Currie ha scritto:
>>> In the midst of building 2.8.0.0 with wxstedit-1.2.4 with mingw and
>>> msys on WinXP using configure...
>>>
>>> When making wxstedit-1.2.4 I ran into the old problem of
>>> wxstedit-1.2.4\samples\stedit\wxstedit.rc  having paths such as
>>> "../../art/pencil.ico" that I must modify to "../art/pencil.ico" to
>>> make the build proceed.
>> I don't understand why that's required: wxstedit.rc indeed needs to use
>> ../.. to reference to the "art" directory... I don't understand why with
>> msys you'd need to use only a single ".."
> 
> The makefile is in a directory at the top level of the distribution;
> i.e., ../configure produces the Makefile line
I don't remember right now if wxCode bakefiles/autoconf stuff fully support the

    mkdir mybuild && cd mybuild && ../configure

trick. Calling configure in the root dir (./configure) doesn't solve the problem?


> srcdir = ..
> 
> and the make target is:
> 
> wxstedit_wxstedit_rc.o: $(srcdir)/./samples/stedit/wxstedit.rc
> 
> So, we are compiling
> 
> ..././samples/stedit/wxstedit.rc
I'd guess you meant .././samples/stedit/wxstedit.rc here

> 
> Now, pencil.ico is in
> 
> ..././art/pencil.ico
> 
> which is
> 
> .../art/pencil.ico
ok, thanks for the explanation.
To fix your problem we'd need to have something like

aaaaaaaa ICON "$(srcdir)/art/pencil.ico"

which is not allowed by other win32 compilers (and wouldn't work even with 
MSYS). We could complicate the build system adding a

#ifndef SRCDIR
     #define SRCDIR ../..
#endif

aaaaaaaa ICON SRCDIR/art/pencil.ico

and then define SRCDIR to be $(srcdir) for MSYS but I don't really know if other 
win32 compilers would like it.


Francesco

Re: [wxlua-users] building 2.8.0.0 with wxstedit-1.2.4

[Doug C. <dou...@gm...>]

On Tuesday, December 26, 2006 Francesco Montorsi wrote: 

> I don't remember right now if wxCode bakefiles/autoconf stuff fully support the

>     mkdir mybuild && cd mybuild && ../configure

> trick.

It seems to work well for wxWidgets and wxLua except for this one
file.

> Calling configure in the root dir (./configure) doesn't solve the problem?

I don't think it would; in that case pencil.ico would be in ./art/pencil.ico

> and then define SRCDIR to be $(srcdir) for MSYS but I don't really know if other
> win32 compilers would like it.

Maybe the Makefile should cd into the $(srcdir)/./samples/stedit
directory and execute $(WINDRES) there. This might make it more
compatible with the "other" IDE.

Is the sample even needed for wxLua? The only reason I'm making it is
for the stedit prerequisite for wxLua. Maybe the sample should not be
in the 'all:' target of wxstedit's Makefile to avoid the problem for
those, like me, who are only trying to get the wxstedit library?

Regards,

e

--
Doug Currie
Londonderry, NH, USA

Re: [wxlua-users] building 2.8.0.0 with wxstedit-1.2.4

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

On 12/25/06, Doug Currie <dou...@gm...> wrote:
> In the midst of building 2.8.0.0 with wxstedit-1.2.4 with mingw and
> msys on WinXP using configure...
>
> Also, in setup.h if I use:
>
> #define STE_USE_LANG_VB          0 // 8
>
> I get:
>
> ./bk-deps g++ -c -o stedit_lib_stelangs.o  -I../include -I/contrib/include -I/usr/local/lib/wx/include/msw-unicode-release-static-2.8 -I/usr/local/include/wx-2.8 -D__WXMSW__ -mthreads  -g0 -O2 .././src/stelangs.cpp
> .././src/stelangs.cpp:4475: error: `STE_WordList1_STE_LANG_VB' was not declared in this scope
> make: *** [stedit_lib_stelangs.o] Error 1

Fixed in CVS now, the HTML lexer uses VB words as a keyword list as
well so the VB keyword list is #ifed for either VB or HTML.

see http://wxcode.cvs.sourceforge.net/wxcode/wxCode/components/wxstedit/src/stelangs.cpp


Thanks,
    John Labenski

Re: [wxlua-users] building 2.8.0.0 with wxstedit-1.2.4

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

On 12/26/06, Doug Currie <dou...@gm...> wrote:
> On Tuesday, December 26, 2006 Francesco Montorsi wrote:
>
> > I don't remember right now if wxCode bakefiles/autoconf stuff fully support the
>
> >     mkdir mybuild && cd mybuild && ../configure
>
> > trick.
>
> It seems to work well for wxWidgets and wxLua except for this one
> file.
>
> > Calling configure in the root dir (./configure) doesn't solve the problem?
>
> I don't think it would; in that case pencil.ico would be in ./art/pencil.ico

Can we not just have no path for pencil.ico and add an -I include
directive for it? I can't test in MSW right now, but IIRC the include
paths work in RC files, but maybe not for icons? Dunno.

wxstedit.rc
pencil16 ICON "pencil.ico"


> > and then define SRCDIR to be $(srcdir) for MSYS but I don't really know if other
> > win32 compilers would like it.
>
> Maybe the Makefile should cd into the $(srcdir)/./samples/stedit
> directory and execute $(WINDRES) there. This might make it more
> compatible with the "other" IDE.
>
> Is the sample even needed for wxLua? The only reason I'm making it is
> for the stedit prerequisite for wxLua. Maybe the sample should not be
> in the 'all:' target of wxstedit's Makefile to avoid the problem for
> those, like me, who are only trying to get the wxstedit library?

It really should be built for people who just want to use wxStEdit
since it's a test to see that things work. Hopefully we can fix this
since the sample program is very small and should compile quickly.

=====================

Franscesco, what is the status of the wxCode bakefiles? What bakefile
should I use to regenerate  wxCode projects in the future?

Thanks,
    John Labenski

Re: [wxlua-users] building 2.8.0.0 with wxstedit-1.2.4

[Francesco M. <f18...@ya...>]

John Labenski ha scritto:
> On 12/26/06, Doug Currie <dou...@gm...> wrote:
>> On Tuesday, December 26, 2006 Francesco Montorsi wrote:
>>
>>> I don't remember right now if wxCode bakefiles/autoconf stuff fully support the
>>>     mkdir mybuild && cd mybuild && ../configure
>>> trick.
>> It seems to work well for wxWidgets and wxLua except for this one
>> file.
>>
>>> Calling configure in the root dir (./configure) doesn't solve the problem?
>> I don't think it would; in that case pencil.ico would be in ./art/pencil.ico
> 
> Can we not just have no path for pencil.ico and add an -I include
> directive for it? I can't test in MSW right now, but IIRC the include
> paths work in RC files, but maybe not for icons? Dunno.
> 
> wxstedit.rc
> pencil16 ICON "pencil.ico"
I don't know if this will work... probably it does only if a copy of 
pencil.ico is in the same folder of wxstedit.rc; so I'd suggest to  move 
wxstedit.rc in art\ (if the -I trick fails).


>>> and then define SRCDIR to be $(srcdir) for MSYS but I don't really know if other
>>> win32 compilers would like it.
>> Maybe the Makefile should cd into the $(srcdir)/./samples/stedit
>> directory and execute $(WINDRES) there. This might make it more
>> compatible with the "other" IDE.
>>
>> Is the sample even needed for wxLua? The only reason I'm making it is
>> for the stedit prerequisite for wxLua. Maybe the sample should not be
>> in the 'all:' target of wxstedit's Makefile to avoid the problem for
>> those, like me, who are only trying to get the wxstedit library?
if you're trying to get only the wxstedit library, then even if the 
sample fails to compile, you should already have the library correctly 
placed in lib\...


> It really should be built for people who just want to use wxStEdit
> since it's a test to see that things work. Hopefully we can fix this
> since the sample program is very small and should compile quickly.
> 
> =====================
> 
> Franscesco, what is the status of the wxCode bakefiles? What bakefile
> should I use to regenerate  wxCode projects in the future?
wxCode bakefiles are ok. The problem is that the WINDRES-hack has been 
added to wxpresets "recently" (I'd say 5-6 months ago) and maybe the 
change didn't go in wx 2.6.3 and maybe you used those wxpresets together 
with the wxCode bakefiles to regen wxstedit makefiles....

to see if your wx presets are recent enough look at wx_unix.bkl; they 
should contain a line like:

<set var="WINDRES">$(WX_RESCOMP)</set>


Francesco