dejan2609 commented on a change in pull request #10606:
URL: https://github.com/apache/kafka/pull/10606#discussion_r633003316



##########
File path: gradle/wrapper/gradle-wrapper.properties
##########
@@ -1,5 +1,6 @@
 distributionBase=GRADLE_USER_HOME
 distributionPath=wrapper/dists
-distributionUrl=https\://services.gradle.org/distributions/gradle-6.8.3-all.zip
+distributionSha256Sum=13bf8d3cf8eeeb5770d19741a59bde9bd966dd78d17f1bbad787a05ef19d1c2d

Review comment:
       Hello @lbradstreet and thanx for dropping by 😃 !
   
   I am (was) aware of this because I am using both Android Studio and IntelliJ 
IDEA (although with Android plugin disabled) on a daily basis. 
   However, I decided to include checksum validation, herewith rationale: 
   - Gradle 7.0 git tag https://github.com/gradle/gradle/releases/tag/v7.0.0 
notes mentions this: 
   ```
   Upgrade Instructions
   
   Switch your build to use Gradle 7.0 by updating your wrapper:
   
   ./gradlew wrapper --gradle-version=7.0 
--gradle-distribution-sha256-sum=eb8b89184261025b0430f5b2233701ff1377f96da1ef5e278af6ae8bac5cc305
   ```
   - issue is fixed and it will be released soon:
      - git commit: 
https://github.com/JetBrains/android/commit/1a79d8e81b5d5b021161a3456ec98406c0eed0ad
      - ticket https://youtrack.jetbrains.com/issue/IDEA-266355 mentions that 
solution is included into IntelliJ IDEA build **_211.7277_** 
   - this is not an Android project after all
   
   
   All-in-all: given a fact that current IntelliJ IDEA version is 2021.1.1:
   ```
   IntelliJ IDEA 2021.1.1 (Ultimate Edition)
   Build #IU-211.7142.45, built on April 30, 2021
   ```   
   we can expect that next Intellij IDEA patch version 2021.1.2 will resolve 
this.
   
   
   
   
   
   
   
   
   
   
   
   
   
   
   
   
   




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to