Re: 8228338: tools/pack200/TimeStamp.java fails with NullPointerException

2019-07-17 Thread Brian Burkhalter
A test job I launched before the CI tests just finished now and it looks like 
this catches all the failures but let’s wait until the CI is done to be certain.

Thanks,

Brian

> On Jul 17, 2019, at 5:09 PM, Lance Andersen  wrote:
> 
> All good Brian, I agree and the latest update also looks OK
> 
> 
>> On Jul 17, 2019, at 8:04 PM, Brian Burkhalter > > wrote:
>> 
>> Sorry Lance but there was another failure which I’ve lumped into the same 
>> issue and updated the webrev in place. I think it would be better to wait 
>> until tiers 1-3 are done before looking at it to avoid more spinning.



Re: 8228338: tools/pack200/TimeStamp.java fails with NullPointerException

2019-07-17 Thread Lance Andersen
All good Brian, I agree and the latest update also looks OK


> On Jul 17, 2019, at 8:04 PM, Brian Burkhalter  
> wrote:
> 
> Sorry Lance but there was another failure which I’ve lumped into the same 
> issue and updated the webrev in place. I think it would be better to wait 
> until tiers 1-3 are done before looking at it to avoid more spinning.
> 
> Thanks,
> 
> Brian
> 
>> On Jul 17, 2019, at 4:17 PM, Lance Andersen > > wrote:
>> 
>> Looks OK Brian
>>> On Jul 17, 2019, at 7:14 PM, Brian Burkhalter >> > wrote:
>>> 
>>> https://bugs.openjdk.java.net/browse/JDK-8228338 
>>> 
>>> http://cr.openjdk.java.net/~bpb/8228338/webrev.00/ 
>>> 
>>> 
>>> Another slip up due to not having tested the JDK-8067801 patch sufficiently.
>>> 
>>> Thanks,
>>> 
>>> Brian
>> 
>>  
>> 
>>   
>> 
>>  Lance Andersen| 
>> Principal Member of Technical Staff | +1.781.442.2037
>> Oracle Java Engineering 
>> 1 Network Drive 
>> Burlington, MA 01803
>> lance.ander...@oracle.com 
>> 
>> 
>> 
> 

 
  

 Lance Andersen| 
Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering 
1 Network Drive 
Burlington, MA 01803
lance.ander...@oracle.com 





Re: 8228338: tools/pack200/TimeStamp.java fails with NullPointerException

2019-07-17 Thread Brian Burkhalter
Sorry Lance but there was another failure which I’ve lumped into the same issue 
and updated the webrev in place. I think it would be better to wait until tiers 
1-3 are done before looking at it to avoid more spinning.

Thanks,

Brian

> On Jul 17, 2019, at 4:17 PM, Lance Andersen  wrote:
> 
> Looks OK Brian
>> On Jul 17, 2019, at 7:14 PM, Brian Burkhalter > > wrote:
>> 
>> https://bugs.openjdk.java.net/browse/JDK-8228338 
>> 
>> http://cr.openjdk.java.net/~bpb/8228338/webrev.00/
>> 
>> Another slip up due to not having tested the JDK-8067801 patch sufficiently.
>> 
>> Thanks,
>> 
>> Brian
> 
>  
> 
>   
> 
>  Lance Andersen| 
> Principal Member of Technical Staff | +1.781.442.2037
> Oracle Java Engineering 
> 1 Network Drive 
> Burlington, MA 01803
> lance.ander...@oracle.com 
> 
> 
> 



Re: 8228338: tools/pack200/TimeStamp.java fails with NullPointerException

2019-07-17 Thread Lance Andersen
Looks OK Brian
> On Jul 17, 2019, at 7:14 PM, Brian Burkhalter  
> wrote:
> 
> https://bugs.openjdk.java.net/browse/JDK-8228338
> http://cr.openjdk.java.net/~bpb/8228338/webrev.00/
> 
> Another slip up due to not having tested the JDK-8067801 patch sufficiently.
> 
> Thanks,
> 
> Brian

 
  

 Lance Andersen| 
Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering 
1 Network Drive 
Burlington, MA 01803
lance.ander...@oracle.com 





8228338: tools/pack200/TimeStamp.java fails with NullPointerException

2019-07-17 Thread Brian Burkhalter
https://bugs.openjdk.java.net/browse/JDK-8228338
http://cr.openjdk.java.net/~bpb/8228338/webrev.00/

Another slip up due to not having tested the JDK-8067801 patch sufficiently.

Thanks,

Brian