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

Adds QNX to BSD platform check in sed function #446

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@rcmadruga

rcmadruga commented Sep 30, 2011

QNX's sed also need *BSD workaround

@ghost ghost assigned bitprophet Sep 30, 2011

@bitprophet

This comment has been minimized.

Member

bitprophet commented Sep 30, 2011

Thanks for the patch! Also tickled to see somebody targeting QNX, I haven't used that in years. (Not intended as a slight, I think it's a really neat piece of tech, just never had a good reason to use it personally.)

@rcmadruga

This comment has been minimized.

rcmadruga commented Sep 30, 2011

Another thing to make Fabric work with QNX is change the env.shell:

env.shell = "/bin/sh -l -c"

as it does not have bash as the default shell.

@rcmadruga rcmadruga closed this Sep 30, 2011

@rcmadruga rcmadruga reopened this Sep 30, 2011

@bitprophet

This comment has been minimized.

Member

bitprophet commented Sep 30, 2011

We cannot change the default value of env.shell w/o breaking backwards compat, and the average user's expectations -- but it's a known thing that users of BSD and other non-Linux Unixes often need to tweak it.

At some point I might need to update the install/about docs with some "Platform-specific errata" like this, to make it a little easier for people to see the pitfalls right away. (Purely organizational -- all known such pitfalls are documented, just not in a "Here are the problems with QNX, here are the problems with FreeBSD" way.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment