Skip to content

Use portable shebang for bash scripts (backport #542)#543

Merged
j-rivero merged 1 commit intogz-cmake5from
mergify/bp/gz-cmake5/pr-542
Apr 13, 2026
Merged

Use portable shebang for bash scripts (backport #542)#543
j-rivero merged 1 commit intogz-cmake5from
mergify/bp/gz-cmake5/pr-542

Conversation

@mergify
Copy link
Copy Markdown
Contributor

@mergify mergify bot commented Apr 13, 2026

🦟 Bug fix

Summary

Replace #!/bin/bash with portable #!/usr/bin/env bash in all bash scripts for portability across systems where bash may not be installed at /bin/bash (e.g. NixOS).

This came up while packaging gazebo for nixpkgs, where builds fail because /bin/bash does not exist.

Checklist

  • Signed all commits for DCO
  • Added a screen capture or video to the PR description that demonstrates the fix (as needed)
  • Added tests
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • Consider updating Python bindings (if the library has them)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • Updated Bazel files (if adding new files). Created an issue otherwise.
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers
  • Was GenAI used to generate this PR? If so, make sure to add "Generated-by" to your commits. (See this policy for more info.)

Generated-by: Claude Code

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining `Signed-off-by` and `Generated-by` messages.


This is an automatic backport of pull request #542 done by Mergify.

Replace #!/bin/bash with #!/usr/bin/env bash for portability across
systems where bash may not be installed at /bin/bash (e.g. NixOS,
some BSDs, non-standard installations).

Generated-by: Claude <noreply@anthropic.com>

Signed-off-by: Taylor Howard <taylorhoward@me.com>
(cherry picked from commit d67ce48)
@mergify mergify bot requested review from j-rivero and scpeters as code owners April 13, 2026 06:29
@github-actions github-actions bot added the 🪵 jetty Gazebo Jetty label Apr 13, 2026
@mergify mergify bot mentioned this pull request Apr 13, 2026
11 tasks
@j-rivero j-rivero merged commit f887e3c into gz-cmake5 Apr 13, 2026
9 checks passed
@github-project-automation github-project-automation bot moved this from Inbox to Done in Core development Apr 13, 2026
@j-rivero j-rivero deleted the mergify/bp/gz-cmake5/pr-542 branch April 13, 2026 11:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🪵 jetty Gazebo Jetty

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants