search

drlivingston / kr

Clojure API for RDF and SPARQL - provides consistent access to APIs including Jena and Sesame
56 stars 17 forks source link

Using Jena 3.0.0 #15

Open nicolasGuillouet opened 8 years ago

nicolasGuillouet commented 8 years ago

Mainly changed namespaces from com.hp.hpl.jena. to org.apache.jena.* Add to modify literal-type-or-language on kr-jena/kr-jena-core/src/main/clojure/edu/ucdenver/ccp/kr/jena/rdf.clj because getDatatypeURI seems to return always something on the Jena 3.

drlivingston commented 8 years ago

Thanks for the patch, I'll take a look at it latest this weekend. Sorry I've been a little slow in my current condition (recovering from shoulder surgery). Kevin On Nov 13, 2015 04:13, "nicolasGuillouet" notifications@github.com wrote:

Mainly changed namespaces from com.hp.hpl.jena. to org.apache.jena.* Add to modify literal-type-or-language on kr-jena/kr-jena-core/src/main/clojure/edu/ucdenver/ccp/kr/jena/rdf.clj

because getDatatypeURI seems to return always something on the Jena 3.

You can view, comment on, or merge this pull request online at:

https://github.com/drlivingston/kr/pull/15 Commit Summary

  • Using Jena 3.0.0

File Changes

Patch Links:

— Reply to this email directly or view it on GitHub https://github.com/drlivingston/kr/pull/15.

drlivingston commented 8 years ago

And don't feel bad about sending "friendly reminders" if something seems forgotten, it's probably not intentional just out of sight out of mind... On Nov 18, 2015 06:50, "Kevin Livingston" kevinlivingston@gmail.com wrote:

Thanks for the patch, I'll take a look at it latest this weekend. Sorry I've been a little slow in my current condition (recovering from shoulder surgery). Kevin On Nov 13, 2015 04:13, "nicolasGuillouet" notifications@github.com wrote:

Mainly changed namespaces from com.hp.hpl.jena. to org.apache.jena.* Add to modify literal-type-or-language on kr-jena/kr-jena-core/src/main/clojure/edu/ucdenver/ccp/kr/jena/rdf.clj

because getDatatypeURI seems to return always something on the Jena 3.

You can view, comment on, or merge this pull request online at:

https://github.com/drlivingston/kr/pull/15 Commit Summary

  • Using Jena 3.0.0

File Changes

Patch Links:

— Reply to this email directly or view it on GitHub https://github.com/drlivingston/kr/pull/15.

nicolasGuillouet commented 8 years ago

Ok, no problem ;-)

drlivingston commented 8 years ago

Thanks for the reminder, I will make time this week to merge this and maybe look into updating the Sesame deps too.

Sorry the holidays just didn't provide the free time I thought they would.

Kevin

On Wed, Dec 30, 2015 at 4:39 PM, nicolasGuillouet notifications@github.com wrote:

Ok, no problem ;-)

— Reply to this email directly or view it on GitHub https://github.com/drlivingston/kr/pull/15#issuecomment-168021059.