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

Check if the path is absolute using path library #126

Merged
merged 1 commit into from
Jul 18, 2014
Merged

Check if the path is absolute using path library #126

merged 1 commit into from
Jul 18, 2014

Conversation

IonicaBizau
Copy link
Contributor

Removed UNIX specific check by replacing it with a check using path methods.

This fixes #125.

Removed UNIX specific check by replacing it with a check using `path` methods.
@UltCombo
Copy link

Nice, this fix seems to work fine. =]

@IonicaBizau
Copy link
Contributor Author

Glad to hear that. 😄

@UltCombo
Copy link

Until now, I thought the bug was at the path.join(pathname || '.', file) part, but of course, that part is never reached for absolute paths which is case here. That is the only reason for that ternary operator, appending pathname only when file is not an absolute path. Oh gawd, no idea how I couldn't see that before. Thanks for the nice patch. =]

@phstc phstc merged commit d1b79fc into cloudhead:master Jul 18, 2014
@phstc
Copy link
Collaborator

phstc commented Jul 18, 2014

@IonicaBizau 🏆 Added to master and released 0.7.4 with this change.

@IonicaBizau
Copy link
Contributor Author

You're welcome. I'm happy that I fixed this bug. 😄

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.

Doubled up the filepaths in Windows
3 participants