[prev in list] [next in list] [prev in thread] [next in thread]
List: openjdk-serviceability-dev
Subject: Re: RFR(S): 8165490: [TESTBUG] sun/tools/jps/TestJpsJar.java still fails with ClassNotFoundException
From: Dmitry Dmitriev <dmitry.dmitriev () oracle ! com>
Date: 2016-09-06 14:45:46
Message-ID: e3cbafa7-717d-18b3-bb55-c6e1c643a16b () oracle ! com
[Download RAW message or body]
FYI: the problem is with new JDK 9 multi-release jar tool which ignores
second(and others) '-C <folder> .' arguments. Bug: JDK-8165500 "jar not
proceed second(and others) -C <dir> . arguments"
<https://bugs.openjdk.java.net/browse/JDK-8165500>
Dmitry
On 06.09.2016 15:35, Dmitry Dmitriev wrote:
> Hello, I withdraw this patch because it's wrong. Thanks!
>
> On 06.09.2016 14:43, Dmitry Dmitriev wrote:
>> Hello,
>>
>> Please review fix for jdk/test/sun/tools/jps tests.
>> sun/tools/jps/TestJpsJar.java still fails with
>> ClassNotFoundException: jdk.testlibrary.ProcessTools. The problem is
>> that TestJpsJar not add class path to the process builder and fails
>> to find test library classes if these classes were compiled before
>> with @build directive.
>>
>> In this fix I also removed unnecessary build directives from
>> sun/tools/jps/ tests.
>>
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8165490
>> webrev.00: http://cr.openjdk.java.net/~ddmitriev/8165490/webrev.00/
>> <http://cr.openjdk.java.net/%7Eddmitriev/8165490/webrev.00/>
>> Testing: locally, RBT(in progress)
>>
>> Thanks,
>> Dmitry
>
[Attachment #3 (text/html)]
<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
FYI: the problem is with new JDK 9 multi-release jar tool which
ignores second(and others) '-C <folder> .' arguments. Bug: <a
href="https://bugs.openjdk.java.net/browse/JDK-8165500">JDK-8165500
"jar not proceed second(and others) -C <dir> . arguments"</a><br>
<br>
Dmitry<br>
<br>
<div class="moz-cite-prefix">On 06.09.2016 15:35, Dmitry Dmitriev
wrote:<br>
</div>
<blockquote
cite="mid:b23d32d8-4504-b2ed-e059-f5b092bf17f8@oracle.com"
type="cite">Hello, I withdraw this patch because it's wrong.
Thanks!
<br>
<br>
On 06.09.2016 14:43, Dmitry Dmitriev wrote:
<br>
<blockquote type="cite">Hello,
<br>
<br>
Please review fix for jdk/test/sun/tools/jps tests.
sun/tools/jps/TestJpsJar.java still fails with
ClassNotFoundException: jdk.testlibrary.ProcessTools. The
problem is that TestJpsJar not add class path to the process
builder and fails to find test library classes if these classes
were compiled before with @build directive.
<br>
<br>
In this fix I also removed unnecessary build directives from
sun/tools/jps/ tests.
<br>
<br>
JBS: <a class="moz-txt-link-freetext" \
href="https://bugs.openjdk.java.net/browse/JDK-8165490">https://bugs.openjdk.java.net/browse/JDK-8165490</a>
<br>
webrev.00:
<a class="moz-txt-link-freetext" \
href="http://cr.openjdk.java.net/~ddmitriev/8165490/webrev.00/">http://cr.openjdk.java.net/~ddmitriev/8165490/webrev.00/</a>
<a class="moz-txt-link-rfc2396E" \
href="http://cr.openjdk.java.net/%7Eddmitriev/8165490/webrev.00/"><http://cr.openjdk.java.net/%7Eddmitriev/8165490/webrev.00/></a>
<br>
Testing: locally, RBT(in progress)
<br>
<br>
Thanks,
<br>
Dmitry
<br>
</blockquote>
<br>
</blockquote>
<br>
</body>
</html>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic