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

cronic script problem on FreeBSD #3048

Closed
mayheart opened this issue Feb 20, 2016 · 5 comments
Closed

cronic script problem on FreeBSD #3048

mayheart opened this issue Feb 20, 2016 · 5 comments

Comments

@mayheart
Copy link

For cronic can we get the script helper changed from

!/bin/bash to

!/usr/bin/env bash

This shouldn't break the script on other platforms.

@murrant
Copy link
Member

murrant commented Oct 10, 2016

@mayheart are you running multiple versions of bash or does bash on FreeBSD just have an odd path? Perhaps linking bash to /bin/bash might be a better solution.

@mayheart
Copy link
Author

bash isn't part of the default OS on FreeBSD, it's an additional package which gets installed to /usr/local/bin

@murrant
Copy link
Member

murrant commented Oct 10, 2016

Wouldn't it be better to just ln -s /usr/local/bin/bash /bin/bash?

I usually only see /usr/bin/env used when the executable can be switched by a tool (such as python). This does add the ability for bash to be found anywhere in $PATH, which could lead to unexpected or insecure behavior.

Just want to make sure before accepting this change...

@laf
Copy link
Member

laf commented Oct 11, 2016

We use env in all other scripts hence the initial suggestion I expect. I'd personally not be concerned over $PATH misuse.

@murrant
Copy link
Member

murrant commented Oct 13, 2016

sounds fine.

@laf laf closed this as completed in #4752 Oct 13, 2016
@lock lock bot locked as resolved and limited conversation to collaborators May 18, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants