Skip to content

Drop fromEnv toolchain restriction for local environment - #9950

Merged
bric3 merged 3 commits into
masterfrom
bdu/drop-fromEnv-toolchain-restriction
Nov 20, 2025
Merged

Drop fromEnv toolchain restriction for local environment#9950
bric3 merged 3 commits into
masterfrom
bdu/drop-fromEnv-toolchain-restriction

Conversation

@bric3

@bric3bric3 commented Nov 13, 2025

Copy link
Copy Markdown
Contributor

What Does This Do

Drop the restrictions on the toolchain lookup features. However maintain the restrictions when running in CI.

This reduce the friction when setting up a new project ; this still requires to have the environment variables when using the testJvm property.

Motivation

In order to allow running on later JDK, we need to drop some constraints on which JDK to look for. This will help in particular

Maybe we can even drive the actual JVM used by gradle with

Additional Notes

A follow-up PR, removes the need to have JAVA_xx_HOME environment variables locally, to use the testJvm feature.

Contributor Checklist

Jira ticket: [PROJ-IDENT]

@bric3bric3 changed the title Drop from env toolchain restriction for local environmentDrop fromEnv toolchain restriction for local environmentNov 13, 2025
@bric3
bric3 marked this pull request as ready for review November 13, 2025 15:30
@bric3
bric3 requested review from a team as code ownersNovember 13, 2025 15:30
@bric3
bric3 requested review from TonyCTHsu and removed request for a teamNovember 13, 2025 15:30
@bric3bric3 added tag: no release notes Changes to exclude from release notes comp: tooling Build & Tooling labels Nov 13, 2025

@PerfectSlayerPerfectSlayer left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

💭 thought: ‏Interesting. I should keep a look at it to update the onboarding documentation and setup scripts. Feel free to remind me to update them when you think we are ready to drop the actual requirements.

Comment thread.gitlab-ci.yml
@AlexeyKuznetsov-DD

Copy link
Copy Markdown
Contributor

💭 thought: ‏Interesting. I should keep a look at it to update the onboarding documentation and setup scripts. Feel free to remind me to update them when you think we are ready to drop the actual requirements.

I agree on that, we need to update our readme files about fact that we need JAVA_xx_HOME for some tests.

@bric3

Copy link
Copy Markdown
ContributorAuthor

I agree on that, we need to update our readme files about fact that we need JAVA_xx_HOME for some tests.

@AlexeyKuznetsov-DD You're fast :) I was working on this (as you saw). The #9968 is stacked on top of this PR.

I should keep a look at it to update the onboarding documentation and setup scripts. Feel free to remind me to update them when you think we are ready to drop the actual requirements.

I intentionally left this as a left-over as I'd like things to settle down before adapting the onboarding doc. The good thing is that currently it shouldn't require to change on laptop config.

@bric3
bric3 merged commit 533755a into masterNov 20, 2025
551 of 553 checks passed
@bric3
bric3 deleted the bdu/drop-fromEnv-toolchain-restriction branch November 20, 2025 16:58
@github-actionsgithub-actionsBot added this to the 1.57.0 milestone Nov 20, 2025
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp: toolingBuild & Toolingtag: no release notesChanges to exclude from release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@bric3@AlexeyKuznetsov-DD@PerfectSlayer@sarahchen6