search

haskell-infra / hackage-trustees

Issue tracker for Hackage maintainance and trustee operations
https://hackage.haskell.org/packages/trustees/
42 stars 7 forks source link

th-lift-instances broken due to latest text release #235

Closed vmchale closed 5 years ago

vmchale commented 5 years ago

Issue first reported here: https://github.com/bennofs/th-lift-instances/issues/15

There is a PR to fix it currently open.

RyanGlScott commented 5 years ago

mustache-2.3.0 also fails in a similar way:

[3 of 8] Compiling Text.Mustache.Internal.Types ( src/Text/Mustache/Internal/Types.hs, dist/build/Text/Mustache/Internal/Types.o )

src/Text/Mustache/Internal/Types.hs:350:10: error:
    Duplicate instance declarations:
      instance Lift Text
        -- Defined at src/Text/Mustache/Internal/Types.hs:350:10
      instance Lift Text -- Defined in ‘Data.Text’
    |
350 | instance Lift Text where
    |          ^^^^^^^^^

A fix has been applied in the upstream repo, but the version bounds for text on the 2.3.0 Hackage release still need to be revised.

phadej commented 5 years ago

Revisions made for th-lift-instances

RyanGlScott commented 5 years ago

mustache-2.3.0 hasn't been revised, as far as I can tell?

phadej commented 5 years ago

@RyanGlScott this is th-lift-instances broken due to latest text release issue. I simply missed your comment.

(EDIT: mustache has missing bounds on either among other issues, so it's really worth own issue if something is using it).