Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix java 14 url to point to always existing version #960

Merged
merged 2 commits into from
Feb 13, 2020

Conversation

ThadHouse
Copy link
Contributor

This should mean we no longer need to update the version every time oracle removes one.

@codecov-io
Copy link

Codecov Report

Merging #960 into master will increase coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #960      +/-   ##
============================================
+ Coverage     52.55%   52.57%   +0.02%     
  Complexity        1        1              
============================================
  Files           331      331              
  Lines          8968     8968              
  Branches        569      569              
============================================
+ Hits           4713     4715       +2     
  Misses         4049     4049              
+ Partials        206      204       -2

@@ -234,7 +234,7 @@ open class JpackageExec : DefaultTask() {
fileAssociations.ifPresent { propsFile ->
args.addAll("--file-associations", propsFile.asFile.absolutePath)
}
args.addAll("--package-type", installerType.get())
args.addAll("--type", installerType.get())
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this change necessary?

Copy link
Contributor Author

@ThadHouse ThadHouse Feb 8, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because they keep changing flags in jpackage. That's why we're going to a fixed version we're hosting. At least until final java 14 comes out.

Copy link
Member

@JLLeitschuh JLLeitschuh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work. Thanks for fixing this!!!!

@JLLeitschuh JLLeitschuh merged commit 5f463c0 into WPIRoboticsProjects:master Feb 13, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants