[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#61789] ‘screen-locker-service’ deprecation
From: |
Ludovic Courtès |
Subject: |
[bug#61789] ‘screen-locker-service’ deprecation |
Date: |
Mon, 06 Mar 2023 23:32:14 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) |
Hi,
Bruno Victal <mirai@makinata.eu> skribis:
> On 2023-03-03 16:45, Ludovic Courtès wrote:
>> Bruno Victal <mirai@makinata.eu> skribis:
>>
>>> * doc/guix.texi (X Window): Replace 'screen-locker-service' with
>>> 'screen-locker-service-type'.
>>> Document <screen-locker-configuration>.
>>> * gnu/services/desktop.scm (desktop-services-for-system): Use
>>> screen-locker-service-type.
>>> * gnu/services/xorg.scm: Export accessors for <screen-locker-configuration>.
>>> (<screen-locker>): Rename to ...
>>> (<screen-locker-configuration>): ... this.
>>> (<screen-locker-configuration>)[empty?]: Rename to ...
>>> (<screen-locker-configuration>)[allow-empty-password?]: ... this.
>>> (screen-locker-pam-services): Update record name.
>>> (screen-locker-setuid-programs): Update accessor name.
>>> (screen-locker-service): Deprecate procedure.
>>
>> Also leaving this one out: there’s a bit too much going on here, and I
>> think we should discuss it separately (it’s quite different from the
>> old-style service procedures.)
>
> The changes essentially amount to a rename of the configuration record-type
> (with backward compatibility)
> and making explicit what the 'screen-locker-service' procedure does behind
> the scenes with 'screen-locker-service-type'.
>
> Do you have another plan in mind to split or refactor this change?
Actually no. :-) Just thinking we should discuss it separately because
it looks less “obvious” to me than the other patches.
Ludo’.