Skip to content

Fix bash shebang for build_frontend_islands

James Fargherrequested to merge
jf_fix_bash_script_shebang into master

What does this MR do and why?

Using /bin/bash in script shebangs isn't portable. This means in nixos, for example, the script fails with:

2025-10-01_21:27:58.49211 webpack                 : $ scripts/build_frontend_islands && NODE_OPTIONS="${NODE_OPTIONS:=--max-old-space-size=5120}" node scripts/frontend/webpack_dev_server.js
2025-10-01_21:27:58.50775 webpack                 : /bin/sh: scripts/build_frontend_islands: /bin/bash: bad interpreter: No such file or directory
2025-10-01_21:27:58.51746 webpack                 : error Command failed with exit code 126.
2025-10-01_21:27:58.51748 webpack                 : info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

So this MR converts it with the portable version. If we wanted to be even more portable, we might rewrite these scripts to use /bin/sh instead.

References

Screenshots or screen recordings

Before After

How to set up and validate locally

MR acceptance checklist

Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports