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

Resolve bash executable path through the PATH variable #47

Closed
wants to merge 1 commit into from

Conversation

yurivict
Copy link
Contributor

@yurivict yurivict commented Apr 4, 2024

No description provided.

@eisenhauer
Copy link
Member

I'm less clear on the need for this. As far as I know /bin/bash exists most everywhere (and certainly on the systems where the ci scripts are run. Is there somewhere that /bin/bash is unavailable which causes problems in dill-config?

@yurivict
Copy link
Contributor Author

yurivict commented Apr 4, 2024

As far as I know /bin/bash exists most everywhere ...

No.

On FreeBSD and all *BSDs bash is in /usr/local/bin/bash.

bash in general can be installed anywhere, but is usually installed in /bin/bash.

@yurivict
Copy link
Contributor Author

yurivict commented Apr 4, 2024

bash is not a part of the operating system.

bash is one of shells. It is just another package that is usually installed by users. There are other shells as well.

@eisenhauer
Copy link
Member

Yeah, I did BSD 4.2 on a VAX back in the day, but of course that was before bash. I'm happy to accept the change to dill-config.in, since that will run in a target environment, but I'll keep the direct reference to /bin/bash in the CI platforms until such a time as we're running CI on a platform that needs the change.

@yurivict
Copy link
Contributor Author

yurivict commented Apr 4, 2024

Ok.
I'd close this then since this isn't critical.

@yurivict yurivict closed this Apr 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants