Skip to content

Fixed: Correct the Unix command for downloading the Gradle wrapper in README (OFBIZ-13417)#1230

Merged
jacopoc merged 1 commit into
apache:trunkfrom
visvendra1991:OFBIZ-13417
May 21, 2026
Merged

Fixed: Correct the Unix command for downloading the Gradle wrapper in README (OFBIZ-13417)#1230
jacopoc merged 1 commit into
apache:trunkfrom
visvendra1991:OFBIZ-13417

Conversation

@visvendra1991

Copy link
Copy Markdown
Contributor

Fixed: Correct the Unix command for downloading the Gradle wrapper in README

(OFBIZ-13417)

Explanation

The "Quick start" section of README.md documented the Unix command to download the Gradle wrapper as an absolute path:

Unix-like OS: `/gradle/init-gradle-wrapper.sh`

The leading slash makes it an absolute path, so running it from the OFBiz home directory fails with No such file or directory.

This changes it to a relative path with the sh prefix:

Unix-like OS: `sh gradle/init-gradle-wrapper.sh`

which matches the script's own --help usage text (gradle/init-gradle-wrapper.sh prints Usage: sh gradle/init-gradle-wrapper.sh [--help] [--upgrade]) and is consistent with the Windows command shown just above it.

Thanks

Comment thread README.md Outdated
… README (OFBIZ-13417)

The "Quick start" section of README.md documented the Unix command to
download the Gradle wrapper as an absolute path (/gradle/init-gradle-wrapper.sh),
which makes it fail with "No such file or directory" when run from the
OFBiz home directory.

Use the relative executable form (./gradle/init-gradle-wrapper.sh), consistent
with the other Unix-like commands in the README (e.g. ./gradlew,
./pullPluginSource.sh). The script is executable, so it runs directly.
@jacopoc jacopoc self-assigned this May 21, 2026
@jacopoc jacopoc merged commit 8cb2027 into apache:trunk May 21, 2026
4 checks passed
@jacopoc

jacopoc commented May 21, 2026

Copy link
Copy Markdown
Contributor

Thank you @visvendra1991, I have merged your Pull Request!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants