Skip to content

Use portable shebang for bash scripts#542

Merged
scpeters merged 1 commit intogazebosim:mainfrom
taylorhoward92:portable-shebangs
Apr 13, 2026
Merged

Use portable shebang for bash scripts#542
scpeters merged 1 commit intogazebosim:mainfrom
taylorhoward92:portable-shebangs

Conversation

@taylorhoward92
Copy link
Copy Markdown
Contributor

@taylorhoward92 taylorhoward92 commented Apr 12, 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.

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).

Signed-off-by: Taylor Howard <taylorhoward@me.com>
Generated-by: Claude <noreply@anthropic.com>
@taylorhoward92 taylorhoward92 marked this pull request as ready for review April 12, 2026 11:18
@taylorhoward92 taylorhoward92 changed the title Use portable shebang (#!/usr/bin/env bash) for bash scripts Use portable shebang for bash scripts Apr 12, 2026
@github-project-automation github-project-automation bot moved this from Inbox to In review in Core development Apr 13, 2026
@scpeters scpeters merged commit d67ce48 into gazebosim:main Apr 13, 2026
8 checks passed
@github-project-automation github-project-automation bot moved this from In review to Done in Core development Apr 13, 2026
@scpeters
Copy link
Copy Markdown
Member

https://github.com/Mergifyio backport gz-cmake5 gz-cmake4 gz-cmake3 ign-cmake2

@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Apr 13, 2026

backport gz-cmake5 gz-cmake4 gz-cmake3 ign-cmake2

✅ Backports have been created

Details

mergify bot pushed a commit that referenced this pull request Apr 13, 2026
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 bot pushed a commit that referenced this pull request Apr 13, 2026
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 bot pushed a commit that referenced this pull request Apr 13, 2026
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 bot pushed a commit that referenced this pull request Apr 13, 2026
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)
j-rivero added a commit that referenced this pull request Apr 13, 2026
Use portable shebang for bash scripts (backport #542)
j-rivero added a commit that referenced this pull request Apr 13, 2026
Use portable shebang for bash scripts (backport #542)
j-rivero added a commit that referenced this pull request Apr 13, 2026
Use portable shebang for bash scripts (backport #542)
j-rivero added a commit that referenced this pull request Apr 13, 2026
Use portable shebang for bash scripts (backport #542)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants