On 7/16/2012 10:52 AM, Kelly O'Hair wrote:
Minor change to the get_source/hgforest scripts to allow for trailing //
Plus adding the whitespace normalizer perl script to the openjdk7updates forest.
7184406: Adjust get_source/hgforest script to allow for trailing // characters
http://cr.openjdk.java.net/~ohair/openjdk8/get_source_double_slash/webrev/
Question
101 pull_newrepo="`echo ${pull_default}/${i} | sed -e
's@\([^:]/\)//*@\1@g'`"
102 cline="hg clone ${pull_newrepo} ${i}"
Shouldn't the extra trailing slashes be removed from ${i} as well? Or
it's just because the values of ${i} are set only in the script itself
and won't have trailing /?
Otherwise the changes look fine.
Dave
6625113: Add the normalize and rmkw perl script to the openjdk repository or
openjdk site?
7184406: Adjust get_source/hgforest script to allow for trailing // characters
http://cr.openjdk.java.net/~ohair/openjdk7u/get_source_normalizer/webrev/
-kto
