instance Eq1/2 etc are only available with base-4.9

[phadej]

This is not indicated in the documentation of these instances, neither in the changelog.

Therefore I was fooled into making broken hashable release.

The easiest "fix" would be to drop GHC-7 support.

I'd also welcome removal of all other conditional interfaces, even at the cost of removal of old GHC support. (especially if it's only GHC-7).

[phadej]

I'm inclined to make restricting base >=4.9 revisions for all containers since 0.5.9.1.

[treeowl]

Eq1 and Eq2 don't exist in earlier versions of base, do they? Different (superior, IMO) classes by the same names existed elsewhere.

[treeowl]

@phadej , why would you do that?

[phadej]

Because the conditional API makes it hard for me to support ranges of container versions. Or I'll just drop hashable GHC-7 support.

EDIT: I'll just do that. People need unordered-containers supporting GHC-8.0 support to use hashable-1.4 anyway.

[treeowl]

I don't know how you expect containers to instantiate a class that doesn't exist.

[phadej]

The Eq1 exists in transformers (EDIT: with old base), and for non-boot libraries using then (Eq1 from base or transformers) interchangeably is (almost) transparent. Ditto hashable issue.

[treeowl]

Okay, well, you're just going to have to test for both base and containers versions to use instances in containers of classes in base. That's normal.

[phadej]

I see this won't be fixed on containers side.

Please consider dropping old GHC support. It would make your life easier as well.

[treeowl]

I am all in favor of doing so.