On Mon, May 23, 2016 at 6:37 PM, Laurent Bigonville wrote: > On Tue, 21 Jul 2015 16:40:25 +0200 Tormod Volden > wrote: >> >> > When running GNOME Shell and xscreensaver modules such as >> > xscreensaver-data- >> > extra and others are added to xscreensaver, the GNOME Application Menu >> > list >> > gets cluttered up with useless icons for each of the many Display Modes >> > that >> > are installed. >> >> I think this will be fixed now by not displaying the hacks in GNOME >> but only in MATE. >> >> >> http://anonscm.debian.org/cgit/collab-maint/xscreensaver.git/commit/?id=47dfa5456abac1773b18a9607a59bdf58ca3fc14 > > Are you sure this is the correct fix?
Well, it was /a/ correct /fix/ for the problem that, functionally, Gnome was renamed to Mate, and something else got the name Gnome. However, whether it is the final /solution/, and for the issue at hand in this bug report, I am not sure. > MATE already has a screensaver (mate-screensaver). > > OTHO, xfce seems to rely on xscreensaver, so shoudn't it "OnlyShowIn=XFCE;" > instead? These desktop files have nothing to do with having or using or relying on xscreensaver. xscreensaver does not use them at all. IIRC they got introduced because the xscreensaver hacks could then be used by e.g. gnome-screensaver. BTW, is mate-screensaver what was gnome-screensaver before? Furthermore, these hacks are freestanding applications that can run on their own and display something funny or beautiful on the screen. It therefore would make sense that they have desktop files. The desktop files have a "Screensaver" category, so it is up the fancy file/menu manager to display them or not, or show them in a separate category. So this "cluttering up" issue is really a defect in the affected desktop environment. As I said xscreensaver does not use them, so for xscreensaver itself we could stop shipping them altogether. But I think it is the choice of the users, if they choose to install these hacks, it is in principle a plus that they can browse them in some sort of file/app view without having to go to the command line or look at the binaries in /usr/lib/xscreensaver. Cheers, Tormod > Cheers, > > Laurent Bigonville
