Re: Compile error in current CVS, src/server.c:5500

[Derek Robert Price]

Brian Murphy wrote:

> Derek Robert Price wrote:
>
> > I think you resent your previous patch.  I added a comment I missed 
> > the first time through anyhow.  Also, please don't forget your 
> > ChangeLog entry.
> How was the last patch I sent? 


Sorry, thought you were still working from my previous message.  I'd 
still like to see each failed call to PAM output a different error 
message on the CVS side, as per  this message: 
<http://groups.google.com/groups?dq=&hl=en&lr=&ie=UTF-8&oe=UTF-8&frame=right&th=2488a285b730dd65&seekm=mailman.300.1058817765.8231.bug-cvs%40gnu.org#link16>.

Also, please split strings as Mark and I were discussing:

   "this is the first line\n"
   "this is the second line\n"
   "this is the third line\n"
   ...

I'll update HACKING soon to specify this style.

Other than that your patch looked pretty good.

Derek

--
               *8^)

Email: derek@ximbiot.com

Get CVS support at <http://ximbiot.com>!
--
This sentance has threee errors.