apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jim Jagielski <...@jaguNET.com>
Subject Re: cvs commit: apr/build apr_common.m4
Date Thu, 14 Mar 2002 03:27:38 GMT
I'm following up... 

Greg Stein wrote:
> 
> On Thu, Mar 14, 2002 at 12:50:43PM +1000, Brian Havard wrote:
> > On Wed, 13 Mar 2002 20:41:12 -0500 (EST), Jim Jagielski wrote:
> > 
> > >Weird... I'm guessing you don't have problems with the other
> > >usages of ${}... $3 should be special enough not to require.
> > >I'll back this out since shells that *don't* support it
> > >are more broken :)
> > 
> > Thanks (wow, that was quick :)
> > Isn't it an M4 expansion that's going on here though? The $3 represents the
> > third parameter to the APR_PATH_RELATIVE M4 macro, not to the shell script.
> 
> Yup.
> 
> I think we need more info on the original report to figure out what was
> going on, and to find a good way to solve it. We might need an temporary
> shell variable to hold some of the $3 output.
> 
> Another change might be to not use '#' for the sed regex terminators. Use
> something like ','.
> 
> Cheers,
> -g
> 
> -- 
> Greg Stein, http://www.lyra.org/
> 


-- 
===========================================================================
   Jim Jagielski   [|]   jim@jaguNET.com   [|]   http://www.jaguNET.com/
      "A society that will trade a little liberty for a little order
             will lose both and deserve neither" - T.Jefferson

Mime
View raw message