Merge lp:~nik90/ubuntu/precise/transmission/add_quicklist_updated into lp:ubuntu/precise/transmission

Proposed by Nekhelesh Ramananthan
Status: Merged
Merge reported by: Didier Roche-Tolomelli
Merged at revision: not available
Proposed branch: lp:~nik90/ubuntu/precise/transmission/add_quicklist_updated
Merge into: lp:ubuntu/precise/transmission
Diff against target: 37 lines (+22/-0)
2 files modified
gtk/transmission-gtk.desktop.in (+11/-0)
qt/transmission-qt.desktop (+11/-0)
To merge this branch: bzr merge lp:~nik90/ubuntu/precise/transmission/add_quicklist_updated
Reviewer Review Type Date Requested Status
Didier Roche-Tolomelli Approve
Ubuntu branches Pending
Review via email: mp+95084@code.launchpad.net

Description of the change

Transmission quicklist support added with latest desktop specifications like title and Actions.

To post a comment you must log in.
Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

@Michael Hall, I had to create a new branch since I could not upload to my old branch. Hence this new merge proposal as well.

Revision history for this message
Didier Roche-Tolomelli (didrocks) wrote :

Hey Nekhelesh,
thanks for your contribution to ubuntu and for this merge requests :)

Some notes about it:
- we normally comment the changes in the debian/changelog file. For that, we are using debchange -i and write something like:
- move the .desktop file change into something the patch system of the package can deal with

http://developer.ubuntu.com/packaging/html/udd-patchsys.html#edit-patch is something which might interest you in this regard. This is the end result: http://paste.ubuntu.com/855106/

Also, we try to forward the change to Upstream directly. I opened a bug in the GNOME bugzilla tracker with this patch attached. We would really appreciate if you can do that yourself next time and stating it on the merge request).

Thanks again for your contribution there! That's really appreciated :)

Marking manually as merged as I needed to put it in a patch with another branch.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'gtk/transmission-gtk.desktop.in'
--- gtk/transmission-gtk.desktop.in 2011-07-19 10:35:40 +0000
+++ gtk/transmission-gtk.desktop.in 2012-02-29 00:55:20 +0000
@@ -12,3 +12,14 @@
12Categories=Network;FileTransfer;P2P;GTK;12Categories=Network;FileTransfer;P2P;GTK;
13X-Ubuntu-Gettext-Domain=transmission13X-Ubuntu-Gettext-Domain=transmission
14X-AppInstall-Keywords=torrent14X-AppInstall-Keywords=torrent
15Actions=Pause;Minimize;
16
17[Desktop Action Pause]
18Name=Start Transmission with all Torrents Paused
19Exec=transmission-gtk --paused
20OnlyShowIn=Unity
21
22[Desktop Action Minimize]
23Name=Start Transmission Minimized
24Exec=transmission-gtk --minimized
25OnlyShowIn=Unity
1526
=== modified file 'qt/transmission-qt.desktop'
--- qt/transmission-qt.desktop 2011-01-28 21:45:01 +0000
+++ qt/transmission-qt.desktop 2012-02-29 00:55:20 +0000
@@ -9,3 +9,14 @@
9MimeType=application/x-bittorrent;9MimeType=application/x-bittorrent;
10Categories=Network;FileTransfer;P2P;Qt;10Categories=Network;FileTransfer;P2P;Qt;
11X-AppInstall-Keywords=torrent11X-AppInstall-Keywords=torrent
12Actions=Pause;Minimize;
13
14[Desktop Action Pause]
15Name=Start Transmission with all Torrents Paused
16Exec=transmission-gtk --paused
17OnlyShowIn=Unity
18
19[Desktop Action Minimize]
20Name=Start Transmission Minimized
21Exec=transmission-gtk --minimized
22OnlyShowIn=Unity

Subscribers

People subscribed via source and target branches