The change looks ok to me Felix.
-Chris.
On 21/01/16 21:41, Felix Yang wrote:
Hi,
please review a minor fix for com/sun/jndi/dns/Test6991580.java, which
fails to compile test/com/sun/jndi/dns/IPv6NameserverPlatformParsingTest.java.
It is necessary to explicitly add exports for com.sun.jdn.dns from module
idk.naming.dns.
Bug: https://bugs.openjdk.java.net/browse/JDK-8133035
Fix:
--- a/test/com/sun/jndi/dns/Test6991580.java Tue Jan 19 08:37:11 2016 +0000
+++ b/test/com/sun/jndi/dns/Test6991580.java Thu Jan 21 13:29:52 2016 -0800
@@ -1,6 +1,6 @@
/*
- * Copyright (c) 2015, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2015, 2016, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
@@ -32,9 +32,10 @@
/**
* @test
- * @bug 6991580 8080108
+ * @bug 6991580 8080108 8133035
* @requires os.family != "windows"
* @summary IPv6 Nameservers in resolv.conf throws NumberFormatException
+ * @modules jdk.naming.dns/com.sun.jndi.dns
* @build IPv6NameserverPlatformParsingTest
* @run main/manual Test6991580
*/
Thanks,
Felix
