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(walk): match filesystem casing for entries on macOS #33

Merged
merged 2 commits into from
Sep 27, 2018

Conversation

haltcase
Copy link
Owner

Closes #32

We now use a call to expandFilename to get the path's casing as it exists in the filesystem. This has refactored things a bit so that instead of tracking relative paths and converting them to absolutes, we use absolutes and convert them relatives.

This PR also introduces macOS testing on travis against both stable & devel versions of nim just like the linux tests. Stable Nim (< 0.19.0) on macOS will currently fail — this is because FileSystemCaseSensitive returned true for macOS until nim-lang/Nim#8411 which is in Nim >= 0.19.0 (currently devel but soon to be stable).

All this means for users is that the casing of returned filesystem entries doesn't quite match the actual filesystem — the results are otherwise correct, so I don't consider this much of a showstopper.

@haltcase haltcase added the type: bug Something isn't working as intended or expected. label Sep 27, 2018
@haltcase
Copy link
Owner Author

Nim 0.19.0 has been released and is now the stable version which means this PR green lights 🎉 Merging.

@haltcase haltcase merged commit d9d1175 into master Sep 27, 2018
@haltcase haltcase deleted the travis-macos branch September 28, 2018 04:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something isn't working as intended or expected.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant