<div dir="ltr"><div>Hi,</div><div><br></div><div>><span class="gmail-im"></span> You can speak for yourself, of course. Could you be more specific in that what kind of small(er) steps you would prefer? (I edited the merge request BTW)</div><div><br></div><div>I would prefer not to meddle with DPMS and not add too many options that take the special behavior of many screensavers/lockers into account, because it implies we test them all. (And yes, I only speak for myself, even if I misleadingly said "we" in my previous message. I was merely relating to what I remember from the project's history. If anyone else wants to review/merge that's fine!)</div><div><br></div><div>> <span class="gmail-im"></span>The script has GPL license so it is not much different than C code in that sense. I agree that writing a portable shell script is demanding, but I have tried to do it anyway. There are numerous issues concerning desktop locking in xfce4-power-manager, xfce4-session and xfce4-screensaver projects. Who takes responsibility of them? My merge request actually aims to resolve some of those issues.</div><div><br></div><div>My comment was less about whether it's shell or C, but that I'd prefer that e.g. whether to handle DPMS or not when locking or idling should be decided by the screensaver/locker, not by Xfce and through some hidden settings that we wouldn't want to expose to the user. (At least some screenlockers do seem to take that into account and it does make some sense, as they control when the systems goes into locked state and comes out of it.)</div><div><br></div><div>I don't mind resolving or reducing the overlaps between session or power-manager. The "lock command" setting I introduced with Erik and the shared "Lock screen when system goes to sleep" setting between the two components were some baby steps in that direction already. For me personally it's just a matter of time and motivation...</div><div><br></div><div>Cheers</div><div>Simon<br></div><div><br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, May 2, 2021 at 12:21 PM Jarno Suni <<a href="mailto:sunijarno@gmail.com">sunijarno@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Thu, Apr 29, 2021 at 10:41 AM Simon Steinbeiss <<a href="mailto:simon@xfce.org" target="_blank">simon@xfce.org</a>> wrote:<br>
<br>
> Regarding your topic - I'm aware of the long-time pending patches for xflock4, but there were always reservations about building an "intelligent lock script". Simply summarized: putting something as sensitive as locking on the shoulders of a (then: complex) bash script doesn't sound like something we want to maintain or be responsible for. So we basically stuck to the "simple/dumb" solution that xflock4 is today.<br>
<br>
The script has GPL license so it is not much different than C code in<br>
that sense. I agree that writing a portable shell script is demanding,<br>
but I have tried to do it anyway. There are numerous issues concerning<br>
desktop locking in xfce4-power-manager, xfce4-session and<br>
xfce4-screensaver projects. Who takes responsibility of them? My merge<br>
request actually aims to resolve some of those issues.<br>
_______________________________________________<br>
Xfce4-dev mailing list<br>
<a href="mailto:Xfce4-dev@xfce.org" target="_blank">Xfce4-dev@xfce.org</a><br>
<a href="https://mail.xfce.org/mailman/listinfo/xfce4-dev" rel="noreferrer" target="_blank">https://mail.xfce.org/mailman/listinfo/xfce4-dev</a></blockquote></div>