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

Fixing file path references on windows and root folder scenarios #139

Closed
wants to merge 3 commits into from

Conversation

adamcarr
Copy link

@adamcarr adamcarr commented Mar 4, 2016

This is related to issue #138

@adamcarr
Copy link
Author

adamcarr commented Mar 5, 2016

Apparently these are failing on linux. I'll test this out on linux and submit a fix this weekend.

This breaks on node v5.7.0 but works on v5.7.1 as they fixed an issue
with Path.relative.
@adamcarr
Copy link
Author

adamcarr commented Mar 5, 2016

his is breaking on linux with node v5.7.0. I don't have a good testing ground for 5.7.1 at this point but will see what I can do. Path.dirname('c:\\someWindowsPath') is coming up as '.' so all tests assume relative. Maybe this means we should fork the tests or omit certain ones on linux? Cross-plat filesystem tests are a PITA.

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