[GitHub] acoburn opened a new pull request #46: Make it possible to build Juneau on JDK 11

[GitBox <gi...@apache.org>]

acoburn opened a new pull request #46: Make it possible to build Juneau on JDK 11
URL: https://github.com/apache/juneau/pull/46
 
 
   This PR makes it possible to build and test Juneau on JDK 11. It does not change the source target from Java 8.
   
   There are a few categories of changes here.
   1. JaCoCo is updated to 0.8.2. The newer version is required for Java 11
   2. JEE modules are explicitly added as dependencies. Starting with JDK 9, `javax.xml.bind` and `javax.activation` packages are no longer included in the Java SE, so they need to be explicitly added.
   3. `URLClassLoader::findResource` is removed in favor of `Class::getResource`. With Java 9+, you can no longer cast a `ClassLoader` to a `URLClassLoader`.
   4. Explicit imports from `java.beans.*`: Java 9+ introduces some new class names in the `java.beans` package which conflict with other imports (e.g. `o.a.j.annotation.BeanProperty`). By avoiding the star import from the `java.beans` package, one can avoid the ambiguous Class names.
   5. The DateFormatter defaults have changed to use CLDR by default (see [JEP 252](https://openjdk.java.net/jeps/252)). In practice, this means that many of the locale-aware tests in `juneau-core-tests` fail because they include a comma or other character between the date and the time part of the resulting String. By setting a system property in the surefire plugin, one can ensure that the effective value of `java.locale.providers` is the same across JDK versions

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services