Re: [PATCH] Add PAM support

[Derek R. Price]

Steve McIntyre wrote:

> The patch here adds PAM support to CVS, so it now works for us in an
> all-PAM (LDAP) environment:

Did I miss something?  It appears you've added some includes and two new
functions, but no calls to these functions.

Also, though we can probably dispense with the test cases, the HAVE_PAM should
be tied into the configure script and this deserves comments in the news file.

Derek

--
Derek Price                      CVS Solutions Architect ( http://CVSHome.org )
mailto:dprice@collab.net         CollabNet ( http://collab.net )
--
I have heard of your paintings too, well enough.
God has given you one face, and you make yourselves
another.  You jig and amble, and you lisp.  You nick-
name God's creatures and make your wantonness your
ignorance.  Go to, I'll no more on't.  It hath made me
mad.  I say we will have no more marriage.  Those that
are married already - all but one - shall live.  The rest
shall keep as they are.  To a nunnery, go.

     - Hamlet, Act III, Scene 1, Lines 143-150