RFR(S): 8043915: Tests get ClassNotFoundException: com.oracle.java.testlibrary.StreamPumper
peter.allwin at oracle.com
Mon Jun 2 13:15:57 UTC 2014
On 02 Jun 2014, at 11:35, Yekaterina Kantserova <yekaterina.kantserova at oracle.com> wrote:
> Peter, thanks for your review! See my comments inlined.
> On 05/30/2014 02:08 PM, Peter Allwin wrote:
>> Hi Katja,
>> Is it necessary to @build classes used in @run statements? I see you added it to some but GetObjectSizeOverflow.java is missing ClassFileInstaller.
> No it is not. Here is an explanation from Jon:
> "The class specified in an "@run main" directive is subject to an implicit "@build" meaning that it will be compiled if needed."
> But it does no harm and my intention was to change as little as possible.
Ah, thanks for the clarification!
> New webrev can be found here: http://cr.openjdk.java.net/~ykantser/8043915/webrev.01/
Looks good to me!
>> Other than that it looks good!
>> On 28 May 2014, at 13:58, Yekaterina Kantserova <yekaterina.kantserova at oracle.com> wrote:
>>> Could I please have a review of this fix.
>>> webrev: http://cr.openjdk.java.net/~ykantser/8043915/webrev.00/
>>> bug: https://bugs.openjdk.java.net/browse/JDK-8043915
>>> When using @library in a JTreg test even @build need to be specify for all library files used by the test. If @build is not specified it can lead to intermittent failures when for example running tests concurrently, since javac implicit compilation and @library and -concurrency don't play well together.
>>> Verified locally.
More information about the hotspot-compiler-dev