[12] RFR 8208363: test/jdk/java/lang/Package/PackageFromManifest.java missing module dependencies declaration
mandy chung
mandy.chung at oracle.com
Fri Jul 27 16:02:29 UTC 2018
Looks fine. This test uses JarUtils instead of jar tool and so
it does not depend on jdk.jar.
Mandy
On 7/27/18 1:42 AM, Chris Yin wrote:
> Please review below minor change to add missing module dependencies declaration, thanks
>
> bug: https://bugs.openjdk.java.net/browse/JDK-8208363
>
> Changes:
>
> diff -r 65fc31d1042b test/jdk/java/lang/Package/PackageFromManifest.java
> --- a/test/jdk/java/lang/Package/PackageFromManifest.java Thu Jul 26 16:22:58 2018 -0700
> +++ b/test/jdk/java/lang/Package/PackageFromManifest.java Fri Jul 27 15:46:34 2018 +0800
> @@ -29,6 +29,7 @@
> * same package if multiple jars). Then verify package versioning info
> * @library /lib/testlibrary
> * @library /test/lib
> + * @modules jdk.compiler
> * @run main PackageFromManifest setup test
> * @run main PackageFromManifest runJar test1.jar
> * @run main PackageFromManifest runJar test1.jar test2.jar foo.Foo1
>
>
> Regards,
> Chris
>
More information about the core-libs-dev
mailing list