Merge lp:~ken-vandine/indicator-me/gtk3 into lp:indicator-me

Proposed by Ken VanDine
Status: Merged
Approved by: Ted Gould
Approved revision: 150
Merged at revision: 150
Proposed branch: lp:~ken-vandine/indicator-me/gtk3
Merge into: lp:indicator-me
Diff against target: 134 lines (+43/-13)
3 files modified
configure.ac (+35/-10)
src/about-me-menu-item.h (+1/-1)
src/indicator-me.c (+7/-2)
To merge this branch: bzr merge lp:~ken-vandine/indicator-me/gtk3
Reviewer Review Type Date Requested Status
Ted Gould (community) Approve
Review via email: mp+65543@code.launchpad.net

Description of the change

GTK3 port

To post a comment you must log in.
Revision history for this message
Ted Gould (ted) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'configure.ac'
--- configure.ac 2011-06-22 17:36:14 +0000
+++ configure.ac 2011-06-22 18:32:53 +0000
@@ -6,7 +6,7 @@
6AC_PREREQ(2.53)6AC_PREREQ(2.53)
77
8AM_CONFIG_HEADER(config.h)8AM_CONFIG_HEADER(config.h)
9AM_INIT_AUTOMAKE(indicator-me, 0.3.0)9AM_INIT_AUTOMAKE(indicator-me, 0.2.90)
1010
11AM_MAINTAINER_MODE11AM_MAINTAINER_MODE
1212
@@ -30,6 +30,7 @@
30###########################30###########################
3131
32GTK_REQUIRED_VERSION=2.1232GTK_REQUIRED_VERSION=2.12
33GTK3_REQUIRED_VERSION=3.0
33INDICATOR_REQUIRED_VERSION=0.3.034INDICATOR_REQUIRED_VERSION=0.3.0
34GWIBBER_REQUIRED_VERSION=0.0.735GWIBBER_REQUIRED_VERSION=0.0.7
35DBUSMENUGLIB_REQUIRED_VERSION=0.3.9536DBUSMENUGLIB_REQUIRED_VERSION=0.3.95
@@ -39,18 +40,38 @@
39INDICATOR_DISPLAY_OBJECTS=0.140INDICATOR_DISPLAY_OBJECTS=0.1
40GIO_UNIX_REQUIRED_VERSION=2.2241GIO_UNIX_REQUIRED_VERSION=2.22
4142
42PKG_CHECK_MODULES(APPLET, gtk+-2.0 >= $GTK_REQUIRED_VERSION
43 indicator >= $INDICATOR_REQUIRED_VERSION
44 libido-0.1 >= $INDICATOR_DISPLAY_OBJECTS
45 dbus-glib-1
46 dbusmenu-glib-0.4 >= $DBUSMENUGLIB_REQUIRED_VERSION
47 dbusmenu-gtk-0.4 >= $DBUSMENUGTK_REQUIRED_VERSION)
48PKG_CHECK_MODULES(MESERVICE, dbusmenu-glib-0.4 >= $DBUSMENUGLIB_REQUIRED_VERSION43PKG_CHECK_MODULES(MESERVICE, dbusmenu-glib-0.4 >= $DBUSMENUGLIB_REQUIRED_VERSION
49 glib-2.0 >= $GLIB_REQUIRED_VERSION44 glib-2.0 >= $GLIB_REQUIRED_VERSION
50 indicator >= $INDICATOR_REQUIRED_VERSION45 dbus-glib-1
46 indicator3 >= $INDICATOR_REQUIRED_VERSION
51 gwibber-0.1 >= $GWIBBER_REQUIRED_VERSION47 gwibber-0.1 >= $GWIBBER_REQUIRED_VERSION
52 gio-unix-2.0 >= $GIO_UNIX_REQUIRED_VERSION48 gio-unix-2.0 >= $GIO_UNIX_REQUIRED_VERSION
53 telepathy-glib >= $TELEPATHYGLIB_REQUIRED_VERSION)49 telepathy-glib >= $TELEPATHYGLIB_REQUIRED_VERSION)
50AC_SUBST(MESERVICE_CFLAGS)
51AC_SUBST(MESERVICE_LIBS)
52
53AC_ARG_WITH([gtk],
54 [AS_HELP_STRING([--with-gtk],
55 [Which version of gtk to use for the indicator @<:@default=3@:>@])],
56 [],
57 [with_gtk=3])
58
59AS_IF([test "x$with_gtk" = x3],
60 [PKG_CHECK_MODULES(APPLET, gtk+-3.0 >= $GTK3_REQUIRED_VERSION
61 indicator3 >= $INDICATOR_REQUIRED_VERSION
62 libido3-0.1 >= $INDICATOR_DISPLAY_OBJECTS
63 dbusmenu-gtk3-0.4 >= $DBUSMENUGTK_REQUIRED_VERSION)
64 ],
65 [test "x$with_gtk" = x2],
66 [PKG_CHECK_MODULES(APPLET, gtk+-2.0 >= $GTK_REQUIRED_VERSION
67 indicator >= $INDICATOR_REQUIRED_VERSION
68 libido-0.1 >= $INDICATOR_DISPLAY_OBJECTS
69 dbusmenu-gtk-0.4 >= $DBUSMENUGTK_REQUIRED_VERSION)
70 ],
71 [AC_MSG_FAILURE([Value for --with-gtk was neither 2 nor 3])]
72)
73AC_SUBST(APPLET_CFLAGS)
74AC_SUBST(APPLET_LIBS)
5475
55###########################76###########################
56# Check to see if we're local77# Check to see if we're local
@@ -66,9 +87,12 @@
66if test "x$with_localinstall" = "xyes"; then87if test "x$with_localinstall" = "xyes"; then
67 INDICATORDIR="${libdir}/indicators/3/"88 INDICATORDIR="${libdir}/indicators/3/"
68 INDICATORICONSDIR="${datadir}/indicator-applet/icons/"89 INDICATORICONSDIR="${datadir}/indicator-applet/icons/"
90elif test "x$with_gtk" = x2; then
91 INDICATORDIR=`$PKG_CONFIG --variable=indicatordir indicator`
92 INDICATORICONSDIR=`$PKG_CONFIG --variable=iconsdir indicator`
69else93else
70 INDICATORDIR=`$PKG_CONFIG --variable=indicatordir indicator`94 INDICATORDIR=`$PKG_CONFIG --variable=indicatordir indicator3`
71 INDICATORICONSDIR=`$PKG_CONFIG --variable=iconsdir indicator`95 INDICATORICONSDIR=`$PKG_CONFIG --variable=iconsdir indicator3`
72fi96fi
73AC_SUBST(INDICATORDIR)97AC_SUBST(INDICATORDIR)
74AC_SUBST(INDICATORICONSDIR)98AC_SUBST(INDICATORICONSDIR)
@@ -144,4 +168,5 @@
144Me Indicator Configuration:168Me Indicator Configuration:
145169
146 Prefix: $prefix170 Prefix: $prefix
171 GTK: $with_gtk
147])172])
148173
=== modified file 'src/about-me-menu-item.h'
--- src/about-me-menu-item.h 2010-03-11 12:20:48 +0000
+++ src/about-me-menu-item.h 2011-06-22 18:32:53 +0000
@@ -28,7 +28,7 @@
28#ifndef __ABOUT_ME_MENU_ITEM_H__28#ifndef __ABOUT_ME_MENU_ITEM_H__
29#define __ABOUT_ME_MENU_ITEM_H__29#define __ABOUT_ME_MENU_ITEM_H__
3030
31#include <gtk/gtkmenuitem.h>31#include <gtk/gtk.h>
3232
33G_BEGIN_DECLS33G_BEGIN_DECLS
3434
3535
=== modified file 'src/indicator-me.c'
--- src/indicator-me.c 2011-04-14 14:56:04 +0000
+++ src/indicator-me.c 2011-06-22 18:32:53 +0000
@@ -28,7 +28,12 @@
28#include <gtk/gtk.h>28#include <gtk/gtk.h>
29#include <gio/gio.h>29#include <gio/gio.h>
30#include <libdbusmenu-glib/menuitem.h>30#include <libdbusmenu-glib/menuitem.h>
31
32#if GTK_CHECK_VERSION(3, 0, 0)
33#include <libdbusmenu-gtk3/menu.h>
34#else
31#include <libdbusmenu-gtk/menu.h>35#include <libdbusmenu-gtk/menu.h>
36#endif
3237
33#include <libindicator/indicator.h>38#include <libindicator/indicator.h>
34#include <libindicator/indicator-object.h>39#include <libindicator/indicator-object.h>
@@ -335,7 +340,7 @@
335}340}
336341
337static void342static void
338item_destroyed_cb (GtkObject *item,343item_destroyed_cb (DbusmenuMenuitem *item,
339 gpointer user_data)344 gpointer user_data)
340{345{
341 g_signal_handlers_disconnect_by_func (user_data,346 g_signal_handlers_disconnect_by_func (user_data,
@@ -479,7 +484,7 @@
479 DBUSMENU_ENTRY_MENUITEM_PROP_HINT);484 DBUSMENU_ENTRY_MENUITEM_PROP_HINT);
480485
481 /* enforce style when typing a message */486 /* enforce style when typing a message */
482 if (GTK_WIDGET_HAS_FOCUS (entry)) {487 if (gtk_widget_has_focus (GTK_WIDGET (entry))) {
483 entry_set_style (entry, GTK_STATE_NORMAL);488 entry_set_style (entry, GTK_STATE_NORMAL);
484 entry_hint_set_shown (GTK_WIDGET (entry), FALSE);489 entry_hint_set_shown (GTK_WIDGET (entry), FALSE);
485490

Subscribers

People subscribed via source and target branches