Skip to content
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

Fix README #55

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix README #55

wants to merge 2 commits into from

Conversation

jaanli
Copy link
Contributor

@jaanli jaanli commented May 12, 2024

If this command is run with $(PWD), what it was like before, then this error results:

docker run -p 8080:8080 \
    -v $(PWD)/signal-cli-config:/home/.local/share/signal-cli \
    -e 'MODE=normal' bbernhard/signal-cli-rest-api:0.57
PWD: command not found

Let me know if you have any questions! This is a super helpful utility 🙏

jaanli added 2 commits May 12, 2024 14:34
If this command is run with $(PWD), then the following error results:

```
docker run -p 8080:8080 \
    -v $(PWD)/signal-cli-config:/home/.local/share/signal-cli \
    -e 'MODE=normal' bbernhard/signal-cli-rest-api:0.57
PWD: command not found
```
@filipre
Copy link
Owner

filipre commented May 29, 2024

You are right. It should be either $PWD or $(pwd). It looks like this PR is conflicting with the other one that you opened (and I just merged)

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

Successfully merging this pull request may close these issues.

2 participants