Skip to content
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

Compatibility with Java 9. #31

Open
angelelectrico opened this issue Apr 10, 2018 · 7 comments
Open

Compatibility with Java 9. #31

angelelectrico opened this issue Apr 10, 2018 · 7 comments

Comments

@angelelectrico
Copy link

If you try to compile a Java 9 deep encapsulated module with both code and jdbc you will get the following error:

error: module ModuleName reads package com.j256.ormlite.db from both ormlite.jdbc and ormlite.core

This seems fairly easy to fix, just move the package to the other project or rename it.

Thanks!

@j256
Copy link
Owner

j256 commented Apr 10, 2018

Interesting problem. It can't be moved to core but it can be pushed to a new package in JDBC. Can you move them to com.j256.ormlite.db.jdbc and see if it compiles?

Thanks.

@zanella
Copy link

zanella commented Jun 21, 2018

@j256 moving only jdbc then logger becomes the next issue, and so on, I moved the whole core and jdbc to have new top packages (com.j256.ormlite -> com.j256.ormlite.core), do you think this is an acceptable approach ?

@j256
Copy link
Owner

j256 commented Jun 21, 2018

I think it makes more sense to just move the jdbc packages since core is a lot larger but whatever works for you @zanella in the short-term.

@1fexd
Copy link

1fexd commented Feb 20, 2020

Hello, I had the same problem so I combined to two libraries into one (available here, usable via Gradle/Maven/etc here) works perfectly on Java 11

@mifritscher
Copy link

I hackfixed it as well by simply copying both -android and -jdbc into -core.

@j256
Copy link
Owner

j256 commented Nov 15, 2020

This has been [hopefully] fixed in 5.2. Can someone verify? I'm not working with Java 9 yet.

@j256 j256 closed this as completed Nov 15, 2020
@j256 j256 reopened this Nov 15, 2020
@Jake4-CX
Copy link

This has been [hopefully] fixed in 5.2. Can someone verify? I'm not working with Java 9 yet.

Nope still like this in 6.1 (Java 11)
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants