You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@apr.apache.org by Justin Erenkrantz <ju...@erenkrantz.com> on 2003/02/11 17:51:26 UTC
Re: cvs commit: apr configure.in
[ Resend from correct address... ]
--On Tuesday, February 11, 2003 3:36 PM +0000 jorton@apache.org wrote:
> jorton 2003/02/11 07:36:56
>
> Modified: . configure.in
> Log:
> Capitalize some "Posix"es and s/dnl/# a comment.
Why are you removing the dnl's? Comments should only be dnl not #.
Adding the #'s doesn't help anyone. They'll just be added to the
unintelligble cruft of configure. -- justin
Re: cvs commit: apr configure.in
Posted by Joe Orton <jo...@manyfish.co.uk>.
On Tue, Feb 11, 2003 at 08:51:26AM -0800, Justin Erenkrantz wrote:
> [ Resend from correct address... ]
>
> --On Tuesday, February 11, 2003 3:36 PM +0000 jorton@apache.org wrote:
>
> >jorton 2003/02/11 07:36:56
> >
> > Modified: . configure.in
> > Log:
> > Capitalize some "Posix"es and s/dnl/# a comment.
>
> Why are you removing the dnl's? Comments should only be dnl not #.
> Adding the #'s doesn't help anyone. They'll just be added to the
> unintelligble cruft of configure. -- justin
They can be useful when debugging the configure script, and help make it
a little bit intelligible ;)
joe
Re: cvs commit: apr configure.in
Posted by Aaron Bannert <aa...@clove.org>.
Sometimes it's useful to have comments in the configure cruft, but yeah
the dnl's should stay.
-aaron
On Tuesday, February 11, 2003, at 08:51 AM, Justin Erenkrantz wrote:
> --On Tuesday, February 11, 2003 3:36 PM +0000 jorton@apache.org wrote:
>
>> jorton 2003/02/11 07:36:56
>>
>> Modified: . configure.in
>> Log:
>> Capitalize some "Posix"es and s/dnl/# a comment.
>
> Why are you removing the dnl's? Comments should only be dnl not #.
> Adding the #'s doesn't help anyone. They'll just be added to the
> unintelligble cruft of configure. -- justin