[Top][All Lists]

[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

From: Ludovic Courtès
Subject: Re: [PATCH v2 3/7] gnu: Allow OS configurations to add PAM session modules
Date: Tue, 25 Aug 2015 23:39:48 +0200
User-agent: 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:
>>   (operating-system
>>     ;; ...
>>     (pam-services (map (lambda (service)
>>                          (pam-service
>>                            (inherit service)
>>                            (session (cons ...))))
>>                        (base-pam-services))))
>> 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?


reply via email to

[Prev in Thread] Current Thread [Next in Thread]