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.

Sign up for GitHub

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

Jump to bottom

The logs command doesn't recognize slashes in script path. It should. #786

Open

dirksen wants to merge 1 commit into foreversd:master from dirksen:master

dirksen commented Dec 24, 2015

I often start my node servers using absolute path, like:

forever start /var/www/foo/bar/app.js

But the logs command doesn't recognize slashes in the script path, and this fails:

forever logs /var/www/foo/bar/app.js

This patch should fix this problem

ssendev mentioned this pull request

make logs accept script with slashes in path #795

Closed


          The logs command doesn't recognize slashes in script path. It should.

e8f214f

kibertoad added the tests-needed label

Contributor

kibertoad commented Jan 5, 2019

Thank you for your contribution! Could you please add tests for this change that would ensure original use-cases still work the same way?

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