[7u-dev] Request for approval RR(XS): Test sun/tools/jcmd/jcmd-f.sh failing after JDK-8017011
Seán Coffey
sean.coffey at oracle.com
Tue Feb 18 04:48:27 PST 2014
Results look good Dmitry. Since this is in hotspot category, you can
push to the hsx team forest.
You don't need approval on this mailing list. Alejandro will get fix
into 7u master via the bulk push request.
regards,
Sean.
On 18/02/2014 09:49, Dmitry Samersoff wrote:
> Sean,
>
> OK. Thank you for doing it.
>
> -Dmitry.
>
> On 2014-02-18 13:47, Seán Coffey wrote:
>> Dmitry,
>>
>> I've seen this type of testcase fail alot. I'm going to pull down your
>> patch and run a JPRT job on it. Will get back to you.
>>
>> regards,
>> Sean.
>>
>> On 18/02/2014 09:43, Dmitry Samersoff wrote:
>>> Seán,
>>>
>>> Updated CR with 8-na, updated webrev (in-place, press shift-reload) with
>>> changes to ProblemList.txt
>>>
>>> -Dmitry
>>>
>>>
>>> On 2014-02-18 13:19, Seán Coffey wrote:
>>>> Dmitry,
>>>>
>>>> Is this issue applicable for JDK 8u and JDK 9 ? There's no 8-na label on
>>>> the bug report.
>>>> Also - the testcase is listed in ProblemList.txt and hence doesn't run
>>>> through JPRT.
>>>>
>>>> Are you going to remove it from ProblemList if it's now working ? (i.e.
>>>> all JPRT runs pass)
>>>>
>>>> regards,
>>>> Sean.
>>>>
>>>> On 18/02/2014 07:54, Dmitry Samersoff wrote:
>>>>> Please approve.
>>>>>
>>>>> On 2014-02-14 22:21, Dmitry Samersoff wrote:
>>>>>> Hi Everyone,
>>>>>>
>>>>>> Please review the fix.
>>>>>>
>>>>>> The fix remove a colon (:) from file with reference output.
>>>>>>
>>>>>> http://cr.openjdk.java.net/~dsamersoff/JDK-8019800/webrev.01/
>>>>>>
>>>>>> -Dmitry
>>>>>>
>
More information about the jdk7u-dev
mailing list