Skip to content

Fix handling of GitHub and URL templates on Windows#4423

Merged
snoyberg merged 1 commit intocommercialhaskell:masterfrom
bradrn:fix-4394
Jan 7, 2019
Merged

Fix handling of GitHub and URL templates on Windows#4423
snoyberg merged 1 commit intocommercialhaskell:masterfrom
bradrn:fix-4394

Conversation

@bradrn
Copy link
Copy Markdown
Contributor

@bradrn bradrn commented Nov 29, 2018

Fixes #4394.

On Windows, the path library converts / to \; this messes up URLs and GitHub templates by causing e.g. yesodweb/simple to be interpreted as yesodweb\simple, which is obviously invalid. This has been fixed by adding an extra String field to the appropriate constructor storing the exact template name which the user has passed (with .hsfiles appended if needed).

Copy link
Copy Markdown
Contributor

@snoyberg snoyberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, looks great!

@snoyberg snoyberg merged commit 9900d99 into commercialhaskell:master Jan 7, 2019
@bradrn bradrn deleted the fix-4394 branch January 7, 2019 12:27
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.

Namespaced templates don't work on windows

2 participants