-
Notifications
You must be signed in to change notification settings - Fork 2
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
no longer checks for build receipt resource, skips Install and Download #4
Conversation
set off by a NPE warning and then kind of kept going.
also removes the need to check for that resource file that no longer exists after 2018.
It's no longer included in the current gradle distribution. Breaks support for issue MovingBlocks#2.
makes groovy work with post-2018 gradle. MovingBlocks/groovy-wrapper#4
makes groovy work with post-2018 gradle. MovingBlocks/groovy-wrapper#4
makes groovy work with post-2018 gradle. MovingBlocks/groovy-wrapper#4
makes groovy work with post-2018 gradle. MovingBlocks/groovy-wrapper#4
WrapperExecutor wrapperExecutor = WrapperExecutor.forWrapperPropertiesFile(propertiesFile); | ||
wrapperExecutor.execute( | ||
args, | ||
new Install(logger, new Download(logger, "gradlew", wrapperVersion()), new PathAssembler(gradleUserHome)), |
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.
I axed all this Install
and Download
invocation, but if you want to keep the download-and-install functionality, you leave this in and just replace that wrapperVersion()
with org.gradle.wrapper.Download.UNKNOWN_VERSION: https://github.com/gradle/gradle/blob/00862b8160a09051e8226736314afd16e7f418f7/subprojects/wrapper/src/main/java/org/gradle/wrapper/Download.java#L24
aka "0"
static File getAndVerifyDistributionRoot(File distDir) { | ||
// Extracted from org.gradle.wrapper.Install.getAndVerifyDistributionRoot |
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.
and then if you keep that Install
invocation, there's no reason to have this duplicate copy of this method brought over from the Install class.
Re-adds the ability to rely on an already downloaded Gradle via `gradlew` execution, on top of #4 for better convenience even if breaking backwards compatibility
This will no longer update the gradle distribution when gradle.properties changes. But it also invokes less code to possibly break with gradle changes.
Fixes #3.
and breaks #2
That is, it no longer tries to put commons-io on the bootstrap classpath, because it's not in the current gradle distribution.