commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Phil Steitz <>
Subject Re: [VOTE] Release Apache Commons Pool 2.0 RC2 as 2.0
Date Sat, 02 Nov 2013 22:41:07 GMT
On 11/1/13 5:53 AM, Mark Thomas wrote:
> On 01/11/2013 12:27, Benedikt Ritter wrote:
>> Hello Mark,
>> again building with:
> Thanks again for the testing.
>> Apache Maven 3.1.1 (0728685237757ffbf44136acec0402957f723d9a; 2013-09-17
>> 17:22:22+0200)
>> Maven home: /Applications/dev/maven/apache-maven-3.1.1
>> Java version: 1.7.0_40, vendor: Oracle Corporation
>> Java home:
>> /Library/Java/JavaVirtualMachines/jdk1.7.0_40.jdk/Contents/Home/jre
>> Default locale: de_DE, platform encoding: UTF-8
>> OS name: "mac os x", version: "10.9", arch: "x86_64", family: "mac"
>> - build works fine
>> - compared source archives with tag: content is the same
>> - signatures are ok
>> - Site looks good, the one FindBugs issue has been resolved but there is a
>> new one:
>> new org.apache.commons.pool2.proxy.JdkProxySource(ClassLoader, Class[]) may
>> expose internal representation by storing an externally mutable object into
>> JdkProxySource.interfaces
>> Again I cannot say if this a problem, because I'm not involved in the
>> development of pool. Maybe others can comment on this?
> It is an issue and I have fixed it in trunk. It slipped through as it
> only became an issue when I changed the visibility of the class.
> There were also some niggles with the site build that were fixed post tag.
>> All my points from RC1 have been addressed, thanks for that.
> No problem. Thanks for the review. I've further improved the code
> coverage of the tests since RC2 as well.
>> +1
>> Benedikt
> I'm tempted to push to release RC2 as the issues are minor. However,
> since they have all been fixed in trunk and the release process isn't
> anywhere near as painful as I was expecting, I'll probably aim to do a
> (hopefully final) RC3 later today.

Sorry to be late reviewing this.  I have found a few other small things.

0) We forgot to specify required JDK level anywhere.  Fixed in
trunk.  If you regenerate release notes, this will be fixed.
1) The Ant build is broken due to cglib dependency.  We should
either dump the Ant build or fix it.  I don't have time to fix it
right now, but will try to get to it this weekend if no one beats me
to it.
2) I can't seem to find your signing key.  Is this a new one?  Have
you put it in /dist/commons/KEYS
3) The class diagrams and a lot of other stuff in the site still
needs updating.  Not a release showstopper; but might be best to
just dump this stuff for now.  I will go after this after 1). 

I am having some trouble with the Ubuntu vm that I use to test JDK
1.6.  I will try to get this running so I can test on 1.6.  All of
my testing up to now has been on 1.7.  We should make sure
everything builds and runs OK on 1.6.  Has anyone tested on 1.6?

> Mark
> ---------------------------------------------------------------------
> To unsubscribe, e-mail:
> For additional commands, e-mail:

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message