Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Avoid hardcoding /bin/bash #4320

Closed
ocaisa opened this issue Aug 9, 2023 · 5 comments
Closed

Avoid hardcoding /bin/bash #4320

ocaisa opened this issue Aug 9, 2023 · 5 comments
Milestone

Comments

@ocaisa
Copy link
Member

ocaisa commented Aug 9, 2023

There are a few cases (including the eb wrapper itself) where we hardcode the use of /bin/bash. There are some cases (like EESSI) where this is not ideal and it's better to ask the environment where to find bash.

@ocaisa
Copy link
Member Author

ocaisa commented Aug 9, 2023

There are perhaps even some indications in an EESSI issue that this leads to problems.

@boegel boegel added this to the 4.x milestone Aug 15, 2023
@boegel boegel modified the milestones: 4.x, 5.0 Jan 16, 2024
@boegel boegel added the EasyBuild-5.0 EasyBuild 5.0 label Jan 16, 2024
@boegel
Copy link
Member

boegel commented Jan 16, 2024

/bin/bash is hardcoded in:

  • eb wrapper script
  • in run_shell_cmd
  • in RPATH wrapper script (easybuild/scripts/rpath_wrapper_template.sh.in)

We should use /usr/bin/env bash instead, or resolve path to bash (in run_shell_cmd)

@bedroge
Copy link
Contributor

bedroge commented Jan 16, 2024

There are perhaps even some indications in an EESSI issue that this leads to problems.

We're indeed seeing issues with at-spi2-core in EESSI now, and one of the reasons is that the EB RPATH wrappers have a /bin/bash shebang (and because LD_LIBRARY_PATH is being set to some Gentoo Prefix directories, this doesn't work).

@boegel
Copy link
Member

boegel commented Jan 26, 2024

@ocaisa I guess we can close this now #4441 is merged?

@ocaisa
Copy link
Member Author

ocaisa commented Jan 26, 2024

Yup

@ocaisa ocaisa closed this as completed Jan 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants