indicator for deluge

[Sergey Klimov]

Binary package hint: deluge

deluge needs indicator (via dbus/libindicator i think)

ProblemType: Bug
DistroRelease: Ubuntu 10.04
Package: deluge 1.2.2-2
Uname: Linux 2.6.34-999-generic x86_64
NonfreeKernelModules: nvidia
Architecture: amd64
Date: Sun May 23 20:39:46 2010
InstallationMedia: Ubuntu 10.04 LTS "Lucid Lynx" - Release amd64 (20100429)
PackageArchitecture: all
ProcEnviron:
 LANG=ru_UA.utf8
 SHELL=/bin/bash
SourcePackage: deluge

[Andrew Starr-Bochicchio (andrewsomething)]

Some relevant upstream discussion starting here:

http://<email address hidden>/msg00794.html

[Daniel Añez Scott (danielscott171)]

I was going to file a bug about it, I hope they implement this for maverick.

[Sense Egbert Hofstede (sense)]

I'm working on this, but work is being blocked by bug #608219.

[Sense Egbert Hofstede (sense)]

The attached patch contains the bulk of the code for creating an Application Indicator for Deluge HEAD.

It doesn't work yet, as it depends on bug #608219, and might need some changes after that bug is fixed.

By default the Application Indicator component is disabled and the systemtray extension is used. Set the preference 'enable_application_indicator' to True. You can find it at 'deluge/ui/gtkui/gtkui.py:113'.

[Andrew Resch (andar)]

I'd like to include this upstream, but I am not entirely convinced of the approach taken in this patch. Adding an additional component that is essentially a copy and paste of the existing SystemTray component isn't really ideal. There is way too much code duplication and this will become a maintenance nightmare.

The better approach would be to modify SystemTray to use either a gtkStatusIcon or the Appindicator possibly based on a configuration value or simply in an order of preference, eg, use the Appindicator if the 'import appindicator' succeeds. I haven't taken a hard look at the patch, but it's likely that this could be accomplished with far less new lines of code. There are other options too, you could for instance, create a base class and subclass the SystemTray and Appindicator respectively.

Thanks for your time in adding this functionality, I hope that we can add it to Deluge in the future.

[Karl Lattimer (karl-qdh)]

Thats really not too hard to do, so I'll have a poke at it tomorrow as I've just been given this bug :)

[Johannes Hell (johanneshell)]

Are there any news on indicator support for deluge? It would be ver nice if this was included.

[Sense Egbert Hofstede (sense)]

There is some large work planned to merge the GtkMenu processing code of AppMenu and Application Indicators, which will probably solve a lot of bugs we are experiencing right now, and which are also blocking this indicator.

I have already got an indicator that is almost finished, it is just buggy. When the bugs are gone I expect it to be very easy to finish it and submit the Deluge Application Indicator.

[Karl Lattimer (karl-qdh)]

latest patch, this almost works around the download/upload speed menu items not having the submenus but still those submenus emit signals on the wrong radio menu item.

This patch also merges together the systemtray and appindicator modules as they were far too similar.

[Karl Lattimer (karl-qdh)]

This bug is related to; https://bugs.launchpad.net/indicator-application/+bug/608219

[Karl Lattimer (karl-qdh)]

@Andrew Resch: Is the patch in comment #10 more to your liking, would you care to accept this upstream?

[Anatoly Kudinov (zz)]

This is already in 1.3-stable branch http://dev.deluge-torrent.org/ticket/1514#comment:3, but deb is not updated since patch inclusion so i cant say if it works or not

[Andrew Starr-Bochicchio (andrewsomething)]

It should be included in the next Deluge point release 1.3.2. Here's the commit:

http://git.deluge-torrent.org/deluge/commit/?h=1.3-stable&id=fd3a33af03b96a868545fa20e9f67af7d1a1b157

I'll look into backporting this to Ubuntu.

[Launchpad Janitor (janitor)]

This bug was fixed in the package deluge - 1.3.1-1ubuntu2

---------------
deluge (1.3.1-1ubuntu2) natty; urgency=low

  * Backport indicator support from upstream 1.3.x
    stable branch (LP: #584669).
 -- Andrew Starr-Bochicchio <email address hidden> Sun, 13 Mar 2011 14:36:24 -0400