Travis: Add JDK 9 to matrix of CI builds

Description

Recently we declared compatibility of IzPack to JDK 9.
Unfortunately, in our Travis CI builds, on activating Oracle JDK 9 the build fails for this JDK due to missing JAXB module in classpath.

Try to fix it.

Environment

None

Activity

Show:
René Krell
March 29, 2018, 1:40 PM
Edited

It seems we will have to remove the built-in support for Substance, which will be a separate issue.
As well as probably call the upcoming version 5.2 instead of 5.1.3.
So far my suggestion.

See also IZPACK-1185. This will also need changes in the Wiki - GUI Preferences.

Reasons:

  • For each JDK we need a different version of Substance

  • There are not all versions of Substance deployed to the Maven Central repositories, and those which there are are deployed by someone else than the developer as fork of the original sources.

Adam Jurcik
March 29, 2018, 11:01 AM

I digged a bit more and found out that acctually it is the InstallationTest.testSubstanceLaf which errors for me. It seems that it errors due to Java 9, please see https://github.com/Insubstantial/insubstantial/issues/56.
While debugging I came to the same place as mochaman in his comment (3 Oct 2017).

So it seems, that Substance will not be available on Java 9 until we use a different version of the library or we patch it for ourselves.

Patrick Reinhart
March 29, 2018, 6:05 AM

@Adam Jurcik I would suggest to create a jdk9 branch on the main project and I will then create a PR containing my current changes....

René Krell
March 28, 2018, 11:31 PM

The InstallationTest fails also for me localy on the current IzPack master.

You probably run on a different OS and /or other environment.
There is a discussion about this here.
Can you play a bit with the version of the Substance L&F library versions?
Original development site: https://github.com/kirill-grouchnikov/substance/releases
Unfortunately, in Maven Central there is only 7.3 as the latest version:

You might also recompile the latest one from the original Github repository.
Unfortunately the API changed.
I'm not sure Substance is the way for us in future or whether we will have to kick it out.

Adam Jurcik
March 28, 2018, 10:43 PM

The InstallationTest fails also for me localy on the current IzPack master.

Fixed

Assignee

Patrick Reinhart

Reporter

René Krell

Impact

Code optimizations and cleanup - safe for all environments

Components

Sprint

None

Affects versions

Priority

Medium