FS#54028 - [syncthing-gtk] Add libappindicator-gtk3 as an optional dependecy

Attached to Project: Community Packages
Opened by Chris Sakalis (ChrisS) - Saturday, 13 May 2017, 13:12 GMT
Last edited by Jaroslav Lichtblau (Dragonlord) - Saturday, 02 December 2017, 19:38 GMT
Task Type Bug Report
Category Packages
Status Closed
Assigned To Jaroslav Lichtblau (Dragonlord)
Architecture All
Severity Very Low
Priority Normal
Reported Version
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 1
Private No

Details

Description:

In order to have the tray icon appear in Plasma 5 libappindicator-gtk3 is required, as seen in [the bugtracker upstream](https://github.com/syncthing/syncthing-gtk/issues/297). Maybe it could be added as an optional dependency?

Additional info:

syncthing 0.14.27-1
syncthing-gtk 1:0.9.2.4-1
libappindicator-gtk3 12.10.0-12

Steps to reproduce:

1. Launch syncthing-gtk on Plasma 5 without having libappindicator-gtk3 installed.
This task depends upon

Closed by  Jaroslav Lichtblau (Dragonlord)
Saturday, 02 December 2017, 19:38 GMT
Reason for closing:  Implemented
Additional comments about closing:  Added as optdep in SVN trunk
Comment by Balló György (City-busz) - Tuesday, 16 May 2017, 09:22 GMT
I don't think that adding libappindicator-gtk3 as optional dependency is a good idea. The application should work without this Ubuntu-related library. It's a bug somewhere in the upstream code that needs to be fixed.
Comment by Doug Newgard (Scimmia) - Monday, 02 October 2017, 13:28 GMT
It is not a bug upstream, and this is not an Ubuntu specific thing. SNI is a freedesktop/KDE thing that just happened to be adopted first by Ubuntu.
Comment by Balló György (City-busz) - Monday, 02 October 2017, 14:48 GMT
Plasma supports XEmbed tray icons with XEmbed SNI Proxy. [1] If it does not work, then it should be a bug either in syncthing-gtk or in plasma-workspace. libappindicator [2] is an Ubuntu-specific implementation of SNI support (designed for the now dead Unity desktop), so I think we should avoid it.

[1] https://github.com/KDE/plasma-workspace/tree/master/xembed-sni-proxy
[2] https://launchpad.net/libappindicator
Comment by Doug Newgard (Scimmia) - Tuesday, 03 October 2017, 03:49 GMT
Plasma natively supports SNI, and Enlightenment *ONLY* supports SNI. An optional dep is a very valid request.

Loading...