On 4/26/16 8:50 PM, Felix Yang wrote:
Hi Amy,
thanks for pointing this out. Updated webrev:
http://cr.openjdk.java.net/~xiaofeya/8154733/webrev.01/
Thank you Felix for the updated webrev. This looks clear and good :-)
Just one question, does this fix for all tests for jdk_rmi test group?
jdk/test/java/rmi/reliability/juicer/AppleUserImpl.java
does not declare java.logging dependency.
Seems same to jdk/test/sun/rmi/runtime/Log/checkLogging/CheckLogging.java
I did not do full check though. (It's fine if other tests are tracked by
separate bugid.)
As I'm not reviewer, please wait for reviewer's feedback.
Thanks,
Amy
Felix
On 2016/4/26 17:21, Amy Lu wrote:
Hi, Felix
With modules declares in TEST.propertiesshould avoid have to modify
test file one by one... Maybe I missed things please correct me.
Example:
$ cat jdk/test/java/rmi/TEST.properties
modules = java.rmi
Thanks,
Amy
On 4/26/16 4:53 PM, Felix Yang wrote:
Hi all,
please review the fix to explicitly declare module dependencies
for rmi tests.
Bug: https://bugs.openjdk.java.net/browse/JDK-8154733
Webrev: http://cr.openjdk.java.net/~xiaofeya/8154733/webrev.00/
Thanks,
Felix