Skip to content

Conversation

@dpwright
Copy link

These are the minimum required fixes to get greencard building under GHC
7.10. As noted in the comments around the existing Functor instances,
though, this may cause compatibility issues with earlier compilers and
indeed other, non-GHC Haskell 98 compilers.

I am submitting this commit not with a view to having it accepted as is
but as a starting-point for a conversation about how to move forward
with supporting the latest GHC alongside earlier versions.

These are the minimum required fixes to get greencard building under GHC
7.10.  As noted in the comments around the existing Functor instances,
though, this may cause compatibility issues with earlier compilers and
indeed other, non-GHC Haskell 98 compilers.

I am submitting this commit not with a view to having it accepted as is
but as a starting-point for a conversation about how to move forward
with supporting the latest GHC alongside earlier versions.
@dpwright
Copy link
Author

As noted in the commit message, I've just made the simplest possible changes here to get it to compile under the latest GHC, but these changes can't go in as-is because they will have rather large compatibility implications.

The simplest way to resolve these would be to wrap the new instances in CPP guards, hiding them from older versions. I note, though, that the existing Functor instances were commented out rather than using CPP, so I'm not sure if that's a direction you want to go in.

Incidentally, what were the compatibility issues that those Functor instances introduced? I think that must have been before my time...

@sinelaw
Copy link

sinelaw commented Aug 26, 2015

+1!!

@cotrone
Copy link

cotrone commented Jun 9, 2016

I ran in to this issue as well. How can I help make this happen?

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.

3 participants