On 3/15/18 3:40 PM, Alexandre (Shura) Iline wrote:
Hi,

Please take a quick look on fix adding missing module dependencies to tests in 
:tier1 jdk tests.

Webrev: http://cr.openjdk.java.net/~shurailine/8199616/webrev.00

Looks okay.   But I'm surprised that test/jdk/java/lang/ProcessHandle/OnExitTest.java requires
   44  * @modules jdk.management

Looks like jdk.management get pulled by the test library.  If so, we should file JBS issue to fix the test library to eliminate such dependency.

Mandy

Reply via email to