Hi Mike ,

On 09/05/14 08:25, Mike Duigou wrote:
On May 8 2014, at 22:35 , Chris Hegarty <chris.hega...@oracle.com> wrote:

Looks fine to me Mike.

Trivially while you are there (not related to your changes), or i can do it 
separately, --sequentially now runs with up to two parallel commands.
Ah sorry, I know why that is. It is due to shell having less-than but no 
less-than-or-equal-to operator. I can fix this easily and will do so before 
pushing. It's simply a matter of reducing at_a_time to 0. This will look odd 
but solve the problem.

You didn't mean 'test -le' for the less-than-or-equal-to operator, did you ?

-Chris

On 9 May 2014, at 03:19, Mike Duigou <mike.dui...@oracle.com> wrote:

Hello all;

This issue is a follow-on bug fix to JDK-8041151 (Improve hgforest.sh concurrency). That 
changeset introduced use of fifos for monitoring sub-shell completion and included an 
alternative implementation for configurations which did not properly support fifos. This 
changeset detects another unsupported configuration, older shells which run the 
"read" command using a sub-shell.

I've tested the change on Solaris 10u11, Solaris 11u1, Ubuntu Linux 13.10 x64, 
MacOS 10.9. Of these only the Bash 3.2 shell on Solaris 10u11 does not support 
FIFOs. I have not tested this specific change on Cygwin as FIFOs were already 
disabled on Cygwin.

https://bugs.openjdk.java.net/browse/JDK-8042810
http://cr.openjdk.java.net/~mduigou/JDK-8042810/0/webrev/

Mike

Best 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

Reply via email to