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

Dave Pointon dpointo8 at linux.vnet.ibm.com
Wed Dec 4 11:48:25 UTC 2013

On Wed, 2013-12-04 at 10:45 +0100, Erik Joelsson wrote:

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

I concur with you Erik, character classes are only proven to work with/in perl 5 across all platforms. I suggest that the need to parse whitespace can be eliminated entirely by using awk(1) with a field-delimiter of '=' thus realizing a simpler solution all together, viz ...

if [[ -z "$OUTPWS" ]]; then OUTPWS=awk -F'=' '/^default(-push)/{print $NF}' .hg/hgrc || $FILTER fi

Rgds ,

-- Dave Pointon FIAP MBCS

Now I saw, tho' too late, the folly of beginning a work before we count the cost and before we we judge rightly of our strength to go thro' with it - Robinson Crusoe

More information about the build-dev mailing list