From: Olaf H. <ha...@in...> - 2009-07-25 11:41:10
|
Hey Tim, On Friday 24 July 2009 12:10:41 Tim Pizey wrote: > [...] > I have added the patch to the tracker, but also attach here. The patch works with Jena 2.5.7. The change in tests/de/fuberlin/wiwiss/ng4j/sparql/MyResultSet.java is not necessary. When you edit a source file and want to commit it please respect and keep the indentation style in this file, also for new methods. With NamedGraphDataset.close you didn't do so. > What I have not done is changed any jars in /lib. > Those changes would be to update antlr to version 2.7.7 > and replace jena.jar and jenatest.jar with jena 2.5.7 jars Don't forget arq.jar > The bouncycastle jars should not be different, we are just using them > directly fromn the maven repository now. What do you mean by "we" here? Remember, not everyone is using Maven (e.g. me). Maven shouldn't be a requirement to compile NG4J. > I don't know if you are happy to continue to deploy to Melati.org or > whether there is a desire to get the ng4j jars into the main maven repos. > It is actually a bit of work to get them in and keep them updated. > (See the bouncy castle process) ??? I was not aware that I/we "deploy to Melati.org." I do not even know what Melati.org is. Seriously, I cannot answer this question. You are the Maven expert. What do you say? > My sourceforge id is timp I added you as a developer to the NG4J SourceForge project. This means you have write access to CVS now. Welcome! Greetings, Olaf |