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
Fix incorrect Gradle root build directory location #11976
Conversation
I also have prepared a branch merging into |
3e2b4db
to
91dcb8a
Compare
@@ -334,8 +337,10 @@ RepositoryBlacklister createRepositoryBlacklister() { | |||
return new ConnectionFailureRepositoryBlacklister(); | |||
} | |||
|
|||
StartParameterResolutionOverride createStartParameterResolutionOverride(StartParameter startParameter) { | |||
return new StartParameterResolutionOverride(startParameter); | |||
StartParameterResolutionOverride createStartParameterResolutionOverride(StartParameter startParameter, BuildLayoutFactory buildLayoutFactory) { |
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.
Can you use BuildLayout
here? It should be a build scope service created in this same way.
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.
Nope, BuildLayout
is not available at this stage.
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.
Are there any integration tests that check that the verification checks work from subprojects? Was there also a problem here where we wouldn't honor the verification file if the build was run from a subproject?
Verification is not public in 6.1 so it doesn't matter really. |
This would cause a couple of bugs: - a `gradle` directory would be created for each build even if they didn't have one - a `gradle` directory would be created for subprojects if the gradle command was executed from this directory (#11971) In addition, this would cause the verification metadata file to be ignored if running from a subproject directory.
91dcb8a
to
aecda42
Compare
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 as a fix for 6.1 👍
Context
Fixes #11971