<AWT Dev> Changes fro JDK-8075327: moving jdk testlibraty files duplicated in hotspot to the common test repository

Chris Hegarty chris.hegarty at oracle.com
Wed May 13 16:48:02 UTC 2015


Alexander,

> On 13 May 2015, at 16:46, Alexander Kulyakhtin <alexander.kulyakhtin at oracle.com> wrote:
> 
> Hi Chris,
> 
> 
>> I suspect that these changes are best going directly into jdk9/dev, as 
>> opposed to a a downstream forest.
> Yes, they are going directly to jdk9/dev, I forgot to add the group, adding now.
> 
>> In may places '@library /lib/testlibrary ...' remains. Is this 
>> redundant, in many tests? If so, it be removed.
> No, this is mostly not redundant, as the jdk/testlibrary comntains many files specific for jdk only. Only a relatively small number of files are duplicated between hotspot and jdk.

Right, but mostly is not all. For example test/com/sun/net/httpserver/Test1.java should no longer include @library /lib/testlibrary. It depends only on SimpleSSLContext which is being moved.  It is easy to see this by looking at the imports. 

Whatever script you have for updating these tests should also remove redundant values in the @library tag. I suspect there are a significant number of these ( a lot of tests just use one library class ).

Just on that, is it for a library class to indicate that it depends on a class from another library? I believe this is the case for some library utilities. How can a test know this. It needs to be a library class that indicates this dependency.I see nothing in the webrev for this ( or maybe I missed it ).

-Chris.

> 
>> The changes are to the 'test' directory in the "top" repo? You are not 
>> proposing to add a new repo, right?
> No, we are not proposing a new repo. The changes are in the 'top' repo.
> 
>> test/lib/testlibrary/jdk/testlibrary/RandomFactory.java was updated 
>> recently in jdk9/dev. The version in your webrev is a little out of date.
> We are going to upmerge the changes then
> 
>> Is there any special update needed to jtreg to support this?
> For thse changes no jtreg update is required.
> However, there are plans for the jtreg to prohibit referencing any @libraries above the repo root unless such libraries are directly specified in TEST.properties
> If jtreg thus prohibits  our using /../../test/lib (because it's above the root) we will have to additionally make a change to jdk and hotspot TEST.properties 
> 
> Best regards,
> Alexander



More information about the jdk9-dev mailing list