Skip to content
This repository has been archived by the owner on Jan 23, 2023. It is now read-only.

The sed command requires an argument for the -i parameter on Mac OSX even if the argument is an empty string. This argument is not required on Linux. #119

Closed
wants to merge 1 commit into from

Conversation

jessebeach
Copy link

I was trying to build the create library on a Mac and I kept running into this error

Executing sed -i 's/{{ VERSION }}/1.0.0alpha4/' /Users/jbeach/code/bergie/create/examples/create.js
{ [Error: Command failed: sed: 1: "/Users/jbeach/code/berg ...": invalid command code j
] killed: false, code: 1, signal: null }
sed: 1: "/Users/jbeach/code/berg ...": invalid command code j

A little googling lead me to this note on stackoverflow about the sed command requiring an argument to the -i parameter on Mac, but not Linux.

http://stackoverflow.com/questions/4247068/sed-command-failing-on-mac-but-works-on-linux

Hopefully this patch will run on Linux as well. I can't test it locally not having a Linux install.

…even if the argument is an empty string. This argument is not required on Linux.
@bergie
Copy link
Owner

bergie commented Oct 14, 2012

Travis says:

Error: Command failed: sed: can't read s/{{ VERSION }}/1.0.0alpha4/: No such file or directory

@jessebeach
Copy link
Author

I wonder if it's just completely incompatible across the operating systems. Well, I can keep my small changes locally to build the file.

@bergie bergie closed this in 39a9915 Oct 17, 2012
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