RFR: JDK-8029512 : webrev.ksh: fix parsing of parent repo paths from hgrc file

Erik Joelsson erik.joelsson at oracle.com
Wed Dec 4 09:45:51 UTC 2013


Hello Mike,

I think sed on some platforms doesn't support the [[:space:]] 
expression. I tried the line on a Solaris machine and it didn't work.

/Erik

On 2013-12-04 06:16, Mike Duigou wrote:
> Hello all;
>
> This change addresses a problem discovered by Dave Pointon.
>
> http://cr.openjdk.java.net/~mduigou/JDK-8029512/0/webrev/
>
> If the .hg/hgrc file contains (perfectly legal) spaces around the = sign of an assignment then the current script will fail to correctly read the path definition. This patch corrects the parsing of the hgrc file and also protects against empty results.
>
> I plan to commit this to the JDK 9 repo first but expect that it should be backported to JDK 8 and JDK 7 repos.
>
> Mike
>




More information about the build-dev mailing list