greenaddress / abcore

ABCore - Android Bitcoin Core

Home Page:http://abco.re

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

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

emanuelb opened this issue · comments

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)