search

basvandijk / case-insensitive

Case insensitive string comparison
Other
25 stars 19 forks source link

NoImplicitPrelude breaks GHC 6.12.1 #8

Closed singpolyma closed 9 years ago

singpolyma commented 11 years ago

I'm trying to build an app to deploy to Debian stable on production, and this is the version of GHC it has. Removing the NoImplicitPrelude extension makes the library work again (it complains about no fromInteger and >>)

basvandijk commented 11 years ago

Could you check if it the ghc-6.12.1 branch builds on GHC-6.12.1?

singpolyma commented 11 years ago

Yes, this branch fixes the issue (though I'm unclear on why NoImplicitPrelude is needed in this case)

basvandijk commented 11 years ago

Yes, this branch fixes the issue

Thanks, for testing this. I released a 1.0.0.1 version to Hackage.

(though I'm unclear on why NoImplicitPrelude is needed in this case)

I just like to be 100% explicit about what I import.

singpolyma commented 11 years ago

@basvandijk ... but you are 100% explicit. The moment you insert an import Prelude (...) line, like the ones you have, the implicit Prelude import is no longer present, and gets replaced with the explicit one.