On Tue, 16 Mar 2010 22:14:06 +0100
Christoph Wickert <christoph.wickert(a)googlemail.com> wrote:
How about removing the screensaver part from xinitrc and let
xscreensaver start through /etc/xdg/autostart? Then we can just
rm /etc/xdg/autostart/xscreensaver-autostart.desktop in the livesys
initscript.
I just make a ~/.xscreensaverrc for the liveuser that disables lock.
Do you see any issues with this?
(see my commit to the kickstart).
> > No, we patched it to prefer gnome-screensaver back in
4.4.3.
> > Check the xfce-utils-4.4.3-gnome-screensaver.patch in xfce-utils.
> > The patch not only changes the default from x-ss to g-ss but also
> > makes xflock check which one is running instead of which one is
> > installed. We should at least keep that part because it fixes
> >
http://bugzilla.xfce.org/show_bug.cgi?id=3770
>
> Yeah, so we need to revert it back to prefer xscreensaver. I can
> look at doing this soon.
I have cleaned up the patch, it only affects xflock now.
Excellent.
> > > 3. Do we need to make any changes to the prefs setup
to only
> > > deal with xscreensaver?
> >
> > For me the question is: Do we want /etc/xdg/xfce4/xinitrc to
> > start a screensaver? Both x-ss and g-ss already have a desktop
> > file in /etc/xdg/autostart/.
>
> Good point.
Yeah, but unfortunately both have "OnlyShowIn=GNOME;", but we can
change this easily.
Can you file bugs on it? Or would you like me to?
Oh, I guess we only want xscreensaver fixed. :)
This would work nicely to not allow people to accidentally use
gnome-screensaver. ;)
Not our fault. People than can at least configure whether they want
to
start a screensaver or not. Currently they need a custom version if
xinitrc in their homedir and this is not very comfortable.
So my suggestion is to remove all the screensaver stuff from the
startup script. Everybody fine with it? If not I will go ahead.
Sounds ok to me. If we can get the OnlyShowIn fixed at the same time in
xscreensaver that would be ideal.
Let me know if there is anything more I can do from here.
Regards,
Christoph
kevin