Return-Path: Delivered-To: new-httpd-archive@hyperreal.org Received: (qmail 22998 invoked by uid 6000); 11 Apr 1998 15:00:57 -0000 Received: (qmail 22991 invoked from network); 11 Apr 1998 15:00:55 -0000 Received: from eastwood.aldigital.algroup.co.uk (194.128.162.193) by taz.hyperreal.org with SMTP; 11 Apr 1998 15:00:55 -0000 Received: from freeby.ben.algroup.co.uk (freeby.ben.algroup.co.uk [193.133.15.6]) by eastwood.aldigital.algroup.co.uk (8.6.12/8.6.12) with ESMTP id PAA08251 for ; Sat, 11 Apr 1998 15:00:08 GMT Received: from algroup.co.uk (naughty.ben.algroup.co.uk [193.133.15.107]) by freeby.ben.algroup.co.uk (8.6.12/8.6.12) with ESMTP id PAA14600 for ; Sat, 11 Apr 1998 15:59:55 +0100 Message-ID: <352F8558.EBD3575F@algroup.co.uk> Date: Sat, 11 Apr 1998 15:59:36 +0100 From: Ben Laurie Organization: A.L. Digital Ltd. X-Mailer: Mozilla 4.04 [en] (WinNT; I) MIME-Version: 1.0 To: new-httpd@apache.org Subject: Re: TestCompile, HP-UX and -r/-x References: <199804110058.UAA07677@devsys.jaguNET.com> Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7bit Sender: new-httpd-owner@apache.org Precedence: bulk Reply-To: new-httpd@apache.org Jim Jagielski wrote: > > Ben Laurie wrote: > > > > Jim Jagielski wrote: > > > > > > Here's a quick little fix... > > > > > > Index: src/helpers/TestCompile > > > =================================================================== > > > RCS file: /export/home/cvs/apache-1.3/src/helpers/TestCompile,v > > > retrieving revision 1.9 > > > diff -u -r1.9 TestCompile > > > --- TestCompile 1998/03/05 12:42:30 1.9 > > > +++ TestCompile 1998/04/10 22:55:52 > > > @@ -120,8 +120,9 @@ > > > # Now run that Makefile > > > eval "make $TARGET >/dev/null $ERRDIR" > > > > > > -# And see if dummy exists, if so, then we assume the > > > -# condition we are testing for is good > > > -if [ -f $TARGET ]; then > > > +# And see if dummy exists and is executable, if so, then we > > > +# assume the condition we are testing for is good > > > +TESTTARG=`PATH=".:$PATH" ./PrintPath $TARGET` > > > > Surely this should just be .? > > > > I'm guessing you mean the PATH part? Well, the problem is that > PrintPath uses 'cat' instead of just '/bin/cat' (ie: it > internally uses $PATH) so if we set PATH to '.' then PrintPath > will fail because ./cat doesn't exist. We could assume cat lives > in /bin but... > > That's why it does the mucking around... if we could assume all > cats were in /bin we could fix PrintPath and just > > if PATH=`pwd` ./PrintPath -s $TARGET > > which would be cleaner... But I wouldn't want to assume that > and cause Configure to start barfing on weird systems where it > worked before (and we'll need to worry about where it is... Ugg). Hmmm ... guess we need to set something different, like: PRINT_PATH=`pwd` ./PrintPath -s $TARGET and have PrintPath mess with PATH as appropriate. The snag being, of course, that if $TARGET happens to be on the path somewhere it all goes horribly wrong. Cheers, Ben. -- Ben Laurie |Phone: +44 (181) 735 0686| Apache Group member Freelance Consultant |Fax: +44 (181) 735 0689|http://www.apache.org and Technical Director|Email: ben@algroup.co.uk | A.L. Digital Ltd, |Apache-SSL author http://www.apache-ssl.org/ London, England. |"Apache: TDG" http://www.ora.com/catalog/apache