Use the -e switch to echo on platforms where it is needed #4975

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants
@mgalgs

mgalgs commented Sep 5, 2012

Round two for this guy: twitter#4620

Some implementations of echo (like /bin/echo on Linux, as well as the bash builtin echo command) require the -e switch to enable interpretation of escape sequences (like \n, \033[32m, etc).

Use the -e switch to echo on platforms where it is needed
Some implementations of echo (like /bin/echo on Linux, as well as the
bash builtin echo command) require the `-e` switch to enable
interpretation of escape sequences (like \n, \033[32m, etc).
@travisbot

This comment has been minimized.

Show comment Hide comment
@travisbot

travisbot Sep 5, 2012

This pull request passes (merged e6ff14f into df8e3c6).

This pull request passes (merged e6ff14f into df8e3c6).

@mdo

This comment has been minimized.

Show comment Hide comment
@mdo

mdo Dec 8, 2012

Owner

Closing this out as it's against an old branch and is failing tests. Please resubmit after 2.2.2-wip goes out. Thanks!

Owner

mdo commented Dec 8, 2012

Closing this out as it's against an old branch and is failing tests. Please resubmit after 2.2.2-wip goes out. Thanks!

@mdo mdo closed this Dec 8, 2012

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