[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 8/9] services: Add sddm service.
From: |
Ludovic Courtès |
Subject: |
Re: [PATCH 8/9] services: Add sddm service. |
Date: |
Sat, 03 Sep 2016 15:32:36 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Hi!
address@hidden (宋文武) skribis:
> [...]
>> +
>> +(define (sddm-activation config)
>> + "Return the activation gexp for CONFIG."
>> + #~(begin
>> + (symlink #$(sddm-configuration-file config) "/etc/sddm.conf")))
>
> I don't know the behavior of activation when reconfigure services,
‘guix system reconfigure’ runs the new system’s activation script.
> but this `symlink' call will fail when the file already exists.
Indeed.
> And to put files to /etc, it's better to extend the `etc-service-type'.
Agreed.
However, it’s even better to avoid populating /etc altogether; ideally
/etc should contain only global configuration files, such as
nsswitch.conf.
Could you instead pass the configuration file via a command-line option
or environment variable?
> Oh, finally we get a modern display manager, thanks!
Seconded. Thank you!
Ludo’.
[PATCH 8/9] services: Add sddm service., David Craven, 2016/09/01
[PATCH 3/9] services: Export initrc., David Craven, 2016/09/01
[PATCH 5/9] gnu: Add prefix to license imports in (gnu packages display-managers)., David Craven, 2016/09/01