! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
xflock4: Add X11R6 to PATH and add support for xidle


Description Landry Breuil editbugs 2012-05-04 20:55:49 CEST
On OpenBSD (and maybe other systems) xscreensaver-command/gnome-screensaver-command are in usr/local/bin and xset/xlock are in usr/X11R6/bin. As of now xflock4 fails to find them and is thus useless.

While here add support for xidle, which is in OpenBSD's base system and also packaged (at least) in FreeBSD and Gentoo.
Comment 1 Landry Breuil editbugs 2012-05-04 20:56:53 CEST
Created attachment 4394 
add X11R6/local to PATH and add support for xidle
Comment 2 Guido Berhoerster 2012-05-06 11:39:28 CEST
This approach is broken, pkill is not only non-portable but should also never be used from a script like this because the argument is interpreted as a regular expression which may match any process whose command contains the given substring "xidle". Apart from that using xidle is superfluous, it cannot lock the screen itself anyway and relies on something like xlock and it might even execute some other command given by the user which does not lock the screen at all.
Comment 3 Landry Breuil editbugs 2012-05-08 12:02:08 CEST
Created attachment 4407 
add /usr/{local,X11R6}/bin to PATH

Okay, makes sense. Let's just fix the PATH then.
Comment 4 Jarno Suni 2016-01-28 21:15:24 CET
Are those additional PATH items not needed anymore? They seem to have been removed from Git.
Comment 5 Jarno Suni 2016-01-29 10:09:43 CET
Would fixing Bug 12282 do?
Comment 6 Skunnyk editbugs 2019-04-29 00:32:07 CEST
I this issue is fixed with 4.13.x. Now xflock call xfconf for the default screensaver to run, so you can set whatever binary you want: https://git.xfce.org/xfce/xfce4-session/tree/scripts/xflock4#n27
Comment 7 Landry Breuil editbugs 2019-04-29 08:24:23 CEST
saome comment as https://bugzilla.xfce.org/show_bug.cgi?id=12282#c9 :) here xfce4-screensaver is in usr/local/bin, so if xfconf has no value, xflock will fail to find it..
Comment 8 Skunnyk editbugs 2019-05-13 13:37:10 CEST

*** This bug has been marked as a duplicate of bug 12282 ***

Bug #8830

Reported by:
Landry Breuil
Reported on: 2012-05-04
Last modified on: 2019-05-13


CC List:
5 users




add X11R6/local to PATH and add support for xidle (925 bytes, patch)
2012-05-04 20:56 CEST , Landry Breuil
landry : review? ( nick )
add /usr/{local,X11R6}/bin to PATH (680 bytes, patch)
2012-05-08 12:02 CEST , Landry Breuil
landry : review? ( nick )

Additional information