[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 3/7] gnu: Allow OS configurations to add PAM session modul
Re: [PATCH v2 3/7] gnu: Allow OS configurations to add PAM session modules
Tue, 25 Aug 2015 23:39:48 +0200
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux)
Andy Wingo <address@hidden> skribis:
> On Tue 25 Aug 2015 16:55, address@hidden (Ludovic Courtès) writes:
>> Andy Wingo <address@hidden> skribis:
>>> * gnu/services/base.scm (mingetty-service):
>>> * gnu/services/xorg.scm (slim-service):
>>> * gnu/services/ssh.scm (lsh-service):
>>> * gnu/system/linux.scm (unix-pam-service, base-pam-services): Add
>>> #:additional-session-modules keyword argument.
>> I wonder if we really need #:additional-session-modules passed around.
>> My first suggestion would be to do ‘unix-pam-service’ or
>> ‘base-pam-services’ in the OS declaration along these lines:
>> ;; ...
>> (pam-services (map (lambda (service)
>> (inherit service)
>> (session (cons ...))))
>> But maybe that turned out to be inconvenient? If so, perhaps we could
>> solve it by introducing helper procedures, like
>> ‘add-pam-service-session’ or something?
>> Or am I missing something? :-)
>> BTW, I realize we should rename (gnu system linux) to (gnu system pam)
>> and then document it in the manual.
> How would that work for other services like slim, mingetty, etc?
Oh, it wouldn’t.
Just to help me understand, could you explain the typical use case you
have in mind?
[PATCH v2 4/7] gnu: polkit: Use elogind for seat management., Andy Wingo, 2015/08/20