You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@httpd.apache.org by Jeff Trawick <tr...@bellsouth.net> on 2001/05/11 13:28:33 UTC

Re: cvs commit: httpd-2.0 acinclude.m4

fielding@apache.org writes:

> fielding    01/05/10 14:29:12
> 
>   Index: acinclude.m4
>   ===================================================================
>   RCS file: /home/cvs/httpd-2.0/acinclude.m4,v
>   retrieving revision 1.82
>   retrieving revision 1.83
>   diff -u -r1.82 -r1.83
>   --- acinclude.m4	2001/05/10 02:03:15	1.82
>   +++ acinclude.m4	2001/05/10 21:29:10	1.83
>   @@ -385,9 +385,9 @@
>          for p in $ap_ssltk_try; do
>            ap_ssltk_version="`$p/openssl version`"
>            case "$ap_ssltk_version" in
>   -            *[[^0-9a-z.]][[1-9]]* | \
>   -            *[[^0-9a-z.]]0.9.[[6-9]]* | \
>   -            *[[^0-9a-z.]]0.[[1-9]][[0-9]]* )
>   +            *[[!0-9a-z.]][[1-9]]* | \
>   +            *[[!0-9a-z.]]0.9.[[6-9]]* | \
>   +            *[[!0-9a-z.]]0.[[1-9]][[0-9]]* )

Good for you...  I was afraid I'd have to use sed or grep to do the
matching...

-- 
Jeff Trawick | trawickj@bellsouth.net | PGP public key at web site:
       http://www.geocities.com/SiliconValley/Park/9289/
             Born in Roswell... married an alien...


Re: cvs commit: httpd-2.0 acinclude.m4

Posted by "Roy T. Fielding" <fi...@ebuilt.com>.
> >   -            *[[^0-9a-z.]][[1-9]]* | \
> >   -            *[[^0-9a-z.]]0.9.[[6-9]]* | \
> >   -            *[[^0-9a-z.]]0.[[1-9]][[0-9]]* )
> >   +            *[[!0-9a-z.]][[1-9]]* | \
> >   +            *[[!0-9a-z.]]0.9.[[6-9]]* | \
> >   +            *[[!0-9a-z.]]0.[[1-9]][[0-9]]* )
> 
> Good for you...  I was afraid I'd have to use sed or grep to do the
> matching...

What bugs me is that after spending a horrendous amout of time trying
to figure out how to include a space within a pattern, finally coming
up with the above as "good enough", I looked at the generated libtool
and they have the answer:

    *" blah "* )

I must have tried every single option *except* that one.  Now I have to
fix it anyway because that regex will match the date inside the version
string instead of what I intended.

....Roy