On 15/06/2009, Filip Hanik - Dev Lists <devli...@hanik.com> wrote: > Cleaned up and fixed. > > The release is located here: > http://people.apache.org/~fhanik/jdbc-pool/v1.0.4/
NOTICE file is incorrect, it should read: >>> Apache Tomcat JDBC Pool Copyright 2008-2009 The Apache Software Foundation This product includes software developed by The Apache Software Foundation (http://www.apache.org/). <<< [e.g. See http://www.apache.org/dev/release.html#notice-required] This assumes that JDBC Pool was first released in 2008; if not adjust accordingly. Two java files (ResultSet and TestException) don't have AL headers. The jar files don't have NOTICE or LICENSE files. Releases must consist of a source archive; binary archives are optional. The source archive must contain all the items needed to build and test the binary archive, see: http://www.apache.org/dev/release.html#what-must-every-release-contain Therefore the source archive needs to contain the test code. It's not essential, but it's helpful if the jar MANIFEST.MF files contain the following: Built-By: Implementation-Title: Implementation-Vendor: The Apache Software Foundation Implementation-Vendor-Id: org.apache Implementation-Version: Specification-Title: Specification-Vendor: The Apache Software Foundation Specification-Version: X-Compile-Target-JDK: X-Compile-Source-JDK: It might be useful to include the Javadoc in the binary archive. The build.xml defines compile.source=1.5, however some of the classes require 1.6, for example SlowQueryReport uses the generic form of OpenType which was only introduced in 1.6. The build file relies on the following files for testing, however there is no indication where these are to be obtained: c3p0-0.9.1.2.jar mysql-connector-java-5.0.7-bin.jar The test file DefaultTestCase does not define any test cases, so it would help some IDEs if it were marked abstract. The ResultSet and Statement test classes in the test driver directory won't compile when using Eclipse, because Eclipse generates an error for @Override tags applied to methods only defined in interfaces. It's not clear whether this is an Eclipse bug or a Sun Java bug, but it does not really add much to use @Override for interface methods, so perhaps these tags could be removed. > <ballot> > [ ] STABLE - I couldn't find any bugs > [ ] BETA - I found some bugs but not critical > [X] BROKEN - I found some show stoppers Incorrect NOTICE file, missing N&L files Incorrect packging. > </ballot> > > Any comments ? See above. > Thanks, > Filip > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org > For additional commands, e-mail: dev-h...@tomcat.apache.org > > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org