Overall, I think these are very good ideas. Thanks for the proposals. More comments below.
On Thu, Jul 4, 2013 at 9:18 PM, Glen Mazza <[email protected]> wrote: > Hi All, here are the next round of improvements I'd like us to consider > for Roller: > > 1.) Update Spring framework from 2.0.7 & 2.5.6 & 1.2 to the latest RELEASE > version (3.2.3.RELEASE apparently: http://mvnrepository.com/** > artifact/org.springframework<http://mvnrepository.com/artifact/org.springframework>). > Hopefully we can standardize on one Spring version. > My only concern here is that we might break OpenID support, which relied on one specific version of Spring Security if my memory serves me right. > > 2.) Switch the default JPA framework from Apache OpenJPA (JPA 2.0) to > EclipseLink JPA (JPA 2.1). All three frameworks (including Hibernate) are > now passing JUnit tests, so we have a choice of any of the three. > Sounds like a very good call. I did not realize OpenJPA had grown so stagnant and fallen behind the times. > 3.) Remove DB connectivity info from the persistence.xml file. > > I'd like to remove line #4 (<non-jta-data-source>${** > datasourceName}</non-jta-data-**source>) from our persistence.xml ( > http://svn.apache.org/viewvc/**roller/trunk/app/src/main/** > resources/META-INF/**persistence.xml?revision=**1491090&view=markup<http://svn.apache.org/viewvc/roller/trunk/app/src/main/resources/META-INF/persistence.xml?revision=1491090&view=markup>) > and do this configuration in the roller-custom.properties file instead. > Our code already handles this: > > http://svn.apache.org/viewvc/**roller/trunk/app/src/main/** > java/org/apache/roller/**weblogger/business/jpa/** > JPAPersistenceStrategy.java?**revision=1498037&view=markup#**l98<http://svn.apache.org/viewvc/roller/trunk/app/src/main/java/org/apache/roller/weblogger/business/jpa/JPAPersistenceStrategy.java?revision=1498037&view=markup#l98> Makes sense. > 4.) New version of ROME-Propono with the latest ROME dependencies (fixing > problem mentioned here: http://mail-archives.apache.** > org/mod_mbox/roller-dev/**201306.mbox/%3C51CF6CF4.**3050709%40gmail.com%3E<http://mail-archives.apache.org/mod_mbox/roller-dev/201306.mbox/%3C51CF6CF4.3050709%40gmail.com%3E>). > This would just be a Maven POM change (I can test it out if Dave wouldn't > mind applying the patch to ROME so we can get a release out of it). As an > alternative, we can switch to Apache Abdera if-and-only-if we have no > longer a need for RSS in Roller (is Roller now Atom-only? 5.0.1 seems to > indicate that...) Switching from ROME to Abdera in Roller could end up > being a lot of work though, probably not worth it unless ROME is really > dormant. Roller is not Atom only and supports RSS feeds for blogs and in the planet aggregator, I would not like to lose those features and they cannot be supported with Abdera. I'd be happy to try to push a fix into ROME and see if we can get a release out. - Dave
