You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2019/10/28 21:48:55 UTC

[tomcat] branch 7.0.x updated: Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=63838

This is an automated email from the ASF dual-hosted git repository.

markt pushed a commit to branch 7.0.x
in repository https://gitbox.apache.org/repos/asf/tomcat.git


The following commit(s) were added to refs/heads/7.0.x by this push:
     new a891580  Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=63838
a891580 is described below

commit a891580b3354ba25a8a0326241c49ae32affc603
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Mon Oct 28 22:45:28 2019 +0100

    Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=63838
    
    Add the same options used when Tomcat starts to suppress reflexive
    access warnings when running unit tests from the command line.
---
 build.xml                  | 18 ++++++++++++++++++
 webapps/docs/changelog.xml |  4 ++++
 2 files changed, 22 insertions(+)

diff --git a/build.xml b/build.xml
index e421448..4b49a52 100644
--- a/build.xml
+++ b/build.xml
@@ -205,6 +205,19 @@
              value="-html5"/>
   <property name="java9.javadoc.options" value=""/>
 
+  <available classname="java.lang.reflect.InaccessibleObjectException"
+             property="java9.test.option.1"
+             value="--add-opens=java.base/java.lang=ALL-UNNAMED"/>
+  <property name="java9.test.option.1" value="-Dtest.1=1"/>
+  <available classname="java.lang.reflect.InaccessibleObjectException"
+             property="java9.test.option.2"
+             value="--add-opens=java.base/java.io=ALL-UNNAMED"/>
+  <property name="java9.test.option.2" value="-Dtest.2=2"/>
+  <available classname="java.lang.reflect.InaccessibleObjectException"
+             property="java9.test.option.3"
+             value="--add-opens=java.rmi/sun.rmi.transport=ALL-UNNAMED"/>
+  <property name="java9.test.option.3" value="-Dtest.3=3"/>
+
   <!-- Classpaths -->
   <path id="compile.classpath">
     <pathelement location="${jdt.jar}"/>
@@ -1521,6 +1534,11 @@
         <jvmarg value="-Djava.net.preferIPv4Stack=${java.net.preferIPv4Stack}"/>
         <jvmarg value="-Dapple.awt.UIElement=true"/>
 
+        <!-- Java 9 -->
+        <jvmarg value="${java9.test.option.1}"/>
+        <jvmarg value="${java9.test.option.2}"/>
+        <jvmarg value="${java9.test.option.3}"/>
+
         <classpath refid="tomcat.test.run.classpath" />
 
         <sysproperty key="tomcat.test.temp" value="${test.temp}" />
diff --git a/webapps/docs/changelog.xml b/webapps/docs/changelog.xml
index d459c50..b522a83 100644
--- a/webapps/docs/changelog.xml
+++ b/webapps/docs/changelog.xml
@@ -125,6 +125,10 @@
         a DataSource was configured with a database that did not exist. Patch
         provided by Guoxiong Li. (markt)
       </fix>
+      <fix>
+        <bug>63838</bug>: Suppress reflexive access warnings when running the
+        unit tests on the command line. (markt)
+      </fix>
     </changelog>
   </subsection>
 </section>


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org