cabal init: parse module names with simpleParse instead of fromString #1140

Merged
merged 1 commit into from Dec 10, 2012

Conversation

Projects
None yet
2 participants
Member

byorgey commented Dec 9, 2012

fromString crashes when given an invalid module name so this was causing cabal init to crash if you happened to have directories containing code that weren't valid module names (e.g. containing hyphens). See #1052. This commit fixes cabal init to deal with invalid module names properly.

dcoutts added a commit that referenced this pull request Dec 10, 2012

Merge pull request #1140 from byorgey/master
cabal init: parse module names with simpleParse instead of fromString

@dcoutts dcoutts merged commit d2b81ef into haskell:master Dec 10, 2012

Member

dcoutts commented Dec 10, 2012

Looks fine, thanks for that.

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