Re: [PATCH] use locale-indep. c_is* functions for parsing, not isspace, isprint etc

[Akim Demaille]

Le 3 sept. 2012 à 12:12, Jim Meyering a écrit :

> Akim Demaille wrote:
>> Le 3 sept. 2012 à 11:14, Jim Meyering a écrit :
>> 
>>> Akim Demaille wrote:
>>> 
>>>> I have no idea why it failed.  Could you attach config.log?
>>>> It's unclear to me what happened here, as gt_JAVACOMP
>>>> does not seem to cause the failure.
>>> 
>>> Here you go:
>>> 
>>> <config.log.xz>
>> 
>> I don't see anything special in there :(  It ends by
>> 
>> configure:35991: WARNING: unknown target-version 1.7, please update
>> gt_JAVACOMP macro
>> configure:36032: checking for Java compiler
>> configure:36519: result: no
>> configure:36530: checking for Java virtual machine
>> configure:36749: result: java
>> configure:36872: checking that generated files are newer than configure
>> configure:36878: result: done
>> configure:36982: creating ./config.status
>> […]
> 
> Isn't it just that m4/javacomp.m4 barfs for an unexpected version,
> and is prepared only for 1.6 and older, while I have 1.7?
> 
> When I run sh -x ./configure, I see this:
> 
>  + CLASSPATH_SEPARATOR=:
>  + rm -f conftest.c
>  + source_version=1.3
>  + test -n 1.3
>  + test -n 1
>  + cat
>  + echo 
> 'yzwx!$!I!D,!)!3+!4!5*!6,!4!7,!8!9)!:)!;"!(MeienN"!$FGW"!%Ojab"!2QeibRohZblVYZgb"!%hYei"!9FXQfYpYKgYidKUnleidLGW"!,Ujol_bPegb"!3_jicnbmnpblJfYpY/!*!+)!</!=!>"!=fYpYJmkb_ece_YnejiJpblmeji/!?!@)!A/!B!C"!._jicnbmnpbl"!3fYpYKgYidKSZfb_n"!3fYpYKgYidKUqmnbh"!$jon"!8QfYpYKejKTleinUnlbYhL"!.dbnTljkblnq"!EFQfYpYKgYidKUnleidLGQfYpYKgYidKUnleidL"!6fYpYKejKTleinUnlbYh"!)kleingi"!8FQfYpYKgYidKUnleidLGW!D!(!)!!!!!#!"!*!+!"!,address@hidden"!"!!!&Hu!"r!!!"!.!!!(!"!!!"!+!/!0!"!,!!!F!#!"!!!/s!#5$v!%t!&r!!!"!.!!!,!#!!!$!.!%!"!1!!!#!2'
>  + tr -d '\012\015'
>  + tr '!"#$%&()*+,./0123456789:;<=>address@hidden' 
> '\000\001\002\003\004\005\006\007\010\011\012\013\014\015\016\017\020\021\022\023\024\025\026\027\030\031\032\033\034\035\036\037\040\041\046\050\051\052\056\057\073\074\076\103\106\114\116\117\120\123\124\126\133\141\142\143\144\145\146\147\151\152\154\155\156\157\160\162\163\164\165\166\171\261\262\266\267\270\272\276\312\376\055'
>  ++ unset JAVA_HOME
>  ++ echo 'configure:35984: CLASSPATH=. java conftestver'
>  ++ CLASSPATH=.
>  ++ java conftestver
>  + target_version=1.7
>  + case "$target_version" in
>  + printf '%s\n' 'configure:35991: WARNING: unknown target-version 1.7, 
> please update gt_JAVACOMP macro'
>  + printf '%s\n' 'configure: WARNING: unknown target-version 1.7, please 
> update gt_JAVACOMP macro'
>  configure: WARNING: unknown target-version 1.7, please update gt_JAVACOMP 
> macro

But it's a warning, it is not expected to fail.

       case "$target_version" in
         1.1 | 1.2 | 1.3 | 1.4 | 1.5 | 1.6) ;;
         null)
           dnl JDK 1.1.X returns null.
           target_version=1.1 ;;
         *) AC_MSG_WARN([unknown target-version $target_version, please update 
gt_@&address@hidden macro])
            target_version=1.1 ;;
       esac

That 1.7 is not supported is one thing, that it _fails_ instead of proceeding 
in degraded mode (or not enabling Java tests) is another, much more serious, 
issue.