Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 669502 - =kde-plasma/plasma-meta-5.14.2: change to USE="+legacy-systray" to pull in kde-plasma/xembed-sni-proxy
Summary: =kde-plasma/plasma-meta-5.14.2: change to USE="+legacy-systray" to pull in kd...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Gentoo KDE team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-10-24 14:49 UTC by Till Schäfer
Modified: 2018-10-25 18:04 UTC (History)
1 user (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Till Schäfer 2018-10-24 14:49:58 UTC
It seems that kde-plasma/xembed-sni-proxy is installed separately from kde-plasma/plasma-workspace since plasma 5.14.2. However, that package is not installed by default, the use flag legacy-systray is disabled by default for plasma-meta.

I propose to flip this default (i.e., USE="+legacy-systray" in plasma-meta), since many applications, such as net-ftp/filezilla or net-im/signal-desktop-bin (i.e., electron apps) still rely on it to display their tray icon and it is quite intransparent to the user, why the systray icons do not work any longer. At least a post install message should be issued with some explanation.
Comment 1 Larry the Git Cow gentoo-dev 2018-10-24 15:30:01 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=c9a69082bb6de539de7ae9645359bf8d0a08094a

commit c9a69082bb6de539de7ae9645359bf8d0a08094a
Author:     Andreas Sturmlechner <asturm@gentoo.org>
AuthorDate: 2018-10-24 15:29:06 +0000
Commit:     Andreas Sturmlechner <asturm@gentoo.org>
CommitDate: 2018-10-24 15:29:51 +0000

    kde-plasma/plasma-meta: Set USE +legacy-systray
    
    Closes: https://bugs.gentoo.org/669502
    Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
    Package-Manager: Portage-2.3.50, Repoman-2.3.11

 .../{plasma-meta-5.14.2.ebuild => plasma-meta-5.14.2-r1.ebuild}         | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Comment 2 Andreas Sturmlechner gentoo-dev 2018-10-24 15:30:43 UTC
Thanks, it makes perfect sense.
Comment 3 Ivan Grynko 2018-10-24 16:53:07 UTC
But this makes no sense
        legacy-systray? (
            dev-libs/libappindicator:3
            dev-libs/libappindicator:2

I suppose it would be better to rename USE flag to something else cuz all apps (telegram, skype, slack) display their tray icon with kde-plasma/xembed-sni-proxy installed. So libappindicator dep is is not needed.
Comment 4 Andreas Sturmlechner gentoo-dev 2018-10-24 17:11:17 UTC
(In reply to Ivan Ivanich from comment #3)
> cuz all apps (telegram, skype, slack) display their tray icon with
> kde-plasma/xembed-sni-proxy installed.
...but is that also true of gtk-based packages?
Comment 5 Ivan Grynko 2018-10-24 17:36:13 UTC
(In reply to Andreas Sturmlechner from comment #4)
> (In reply to Ivan Ivanich from comment #3)
> > cuz all apps (telegram, skype, slack) display their tray icon with
> > kde-plasma/xembed-sni-proxy installed.
> ...but is that also true of gtk-based packages?

Slack is gtk based, when I upgraded to plasma-5.14.2 i loose my tray icons for slack and skype then I installed kde-plasma/xembed-sni-proxy manually and all tray icons reappeared.
Comment 6 Andreas Sturmlechner gentoo-dev 2018-10-25 17:32:49 UTC
Let's try without then. Both dev-libs/libappindicator:2 and :3 dependencies are quite old and predate the invention of xembed-sni-proxy.
Comment 7 Larry the Git Cow gentoo-dev 2018-10-25 17:46:06 UTC
The bug has been referenced in the following commit(s):

https://gitweb.gentoo.org/proj/kde.git/commit/?id=f6d28bf38c8eceb246b52a6014b654488eb20fdb

commit f6d28bf38c8eceb246b52a6014b654488eb20fdb
Author:     Andreas Sturmlechner <asturm@gentoo.org>
AuthorDate: 2018-10-25 17:45:50 +0000
Commit:     Andreas Sturmlechner <asturm@gentoo.org>
CommitDate: 2018-10-25 17:45:50 +0000

    kde-plasma/plasma-desktop: Don't mention dev-libs/libappindicator anymore
    
    Bug: https://bugs.gentoo.org/669502
    Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
    Package-Manager: Portage-2.3.51, Repoman-2.3.11

 kde-plasma/plasma-desktop/plasma-desktop-5.14.49.9999.ebuild | 11 -----------
 kde-plasma/plasma-desktop/plasma-desktop-9999.ebuild         | 11 -----------
 2 files changed, 22 deletions(-)

https://gitweb.gentoo.org/proj/kde.git/commit/?id=92b52936f4955ea2d00b95e40ad168b4e9b270c2

commit 92b52936f4955ea2d00b95e40ad168b4e9b270c2
Author:     Andreas Sturmlechner <asturm@gentoo.org>
AuthorDate: 2018-10-25 17:42:19 +0000
Commit:     Andreas Sturmlechner <asturm@gentoo.org>
CommitDate: 2018-10-25 17:42:19 +0000

    kde-plasma/plasma-meta: Finally drop dev-libs/libappindicator
    
    It appears to be unnecessary since the advent of xembed-sni-proxy.
    
    Reported-by: Ivan Ivanich <ivanich@ivanich.org.ua>
    Bug: https://bugs.gentoo.org/669502
    Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
    Package-Manager: Portage-2.3.51, Repoman-2.3.11

 kde-plasma/plasma-meta/plasma-meta-5.14.49.9999.ebuild | 4 ----
 kde-plasma/plasma-meta/plasma-meta-9999.ebuild         | 4 ----
 2 files changed, 8 deletions(-)
Comment 8 Larry the Git Cow gentoo-dev 2018-10-25 18:04:26 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=276183483a6bc9e52b2b7ac4a2c496fd5004f61c

commit 276183483a6bc9e52b2b7ac4a2c496fd5004f61c
Author:     Andreas Sturmlechner <asturm@gentoo.org>
AuthorDate: 2018-10-25 17:45:50 +0000
Commit:     Andreas Sturmlechner <asturm@gentoo.org>
CommitDate: 2018-10-25 18:04:07 +0000

    kde-plasma/plasma-desktop: Don't mention dev-libs/libappindicator anymore
    
    Closes: https://bugs.gentoo.org/669502
    Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
    Package-Manager: Portage-2.3.51, Repoman-2.3.11

 kde-plasma/plasma-desktop/plasma-desktop-5.14.2.ebuild | 11 -----------
 1 file changed, 11 deletions(-)

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=5f0a904bfb8672ba1915f02bc0028b03ef08f8e4

commit 5f0a904bfb8672ba1915f02bc0028b03ef08f8e4
Author:     Andreas Sturmlechner <asturm@gentoo.org>
AuthorDate: 2018-10-25 17:42:19 +0000
Commit:     Andreas Sturmlechner <asturm@gentoo.org>
CommitDate: 2018-10-25 18:04:07 +0000

    kde-plasma/plasma-meta: Finally drop dev-libs/libappindicator
    
    It appears to be unnecessary since the advent of xembed-sni-proxy.
    
    Reported-by: Ivan Ivanich <ivanich@ivanich.org.ua>
    Closes: https://bugs.gentoo.org/669502
    Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
    Package-Manager: Portage-2.3.51, Repoman-2.3.11

 .../{plasma-meta-5.14.2-r1.ebuild => plasma-meta-5.14.2-r2.ebuild}    | 4 ----
 1 file changed, 4 deletions(-)