fix cabal file for ghc-7.8.4 #2

Merged
merged 2 commits into from Feb 19, 2015

Projects

None yet

2 participants

@s9gf4ult
Contributor

No description provided.

@mokus0
mokus0 commented on 2cfe38c Feb 18, 2015

Do you happen to know what version of GHC first accepted this syntax? I'd prefer not to break older versions if possible.

@mokus0
Owner
mokus0 commented Feb 19, 2015

based on a tip from Carter Schonwald, I've set up a multi-ghc travis build to help narrow down this problem. It appears to actually be related to cabal version rather than ghc version, and this fix is at least partially backward compatible. Gonna go ahead and merge it and see what the buildbot says about it.

@mokus0 mokus0 merged commit bcce1c8 into mokus0:master Feb 19, 2015
@s9gf4ult
Contributor

I can surely say, that cabal-1.22 and ghc-7.8 are not compatible with current version.

If this patch is not backward compatible then maybe major version should be changed with dependencies constraints ... or something like that.

@mokus0
Owner
mokus0 commented Feb 19, 2015

It turned out the syntax you switched to is supported by older Cabal versions as well. This is now on Hackage as 0.1.1.3

@s9gf4ult
Contributor

Great, thanks!

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