On Wed, Dec 30, 2009 at 11:31 AM, sebb <seb...@gmail.com> wrote: > On 29/12/2009, Phil Steitz <phil.ste...@gmail.com> wrote: >> Hopefully all problems with JDK versions and the site build have now >> been resolved. As previously discussed, the only difference between >> 1.3 and 1.4 is that the 1.3 sources have been filtered to exclude >> JDBC4 methods. Version 1.3 is for JDK 1.4-1.5 and only builds under >> one of these JDKs. Note that to execute the 1.3 maven build under >> JDK 1.4 you need a 2.0.x version of maven. >> >> Here are the artifacts: >> >> 1.3 (JDBC 3) version: >> http://people.apache.org/~psteitz/dbcp-1.3-rc6 >> http://people.apache.org/~psteitz/dbcp-1.3-rc6/site >> http://people.apache.org/~psteitz/dbcp-1.3-rc6/maven >> http://svn.apache.org/repos/asf/commons/proper/dbcp/tags/DBCP_1_3_RC6/ >> >> 1.4 (JDBC 4) version: >> http://people.apache.org/~psteitz/dbcp-1.4-rc6 >> http://people.apache.org/~psteitz/dbcp-1.4-rc6/site >> http://people.apache.org/~psteitz/dbcp-1.4-rc6/maven >> http://svn.apache.org/repos/asf/commons/proper/dbcp/tags/DBCP_1_4_RC6/ > > Sigs, hashes OK. > Source archives agree with SVN Tag, apart from doap, > release-process.txt and build-1.3.xml. I think these are OK. > > It would be nice if the source manifest included Java source version, > but not essential. > > The source and binary archives both contain the file > > testpool.jocl > > which comes from the src/test directory in SVN. > > As a test file, surely it should not be there? > > Not a blocker, but should be fixed for any future release. > > The 1.3 code builds and tests OK with Java 1.4.2 and 1.5.0, but > generates a lot of stack traces from TestManual.testLogWriter(). > Ideally the output should be suppressed. > > However, compilation fails when using Java 1.6.0. Unfortunately the POM says: > > <!-- Target Java versions are actually 1.4, 1.5 and 1.6 --> > > Also README says: > >>>> > This release of JDBC compiles with and supports JDK 1.4-1.5 (JDBC 3.0) > and JDK 1.6 (JDBC 4.0). The 1.4 binary release requires JDK 1.6 (JDBC 4.0). > The 1.3 binary release was built from filtered versions of the same sources > using JDK 1.5.0_19. > <<< > > The compilation errors seem to be all instances of concrete classes > failing to override abstract methods. > > It may be possible to fix it so that the code will compile OK with > Java 1.6.0, but if not, the descriptions should be updated, as this is > unusual and unexpected behaviour (AFAIK, all other commons components > are upwards compatible).
Additional methods and objects were added to the JDBC contract in JDK 1.6 (known as JDBC 4) which make it impossible to have a version of DBCP which is both compatible with JDK 1.6 and earlier versions. This is the reason Phil is releasing two versions - DBCP 1.4 which is compatible with JDK 1.6 and DBCP 1.3 which is compatible with JDK 1.4/1.5. DBCP 1.3 has been created by automtaically removing the JDK 1.6 methods from the DBCP 1.4 code base. This is something out of our control since the API implemented by DBCP is defined by the JDK but by releasing two versions ebveryone should be satisfied. > I think this warrants a -1. The release notes and readme make it clear that DBCP 1.3 is required for JDK 1.4/1.5 and DBCP 1.4 is required for JDK 1.6. See the paragraph after the one you quoted from the README. I don't believe what you've highlighted is an issue, but even if it is - then its minor and only a documentation one and shouldn't result in us voting down this release. Niall P.S. Had my laptop stolen, so haven't managed to properly review the release yet. > Sorry I was not able to test earlier release candidates. > >> Release notes (common version, ships with both) >> http://people.apache.org/~psteitz/RELEASE-NOTES.txt >> >> Votes, please. This VOTE will close 01-January-2010 03:30 GMT. >> >> [ ] +1 Proceed with release >> [ ] +0 OK >> [ ] -0 OK, but I would prefer... >> [ ] -1 No, showstopper = ... >> >> Thanks! >> >> Phil >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org >> For additional commands, e-mail: dev-h...@commons.apache.org >> >> > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > For additional commands, e-mail: dev-h...@commons.apache.org > > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org
