955159 - Fix compilation of purple/ with glib 2.32 and newer

Status: RESOLVED FIXED

(Chat Core :: General, defect)

Description

[Florian Quèze [:florian]]

*** Original post on bio 1730 at 2012-10-16 22:08:00 UTC ***

*** Due to BzAPI limitations, the initial description is in comment 1 ***

Comment 1

[Florian Quèze [:florian]]

Attachment: Patch

*** Original post on bio 1730 as attmnt 1971 at 2012-10-16 22:08:00 UTC ***

" The recommended way of using GLib has always been to only include the toplevel headers glib.h, glib-object.h, gio.h. Starting with 2.32, GLib enforces this by generating an error when individual headers are directly included. " http://developer.gnome.org/glib/2.32/glib-compiling.html

The attached patch includes changes specific to our usage of libpurple plus an existing meanwhile patch that we found, see:
23:15:22 - unghost: fwiw, https://aur.archlinux.org/packages.php?ID=44209&comments=all has link to corrected meanwhile
23:18:40 - clokep_work: (Or just the patch http://codezen.org/static/fix_glib_includes.patch)
23:25:38 - flo: clokep_work: that patch applies fine for us

Comment 2

[Patrick Cloke [:clokep]]

Comment on attachment 8353730 [details] [diff] [review]
Patch

*** Original change on bio 1730 attmnt 1971 at 2012-10-16 22:16:16 UTC ***

>diff --git a/purple/libpurple/protocols/sametime/meanwhile/mw_srvc_dir.h b/purple/libpurple/protocols/sametime/meanwhile/mw_srvc_dir.h
[...]
> #include <glib.h>
>-#include <glib/glist.h>
>+#include <glib.h>
In a few places you've included glib.h twice.

Comment 3

[Florian Quèze [:florian]]

Attachment: Patch v2

*** Original post on bio 1730 as attmnt 2060 at 2012-11-05 23:31:00 UTC ***

Removed the duplicated glib.h include in 4 files. I hadn't looked at these files, only applied the patch we had found.

Comment 4

[Florian Quèze [:florian]]

Comment on attachment 8353730 [details] [diff] [review]
Patch

*** Original change on bio 1730 attmnt 1971 at 2012-11-05 23:31:53 UTC was without comment, so any subsequent comment numbers will be shifted ***

Comment 5

[Patrick Cloke [:clokep]]

Comment on attachment 8353820 [details] [diff] [review]
Patch v2

*** Original change on bio 1730 attmnt 2060 at 2012-11-06 15:22:24 UTC ***

Looks good, thanks! :)

Comment 6

[Patrick Cloke [:clokep]]

*** Original post on bio 1730 at 2012-11-06 22:54:36 UTC ***

Committed as http://hg.instantbird.org/instantbird/rev/7da3aa835434

Thanks!