Andy I hadn't realised we'd taken over provided the reader and writer pieces ourselves
I already figured out a fix for this bug so I'll port it to our code base Btw I couldn't see any obvious JSON-LD tests in our code base, is this something we would want to migrate from the old jsonld-java-jena module? Cheers, Rob On 22/05/2014 11:20, "Andy Seaborne" <[email protected]> wrote: >On 22/05/14 10:23, Rob Vesse wrote: >> Holger >> >> The jsonld-java module is an external project not affiliated with Apache >> Jena so any bugs with it should be reported to that project on GitHub >> >> Note that our trunk uses that module as a dependency to add JSON-LD >> support to Jena and that will be included in our next release so any >> issues in that module that you can report and help get fixed will feed >> into our use of the module in our next release >> >> Regards, >> >> Rob > >Yes - Jena (dev) has JSON-LD as part of Jena directly and does not >require additional configuration by the user. > >The reader code is at: > >https://svn.apache.org/repos/asf/jena/trunk/jena-arq/src/main/java/org/apa >che/jena/riot/lang/JsonLDReader.java > >Jena's dependency is on the com.github.jsonld-java:jsonld-java module >(the core JSON-LD engine) from that project and does not use the (old) >jsonld-java-jena artifact (integration/jena). > >If there is a bug, could you provide a fix for the code in SVN please? > > Andy > >> >> On 22/05/2014 09:37, "Holger Knublauch" <[email protected]> wrote: >> >>> (I hope this is the right place to ask this question). >>> >>> Looking at >>> >>> >>>https://github.com/jsonld-java/jsonld-java/blob/master/integration/jena/ >>>sr >>> c/main/java/com/github/jsonldjava/jena/JsonLDReader.java >>> >>> it seems that the current JSON-LD reader does not copy the prefix >>> definitions into the target Jena model, even though the writer is >>> correctly creating them. >>> >>> Thanks >>> Holger >>> >> >> >> >> >
