-
Notifications
You must be signed in to change notification settings - Fork 232
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
Make random-beans build on java 9 #243
Comments
Oh great to see random beans on the road to be java 9 ready 😄 We can wait for lombok issue to be resolved. no hurry. |
Sure. Did you take a look at reflection related changes in java 9? Java 9 forbids accessing private members from a different module. This is going to be a pain for random-bean users, because they will have to open up every package (e.g. |
Yes, indeed. When moving to Java 9, we can profit from the major version v4.x to rewrite random beans using modules. It has been a long time that I would like to rewrite it from scratch.. |
Yes that is a possibility, but if I'm not mistaken it will not change the fact, that it will be significantly harder for the user to start using random-beans on java 9. :( |
Yes yes, I should have added "By the way" after "Yes indeed". Rewriting random beans in modules does not change the fact that its users still have to open packages for introspection. |
According to http://mail.openjdk.java.net/pipermail/jigsaw-dev/2017-March/011763.html |
Thank you for the link, I saw it a couple of days ago on reddit. Anyway, this gives us some time (as libraries developers) to adapt random beans (if necessary) in v4 with Jdk 9. |
- update lombok version to 1.16.16 - add java 9 maven profile which add java.xml.bind module for tests - fix two test which behave different on java 9 Random beans can now be build on java 9 with the right MAVEN_OPTS: --add-opens jdk.compiler/com.sun.tools.javac.processing=ALL-UNNAMED --add-opens jdk.compiler/com.sun.tools.javac.util=ALL-UNNAMED --add-opens jdk.compiler/com.sun.tools.javac.code=ALL-UNNAMED --add-opens jdk.compiler/com.sun.tools.javac.main=ALL-UNNAMED --add-opens jdk.compiler/com.sun.tools.javac.tree=ALL-UNNAMED --add-opens jdk.compiler/com.sun.tools.javac.model=ALL-UNNAMED --add-opens jdk.compiler/com.sun.tools.javac.comp=ALL-UNNAMED --add-opens jdk.compiler/com.sun.tools.javac.file=ALL-UNNAMED --add-opens jdk.compiler/com.sun.tools.javac.parser=ALL-UNNAMED
First we need a java 9 compatible lombok version (see: projectlombok/lombok#1315), because currently on Java 9 Build 157 the build fails with:
The text was updated successfully, but these errors were encountered: