Merge lp:~dylanmccall/ubuntu/oneiric/network-manager-applet/lp852961-disable-autostart-for-gnome-shell into lp:ubuntu/oneiric/network-manager-applet

Proposed by Dylan McCall
Status: Superseded
Proposed branch: lp:~dylanmccall/ubuntu/oneiric/network-manager-applet/lp852961-disable-autostart-for-gnome-shell
Merge into: lp:ubuntu/oneiric/network-manager-applet
Diff against target: 56 lines (+17/-7)
3 files modified
debian/changelog (+7/-0)
debian/patches/04-autostart.patch (+9/-7)
nm-applet.desktop.in (+1/-0)
To merge this branch: bzr merge lp:~dylanmccall/ubuntu/oneiric/network-manager-applet/lp852961-disable-autostart-for-gnome-shell
Reviewer Review Type Date Requested Status
Mathieu Trudel-Lapierre Needs Resubmitting
Dylan McCall Pending
Ubuntu branches Pending
Review via email: mp+76335@code.launchpad.net

This proposal supersedes a proposal from 2011-09-21.

Description of the change

This adds an extra rule to nm-applet's autostart stuff so it does not autostart in Gnome Shell, fixing bug #852961.

Sorry about all the weirdness from that first merge request! I forgot to reapply the patches in quilt after making my changes (and edit-patch forgot, too). Out of curiosity, is there something different with the patch system in this package than with others?

At any rate, this merge request is the real one with the puny one line change fitting in an equivalently small diff :)

To post a comment you must log in.
Revision history for this message
Dylan McCall (dylanmccall) wrote : Posted in a previous version of this proposal

Eeek! Sorry about that first diff. First time using edit-patch, I guess :)
New one coming in a second. I'll just kill this one.

review: Needs Resubmitting
Revision history for this message
Mathieu Trudel-Lapierre (cyphermox) wrote :

Well, it's actually the wrong branch too. Please re-do the changes onto lp:~network-manager/network-manager-applet/ubuntu.head, I'll gladly merge these changes.

As for the actual process, it might even be easier that way:

bzr branch lp:~network-manager/network-manager-applet/ubuntu.head nm-applet
cd nm-applet
bzr bd-do
quilt push -a
quilt new 04-autostart.patch
quilt edit nm-applet.desktop.in
quilt refresh
exit 0
 .... you know the drill :)

review: Needs Resubmitting

Unmerged revisions

71. By Dylan McCall

* debian/patches/04-autostart.patch:
  - disable autostart when using gnome-shell (LP: #852961)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2011-07-08 01:07:05 +0000
+++ debian/changelog 2011-09-21 06:18:29 +0000
@@ -1,3 +1,10 @@
1network-manager-applet (0.8.9997+git.20110707t152406.340f695-0ubuntu2) oneiric; urgency=low
2
3 * debian/patches/04-autostart.patch:
4 - disable autostart when using gnome-shell (LP: #852961)
5
6 -- Dylan McCall <dylanmccall@ubuntu.com> Tue, 20 Sep 2011 23:06:34 -0700
7
1network-manager-applet (0.8.9997+git.20110707t152406.340f695-0ubuntu1) oneiric; urgency=low8network-manager-applet (0.8.9997+git.20110707t152406.340f695-0ubuntu1) oneiric; urgency=low
29
3 * upstream snapshot 2011-07-07 15:24:06 (GMT)10 * upstream snapshot 2011-07-07 15:24:06 (GMT)
411
=== modified file 'debian/patches/04-autostart.patch'
--- debian/patches/04-autostart.patch 2011-05-30 13:25:18 +0000
+++ debian/patches/04-autostart.patch 2011-09-21 06:18:29 +0000
@@ -1,8 +1,8 @@
1Index: network-manager-applet-0.8.999+git.20110523t134536.bb7ef97/nm-applet.desktop.in1Index: network-manager-applet/nm-applet.desktop.in
2===================================================================2===================================================================
3--- network-manager-applet-0.8.999+git.20110523t134536.bb7ef97.orig/nm-applet.desktop.in 2011-05-25 21:20:49.000000000 -04003--- network-manager-applet.orig/nm-applet.desktop.in 2011-09-20 23:02:13.934376000 -0700
4+++ network-manager-applet-0.8.999+git.20110523t134536.bb7ef97/nm-applet.desktop.in 2011-05-25 22:20:42.709099830 -04004+++ network-manager-applet/nm-applet.desktop.in 2011-09-20 23:05:38.755287537 -0700
5@@ -6,6 +6,7 @@5@@ -6,7 +6,9 @@
6 Terminal=false6 Terminal=false
7 Type=Application7 Type=Application
8 NoDisplay=true8 NoDisplay=true
@@ -10,10 +10,12 @@
10 X-GNOME-Bugzilla-Bugzilla=GNOME10 X-GNOME-Bugzilla-Bugzilla=GNOME
11 X-GNOME-Bugzilla-Product=NetworkManager11 X-GNOME-Bugzilla-Product=NetworkManager
12 X-GNOME-Bugzilla-Component=general12 X-GNOME-Bugzilla-Component=general
13Index: network-manager-applet-0.8.999+git.20110523t134536.bb7ef97/nm-connection-editor.desktop.in13 X-GNOME-Autostart-enabled=true
14+AutostartCondition=GNOME3 unless-session gnome
15Index: network-manager-applet/nm-connection-editor.desktop.in
14===================================================================16===================================================================
15--- network-manager-applet-0.8.999+git.20110523t134536.bb7ef97.orig/nm-connection-editor.desktop.in 2011-05-25 21:20:49.000000000 -040017--- network-manager-applet.orig/nm-connection-editor.desktop.in 2011-09-20 23:02:13.934376000 -0700
16+++ network-manager-applet-0.8.999+git.20110523t134536.bb7ef97/nm-connection-editor.desktop.in 2011-05-25 22:19:44.349099837 -040018+++ network-manager-applet/nm-connection-editor.desktop.in 2011-09-20 23:02:13.934376000 -0700
17@@ -10,4 +10,4 @@19@@ -10,4 +10,4 @@
18 X-GNOME-Bugzilla-Product=NetworkManager20 X-GNOME-Bugzilla-Product=NetworkManager
19 X-GNOME-Bugzilla-Component=general21 X-GNOME-Bugzilla-Component=general
2022
=== modified file 'nm-applet.desktop.in'
--- nm-applet.desktop.in 2011-05-30 13:25:18 +0000
+++ nm-applet.desktop.in 2011-09-21 06:18:29 +0000
@@ -11,3 +11,4 @@
11X-GNOME-Bugzilla-Product=NetworkManager11X-GNOME-Bugzilla-Product=NetworkManager
12X-GNOME-Bugzilla-Component=general12X-GNOME-Bugzilla-Component=general
13X-GNOME-Autostart-enabled=true13X-GNOME-Autostart-enabled=true
14AutostartCondition=GNOME3 unless-session gnome

Subscribers

People subscribed via source and target branches

to all changes: