[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, December 23, 2020 4:58 PM
> To: 'jdk8u-dev at openjdk.java.net' <jdk8u-dev at openjdk.java.net>
> Subject: [PING] RE: 8180478: tools/launcher/MultipleJRE.sh fails on Windows
> because of extra-''
> 
> 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