RFR: JDK-8146002: Need to support mirrors for bootstrapping Jib

[Tim Bell]

Erik:

Looks good to me as well.

/Tim


> Right, new webrev: http://cr.openjdk.java.net/~erikj/8146002/webrev.03/
>
> /Erik
>
> On 2015-12-22 17:43, Magnus Ihse Bursie wrote:
>> Nice. Just one thing more: please save and restore the old value of 
>> IFS. Things can behave strange when modifying IFS so it's good to 
>> have it changed for as short a period as possible.
>>
>> /Magnus
>>
>>> 22 dec. 2015 kl. 15:33 skrev Erik Joelsson <erik.joelsson at oracle.com>:
>>>
>>> Thanks for fast review!
>>>
>>> Added IFS and verified that it accepts any combination of space and 
>>> comma as separator.
>>>
>>> Webrev: http://cr.openjdk.java.net/~erikj/8146002/webrev.02/
>>>
>>> /Erik
>>>
>>>> On 2015-12-22 15:23, Magnus Ihse Bursie wrote:
>>>> 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.joelsson at 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
>