Tweaking autotools-based buildsystem for W32 compatibility

Tomasz Wasilczyk twasilczyk at pidgin.im
Thu Apr 24 17:21:15 EDT 2014


Hi,

I have gone through these patches and fixed most of the issues. Right 
now, Pidgin (and Finch!) builds and works. However, we don't have an 
installer yet, so it's non trivial to run the build (however, it's easy).

Patches that I marked as "done" (not necessary committed as-is, but the 
issue should be solved): 0008, 0012, 0019, 0020, 0022, 0023, 0025, 0026, 
0027, 0030, 0031, 0041, 0042, 0043, 0044, 0045, 0047, 0048, 0049, 0050, 
0051, 0052, 0054, 0055, 0057, 0059, 0061, 0064, 0066, 0067, 0068. And 
"not necessary": 0006, 0009, 0013, 0016, 0037, 0065.

I decided not to handle few of them because of reasons not related 
directly: 0002 (I think we should drop gtkspell2/3 support, since 
webkitgtk doesn't use it anyway), 0014 (zephyr prpl won't go on win32 
because of fork), 0035 (I don't think spending time on tcl/perl support 
on win32 makes any sense, since gobj.plugins branch is almost ready).

Four patches are "not wanted": 0033 (we don't use %z nor %ll in our code 
and this macro breaks encoding), 0046 (dbus is disabled by default for 
win32 and the patch is described as unfinished anyway), 0053 (we 
discussed it - this patch was a mistake), 0056 (the part about wchar - 
see 0033; the part about [b]slash is committed).


There are also patches I'm not sure about.

0021-add-inet-pton~2-compat.all.patch:
I prefer to use system library here. Even, if the system is win32.

0058-rename-pidgin-to-pidgin3.all.patch:
I don't think there is a need for such rename. But I may be wrong. And 
possibly it might be useful to rename libpurple to libpurple3.

0069-versioned-protolibs.mingw.patch:
I don't think there is any gain from versioning "base" protocol plugins.


And few patches related to FHS, I'm working on right now:

0024-dont-fail-if-~ot-exist.all.patch:
0060-remove-crazy-dll-loading-code.all.patch:
I'm not sure about these, they may be necessary.

0029-correct-runtime-paths.all.patch:
PURPLE_USE_WINPATHS, SSL_CERTIFICATES_DIR - just a todo.
DONT_USE_NICE_PNG_ICON_AS_DEFAULT - I'm not sure if we need this. I 
think .ico files are enough for win32.


A request for other devs: please express your opinion, especially about 
0021, 0058, 0069, 0024, 0060 and 0029 (the patches I'm not sure about).

Tomek

-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 4225 bytes
Desc: Kryptograficzna sygnatura S/MIME
URL: <https://pidgin.im/pipermail/devel/attachments/20140424/61267b8e/attachment.bin>


More information about the Devel mailing list