Merge lp:~chrisccoulson/gnome-media/bug337235 into lp:~ubuntu-desktop/gnome-media/ubuntu

Proposed by Chris Coulson
Status: Merged
Merged at revision: not available
Proposed branch: lp:~chrisccoulson/gnome-media/bug337235
Merge into: lp:~ubuntu-desktop/gnome-media/ubuntu
Diff against target: None lines
To merge this branch: bzr merge lp:~chrisccoulson/gnome-media/bug337235
Reviewer Review Type Date Requested Status
Sebastien Bacher Approve
Review via email: mp+4422@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Sebastien Bacher (seb128) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2009-03-03 22:22:58 +0000
+++ debian/changelog 2009-03-12 21:49:02 +0000
@@ -1,3 +1,11 @@
1gnome-media (2.25.92-0ubuntu2) jaunty; urgency=low
2
3 * debian/patches/04_gst-mixer_help.patch:
4 - Correctly display the help by calling gtk_show_uri
5 as opposed to running xdg-open (LP: #337235).
6
7 -- Chris Coulson <chrisccoulson@googlemail.com> Thu, 12 Mar 2009 21:18:37 +0000
8
1gnome-media (2.25.92-0ubuntu1) jaunty; urgency=low9gnome-media (2.25.92-0ubuntu1) jaunty; urgency=low
210
3 * New upstream version (LP: #337462):11 * New upstream version (LP: #337462):
412
=== added file 'debian/patches/04_gst-mixer_help.patch'
--- debian/patches/04_gst-mixer_help.patch 1970-01-01 00:00:00 +0000
+++ debian/patches/04_gst-mixer_help.patch 2009-03-12 21:49:02 +0000
@@ -0,0 +1,47 @@
1diff -Nur -x '*.orig' -x '*~' gnome-media-2.25.92/gst-mixer/src/window.c gnome-media-2.25.92.new/gst-mixer/src/window.c
2--- gnome-media-2.25.92/gst-mixer/src/window.c 2009-03-03 17:20:53.000000000 +0000
3+++ gnome-media-2.25.92.new/gst-mixer/src/window.c 2009-03-12 21:32:07.000000000 +0000
4@@ -99,34 +99,23 @@
5 }
6
7 static void
8-open_uri (GtkWindow *parent,
9- const char *uri)
10+cb_help (GtkAction *action,
11+ GnomeVolumeControlWindow *win)
12 {
13- GtkWidget *dialog;
14 GdkScreen *screen;
15+ GtkWidget *dialog;
16 GError *error = NULL;
17- gchar *cmdline;
18-
19- screen = gtk_window_get_screen (parent);
20-
21- cmdline = g_strconcat ("xdg-open ", uri, NULL);
22-
23- if (gdk_spawn_command_line_on_screen (screen, cmdline, &error) == FALSE) {
24- dialog = gtk_message_dialog_new (parent, GTK_DIALOG_DESTROY_WITH_PARENT,
25+
26+ screen = gtk_window_get_screen (GTK_WINDOW (win));
27+
28+ if (gtk_show_uri (screen, "ghelp:gnome-volume-control", GDK_CURRENT_TIME,
29+ &error) == FALSE) {
30+ dialog = gtk_message_dialog_new (GTK_WINDOW (win), GTK_DIALOG_DESTROY_WITH_PARENT,
31 GTK_MESSAGE_ERROR, GTK_BUTTONS_CLOSE, "%s", error->message);
32 gtk_dialog_run(GTK_DIALOG(dialog));
33 gtk_widget_destroy(dialog);
34 g_error_free(error);
35 }
36- g_free(cmdline);
37-}
38-
39-
40-static void
41-cb_help (GtkAction *action,
42- GnomeVolumeControlWindow *win)
43-{
44- open_uri (GTK_WINDOW (win), "ghelp:gnome-volume-control");
45 }
46
47 static void

Subscribers

People subscribed via source and target branches