On 24/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.5/

Exactly the same path names were used previously; I assume you are
referring to the following versions of the files:

apache-tomcat-jdbc-1.0.5-bin.tar.gz.md5:808cf400c4f7f4de7294b844c68108fa
apache-tomcat-jdbc-1.0.5-bin.zip.md5:3f20849d6b0dbe29bb9707cd519c456c
apache-tomcat-jdbc-1.0.5-src.tar.gz.md5:6a63d1e77c47c5d6385cf680dac4514c
apache-tomcat-jdbc-1.0.5-src.zip.md5:7b4870d50e498a18014031589b8a88eb

rather than the older ones:

apache-tomcat-jdbc-1.0.5-bin.tar.gz.md5:b6081e6d34a8e9ecd70b505c90e73485
apache-tomcat-jdbc-1.0.5-bin.zip.md5:76cb2efd7ce7093d71e4a989e71d2874
apache-tomcat-jdbc-1.0.5-src.tar.gz.md5:d8d08870f3479080582d3261a4d1afe5
apache-tomcat-jdbc-1.0.5-src.zip.md5:cc6992ff33524f15052f9b72588b628f

==

The source and test source archives contain META-INF/MANIFEST.MF files
which don't belong in a source archive.

The binary archives contain MD5 hashes of all but one of the jars;
again, these don't belong in the archives.

The jars should contain NOTICE and LICENSE files.

There's no easily accessible documentation on how to build and test the code.
If someone is familiar with Ant, they can work out what the targets
do, but the user should not have to do this.

The ant script automatically downloads jars, some of which don't have
Apache Licenses. In particular, the MySQL licence appears to be GPL,
which is not compatible with the AL.

AFAICT, this is specifically forbidden:

http://www.apache.org/legal/3party.html#options-build-may2

The "ant test" target generates a few warnings, e.g.

WARNING: Database connection pool evicter thread interval is set to
lower than 1 second.

Several of the tests fail.

There's no documentation on what database needs to be set up in order
to run the database tests so I don't know if these are due to failure
to set up the database correctly or whether the test failures were
nothing to do with the database.

Ideally, the user should be given the option of running the JDBC tests
against whatever database they prefer. If the tests require tables etc
to be set up, these should either be done as part of the test setup,
or there should be clear documentation on what data needs to be set
up.

>  <ballot>
>  [ ] STABLE - I couldn't find any bugs
>  [ ] BETA   - I found some bugs but not critical
>  [ ] BROKEN - I found some show stoppers
>  </ballot>
>
>  Any comments ?
>
>  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

Reply via email to