apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Brian Havard" <bri...@kheldar.apana.org.au>
Subject Re: cvs commit: apr/build find_apr.m4
Date Fri, 08 Feb 2002 12:46:46 GMT
On 7 Feb 2002 00:38:18 -0000, gstein@apache.org wrote:

>gstein      02/02/06 16:38:18
>
>  Modified:    build    find_apr.m4
>  Log:
>  be a bit more restrictive: look for an exectuable apr-config, rather
>  than just an existing file.
>  
>  Revision  Changes    Path
>  1.8       +1 -1      apr/build/find_apr.m4
>  
>  Index: find_apr.m4
>  ===================================================================
>  RCS file: /home/cvs/apr/build/find_apr.m4,v
>  retrieving revision 1.7
>  retrieving revision 1.8
>  diff -u -r1.7 -r1.8
>  --- find_apr.m4	6 Feb 2002 11:41:57 -0000	1.7
>  +++ find_apr.m4	7 Feb 2002 00:38:18 -0000	1.8
>  @@ -50,7 +50,7 @@
>       if test -x "$withval/bin/apr-config"; then
>         apr_found="yes"
>         apr_config="$withval/bin/apr-config"
>  -    elif test -f "$withval/apr-config"; then
>  +    elif test -x "$withval/apr-config"; then
>         apr_found="yes"
>         apr_config="$withval/apr-config"
>       elif test -x "$withval" && $withval --help > /dev/null 2>&1 ;
then

Was there any actual need to do this? I ask because it breaks the OS/2
build due to the fact that OS/2 has no 'x' bit in the file system. "test
-x" is only true if the file has a .exe extension which apr-config
obviously does not.

-- 
 ______________________________________________________________________________
 |  Brian Havard                 |  "He is not the messiah!                   |
 |  brianh@kheldar.apana.org.au  |  He's a very naughty boy!" - Life of Brian |
 ------------------------------------------------------------------------------


Mime
View raw message