If you set IFS=' ,' before looping over the jib_mirrors variable, you will allow a comma-separated list as well as a space separated, which makes it possible to set using environment variables in a reasonable way. I think. :) I always have to look up the details on IFS.
Otherwise it's looking fine. /Magnus > 22 dec. 2015 kl. 15:13 skrev Erik Joelsson <erik.joels...@oracle.com>: > > Hello, > > If we are to rely on Jib, the whole process needs redundancy in download > locations. For the main downloads, this is already in place, but the weak > link is the initial bootstrapping of the tool itself. I propose to add > support in common/bin/jib.sh for checking a list of mirrors to download the > install script from. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8146002 > Webrev: http://cr.openjdk.java.net/~erikj/8146002/webrev.01/ > > /Erik
