You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@httpd.apache.org by Ryan Bloom <rb...@covalent.net> on 2002/04/18 19:56:36 UTC

RE: cvs commit: httpd-2.0/build fastgen.sh

> jerenkrantz    02/04/18 10:49:42
> 
>   Modified:    build    fastgen.sh
>   Log:
>   The pedant in me wishes to see all extraneous trailing / go away.
>   So, if we are in the top-level makefile, don't add an extra trailing
>   slash to srcdir, builddir, or VPATH.  It's annoying.

WHAT?????  This patch removes an "annoying" trailing slash that doesn't
hurt anything and instead inserts an if condition which means that this
logic must essentially be maintained twice.  Is there a valid technical
reason for this change, or is it personal preference?  If it's personal
preference, then please back it out, this just makes maintenance harder.

Ryan

Re: cvs commit: httpd-2.0/build fastgen.sh

Posted by Justin Erenkrantz <je...@apache.org>.
On Thu, Apr 18, 2002 at 10:56:36AM -0700, Ryan Bloom wrote:
> WHAT?????  This patch removes an "annoying" trailing slash that doesn't
> hurt anything and instead inserts an if condition which means that this
> logic must essentially be maintained twice.  Is there a valid technical
> reason for this change, or is it personal preference?  If it's personal
> preference, then please back it out, this just makes maintenance harder.

IMHO, we should strive for being correct - adding extra slashes is
very sloppy behavior.  I'd prefer us to be not sloppy at the cost of
a little more maintenance - and really how often is this code changed?

I believe that the whole fastgen.sh code is an obstacle to maintaining
our build system.  It attempts to circumvent the autoconf code and
ends up hurting us more than it helps.  -- justin

Re: cvs commit: httpd-2.0/build fastgen.sh

Posted by Justin Erenkrantz <je...@apache.org>.
On Thu, Apr 18, 2002 at 10:56:36AM -0700, Ryan Bloom wrote:
> > jerenkrantz    02/04/18 10:49:42
> > 
> >   Modified:    build    fastgen.sh
> >   Log:
> >   The pedant in me wishes to see all extraneous trailing / go away.
> >   So, if we are in the top-level makefile, don't add an extra trailing
> >   slash to srcdir, builddir, or VPATH.  It's annoying.
> 
> WHAT?????  This patch removes an "annoying" trailing slash that doesn't
> hurt anything and instead inserts an if condition which means that this
> logic must essentially be maintained twice.  Is there a valid technical
> reason for this change, or is it personal preference?  If it's personal
> preference, then please back it out, this just makes maintenance harder.

I hope that my last commit to build/fastgen.sh addressed your 
concerns.  -- justin