Re: [PATCH v2 3/7] gnu: Allow OS configurations to add PAM session modules

[Ludovic Courtès]

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?

Thanks,
Ludo’.