RFR: JDK-8011180 : Sync make/scripts/hgforest.sh to common/bin/hgforest.sh

[Mike Duigou]

When I push the changeset for 8011178 I would also like to refresh/sync the copy of hgforest.sh in make/scripts. I have created an issue for this task:

    JDK-8011180 : Sync make/scripts/hgforest.sh to common/bin/hgforest.sh 

Mike


On Apr 1 2013, at 11:39 , Mike Duigou wrote:

> I've created a bug for this improvement, JDK-8011178, and would like a reviewer for the proposed patch below.
> 
> I will push it through TL if approved as I am not a jdk8/build repo committer.
> 
> Mike
> 
> 
> On Jan 16 2013, at 16:53 , Mike Duigou wrote:
> 
>> With the mac ports version of mercurial (wasn't I just complaining about another mac ports problems yesterday? ;-) ) the path to python in the mercurial launcher has is 
>> 
>> /opt/local/Library/Frameworks/Python.framework/Versions/2.7/Resources/Python.app/Contents/MacOS/Python
>> 
>> Note the capital "P" which is what was running afoul of the hgforest.sh detection. The following patch uses a different technique for checking the python executable.
>> 
>> Mike
>> 
>> PS:- I assume that the make/scripts/hgforest.sh is going away at some point?
>> 
>> diff --git a/common/bin/hgforest.sh b/common/bin/hgforest.sh
>> --- a/common/bin/hgforest.sh
>> +++ b/common/bin/hgforest.sh
>> @@ -51,7 +51,7 @@
>>   bpython="`basename "$python"`"
>> fi
>> 
>> -if [ "python" = "$bpython" -a -x "$python" ] ; then
>> +if [ -x "$python" -a ! -d "$python" -a  "`${python} --version 2>&1 | cut -f 1 -d " "`" == "Python" ] ; then
>>  hg="${python} -u ${whichhg}"
>> else
>>  echo Cannot find python from hg launcher. Running plain hg, which probably has buffered stdout.
>> 
>