You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by ah...@apache.org on 2022/09/24 21:26:46 UTC

[commons-statistics] branch master updated: Remove javadoc links to JDK (reporting)

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

aherbert pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-statistics.git


The following commit(s) were added to refs/heads/master by this push:
     new d1d3902  Remove javadoc links to JDK (reporting)
d1d3902 is described below

commit d1d39023e1e87731b7ef86ea630c964a4d15a4ed
Author: Alex Herbert <ah...@apache.org>
AuthorDate: Sat Sep 24 22:22:56 2022 +0100

    Remove javadoc links to JDK (reporting)
---
 pom.xml | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/pom.xml b/pom.xml
index 384f0ed..c20312a 100644
--- a/pom.xml
+++ b/pom.xml
@@ -339,8 +339,8 @@
           </excludes>
         </configuration>
       </plugin>
-      <!-- MathJax -->
       <plugin>
+        <!-- NOTE: javadoc config must also be set under <reporting> -->
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-javadoc-plugin</artifactId>
         <configuration>
@@ -349,6 +349,7 @@
           <links combine.self="override">
           </links>
           <failOnWarnings>true</failOnWarnings>
+          <!-- Enable MathJax -->
           <additionalOptions>${doclint.javadoc.qualifier} ${allowscript.javadoc.qualifier} -header '&lt;script type="text/javascript" async src="${statistics.mathjax.url}"&gt;&lt;/script&gt;'</additionalOptions>
         </configuration>
       </plugin>
@@ -465,12 +466,17 @@
           </reportSet>
         </reportSets>
       </plugin>
-      <!--  MathJax -->
       <plugin>
+        <!-- NOTE: javadoc config must also be set under <build> -->
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-javadoc-plugin</artifactId>
         <configuration>
+          <!-- Java API links configured in commons-parent create redirect warnings on JDK 17
+            and fail the build. The links are not required as detectJavaApiLink=true (default). -->
+          <links combine.self="override">
+          </links>
           <failOnWarnings>true</failOnWarnings>
+          <!-- Enable MathJax -->
           <additionalOptions>${doclint.javadoc.qualifier} ${allowscript.javadoc.qualifier} -header '&lt;script type="text/javascript" async src="${statistics.mathjax.url}"&gt;&lt;/script&gt;'</additionalOptions>
         </configuration>
       </plugin>