* Charles Wilson wrote on Wed, Apr 25, 2007 at 03:53:47AM CEST:
> Ralf Wildenhues wrote:
>
>> +AS_IF([test -z "$ARGZ_H"],
>> +    [AC_CACHE_CHECK(
>> +        [if argz actually works],
>> +        [lt_cv_sys_argz_works],
>> +        [case $host_os in #(
>
> I like this trick.  Fixes syntax highlighting and paren-matching, right?

Sort of, yes.

>> +     *cygwin*)
>> +       lt_cv_sys_argz_works=no
>> +       if test "$cross_compiling" != no; then
>> +         lt_cv_sys_argz_works="guessing no"
>> +       else
>> +         save_IFS=$IFS
>> +         IFS=-.
>> +         set x `uname -r`
>
> set x `uname -r | $SED -e 's/^\([[0-9\.]]*\).*/\1/'`

Looks good to me.

>> +         if test "$lt_os_major" -gt 1 ||
>> +            { test "$lt_os_major" -eq 1 &&
>> +              { test "$lt_os_minor" -gt 5 ||
>> +                { test "$lt_os_minor" -eq 5 &&
>> +                  test "$lt_os_micro" -gt 24; }; }; }; then
>
> Eeewww. It's more efficient than my version, but ugly!

But it's also more correct than your version!  I think yours mishandled
1.4.25.

> I'll run some tests on a modified version of your patch, and post the 
> revised version with (sigh...) my gamut of test results.

Thanks.

> And, as with gcc 
> testing, I'll revert my cygwin system to autoconf-2.59 first...

I don't think you need to do that.

Cheers,
Ralf


Reply via email to