You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@drill.apache.org by GitBox <gi...@apache.org> on 2022/12/28 15:16:10 UTC
[GitHub] [drill] jnturton opened a new pull request, #2730: DRILL-8378: Support doing Maven releases using modern JDKs
jnturton opened a new pull request, #2730:
URL: https://github.com/apache/drill/pull/2730
# [DRILL-8378](https://issues.apache.org/jira/browse/DRILL-8378): Support doing Maven releases using modern JDKs
## Description
While [DRILL-8113](https://issues.apache.org/jira/browse/DRILL-8113) enabled the building of Drill using a modern JDK, more work is required to enable a Maven release of Drill using a modern JDK. Presently, the Maven Release Plugin will fail on Javadoc generation when run with a newer JDK while it succeeds with JDK 8. The failures are due to dependencies missing from the Maven Javadoc Plugin's config which I assume get treated with a more lenient "warn and skip" policy in the javadoc tool shipped with JDK 8 but cause errors in newer JDKs (in my case OpenJDK 17).
In particular, the presence of the `sourcepath` property in the javadoc plugin's config in the root pom causes the default javadoc:javadoc goal to try to generate docs for our src/test packages. Unlike the javadoc:test-javadoc, the javadoc:javadoc goal does not inherit dependencies declared with `test` scope so it fails to resolve those.
## Documentation
N/A
## Testing
Successfully run maven release:prepare using OpenJDK 17.
Successfully run mvn javadoc:javadoc in the Drill root module using OpenJDK 17 with HTML output generated under each module's target/site/apidocs directory.
--
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.
To unsubscribe, e-mail: dev-unsubscribe@drill.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [drill] cgivre merged pull request #2730: DRILL-8378: Support doing Maven releases using modern JDKs
Posted by GitBox <gi...@apache.org>.
cgivre merged PR #2730:
URL: https://github.com/apache/drill/pull/2730
--
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.
To unsubscribe, e-mail: dev-unsubscribe@drill.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org