Merge lp:~xnox/appmenu-gtk/fix-all-emacs into lp:appmenu-gtk

Proposed by Dimitri John Ledkov
Status: Merged
Approved by: Mathieu Trudel-Lapierre
Approved revision: 169
Merged at revision: 169
Proposed branch: lp:~xnox/appmenu-gtk/fix-all-emacs
Merge into: lp:appmenu-gtk
Diff against target: 14 lines (+1/-3)
1 file modified
src/bridge.c (+1/-3)
To merge this branch: bzr merge lp:~xnox/appmenu-gtk/fix-all-emacs
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Mathieu Trudel-Lapierre Approve
Review via email: mp+156234@code.launchpad.net

Commit message

* Fix appmenu-gtk breaks dynamically created Emacs Gtk+ menus, for emacs* (emacs24 specifically) (LP: #673302)

To post a comment you must log in.
Revision history for this message
Mathieu Trudel-Lapierre (cyphermox) wrote :

Looks fine, approve.

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/bridge.c'
--- src/bridge.c 2013-02-25 20:11:18 +0000
+++ src/bridge.c 2013-03-30 00:46:22 +0000
@@ -971,9 +971,7 @@
971 || (g_strcmp0 (prg, "firefox-bin") == 0)971 || (g_strcmp0 (prg, "firefox-bin") == 0)
972 || (g_strcmp0 (prg, "thunderbird-bin") == 0)972 || (g_strcmp0 (prg, "thunderbird-bin") == 0)
973 || (g_strcmp0 (prg, "Eclipse") == 0)973 || (g_strcmp0 (prg, "Eclipse") == 0)
974 || (g_strcmp0 (prg, "emacs") == 0)974 || g_str_has_prefix (prg, "emacs")
975 || (g_strcmp0 (prg, "emacs23") == 0)
976 || (g_strcmp0 (prg, "emacs23-lucid") == 0)
977 || (g_strcmp0 (prg, "glade") == 0)975 || (g_strcmp0 (prg, "glade") == 0)
978 || (g_strcmp0 (prg, "gnome-panel") == 0)976 || (g_strcmp0 (prg, "gnome-panel") == 0)
979 || (g_strcmp0 (prg, "gwyddion") == 0)977 || (g_strcmp0 (prg, "gwyddion") == 0)

Subscribers

People subscribed via source and target branches