Skip to content

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

Merged
j-rivero merged 1 commit intoign-cmake2from
mergify/bp/ign-cmake2/pr-542
Apr 13, 2026
Merged

Use portable shebang for bash scripts (backport #542)#546
j-rivero merged 1 commit intoign-cmake2from
mergify/bp/ign-cmake2/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:30
@github-actions github-actions bot added the 🏯 fortress Ignition Fortress label Apr 13, 2026
@mergify mergify bot mentioned this pull request Apr 13, 2026
11 tasks
@j-rivero j-rivero merged commit b9b8d25 into ign-cmake2 Apr 13, 2026
8 checks passed
@j-rivero j-rivero deleted the mergify/bp/ign-cmake2/pr-542 branch April 13, 2026 11:14
@github-project-automation github-project-automation bot moved this from Inbox to Done in Core development Apr 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🏯 fortress Ignition Fortress

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants