-
Notifications
You must be signed in to change notification settings - Fork 175
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
module-info.java
missing "exports"
#409
Comments
@triathematician Is this with version 2.12.1? And yes, I think that would be needed indeed. |
yes, version 2.12.1 |
module-info.java
missing "exports"
Fixed as suggested, will be in 2.12.2. Also pushed snapshot for 2.12.2-SNAPSHOT to Sonatype OSS repo if anyone could test (that or just build from 2.12 branch). |
I had the same problem, and although 2.12.0-SNAPSHOT fixes the error inside the module-info.java, my application still cannot see the Kotlin-specific converters like |
@csotiriou I had a typo -- fix will be in 2.12.2, not 2.12.0 that was released earlier. But I am not sure |
Well, I had a typo myself. :) I was actually building 1.12.2-SNAPSHOT. I switched to using com.fasterxml.jackson.module.kotlin.jacksonObjectMapper and it's fine. I am stuck at other issues for the time being - I will post back here if I have any other problems, thanks. |
:) ok makes sense! |
@cowtowncoder I believe I found an error (just started using Java 9 modules, so please bear with me). I believe an additional line should be added: I have a JavaFX application, and I received the following error while using this library with Retrofit
Adding |
that is possible |
@csotiriou Thank you: I added that requires directive; please lmk if there are remaining issues. |
Attempting to use com.fasterxml.jackson.kotlin fails because "Symbol is declared in module 'com.fasterxml.jackson.kotlin' which does not export package 'com.fasterxml.jackson.module.kotlin'"
I believe module-info should have an exports line added:
The text was updated successfully, but these errors were encountered: