wardev commented on a change in pull request #60:
URL:
https://github.com/apache/maven-javadoc-plugin/pull/60#discussion_r487853644
##########
File path:
src/main/java/org/apache/maven/plugins/javadoc/AbstractJavadocMojo.java
##########
@@ -3799,7 +3799,7 @@ else if ( SystemUtils.IS_OS_MAC_OSX &&
!JavaVersion.JAVA_SPECIFICATION_VERSION.i
{
javadocExe = new File( SystemUtils.getJavaHome() + File.separator
+ "bin", javadocCommand );
}
- // On Java 9, this has moved to:
/usr/lib/jvm/java-9-openjdk-amd64/bin/javadoc
+ // On Java 9 the jre subdirectory was removed from the JDK
Review comment:
It was intended to be an addendum to the comment about 15 lines above
above, which I think includes the details your're looking for:
```
// Try to find javadocExe from System.getProperty( "java.home" )
// By default, System.getProperty( "java.home" ) = JRE_HOME and
JRE_HOME
// should be in the JDK_HOME
```
I can add a longer comment if you think it is worthwhile.
```
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
