Skip to content
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

Remove cabal flag - undecidable in or out? #4

Closed
DanBurton opened this issue Aug 8, 2012 · 2 comments
Closed

Remove cabal flag - undecidable in or out? #4

DanBurton opened this issue Aug 8, 2012 · 2 comments

Comments

@DanBurton
Copy link
Owner

Eep, your API changes based on compile-time settings. I think this is
a bad idea, because other packages cannot depend on a flag, so
realistically other packages cannot depend on the instances existing,
so they're nearly useless.

UndecidableInstances is excessively maligned and usually fine anyway.
If it compiles, it won't go wrong.

~ Ben Millwood
http://www.haskell.org/pipermail/haskell-cafe/2012-August/102731.html


I find it unlikely that other packages will depend on Tardis, but in order to make it so they can, this should be addressed in some form, if only in documentation that states "INSTALL WITH FLAG AT YOUR OWN RISK".

@ghost ghost assigned DanBurton Aug 8, 2012
@DanBurton
Copy link
Owner Author

I've added a warning to the .description. I'm leaving this issue open until I've decided for sure whether or not to keep the flag. I'm sort of thinking no.

@bmillwood
Copy link

Sorry for reporting this in the wrong place initially :) but I think many people would agree with me when I say UndecidableInstances isn't very scary.

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

No branches or pull requests

2 participants