Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 275321 - x11-misc/electricsheep doesn't add itself to app-defaults/XScreenSaver
Summary: x11-misc/electricsheep doesn't add itself to app-defaults/XScreenSaver
Status: RESOLVED WONTFIX
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it
URL:
Whiteboard:
Keywords: NeedPatch
Depends on:
Blocks:
 
Reported: 2009-06-24 20:13 UTC by Maik Nijhuis
Modified: 2016-11-09 10:34 UTC (History)
2 users (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 Maik Nijhuis 2009-06-24 20:13:28 UTC
After emerging x11-misc/electricsheep, Electrichsheep does not show up in the screensaver list in xscreensaver-demo when you have no .xscreensaver configuration file. xscreensaver-demo generates a standard .xscreensaver file which does not contain a reference to electricsheep.

When you run xscreensaver-demo -debug you see it only scans the
/usr/share/xscreensaver/config directory for the screensavers listed in .xscreensaver. It does *not* notice other files, like electricsheep.xml.

After adding a line for electricsheep in the programs: section of .xscreensaver, xscreensaver-demo reads electricsheep.xml again and Electricsheep is listed and can be configured, like before. Just copy-paste one of the other lines without GL: in it, and change the command (including '-root') to 'electricsheep'

xscreensaver-5.07 has the same problem.

Reproducible: Always

Steps to Reproduce:
1.emerge xscreensaver and electricsheep
2.Create new user account, or remove/rename your .xscreensaver file.
3.Start xscreensaver-demo

Actual Results:  
No Electricsheep in screensaver list.
Comment 1 Doktor Notor 2010-02-25 22:20:18 UTC
Not fixed with x11-misc/xscreensaver-5.10 either.
Comment 2 Samuli Suominen (RETIRED) gentoo-dev 2010-02-25 22:27:50 UTC
Look at pkg_post{inst,rm} of x11-misc/rss-glx for hint how this should be solved. It's not a bug in xscreensaver.
Comment 3 Doktor Notor 2010-02-25 22:42:28 UTC
(In reply to comment #2)
> Look at pkg_post{inst,rm} of x11-misc/rss-glx for hint how this should be
> solved. It's not a bug in xscreensaver.

Uh, that's just horrible. You are mangling a configuration file installed by another package? o_O 

How's that a solution and what's the xml configuration file (/usr/share/xscreensaver/config/electricsheep.xml) for when xscreensaver ignores it? Broken by design pretty much.
Comment 4 Doktor Notor 2010-02-25 22:49:17 UTC
/usr/share/xscreensaver/config/README

<snip>
This directory contains XML files that describe each of the screenhacks;
the per-hack user interface is constructed based on the things in these
files.  The files are *loaded* at *run-time* by *xscreensaver-demo* (also
known as "the Control Center screensaver properties capplet".)
</snip>

So, this clearly is a xscreensaver bug.
Comment 5 Samuli Suominen (RETIRED) gentoo-dev 2010-02-25 23:13:51 UTC
(In reply to comment #4)
> /usr/share/xscreensaver/config/README
> 
> <snip>
> This directory contains XML files that describe each of the screenhacks;
> the per-hack user interface is constructed based on the things in these
> files.  The files are *loaded* at *run-time* by *xscreensaver-demo* (also
> known as "the Control Center screensaver properties capplet".)
> </snip>
> 
> So, this clearly is a xscreensaver bug.
> 

/usr/share/X11/app-defaults/XScreenSaver:

! This is the list of installed screen saver modes.  See "man xscreensaver"
! for the syntax used here.

[ .. ]

*programs:  

[ .. ]

Not a xscreensaver bug.
Comment 6 Doktor Notor 2010-02-25 23:35:15 UTC
(In reply to comment #5)
> ! This is the list of installed screen saver modes.  See "man xscreensaver"
> ! for the syntax used here.

Yeah, and a couple of lines below it says:

! You can use the `xscreensaver-demo' program to edit the current list of
! screen savers interactively.

which you clearly can't since anything you install beyond the default stuff is wrongly ignored. Also, there's no 'edit list' feature implemented at all in xscreensaver-demo, you can select a screensaver from the built-in hardcoded list and that's it. The "documentation" is useless.

Whatever, I really have better things to do than arguing about screensavers. The only important thing here is that the suggested "solution" is worse than no solution at all. Among others, this nasty hack that works around a clear bug/misdesign of xscreensaver *orphans* the file you are mangling and should be just removed.
Comment 7 Jeroen Roovers (RETIRED) gentoo-dev 2010-02-26 00:43:29 UTC
Despite the availability of electricsheep and rss-glx and other xscreensaver compatible "hacks", xscreensaver's author has never come up with a clean way to integrate those third party "hacks" so the only current option is to mangle that file. If you have a patch to be able to have /usr/share/X11/app-defaults/XScreenSaver include other files, or to make that file obsolete and to just read the invididual .xml files instead, then please send that to xscreensaver's author. Meanwhile, we have a distro to make, and apparently Gentoo isn't the only distro to work around the problem in this manner.
Comment 8 Pacho Ramos gentoo-dev 2016-11-09 10:34:49 UTC
removed