Hi, sorry for the late reply… Enrico I think the latest version should work, and here is the IT (I just didn’t create a PR yet): https://github.com/apache/maven-compiler-plugin/pull/37
MCOMPILER-445/pom.xml succeeds, other tests fail because of an outdated groovy plugin. I will push another commit updating the groovy plugin, if you like. WDYT? Am Do., 7. Jan. 2021 um 14:10 Uhr schrieb Enrico Olivelli < eolive...@gmail.com>: > I am testing latest build of openjdk16, > it seems to me that the problem has been resolved > > Benjamin, > can you please try ? > > probably there is no need for a fix on Maven.... > > Enrico > > Il giorno lun 14 dic 2020 alle ore 22:12 Benjamin Marwell < > bmarw...@apache.org> ha scritto: > > > > I'm pretty sure this can be solved without touching the maven startup > > scripts. > > > And I don't like the idea to hack the script > > > > Agreed. I did not like the idea either, and even worse, I was unable to > > reproduce the results I had. > > > > > > > > Am Mo., 14. Dez. 2020 um 21:58 Uhr schrieb Robert Scholte < > > rfscho...@apache.org>: > > > > > I'm pretty sure this can be solved without touching the maven startup > > > scripts. > > > And I don't like the idea to hack the script because one plugin has > > issues. > > > I expect good help on your question on the jigsaw mailinglist. > > > > > > Robert > > > On 14-12-2020 15:21:04, Benjamin Marwell <bmarw...@apache.org> wrote: > > > I was able to provide a test, but the only solution to the underlying I > > > found was to modify the mvn start script by adding the missing modules > > (or > > > maybe even better: all system modules). > > > > > > From what I could see from slack, plexus compiler would need to be a > > > module, requiring jdk.javadoc. > > > > > > Thus, we could solve it in a maven release 3.6.4 if you wished, or the > > next > > > compiler plugin as soon as the next plexus javac dependency is > available. > > > > > > I wouldn't mind a 3.6.4 release and cherry picking the change to 4.0.0, > > as > > > that would be a quick solution. > > > > > > > > > On Sun, 13 Dec 2020, 17:12 Benjamin Marwell, wrote: > > > > > > > JIRA issue (please kindly review): > > > > https://issues.apache.org/jira/browse/MCOMPILER-445 > > > > > > > > > > > > Am So., 13. Dez. 2020 um 14:07 Uhr schrieb Benjamin Marwell < > > > > bmarw...@apache.org>: > > > > > > > >> > If is has proven itself for jlink, then we know we can do the same > > for > > > >> all other tools. > > > >> > > > >> I tested my PR with a JavaFX app and it did work. But there's no > > release > > > >> yet, only the ITs and my test project. But adding a parameter to > > disable > > > >> the ToolProvider as a fallback should not be a problem. > > > >> > > > >> A test case should be ready by tomorrow morning, but I cannot make > any > > > >> promises about the implementation, as I work on it only in my free > > time. > > > >> Possibly, there is quite a bit of code to refactor as we want to > have > > as > > > >> few code duplication as possible. > > > >> > > > >> Please do ourselves a favour and vote for all MultiRelease (MR) > issues > > > at > > > >> jetbrains. They currently do not support MR projects, and it is a > PIT* > > > to > > > >> develop for MR jars (from an IDE perspective). Thanks. 😉 > > > >> > > > >> Ben > > > >> > > > >> > > > >> > > > >> On Sun, 13 Dec 2020, 13:38 Robert Scholte, wrote: > > > >> > > > >>> Yes, that makes a lot of sense. If is has proven itself for jlink, > > then > > > >>> we know we can do the same for all other tools. > > > >>> If we have a good feeling about the implementation, we could use it > > at > > > >>> reference for other plugins as some kind of pattern. > > > >>> > > > >>> Robert > > > >>> On 13-12-2020 11:39:02, Benjamin Marwell wrote: > > > >>> Robert already suggested to use ToolProvider for the JDK9+ builds. > I > > > >>> created such a patch for jlink and I could create s similar one for > > the > > > >>> compiler and javadoc plugin. This would solve the underlying > problem > > > from > > > >>> my understanding. > > > >>> > > > >>> As fork mode and fork count would not apply, I would suggest that > the > > > >>> ToolProvider is only used if fork mode is "no fork". This way, > > existing > > > >>> configurations are not affected. Does that make sense? > > > >>> > > > >>> Best regards, > > > >>> Ben > > > >>> > > > >>> > > > >>> On Sat, 12 Dec 2020, 20:49 Enrico Olivelli, wrote: > > > >>> > > > >>> > Is anyone interested in helping with this problem? > > > >>> > > > > >>> > Otherwise with the advent of jdk16 we will probably see people > that > > > >>> need to > > > >>> > switch to fork mode for javac, with slower builds, and we will > see > > > >>> > complaints from users > > > >>> > > > > >>> > The problem probably is is plexus compiler and the way we start > > > javac, > > > >>> we > > > >>> > should enable jdk.javadoc module > > > >>> > > > > >>> > Unfortunately I don't have time > > > >>> > > > > >>> > Enrico > > > >>> > > > > >>> > Il Gio 12 Nov 2020, 13:59 Enrico Olivelli ha > > > >>> > scritto: > > > >>> > > > > >>> > > Yes, the problem is about javac with "no-fork + -Xdoclint" > > > >>> > > > > > >>> > > using no-fork is not a good option because it slows down a lot > > big > > > >>> multi > > > >>> > > module projects > > > >>> > > > > > >>> > > Enrico > > > >>> > > > > > >>> > > Il giorno gio 12 nov 2020 alle ore 13:55 Romain Manni-Bucau > > > >>> > > rmannibu...@gmail.com> ha scritto: > > > >>> > > > > > >>> > >> @Graham I guess you can force the compiler to fork and force > > > >>> doclint to > > > >>> > >> none in javadoc plugin config (ensure to use a recent > version). > > > >>> > >> > > > >>> > >> Romain Manni-Bucau > > > >>> > >> @rmannibucau | Blog > > > >>> > >> | Old Blog > > > >>> > >> | Github > > > >>> > >> https://github.com/rmannibucau> | > > > >>> > >> LinkedIn | Book > > > >>> > >> > > > >>> > >> > > > >>> > > > > >>> > > > > > > https://www.packtpub.com/application-development/java-ee-8-high-performance > > > >>> > >> > > > > >>> > >> > > > >>> > >> > > > >>> > >> Le jeu. 12 nov. 2020 à 13:51, Graham Leggett > > > >>> > a > > > >>> > >> écrit : > > > >>> > >> > > > >>> > >> > On 12 Nov 2020, at 14:03, Enrico Olivelli > > > >>> > wrote: > > > >>> > >> > > > > >>> > >> > > I have fallen into this issue about Maven + Maven Compiler > > > >>> Plugin + > > > >>> > >> JDK16 > > > >>> > >> > > > > > >>> > >> > > This is the issue on JDK issue tracking > > > >>> > >> > > https://bugs.openjdk.java.net/browse/JDK-8253996 > > > >>> > >> > > > > > >>> > >> > > Basically -Xdoclint:missing does not work anymore when you > > run > > > >>> javac > > > >>> > >> > inside > > > >>> > >> > > the same JVM as Maven core, because the JVM lacks the > > > >>> jdk.javadoc > > > >>> > >> module. > > > >>> > >> > > If you run javac in "fork" mode the problem is not present > > > >>> because > > > >>> > the > > > >>> > >> > > external "javac" program loads correctly jdk.javadoc > module > > > and > > > >>> is > > > >>> > >> able > > > >>> > >> > to > > > >>> > >> > > execute "-Xdoclint" > > > >>> > >> > > > > > >>> > >> > > it looks like we have to fix it on Maven, I am not sure > the > > > >>> problem > > > >>> > is > > > >>> > >> > > about maven-compiler-plugin or plexus compiler, as it is > > > >>> because the > > > >>> > >> JVM > > > >>> > >> > > that executes Maven core lacks the jdk.javadoc module. > > > >>> > >> > > > > > >>> > >> > > On the JDK side it looks like the issue is to be closed as > > > >>> "works > > > >>> > for > > > >>> > >> me" > > > >>> > >> > > > > > >>> > >> > > > > > >>> > >> > > Thoughts? > > > >>> > >> > > > > >>> > >> > I have been smashing my head against the javadoc plugin and > > > >>> > >> > maven-release-plugin, which keeps failing releases over and > > over > > > >>> again > > > >>> > >> on > > > >>> > >> > the basis that the docs can’t be built. > > > >>> > >> > > > > >>> > >> > In the absence of a way to fix this, if there is a way to > > switch > > > >>> this > > > >>> > >> off > > > >>> > >> > it would help a huge amount. > > > >>> > >> > > > > >>> > >> > Regards, > > > >>> > >> > Graham > > > >>> > >> > — > > > >>> > >> > > > > >>> > >> > > > > >>> > >> > > > > >>> > --------------------------------------------------------------------- > > > >>> > >> > To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org > > > >>> > >> > For additional commands, e-mail: dev-h...@maven.apache.org > > > >>> > >> > > > > >>> > >> > > > > >>> > >> > > > >>> > > > > > >>> > > > >> > > > > > >