michael-o commented on pull request #31: URL: https://github.com/apache/maven-apache-parent/pull/31#issuecomment-755211912
> > > this PR mixes many many different things: > > * create a property to define maven-javadoc-plugin version: no, please don't do that Please explain. > * the hope that it will be sufficient to have reproducible javadoc: IMHO, you're dreaming, there are so many moving parts in javadoc generated html (with links to external components javadoc) that I personnally stopped dreaming about it Likely, but this will at least spare wasteful commits to scmpubsub. > * the question of configuring the reporting section in addition to configuring the plugin, which should not be necessary if pluginManagement is configured since maven-site-plugin 3.4: see https://maven.apache.org/plugins/maven-site-plugin/history.html (some documentation here and there should probably require some updates to explain how things evolved over time) It causes fuzz as you can see. What is your concrete recommendatoin. > all in all, in this PR, I would only keep the timestamp removal in plugin configuration (and not advertise it as "configure javadoc for reproducible builds", because it's only a little part of what is needed for that) > anything else would require separate discussion I ran reword the issue title. Would you keep the Javadoc version or rather remove it? ---------------------------------------------------------------- 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: us...@infra.apache.org
