Re: PAM authentication patch - v2

Brian Murphy wrote:

> This is, I'm afraid, very system dependant. Sun has no /etc/pam.d, just
> an /etc/pam.conf and also requires
> (it seems) a full path to the module:
>
> login auth required /usr/lib/security/$ISA/pam_unix.so.1
>
> the ISA variable is to handle 32 and 64 bit variants. Linux has both
> variants and other systems
> probably have their own quirks. It seems like asking for trouble to try
> and automate this. All
> systems have a default "other" PAM configuration which cvs would use if
> there was no specific
> cvs configuration. 


Hrm.  Okay.  I'm not _too_ worried about incompatible systems at the 
moment, though we could deal with that using configure, at least in 
theory.  I may write one that works on my RH Linux before I commit this. 
 We can let users with other requirements submit patches, perhaps.

> Here is the updated texinfo file.


That looked good.  You missed a few of the texinfo features we use, but 
I'll add the ones I don't miss before I commit this.  :)

Speaking of committing, if I read the discussion correctly and noone 
changed their mind without saying so, we're still at +1 developer votes:

    Larry: +1
    Mark:  -1
    Me:    +1
    _________
    Total: +1


Any further objections to this going into experimental before I test it 
here and commit it?

Derek

-- 
                *8^)

Email: [email protected]

Get CVS support at <http://ximbiot.com>!
-- 
I will not cut corners.
"   "   "   "     "
"   "   "   "     "   ...

          - Bart Simpson on chalkboard, _The Simpsons_