Import of Data.Text.Lazy.Binary about rdf4h

[cordawyn]

Probably somewhat related to this issue is that I can no longer compile the project even at its latest head: c2297f2. Only after I've removed all lines with instance Binary * declarations, I managed to compile it. (It seems import Data.Binary in src/Data/RDF/TriplesGraph.hs doesn't do much as well). How did this "Binary" thing creep into our cute little project? ;-)
It appears, @robstewart57 enabled it for RDF graph serialisation in 2773e4c... Is it still valid, @robstewart57 ?

[robstewart57]

Hi @cordawyn .

Probably somewhat related to this issue is that I can no longer compile the project even at its latest head:

What compiler errors relating to the Binary instances are you getting? I'll take a look into it.

How did this "Binary" thing creep into our cute little project?

The reason for adding Binary instances is to allow users to serialise RDF graphs, for example should they want to persist a graph as a Haskell bytestring to file, or if they wish to transmit an RDF graph between computers over a network with one of the distributed Haskell libraries.

[cordawyn]

Hi @robstewart57! Long time no see :)
Here's the output of make:

Building rdf4h-1.3.2...
Preprocessing library rdf4h-1.3.2...
[ 1 of 14] Compiling Data.RDF.Types   ( src/Data/RDF/Types.hs, dist/build/Data/RDF/Types.o )

src/Data/RDF/Types.hs:514:10:
    No instance for (Binary T.Text)
      arising from a use of `binary-0.7.2.3:Data.Binary.Class.$gdmput'
    Possible fix: add an instance declaration for (Binary T.Text)
    In the expression: (binary-0.7.2.3:Data.Binary.Class.$gdmput)
    In an equation for `put':
        put = (binary-0.7.2.3:Data.Binary.Class.$gdmput)
    In the instance declaration for `Binary PrefixMappings'
make: *** [build] Error 1

(I'm still using good old ghc-7.6.3, hopefully it's not the issue though).

[robstewart57]

OK @cordawyn I've pushed a commit that bumps the lower bounds for text to 1.2.1.0 , which is when the Binary instances were added to that library. See 009d7e7 , which is now HEAD.

[cordawyn]

@robstewart57 That worked for me, thanks!

[robstewart57]

Fixed in 0ffc85b .