Skip to content

Conversation

@BenchmarkingBuffalo
Copy link
Contributor

If the JDK version used to compile the software is not compatible with Java 24, the build fails fast with a hint what has gone wrong. This check happens on the projectsLoaded hook.
Fix #46625

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Aug 12, 2025
If the JDK version used to compile the software is not compatible with Java 24, the build fails fast with a hint what has gone wrong.
This check happens on the 'projectsLoaded' hook.
Fix spring-projectsgh-46625

Signed-off-by: BenchmarkingBuffalo <46448799+BenchmarkingBuffalo@users.noreply.github.com>
@BenchmarkingBuffalo BenchmarkingBuffalo force-pushed the 46625-fail-with-a-useful-error-message-if-gradle-is-run-on-jdk-prior-24 branch from c8e9242 to a6705d5 Compare August 13, 2025 18:26
@snicoll
Copy link
Member

snicoll commented Aug 18, 2025

@BenchmarkingBuffalo thanks for the PR but we'd like to explore another route. I'll update the linked issue accordingly.

@snicoll snicoll closed this Aug 18, 2025
@snicoll snicoll added status: declined A suggestion or change that we don't feel we should currently apply and removed status: waiting-for-triage An issue we've not yet triaged labels Aug 18, 2025
@mhalbritter mhalbritter changed the title Fail fast with explicit error message for incompatible JDK version (currently < 24) Fail with a useful error message if Gradle is run on JDK < 24 Nov 17, 2025
@mhalbritter mhalbritter added type: task A general task and removed status: declined A suggestion or change that we don't feel we should currently apply labels Nov 17, 2025
@mhalbritter mhalbritter added this to the 4.0.0 milestone Nov 17, 2025
@mhalbritter
Copy link
Contributor

Hey @BenchmarkingBuffalo , after some consideration we're happy to accept the pull request. Sorry for the confusion and thanks for your help!

@mhalbritter mhalbritter reopened this Nov 17, 2025
mhalbritter pushed a commit that referenced this pull request Nov 17, 2025
See gh-46818

Signed-off-by: BenchmarkingBuffalo <46448799+BenchmarkingBuffalo@users.noreply.github.com>
philwebb added a commit that referenced this pull request Nov 19, 2025
This reverts commit 4730dcf, reversing
changes made to 024bb72.

See gh-46818
@philwebb philwebb reopened this Nov 19, 2025
@philwebb
Copy link
Member

This unfortunately seems to break Eclipse. I'm going to revert it until I have a bit more time to investigate.

@philwebb philwebb modified the milestones: 4.0.0, 4.x Nov 19, 2025
@snicoll
Copy link
Member

snicoll commented Nov 19, 2025

I am going to close this one as we have a duplicate PR already.

@snicoll snicoll closed this Nov 19, 2025
@snicoll snicoll removed this from the 4.x milestone Nov 19, 2025
@snicoll snicoll added status: declined A suggestion or change that we don't feel we should currently apply and removed type: task A general task labels Nov 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: declined A suggestion or change that we don't feel we should currently apply

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fail with a useful error message if Gradle is run on JDK < 24

5 participants