Received: (qmail 21395 invoked by uid 2012); 11 Nov 1999 18:39:07 -0000
Message-Id: <19991111183907.21390.qmail@hyperreal.org>
Date: 11 Nov 1999 18:39:07 -0000
From: Matthias Lohmann <lohm@lynet.de>
Reply-To: lohm@lynet.de
To: apbugs@hyperreal.org
Subject: Editting variebles in the configure script.
X-Send-Pr-Version: 3.2

>Number:         5291
>Category:       config
>Synopsis:       Editting variebles in the configure script.
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    apache
>State:          open
>Class:          sw-bug
>Submitter-Id:   apache
>Arrival-Date:   Thu Nov 11 10:40:03 PST 1999
>Last-Modified:  Sun Nov 28 03:20:03 PST 1999
>Originator:     lohm@lynet.de
>Organization:
apache
>Release:        1.3.9
>Environment:
Solaris, but does not matter. All Unixes affected.
>Description:
I just discovered a bug in the configure script of apache 1.3.9.
At least it was a bug for me ..............

From line 1034 in the script ............

[...]

OIFS="$IFS"
IFS="$DIFS"
for var in prefix exec_prefix bindir sbindir libexecdir mandir \
           sysconfdir datadir iconsdir htdocsdir cgidir includedir \
           localstatedir runtimedir logfiledir proxycachedir \
           suexec_docroot suexec_logexec; do
    eval "val=\"\$$var\"";
***    [ X$val != X"/" ] && val=`echo $val | sed -e 's:/*$::'`
    eval "$var=\"$val\""
    #   expand value
    eval "val=\$$var"
 

[...]


I changed the line marked with ***. Before my changes it reads 

val=`echo $val | sed -e 's:/*$::'`

But I got into a problem, when one of the variables above was set to
just '/' (for me especially the suexec_docroot), but perhaps
for others too if they like to install (for what reason ever) apache
in / or set other paths to /

I change the line line shown at ***, so that the variable will keep
the '/' if it is explicit set to '/', the trailing '/' are still removed
if the are some.

If you like, please include it in future versions.

Kind regards
Matthias Lohmann
>How-To-Repeat:
Just configure with f.ex. suexec-docroot=/
>Fix:
See my fix in the description.
>Audit-Trail:

From: "Ralf S. Engelschall" <rse@engelschall.com>
To: apbugs@apache.org
Cc:  Subject: Re: config/5291: Editting variebles in the configure script.
Date: Sun, 28 Nov 1999 12:17:35 +0100

 In article <19991111183907.21390.qmail@hyperreal.org> you wrote:
  
 >>Number:         5291
 >>Category:       config
 >>Synopsis:       Editting variebles in the configure script.
 > [...]
 > But I got into a problem, when one of the variables above was set to
 > just '/' (for me especially the suexec_docroot), but perhaps
 > for others too if they like to install (for what reason ever) apache
 > in / or set other paths to /
 
 This is now fixed for Apache 1.3.10.
 Thanks for your feedback.
                                        Ralf S. Engelschall
                                        rse@engelschall.com
                                        www.engelschall.com
>Unformatted:
[In order for any reply to be added to the PR database, you need]
[to include <apbugs@Apache.Org> in the Cc line and make sure the]
[subject line starts with the report component and number, with ]
[or without any 'Re:' prefixes (such as "general/1098:" or      ]
["Re: general/1098:").  If the subject doesn't match this       ]
[pattern, your message will be misfiled and ignored.  The       ]
["apbugs" address is not added to the Cc line of messages from  ]
[the database automatically because of the potential for mail   ]
[loops.  If you do not include this Cc, your reply may be ig-   ]
[nored unless you are responding to an explicit request from a  ]
[developer.  Reply only with text; DO NOT SEND ATTACHMENTS!     ]