-
Notifications
You must be signed in to change notification settings - Fork 25
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
[ttools] com.sun.javadoc was removed in JDK 17 #64
Comments
@olebole , thanks for bringing that to my attention. I'll have a think about what to do. In the mean time, do you happen to know if those classes are available in a debian-licencing-friendly form anywhere? If you don't know, I can go looking myself. |
Do you mean the old API or the new one? I am afraid that com.sun.javadoc is finally gone, so for the meantime I could only patch that out (my understanding is that its use is the creation of the help pages for JEL functions). |
Either or both ... I thought you were saying that jdk.javadoc.doclet had been removed, but re-reading your comment I see you mean jdk.javadoc.doclet is still present. I can take a look at that, it should be OK if it's Java8 compatible, but if not (and given that it seems to cater for modules maybe it won't be) that might be a problem. |
jdk.javadoc.doclet was introduced in Java 9, https://openjdk.org/jeps/221 |
This is going to be a bit of a pain to resolve, unless I move to building with Java 9+, which I don't particularly want to do. I will tackle it at some point, but it won't be immediately. I guess in the mean time that means that the debian build will just have to avoid running the relevant doclets, so the auto-generated documentation (of expression language functions) will be somewhat deficient in the debian version. |
@olebole I have reworked the relevant classes and there are now two alternative doclet implementations that can be used at build time for generating this documentation, one for Java8 and one for Java17+ (in between, either will do). See commits a2d8b5e^.. e90c092. The commit messages should tell you what you need to know, but get back to me if you want more explanation or anything else. Thanks for making me aware of this issue. |
Hi Mark,
I got the bug Debian#1025440 that ttools does not compile anymore on Debian. It turns out that this is due to the import of com.sun.javadoc classes in the uk.ac.starlink.ttools.build package:
The com.sun.javadoc API was replaced with jdk.javadoc.doclet and deprecated some time ago and is now removed with JDK 17, so probably the uk.ac.starlink.ttools.build package needs a rewrite.
The text was updated successfully, but these errors were encountered: