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
Embed Javadoc of dependencies in application #1513
Conversation
Might be an idea to do releases with & without javadocs as I guess many users won't use scripting at all |
Great! Yes, I think we shouldn't increase the download by too much. My suggestion is to use a command line flag e.g.
I think the default list should only add ~10MB or so. This also allows the gradle to be simplified and Note: Currently, I can't build this PR fully with
or (if I delete that directory)
|
I added the command line flags and removed the About the error, I wasn't able to reproduce it but I added a duplicate handling strategy that should fix it. Opening the results of the new task with the javadoc viewer seems to work correctly. |
Does it also happen with
Added with last commit.
There is an automatic redirection that is not triggered on the old Javadoc viewer. It seems to work on the new Javadoc viewer, can you check if it also works on your side? |
Actually this increases the startup time by a few seconds each time I run QuPath after changing something, which is quite annoying. Should we remove that feature? |
Would there be a way to associate it with a task that isn't required for Not sure if this is an option, but If it's not straightforward, I agree that the delay of a few seconds is more annoying than the benefit for the javadocs - so better not to generate them. Edit: simpler alternative to the above might be to just require a separate task to be called if the javadocs should be generated (but which |
You can call the newly created task directly ( |
This PR makes the
jpackage
gradle task put the Javadocs of all dependencies alongside the Javadocs of QuPath when creating the executable (inContent/app/docs
for MacOS for example).This adds 279.8 MB to the size of the executable, so it may be better to skip some dependencies.