Review Request: JDK-8174826 jlink does not provide support for linking in service provider modules

Mandy Chung mandy.chung at oracle.com
Thu Mar 23 18:16:21 UTC 2017


> On Mar 23, 2017, at 11:07 AM, Mandy Chung <mandy.chung at oracle.com> wrote:
> 
>> Instead of "copy-paste" code to run Jlink in tests ProcessTools and OutputAnalyzer can be used from standard test library test/lib/share/classes/jdk/test/lib/process
>> 
> 
> I agree that it’d be good to move it to a test library but this does not need to be the general one since it’s jlink-specific.  jlink has a test library under jdk/tests/tools/lib/tests that needs big cleanup.  At some point we should look at all jlink tests and see what makes sense. 
> 
> For these new tests, I can move the helper class to jdk/tests/tools/lib/tests. 
> 

Having another look at jdk/tests/tools/lib/tests/*, I think it’s probably best to leave these new tests as is and avoid adding more helper classes until they are cleaned up/unified.

Mandy



More information about the jigsaw-dev mailing list