Skip to content
This repository has been archived by the owner on May 27, 2019. It is now read-only.

Tweak install.sh for Homebrew #190

Merged
merged 3 commits into from
Nov 15, 2017
Merged

Tweak install.sh for Homebrew #190

merged 3 commits into from
Nov 15, 2017

Conversation

zsau
Copy link
Contributor

@zsau zsau commented Nov 15, 2017

No description provided.

install.sh Outdated
fi

# Replace path to host
sed -i -e "s/%%replace%%/$ESCAPED_HOST_FILE/" "$TARGET_DIR/$APP_NAME.json"
sed -i '' -e "s/%%replace%%/$ESCAPED_HOST_FILE/" "$TARGET_DIR/$APP_NAME.json"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unfortunately this doesn't work on linux 😞 Is there a way to make it compatible?

sed: can't read : No such file or directory

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Whoops, didn't realize that was a sed BSD-ism. Perhaps instead of first copying the files and then using -i, we could just run sed on the original location and output to NativeMessagingHosts?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sounds like a good approach

@maximbaz
Copy link
Member

awesome, thanks 👍

@maximbaz maximbaz merged commit dfb2478 into browserpass:master Nov 15, 2017
@zsau zsau deleted the homebrew branch November 17, 2017 11:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants