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 path-handling on Windows #121

Merged
merged 2 commits into from Aug 21, 2014
Merged

Fix path-handling on Windows #121

merged 2 commits into from Aug 21, 2014

Conversation

@edbrannin
Copy link
Contributor

@edbrannin edbrannin commented Aug 17, 2014

This has the same goal as #47, with a different approach. I'm not sure yet which is better.

I'm making this PR to make it easier to compare the results of both approaches directly.

edbrannin added 2 commits Aug 14, 2014
nav.FileContext.make_absolute() is only used to compare
links to entries in mkdocs.yml, so it makes sense for it
to only use POSIX-style paths.

Without this, the site fails to build because it checks for
a Windows-style path in a list of POSIX paths in
build.PathToURL.__call__().

This is the last change required to build the mkdocs site
on Windows.
@edbrannin
Copy link
Contributor Author

@edbrannin edbrannin commented Aug 17, 2014

I'll try to add some tests for this (starting with the one in #47) soon.

@d0ugal
Copy link
Member

@d0ugal d0ugal commented Aug 20, 2014

I'll wait for the tests, but otherwise this sounds reasonable and LGTM.

tomchristie added a commit that referenced this pull request Aug 21, 2014
@tomchristie tomchristie merged commit 80c9b28 into mkdocs:master Aug 21, 2014
1 check passed
1 check passed
@tomchristie
continuous-integration/travis-ci The Travis CI build passed
Details
@edbrannin
Copy link
Contributor Author

@edbrannin edbrannin commented Aug 21, 2014

Oh, thank you! I'll keep targeted tests for this on my backlog.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants