-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Fix to avoid incompatibility if compile with jdk9 #6584
Comments
@Craigacp could you please help? |
Sure, that's easy to fix in the build system. How did you manage to skew JAVA_HOME between the build and test if it was in the same command? |
They are on two different machines. We build the jar on Windows then run the tests on Win/Linux/Mac. So very likely the java version mismatches. |
Ok. Basically we need to conditionally set the |
Do we need to continue to support JDK8? |
Personally I would like it to for at least another release, as Tribuo hasn't moved to Java 11 yet, and Tribuo is why I wrote the Java bindings. I'm not sure if our internal users have moved on from Java 8 yet, I'll check. Changing the minimum Java version is probably worth a separate issue and some discussion from the interested parties in Oracle, MS & AWS. I dunno what version of Java DJL uses. Apache Spark only just released support for Java 11 in version 3.0 which came out in the middle of last year. The patch in gradle to make it work when compiling on 9+ but running on 8 is small, I'll work up a PR. |
Describe the bug
See apache/felix#114
Urgency
If there are particular important use cases blocked by this or strict project-related timelines, please share more information and dates. If there are no hard deadlines, please specify none.
System information
To Reproduce
Build onnxruntime with java enabled, and run the tests.
The text was updated successfully, but these errors were encountered: