Skip to content

MINOR: Add shebang to gradlewAll#11373

Merged
mimaison merged 1 commit intoapache:trunkfrom
akatona84:patch-1
Oct 6, 2022
Merged

MINOR: Add shebang to gradlewAll#11373
mimaison merged 1 commit intoapache:trunkfrom
akatona84:patch-1

Conversation

@akatona84
Copy link
Copy Markdown
Contributor

More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.

Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.

Committer Checklist (excluded from commit message)

  • Verify design and implementation
  • Verify test coverage and CI build status
  • Verify documentation (including upgrade notes)

@akatona84
Copy link
Copy Markdown
Contributor Author

@ijuma could you approve this change pls?

Python based build executor died because of it:

OSError: [Errno 8] Exec format error: './gradlewAll'

@akatona84
Copy link
Copy Markdown
Contributor Author

@granthenke , could you take a look as well? Thanks!

@viktorsomogyi
Copy link
Copy Markdown
Contributor

@ijuma would you please quickly review this small pr?

@akatona84
Copy link
Copy Markdown
Contributor Author

@ijuma I've added the same shebang as gradlew has to gradlewAll. Could you check it pls? 🥺 🎅 🎄 😄

@akatona84
Copy link
Copy Markdown
Contributor Author

@omkreddy , @ijuma
could you take a look? pretty small change, pretty pls :)

@viktorsomogyi
Copy link
Copy Markdown
Contributor

@mimaison would you please check this quickly? (I think the test failures are unrelated as this only affects the command line)

Copy link
Copy Markdown
Member

@mimaison mimaison left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mimaison mimaison merged commit 2762ce4 into apache:trunk Oct 6, 2022
guozhangwang pushed a commit to guozhangwang/kafka that referenced this pull request Jan 25, 2023
Reviewers: Mickael Maison <mickael.maison@gmail.com>, Viktor Somogyi-Vass <viktorsomogyi@gmail.com>
rutvijmehta-harness pushed a commit to rutvijmehta-harness/kafka that referenced this pull request Feb 9, 2024
Reviewers: Mickael Maison <mickael.maison@gmail.com>, Viktor Somogyi-Vass <viktorsomogyi@gmail.com>
rutvijmehta-harness added a commit to rutvijmehta-harness/kafka that referenced this pull request Feb 9, 2024
Reviewers: Mickael Maison <mickael.maison@gmail.com>, Viktor Somogyi-Vass <viktorsomogyi@gmail.com>

Co-authored-by: Andras Katona <41361962+akatona84@users.noreply.github.com>
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.

3 participants