Looks good to me. 

/Magnus

> 22 dec. 2015 kl. 17:50 skrev Erik Joelsson <erik.joels...@oracle.com>:
> 
> 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.joels...@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.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
> 

Reply via email to