-
Notifications
You must be signed in to change notification settings - Fork 23
Use environment variables to configure Gradle init scripts #675
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
Conversation
System properties cannot be reliably read from init scripts in certain versions of Gradle. This is not the case for environment variables.
Successful cross platform test run: https://github.com/gradle/gradle-enterprise-build-validation-scripts/actions/runs/11686512948/job/32542510765 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the super quick fix!
One little concern for Windows environment 😛
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
@erichaagdev Regarding System properties cannot be read reliably from init scripts for older versions of Gradle.: I don’t agree with that. It’s only since changing to |
System properties cannot be read reliably from init scripts for older versions of Gradle. This is not the case for environment variables. This PR changes the configuration of the init scripts to be done using environment variables instead of system properties.