Yingchun Lai has uploaded a new patch set (#2) to the change originally created by Alexey Serbin. ( http://gerrit.cloudera.org:8080/19963 )
Change subject: [java] alternative distro for Gradle propdeps plugin ...................................................................... [java] alternative distro for Gradle propdeps plugin This patch addresses Gradle build failure due to inability to fetch the required propdeps plugin from repo.spring.io artifactory instance (a.k.a. repository) anonymously. As it turned out, anonymous access and dependency resolution using in 'plugins-release' has been disabled as of end of January 2023 [1]. Without this patch, if the gradle's cache (usually in $HOME/.gradle) had been cleaned, building Kudu would fail while trying to build gradlew as a part of the build. The error message looked like the following: Could not GET 'https://repo.spring.io/plugins-release/io/spring/gradle/propdeps-plugin/0.0.10.RELEASE/propdeps-plugin-0.0.10.RELEASE.pom'. Received status code 401 from server This patch updates the gradle's configuration to use cn.bestwu.propdeps plugin [2] instead of propdeps, since the former looks to be compatible with propdeps and is still available at https://plugins.gradle.org/m2/ at MavenCentral [3]. Also, cn.bestwu.propdeps is the authoritative result of search for 'propdeps' at plugins.gradle.org [4]. Ideally, it would be great to get rid of Maven-like scopes 'provided' and 'optional', converting them into Gradle's 'compileOnly'. However, that requires many changes in the code because of issues with annotations like @Nonnull, @Nullable, etc. That will be taken care of in a separate changelist, while this patch fixes Kudu build failure. [1] https://spring.io/blog/2022/12/14/notice-of-permissions-changes-to-repo-spring-io-january-2023 [2] https://plugins.gradle.org/plugin/cn.bestwu.propdeps [3] https://mvnrepository.com/artifact/gradle.plugin.cn.bestwu.gradle/propdeps-plugin [4] https://plugins.gradle.org/search?term=propdeps Change-Id: Ied311c2a3489b707024d2873a70fd5d84e51844d Reviewed-on: http://gerrit.cloudera.org:8080/19961 Reviewed-by: Abhishek Chennaka <achenn...@cloudera.com> Tested-by: Abhishek Chennaka <achenn...@cloudera.com> Tested-by: Alexey Serbin <ale...@apache.org> --- M java/buildSrc/build.gradle M java/gradle/scopes.gradle 2 files changed, 5 insertions(+), 6 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/63/19963/2 -- To view, visit http://gerrit.cloudera.org:8080/19963 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.17.x Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ied311c2a3489b707024d2873a70fd5d84e51844d Gerrit-Change-Number: 19963 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <ale...@apache.org> Gerrit-Reviewer: Abhishek Chennaka <achenn...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yingchun Lai <laiyingc...@apache.org>
