[PING] RE: 8180478: tools/launcher/MultipleJRE.sh fails on Windows because of extra-''

[yano-masanori at fujitsu.com]

Hello,

Please reply if anyone can be a sponsor.

Regards,
Masanori Yano

> -----Original Message-----
> From: Yano, Masanori/矢野 公規
> Sent: Wednesday, November 11, 2020 4:06 PM
> To: 'jdk8u-dev at openjdk.java.net' <jdk8u-dev at openjdk.java.net>
> Subject: 8180478: tools/launcher/MultipleJRE.sh fails on Windows because of
> extra-''
> 
> Hello.
> 
> I would like to contribute for JDK-8180478.
> 
> The test of tools/launcher/MultipleJRE.sh fails on the JDK8 for the Windows because
> the 'RELEASE' variable contains the carriage return.
> 
> The 'RELEASE' variable is created from the third word in the first line of the "java
> -version" command.
> But this is a last word of the line, so the 'RELEASE' variable contains the carriage
> return.
> The 'RELEASE' variable is compared to the actual java version, but does not match
> because of the carriage return difference.
> 
> Since JDK 11, this is no longer a problem because the date has been added as a fourth
> word.
> Therefore, the test for JDK8 should be fixed to remove the carriage returns.
> 
> Please sponsor the following change.
> 
> diff -r 4c5dceabd4c6 test/tools/launcher/MultipleJRE.sh
> --- a/test/tools/launcher/MultipleJRE.sh        Mon Apr 21 14:35:14 2014 +0400
> +++ b/test/tools/launcher/MultipleJRE.sh        Tue Nov 10 20:08:05 2020
> +0900
> @@ -308,7 +308,7 @@
>  # Main test sequence starts here
>  #
>  RELEASE=`$JAVA -version 2>&1 | head -n 1 | cut -d ' ' -f 3 | \
> -  sed -e "s/\"//g"`
> +  sed -e "s/\"//g" | sed -e "s/\r//g"`
>  BASE_RELEASE=`echo $RELEASE | sed -e "s/-.*//g"`
> 
>  #
> 
> Regards,
> Masanori Yano