Hi Andy,
On Wed, Jan 15, 2014 at 2:52 PM, Andy Seaborne <[email protected]> wrote: > On 10/01/14 11:25, Andy Seaborne wrote: >> >> I don't think this is ready for release. >> >> Either we need to delay 2.11.1 or make it wait until the next release. >> >> Removing from this release seems the better course to me so there is no >> time issue on cleaning it up. >> >> >> 1/ I fixed up all the source files yesterday assuming it was just an >> oversight. >> >> 2/ LICENSE wrong. I can fix that for the AL part but whether there is >> more to add because ... >> >> 3/ there is no NOTICE >> >> I see org.eclipse in the dependency tree so presumably EPL licensed >> which needs mentioning. >> >> Not fixed. I don't know how the dependencies are used. >> >> 4/ Package space is org.openjena (minor but easier to fix now?) >> >> If cleanup is going to take a while, we ought to move it out of trunk >> for the moment. >> >> Andy >> > > Move to /Experimental/jena-maven-tools - this is very far from ideal but if > it's in /trunk, it's in the formal source-release artifact, which it can't > be and I don't have time at the moment to sort the L&N out. Therefore, it > would hold up the release for a significant period of time unless moved. > > Is there any reason to leave the code in org.openjena? Is so, what about > placeholder classes redriecting to the org.apache.jena ones? I have: * moved the code to org.apache.jena * added the missing license headers * removed some of the obsolete code (src/site, for example) * run Rat: ian@ian-desktop $ mvn apache-rat:check [INFO] Scanning for projects... [INFO] [INFO] ------------------------------------------------------------------------ [INFO] Building Jena plugins for Apache Maven,including schemagen 0.4-SNAPSHOT [INFO] ------------------------------------------------------------------------ [INFO] [INFO] --- apache-rat-plugin:0.10:check (default-cli) @ jena-maven-tools --- [INFO] 51 implicit excludes (use -debug for more details). [INFO] Exclude: **/target/**/* [INFO] Exclude: **/*.log [INFO] 31 resources included (use -debug for more details) [INFO] Rat check: Summary of files. Unapproved: 0 unknown: 0 generated: 0 approved: 29 licence. [INFO] ------------------------------------------------------------------------ [INFO] BUILD SUCCESS [INFO] ------------------------------------------------------------------------ [INFO] Total time: 1.543s [INFO] Finished at: Wed Jan 15 15:14:56 GMT 2014 [INFO] Final Memory: 11M/216M [INFO] ------------------------------------------------------------------------ [~/workspace/jena/jena-maven-tools] I can't commit the code at the moment, due to a conflict in svn. I'll try to sort that out now. Ian
