Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Resolve paths, rather than appending to cwd, for full path support. #81

Merged
merged 1 commit into from Apr 23, 2012

Conversation

2 participants
Contributor

jaredhanson commented Apr 9, 2012

This is a full fix for issue #78

It resolves arguments (rather than appending them to cwd), to test if they exist. For example, prior to the patch, this would work:

$ nodemon debug.js

but this wouldn't:

$ nodemon /Users/jaredhanson/Projects/test/debug.js

This was because the existsSync check was testing for the existence of /Users/jaredhanson/Projects/test//Users/jaredhanson/Projects/test/debug.js (cwd appended to the absolute path argument).

FYI, this is being run in a directory that contains a package.json file that does not have a main property.

remy added a commit that referenced this pull request Apr 23, 2012

Merge pull request #81 from jaredhanson/full-path
Resolve paths, rather than appending to cwd, for full path support.

@remy remy merged commit 76420d3 into remy:master Apr 23, 2012

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