Skip to content

Make gradlew executable (mode 100755) - #6

Merged
tdaly61 merged 2 commits into
openMF:devfrom
GiulioRinalduzzi:fix/gradlew-exec-bit
Aug 24, 2026
Merged

tdaly61 merged 2 commits into
openMF:devfrom
GiulioRinalduzzi:fix/gradlew-exec-bit

Conversation

@GiulioRinalduzzi

Copy link
Copy Markdown
Contributor

gradlew is committed with mode 100644, so on Linux and macOS the build fails with permission denied before Gradle starts. Same failure @tdaly61 hit on paymenthub-ee-vouchers and fixed in 9d9c849; this is the last paymenthub-ee-* repo left with the wrong mode.

Cause: the wrapper was added from Windows, where git has core.filemode=false and cannot record the executable bit — a green local build there says nothing about the mode. In this repo specifically the file came in through the multi-module bootstrap commit rather than from the original repo's git objects, which is why it landed 100644 here and 100755 in the other repos.

Change: git update-index --chmod=+x gradlew. No content change — same blob f5feea6, only the index mode. Diff is 1 file changed, 0 insertions(+), 0 deletions(-).

Note that nothing in CI would have caught this: the repo has no workflow that runs ./gradlew on Linux (only cla-check.yml, and CircleCI isn't wired up), so the first Linux execution is always a human's. Happy to add the GitHub Actions test.yml here as a follow-up if you want that closed properly.

gradlew was committed with mode 100644, so on Linux and macOS the build
fails with 'permission denied' before Gradle even starts. It was added
from Windows, where git has core.filemode=false and cannot record the
executable bit; a green build there says nothing about the mode.

Same fix Tom applied to paymenthub-ee-vouchers in 9d9c849. Of all the
paymenthub-ee-* repos this is the last one left with the wrong mode.

No file content changes: same blob, only the index mode.
@GiulioRinalduzzi
GiulioRinalduzzi requested a review from a team August 7, 2026 07:25
@tdaly61
tdaly61 merged commit d2ea911 into openMF:dev Aug 24, 2026
1 check passed
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