Skip to content
This repository has been archived by the owner on Mar 6, 2019. It is now read-only.
This repository is currently being migrated. It's locked while the migration is in progress.

Do not suggest to potentially use builtin echo #5

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

andreaspe
Copy link

From man(1) echo on MacOS:
Some shells may provide a builtin echo command which is similar or
identical to this utility. Most notably, the builtin echo in sh(1) does
not accept the -n option.

Following the setup instructions of the chart it's quite hard to spot
if builtin echo is used which is why it might be better to use absolute
path.

From man(1) echo on MacOS:
  Some shells may provide a builtin echo command which is similar or
  identical to this utility.  Most notably, the builtin echo in sh(1) does
  not accept the -n option.

Following the setuo instructions of the chart it's quite hard to spot
if builtin echo is used which is why it might be better to use absolute
path.
@till
Copy link

till commented Oct 31, 2018

I can confirm what @andreaspe added here.

Once I started switching to bash, it worked as intended. Before, it basically send a base64 encoded value prefixed with -n to my k8s cluster.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants