-
-
Notifications
You must be signed in to change notification settings - Fork 155
doc-jar fails on OpenJDK 9+ #376
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Can't reproduce, please provide a simple sample project to reproduce (not your full multi-module project with tons of plugins). |
@slandelle is the output from Travis CI with logs, which shows JDK 1.8 passing, but JDK 9, 10, and 11 failing with that error helpful? You can see it here - https://travis-ci.org/digital-preservation/csv-validator/builds/623143738 |
As I said, your project is way too complicated and involves lots of plugins to be able to investigate in isolation. |
@slandelle I see exactly the same failure with the same stack trace using scala-maven-plugin 4.3.0 with the prj_multi_modules but using Scala 2.11.12 and Java 11. Scala 2.12.X works fine though. |
I think this is an upstream problem in Scala 2.11 (and presumably 2.10 as well), rather than a Maven problem per se: scala/bug#10603 that ticket also recommends adding |
The |
Can you provide a simple sample project that demonstrates this? |
@SethTisue you can take your sample project samples/prj_multi_modules and just change the Scala version to 2.11.12. I tried with and without -nobootcp, no difference. But as you mentioned before, this is probably Scala 2.11 issue. |
Generally speaking, it seems scala-compiler 2.11 doesn't work properly with Java 11. Switching to 2.11.11 here and running
Anyway, I'm closing this issue as that's really not something that can be fixed here, it could only be fixed in . Now, let's be realistic:
We can safely say Scala 2.11 has reached end-of-life and Java 11 support is very unlikely to happen. And IMHO, it shouldn't. Libraries are dropping Scala 2.11 support as well: Your Scala version has been dead for 2.5 years, time to upgrade, people! |
I think Stephane is right to close the ticket, but we can still discuss workarounds. Stephane writes:
My working hypothesis is that it can be always be made to work by passing
True, but adding @asmotrich writes:
that wasn't my experience. when I tried it, SethTisue@217d282 fixes both you say it doesn't work for you in your project. perhaps you are doing something differently? |
On my side, it didn't work but I configured at execution level instead of root one, which might have been wrong (I never remember how this works...). |
Exactly the same in my case: I put configuration at the execution level and
it did not work, once I followed recommendations in SethTisue@217d282
<SethTisue@217d282>
and
put it at the root level everything worked correctly. This is perfectly
functional workaround, end of story IMHO.
I figured a way to move my environment forward to Scala 2.12 and everything
seems to be in good order.
Thanks!
…On Tue, Dec 17, 2019 at 7:21 AM Stephane Landelle ***@***.***> wrote:
you say it doesn't work for you in your project. perhaps you are doing
something differently?
On my side, it didn't work but I configured at execution level instead of
root one, which might have been wrong (I never remember how this works...).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#376?email_source=notifications&email_token=AFMMEQBFEQX5LQZF6OBCCNLQZDVBJA5CNFSM4JY5T3ZKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHCXEPQ#issuecomment-566587966>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AFMMEQBATZPXLHGN4MU3MZDQZDVBJANCNFSM4JY5T3ZA>
.
|
The error is:
Adding the option
-nobootcp
to the args fordoc-jar
seems to be a viable workaround, i.e.:The text was updated successfully, but these errors were encountered: