Hello all; I have updated the webrev after additional testing and cleanup. I found a couple of additional minor problems and added a feature that had been requested. Some of the default behaviour can now be controlled using environment variables. It had been pointed out that being able to run commands as HGFOREST_SEQUENTIAL=true sh get_source.sh would have provided a good work around on broken platforms. I ended up adding a lot of source comments since testing took a while. Maybe someone will find the environment variables useful.
I also cleaned up a number of "buglets" and bashisms such as referencing undefined variables, previously failing to report result code of root repo and use of $() in "serve" command. http://cr.openjdk.java.net/~mduigou/JDK-8042810/1/webrev/ Mike On May 8 2014, at 19: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