greenaddress / abcore

ABCore - Android Bitcoin Core
http://abco.re
MIT License
164 stars 55 forks source link

Upgrade gradlew wrapper scripts (./gradlew & ./gradlew.bat) #125

Open emanuelb opened 3 years ago

emanuelb commented 3 years ago

Description

The ./gradlew & ./gradlew.bat files are old, the ./gradlew require bash dependency which is removed in newer version, gradle version in distributionUrl (in file gradle/wrapper/gradle-wrapper.properties) was updated in the past without updating the related gradle scripts.

Expected behavior

newer versions of the scripts are used which avoid the bash dependency, future updates to gradle will include updates to scripts as well.

Additional info

running:

./gradlew wrapper --gradle-version 6.8.3

which is the command to upgrade the gradle wrapper: https://docs.gradle.org/current/userguide/gradle_wrapper.html#sec:upgrading_wrapper

will result in changes in both gradlew and gradlew.bat files.

the history of ./gradle/wrapper/gradle-wrapper.properties shows that the gradle version was updated several times, but in each update the related wrapper files wasn't updated, thus in the future it's also recommended to run the above command when upgrading the gradle version (as it's change the gradle version in distributionUrl as well)