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

Support running Kythe on recent JVMs. #7703

Merged
merged 3 commits into from May 15, 2019

Conversation

Projects
None yet
2 participants
@benjyw
Copy link
Contributor

commented May 10, 2019

Kythe was stuck requiring JDK8 for a long time.
It now supports up to at least JDK11. This change
upgrades to a recent version, and special-cases
some bootclasspath magic required only on JDK8.

Support running Kythe on recent JVMs.
Kythe was stuck requiring JDK8 for a long time.
It now supports up to at least JDK11, so this
change enables that use.

@benjyw benjyw requested a review from stuhood May 10, 2019

@benjyw benjyw force-pushed the benjyw:update_kythe branch from a3e5589 to 90280ba May 12, 2019

@benjyw benjyw merged commit 29d5bec into pantsbuild:master May 15, 2019

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@benjyw benjyw deleted the benjyw:update_kythe branch May 15, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.