Skip to content

Fix multi-release JAR test on JDK 15 #21605

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

Closed

Conversation

dreis2211
Copy link
Contributor

@dreis2211 dreis2211 commented May 28, 2020

Hi,

I just did a quick for #21604 to see if everything works on JDK 15 with the custom buildJavaHome property I introduced for JDK 14.

This PR fixes a test failure in JarFileTests.multiReleaseEntry.

I'm wondering if we should add some entries in advance already for JDK 16 & 17 ;-)

Cheers,
Christoph

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label May 28, 2020
@snicoll snicoll added type: task A general task and removed status: waiting-for-triage An issue we've not yet triaged labels May 28, 2020
@snicoll snicoll added this to the 2.4.x milestone May 28, 2020
@snicoll
Copy link
Member

snicoll commented May 28, 2020

I'm wondering if we should some entries in advance already for JDK 16 & 17 ;-)

Ah! I don't think we should do that as we'll have to do a check every time we need to support a new JDK anyway.

@dreis2211
Copy link
Contributor Author

I agree - I just wanted to mention it at least once. ;)

@dreis2211 dreis2211 mentioned this pull request Jun 5, 2020
@snicoll snicoll self-assigned this Jun 5, 2020
@snicoll snicoll modified the milestones: 2.4.x, 2.4.0-M1 Jun 5, 2020
snicoll pushed a commit that referenced this pull request Jun 5, 2020
@snicoll snicoll closed this in 78165c6 Jun 5, 2020
dreis2211 added a commit to dreis2211/spring-boot that referenced this pull request Sep 22, 2020
dreis2211 added a commit to dreis2211/spring-boot that referenced this pull request Sep 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: task A general task
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants