[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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

From: Derek Robert Price
Subject: Re: Compile error in current CVS, src/server.c:5500
Date: Wed, 23 Jul 2003 14:51:06 -0400
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.4) Gecko/20030624 Netscape/7.1

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.



Email: derek@ximbiot.com

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

reply via email to

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