On Tue, 19 Feb 2008 02:25:38 +0200, Yura Siamashka yurand2@gmail.com wrote:
Hi,
thanks for the patch.
- scintilla/ScintillaGTK.cxx, Windows.h should be changed to windows.h
since cross compiler is still case sensitive. This should be probably reported to Scintilla.
I'll sent a patch to the Scintilla mailing list.
- Does windows version need GEANY_PREFIX, GEANY_DATADIR, GEANY_LIBDIR,
GEANY_LOCALEDIR? If so need to define them probably in configure.in. For now I commented them.
On Windows, Geany is installed (and configured) to use relative paths. These macros are defined in src/makefile.win32, we should define them in src/Makefile.am for cross-compiling, too.
- Using hardcoded i386-mingw32msvc as cross-compile target is not very
nice. I replaced /usr/local/cross-tools/bin/i386-mingw32msvc-windres with $(host)-windres in makefile.am.
Yes, I used this because I didn't expect anyone else than me would use this and hardcoding the paths was just easier :D.
- Windows doesn't have fnmatch, I replaced it's usage with modified
function I found in one of the files. But this is really smell bad.
I'm not yet sure why this happens but fnmatch is available via liberty, IIRC. At least when I wrote the cross-compiling stuff, it worked. Right now, I have some other problems which prevent me from further testing, e.g. configure doesn't detect the host correctly. And I don't have the time to test further now. I applied your patch partyl, thanks and I will continue testing on Friday or Saturday.
p.s I was able to run geany in wine, but something nasty happend with fonts. I see only rectangles. I guess I will find it tomorrow.
Hehe, Geany for Windows in Wine on Linux ;-). I've no idea what went wrong there but I wouldn't spend too much time in solving problems in this environment...
p.p.s. At download page http://geany.uvena.de/Download/Releases written that geany-0.13_setup.exe contains GTK 2.10, but installer said it is 2.12.
Fixed, thanks.
Regards, Enrico