On 2/20/11 10:15 AM, Luc Maisonobe wrote: > Tag: http://svn.apache.org/repos/asf/commons/proper/math/tags/MATH_2_2_RC2/ > > Distributions: http://people.apache.org/~luc/math-2.2-RC2/ > > Maven artifacts: http://people.apache.org/~luc/math-2.2-RC2/maven/ > > Documentation bundled with the binary distribution: > http://people.apache.org/~luc/math-2.2-RC2/docs/ > > Output of maven:site run against the source distribution: > http://people.apache.org/~luc/math-2.2-RC2/site/ > > Clirr report: > http://people.apache.org/~luc/math-2.2-RC2/site/clirr-report.html > > Release notes: http://people.apache.org/~luc/math-2.2-RC2/RELEASE-NOTES.txt > > Votes, please. This vote will close in 72 hours, 2011-02-23T16:00:00 UTC > > [ ] +1 Release these artifacts > [ ] +0 OK, but... > [ ] -0 OK, but really should fix... > [ ] -1 I oppose this release because... > > Thanks! > > Luc > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > For additional commands, e-mail: dev-h...@commons.apache.org > > +1
Nice work! I have a couple of non-showstopper comments in addition to the release notes comments elsewhere. I do not see any of these as show-stopping. 0) Sigs and hashes are good. Make sure to add your key to KEYS. 1) org.apache.commons.math.util.FastMathStrictComparisonTest produces console output (System.out) 2) You have done a great job maintaining LICENSE.txt and NOTICE.txt. The content is over my head, personally, so it would be great if one of our more legally inclined brethren could take a look at these files and make sure all looks fine. 3) Same request as 2) for the osgi stuff in the Manifest. Has not changed, so should be OK I tested the build in the following environments, always using Maven 2.2.1 for the Maven build tests. Mac OSX 10.6.6 java version "1.6.0_22" Java(TM) SE Runtime Environment (build 1.6.0_22-b04-307-10M3261) Java HotSpot(TM) 64-Bit Server VM (build 17.1-b03-307, mixed mode) No problems with Ant or Maven build Ubuntu Linux 10.10 (running in "Oracle" VirtualBox) java version "1.5.0_19" Java(TM) 2 Runtime Environment, Standard Edition (build 1.5.0_19-b02) BEA JRockit(R) (build R27.6.5-32_o-121899-1.5.0_19-20091001-2113-linux-ia32, compiled mode) Test failures showing a JDK bug. This is clearly a JRockit bug. (The "expected" value is the one returned by Math.min, which should be NaN.) [junit] Testcase: testMinMaxDouble took 0.491 sec [junit] FAILED [junit] min(NaN, 1.0) expected:<1.0> but was:<NaN> [junit] junit.framework.AssertionFailedError: min(NaN, 1.0) expected:<1.0> but was:<NaN> [junit] at org.apache.commons.math.util.FastMathTest.testMinMaxDouble(FastMathTest.java:63) [junit] [junit] Testcase: testMinMaxFloat took 0.024 sec [junit] FAILED [junit] min(NaN, 1.0) expected:<1.0> but was:<NaN> [junit] junit.framework.AssertionFailedError: min(NaN, 1.0) expected:<1.0> but was:<NaN> [junit] at org.apache.commons.math.util.FastMathTest.testMinMaxFloat(FastMathTest.java:96) Ubuntu Linux 10.10 (running in "Oracle" VirtualBox) java version "1.5.0_22" Java(TM) 2 Runtime Environment, Standard Edition (build 1.5.0_22-b03) Java HotSpot(TM) Client VM (build 1.5.0_22-b03, mixed mode, sharing) No problems --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org