-Wall -Werror

[Simon Peyton-Jones]

Ashley

| Duncan Coutts wrote:
| >> $ head -1 time/Data/Time/Calendar/Gregorian.hs
| >> {-# OPTIONS -Wall -Werror #-}
| >>
| >> (in actual fact, we don't even build GHC/bootlibs with -Werror except
| >> when validating).
| >
| > Ah, so that's the culprit.
|
| I prefer this, actually, as we get to discover issues sooner rather than
| later. Though putting it in the .cabal file or wherever might be better.

That's fine.  But then you can choose

a) add -fno-warn-orphans
or
b) move the (Show Day) instance to the module declaring Day.

I'm not sure which is best for you, but you're the package author so you get to decide!

Regardless, it'd help if you felt able to do one or the other, because currently the package won't compile at all.

Simon