Skip to content

Conversation

@dreis2211
Copy link
Contributor

@dreis2211 dreis2211 commented Mar 5, 2021

Hi,

unfortunately, the first pipeline run on JDK 16 failed due to the gradle-plugin not being built which the spring-boot-launch-script-tests and spring-boot-loader-tests depend upon. I've missed this on local tests, because previous runs populated my Gradle caches. I'm sorry for that.

As this is only a temporary solution until Gradle 7 is out, I chose to pragmatically disable things for now. There are other ways to fix this. E.g. by extending the toolchain DSL to enforce the compilation step, which would cause the spring-boot-gradle-plugin JAR to be built. Just let me know if you prefer that and I will play around with that idea.

Cheers,
Christoph

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Mar 5, 2021
@dreis2211 dreis2211 changed the base branch from master to 2.4.x March 5, 2021 19:54
@dreis2211
Copy link
Contributor Author

On a related note: Are you planning to eventually upgrade to Gradle 7 in 2.4.x at all?

@snicoll
Copy link
Member

snicoll commented Mar 6, 2021

On a related note: Are you planning to eventually upgrade to Gradle 7 in 2.4.x at all?

We're running against the 7.0 milestones on master only at this point.

@snicoll snicoll removed the status: waiting-for-triage An issue we've not yet triaged label Mar 6, 2021
@snicoll snicoll self-assigned this Mar 6, 2021
@snicoll snicoll added the type: task A general task label Mar 6, 2021
@snicoll snicoll added this to the 2.4.4 milestone Mar 6, 2021
snicoll pushed a commit that referenced this pull request Mar 6, 2021
@snicoll snicoll closed this in 8ac6482 Mar 6, 2021
@snicoll
Copy link
Member

snicoll commented Mar 6, 2021

Thanks for the follow-up Christoph. Let's see how it goes and we can refine if needed.

@dreis2211
Copy link
Contributor Author

And we have a green build ladies and gentlemen.

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