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

Travis configuration improvements #37

Merged
merged 1 commit into from
Mar 14, 2019

Conversation

wborn
Copy link
Member

@wborn wborn commented Mar 12, 2019

Similar to openhab/openhab-core#643, I've also improved the openhab-webui Travis configuration.


This PR makes it possible again to follow progress of a Travis build while it's running. While building it shows the reactor progress information while making sure the build doesn't timeout by appending zero width characters every 3 minutes.

While I was at it I've made some other configuration improvements as well:

  • Uses the new Xenial Travis images which easily allows for running Java 11 builds
  • Builds using OpenJDK 8 instead of Oracle JDK 8. Oracle JDK 8 is no longer available in the Xenial images. I've done a lot of builds already with it and it works equally well.
  • Adds a new OpenJDK 11 build which is allowed to fail. This makes it easy to see what needs to be done for successfully compiling the code with Java 11. It disables the enforcer for the Java 11 build.
  • On successful builds only the reactor summary is printed (instead of the most recent 200 lines)
  • Removes JVM options that are no longer supported in Java 8 and which result in warnings:
    -XX:PermSize=512m -XX:MaxPermSize=1g
  • Fixes/adds missing NPM caches for the various UIs

The build times remain the same with this configuration.

Signed-off-by: Wouter Born <[email protected]>
@wborn wborn requested a review from a team as a code owner March 12, 2019 12:18
@wborn
Copy link
Member Author

wborn commented Mar 12, 2019

Can you also enable Travis for this repository? Looks like there also isn't a Jenkins PR build yet.

@kaikreuzer kaikreuzer merged commit 042899c into openhab:master Mar 14, 2019
@wborn wborn deleted the update-travis-config branch March 15, 2019 01:56
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

Successfully merging this pull request may close these issues.

2 participants