Looking good so far Andy! I had a look at the imports and Eclipse is doing a 
great job! 

By the way, do you have a custom coding style for Eclipse? I noticed that the 
Fix imports kept the space between the last character and the ";" character. If 
you could share your Eclipse coding style I'd start using too to ease merging 
of patches.
The branch isn't building yet as there are other references to the old package 
[1]. But after fixing some locally-only in assembler files, some tests like 
TestTDBAssembler start passing again, so looks like the rest of the process 
won't give much trouble (hopefully :)
CheersBruno

[1] https://gist.github.com/kinow/fa054b93df33bc12fb0d

      From: Andy Seaborne <[email protected]>
 To: [email protected] 
 Sent: Friday, April 24, 2015 8:42 AM
 Subject: Jena3 bootstrap process notes
   
Brief progress reports for the bulk conversion steps:

* Tag the split point.
* Branches jena2 and jena3 setup
* Package renaming done
* jena-core and jena-arq tests run inside Eclipse, not outside (new 
version numbering not redone yet)

POM changes and build next.


Method:

All code modules, open in Eclispe.  Not intermediate POMs (parent, 
jena-jdbc etc etc), not jena-shaded-guava (if open, it confused Eclipse 
for building ARQ).

rename src/main/java and then src/test/java for jena-core, jena-arq, 
jena-tdb, jena-sdb.

Track down a few other com.hp.hpl.jena uses in strings, comments and 
jena-test-log4j.properties.

Odd things:

It is better to do the bulk conversion before a git-synchronization in 
egit.  Without egit setup, the renames were faster and more reliable

With egit sync'ed, the rename was much slower and I had one Eclipse lock 
up as egit got sorted out.

It seems to be the same thing as using the command line tools only, 
which in the end I used to commit and push.

A few (<5) confusions when classes of the same name were in scope: 
Statement (java.sql. and org.apache.jena.rdf.model) in SDB and jena-jdbc.

A few cases of unused imports introduced.

An old ResultSetRewindable was around (deleted).


   

Reply via email to