-
Notifications
You must be signed in to change notification settings - Fork 76
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
Topic/modularize #577
Topic/modularize #577
Conversation
* wip: Fix module running: * downgraded logback version : https://jira.qos.ch/browse/LOGBACK-1491 * add required github.api and com.fasterxml.databind(required github.api) * cherrypick module-info.java from old PR for java 11 * fix mainClassName (mainModule makes automatically) * wip: Fix test running: add implicit exports module for jupiter module (gradle try run tests in module path, if without it) Some refactoring for module-info.java * wip: ignore module-info.java for checkstyle * Update module-info.java with commonmark
Trying this out. Observations so far:
also, not related to packaging, but I'm worried the way BundleUtils and LauncherManagedDirectory obfuscates the resource identifier enough that nothing ever noticed that resource doesn't exist. |
`{}` is for logs. String.format uses printf-style `%` placeholders.
apparently trying to create a directory on a read-only filesystem is an UnsupportedOperation, not a IOException.
Just for the sake of experimentation, to see how far it would get, I commented out chunks of LauncherInitTask until it wasn't trying to do impossible filesystem things anymore. And ... it works? it's a bit dodge, as I haven't been too careful about which parts of the init I cut out, and I suspect it's currently trying to show me an error dialog that's zero pixels wide, but it does display the main stage and attempt to run the game. I think that's your proof-of-concept. Also, after all that worry we had about finding a jdk distribution with javafx, Idea is telling me in modules-info that there are two things providing the javafx modules. I think we must get one from liberica and one from the gradle javafx plugin. |
I have been poking around with this modularization stuff in an effort to see if it's a blocker for merging #562, and I have not been having a good day of it. From the CodeFX blog, by the author of the book The Java Module System:
which led me to a discussion on IDEA's tracker about testing, which seems to be far from resolved. and this article on Testing in the Modular World. … It's left me feeling resigned and pessimistic about this. Earlier, I thought this was going to take some work, and I had my doubts about the return on investment, but I honestly didn't think it was going to be this hard. I thought we were going to have to learn some things and some new tools but the path would be well-charted. And now, well… I'm reminded of Python 3 a few years back. When there was one big group of people saying "Python 3 has been out for five years, get on board with the new program already!", and another group of people who had a lot of code to keep running and they and had no desire to dive in to compatibility-breaking changes with a rocky transition plan. |
So. Maybe I am having a bad day, and this will all look clearer on another day to someone with brighter eyes. But right now I am trying to figure out next steps for the other branches.
then either
any other suggestions? |
No description provided.