Skip to content
This repository has been archived by the owner on Oct 12, 2022. It is now read-only.

Support Java 9+ #77

Merged
merged 5 commits into from
May 7, 2018
Merged

Support Java 9+ #77

merged 5 commits into from
May 7, 2018

Conversation

novak
Copy link
Contributor

@novak novak commented Mar 28, 2018

There was a change made to the oauth2-useragent dependency to fix an
issue with parsing newer versions of Java. This change updates to that
newer version to include the fix.

Additionally, it appears javax.xml.bind is no longer included with Java
9. This has been included as a maven dependency to resolve class def
errors once the oauth2-useragent error was resolved.

novak added 5 commits March 28, 2018 14:08
There was a change made to the oauth2-useragent dependency to fix an
issue with parsing newer versions of Java. This change updates to that
newer version to include the fix.

Additionally, it appears javax.xml.bind is no longer included with Java
9. This has been included as a maven dependency to resolve class def
errors once the oauth2-useragent error was resolved.
Need to use an older version of maven if it will use jdk6.

Relevant issue on Github for Travis: travis-ci/travis-ci#8452
@novak
Copy link
Contributor Author

novak commented Mar 28, 2018

The extra commits were an attempt at getting jdk6 working on Travis, definitely causes some issues. Using jdk7 builds successfully.

I can close and open two pull requests if that is preferred.

@novak
Copy link
Contributor Author

novak commented Apr 18, 2018

@davidstaheli is it possible to get a release with this PR merged?

@dalemyers
Copy link

Bumping this again since we'd like this merged also.

@dalemyers
Copy link

We'd still like this merged. @davidstaheli is this possible?

@davidstaheli davidstaheli merged commit 57155d9 into microsoft:master May 7, 2018
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants