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 windows cwd name issue in includes #54

Closed
wants to merge 4 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@ianturton
Contributor

ianturton commented Jun 5, 2018

When including a file inside a mapfile on a windows machine the current working directory is wrong as windows doesn't end it's paths with a '/' so os.split treats the directory name as a file and removes it.

This PR fixes the issue by adding a '/'.

@ianturton ianturton referenced this pull request Jun 6, 2018

Closed

Incudes fail on windows #55

geographika added a commit that referenced this pull request Jun 11, 2018

@geographika

This comment has been minimized.

Owner

geographika commented Jun 11, 2018

Updated with os.sep and a test case in master, so closing this.

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