Add Gradle 9.0 support #768
Conversation
…nd distsDirName property - Remove --build-file argument from gradle.py as it was deprecated in Gradle 8 and removed in Gradle 9 - Update lambda-build-init.gradle to use 'distributions' instead of deprecated distsDirName property - Update all related unit tests to reflect the command line changes - All unit and integration tests now pass with Gradle 9.0 Fixes: Unknown command-line option '--build-file' error when using Gradle 9.0
github-actions
bot
added
area/workflow/java_gradle
pr/external
stage/needs-triage
| @@ -33,7 +33,7 @@ def build(self, source_dir, build_file, cache_dir=None, init_script_path=None, p | |||
| if not self.os_utils.exists(build_file): | |||
| raise BuildFileNotFoundError(build_file) | |||
|
|
|||
| args = ["build", "--build-file", build_file] | |||
There was a problem hiding this comment.
Random thoughts here.
I see that now the build_file parameter is not used anywhere in the method anymore, so maybe we could remove it from the function signature: build(self, source_dir, cache_dir=None, init_script_path=None, properties=None)
If we do that, we would probably want to update the rest of the places where this is used, like the actions file and remove it from the JavaGradleBuildAction fields, and then remove it from the workflow file where this is called.
This would potentially break users if they're using these classes directly instead of using the LambdaBuilder object that SAM CLI uses, but I guess if that's the case, they can "not upgrade to the new lambda-builders version until they remove this extra parameter from their code".
Nah, but I think at the end of the day, it's okay to leave it like this. Maybe it could be better to add a comment here just mentioning that we're not using the build_file parameter because Gradle doesn't support it anymore, but outside of that, I think we're okay.
Add Gradle 9.0 support by removing deprecated --build-file argument and distsDirName property
Fixes: Unknown command-line option '--build-file' error when using Gradle 9.0
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.