RFR[10] 8193460 : Take tools/launcher/TestXcheckJNIWarnings.java back off the ProblemList

[David Holmes]

On 14/12/2017 12:53 PM, David Holmes wrote:
> Brent,
> 
> I'm going to manually import the missing changes from jdk/hs to jdk/jdk. 
> I've conferred with Jesper and this shouldn't cause any problems. Though 
> there may be a rather large merge changeset due to the just pushed 
> changes to jdk/jdk. <sigh>

Unfortunately this was not possible as the changeset for 8190984 
conflicts with this changeset (8193460). So the test will fail again in 
jdk/jdk until the changes make their way up from jdk/hs. If that is a 
concern then you will need to backout this changeset so that the test is 
problem-listed again.

David

> David
> 
> On 14/12/2017 9:28 AM, David Holmes wrote:
>> On 14/12/2017 9:24 AM, Brent Christian wrote:
>>> On 12/13/17 3:17 PM, David Holmes wrote:
>>>>>
>>>>> I already took care of this as part of the fix for 8190984.
>>>
>>> OK, thanks.
>>> Well, crud - my apologies to the person doing the merge later.  :\
>>>
>>> (I saw that the push went to jdk/hs, but didn't look closely.  I 
>>> guess you must have updated from jdk/jdk first...)
>>
>> Yes. Doing the pull down from jdk/jdk doesn't cause any issues.
>>
>>>> Now the test will run again in jdk/jdk CI but the fixes are still 
>>>> only in jdk/hs.
>>>
>>> Yeah :{
>>> Hopefully not for too long...
>>
>> At least a week I think as the next push up happens after the next 
>> hotspot PIT.
>>
>> David
>>
>>> -Brent