* PCMan <pcman...@gmail.com> [2010-09-14 15:48]:
> > No, running "gnome-screensaver-command -l" without
> > gnome-screensaver running will print
> > **Message: Screensaver is not running!
> > and return exit code 1.
> This is not the case on my box.
> It tried to launch gnome-screensaver even if this command failed.
> I'm using ubuntu 10.04.
> 
> Please read the source code of gnome-screesaver-command.
> Is doesn't do what you said.
> 
> http://git.gnome.org/browse/gnome-screensaver/tree/src/gnome-screensaver-command.c
> 
> It exit with code 1 only when you pass wrong arguments, when dbus is
> not available, and when you query the version number.
> It return 0 even if gnome-screensaver is not running.
> The dbus call, however, will cause the gnome-screensaver service being 
> launched.
> I don't know the version of gnome-screensaver you're using, but from
> the latest code in git, it doesn't not work the way we want.

I admit you are right on this one, the DBus autostart seems to
have been introduced in April:
http://git.gnome.org/browse/gnome-screensaver/commit/?id=47155576403fea9339a1f702e3dc63455b763fb8

They more or less silently changed the behavior of
gnome-screensaver-command breaking shell scripts (including one
of mine for inhibiting the screensaver while running an
application) relying on the previous behavior which was modeled
after xscreensaver-command, just lovely.

If someone has gnome-screensaver installed and is running another
screensaver I don't see any way to tell from a shellscript which
one to use now.  Looking for a process is just plainly broken and
unreliable by principle, e.g. someome might suspend before the
session has started gnome-screensaver, see
https://bugzilla.gnome.org/show_bug.cgi?id=616225

For lxsession I'd propose to use my script and remove
gnome-screensaver-command from the list.  If someone wants to use
it, it can still be added as an explicit locking command in the
config file.

Since I find changing the cli silently in a minor version upgrade
unacceptable I am also going to file a bug against
gnome-screensaver.
-- 
Guido Berhoerster

------------------------------------------------------------------------------
Start uncovering the many advantages of virtual appliances
and start using them to simplify application deployment and
accelerate your shift to cloud computing.
http://p.sf.net/sfu/novell-sfdev2dev
_______________________________________________
Lxde-list mailing list
Lxde-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/lxde-list

Reply via email to