P.S. Sorry it wasn't my intention to be -ve, I checked out math hoping to vote +1. One thing I meant to mention - I was impressed with the Math User Guide and I think it raises the bar for other components :-)
Niall ----- Original Message ----- From: "Niall Pemberton" <[EMAIL PROTECTED]> Sent: Thursday, December 08, 2005 1:18 AM > I'm -1 on this release candidate. The maven.xml is not up to date - it > doesn't have the change that was made to include the "checkstyle.xml" and > "license-header.txt" files in the source distro and hence those files are > not in RC4 source distro and the maven.xml is the wrong version. I've looked > at the "trunk", "branches/MATH_1_1" and "tags/MATH_1_1_RC4" and that change > is in all of them. This makes me believe that RC4 was built with the wrong > source and IMO questions the validity of the whole of RC4 - what else might > be the wrong version? > > If it hadn't been for the above issue I would have voted +0 on this release > candidate because of the following: > > * The "xdocs" and "release-notes.jsl" are missing from source distribution > which prevents the maven build (i.e. "maven dist") from working with the > source distro. I realixe that typically it wouldn't work anyway, because our > build system requires "commons-build" - however, if a user resolves that and > wants to build from the source distro IMO they should be able to. > > I also have the following observations - none of which would have prevented > me from voting +1: > > * The url on the issue tracking page is, IMO, ugly - for validator and > resources I used a page based on the Commons IO one (but including the > features provided by the Math url) and IMO its pretty good. > > * My preference is to include the maven plugin dependencies (with comments) > that the site/doc generation requires - otherwise the jelly errors are > confusing for someone whose maven distro isn't up to the required versions. > > * According to the maven guys its easier for maven 2 users if > artifactId/groupId is used for dependencies, rather than id. > > I've created a patch to resolve the above (sorry its against the "trunk", > not MATH_1_1 branch) which is attached to Bug 37833 > http://issues.apache.org/bugzilla/show_bug.cgi?id=37833 > > Niall > > ----- Original Message ----- > From: "Phil Steitz" <[EMAIL PROTECTED]> > Sent: Wednesday, December 07, 2005 4:34 PM > > > This vote is about to close. Pls [math] and other commons committers, weigh > in. > > Thanks! > > Phil > > On 12/4/05, Phil Steitz <[EMAIL PROTECTED]> wrote: > > There have been no problems reported with math 1.1 RC4, other than some > > small javadoc fixes, which have been applied to the release branch. > > Assuming a positive vote, I will cut a new signed release, including > > these fixes. > > > > Release notes are here: > > > > http://people.apache.org/~psteitz/commons-math/1-1-rc4/RELEASE-NOTES.txt > > > > And apidocs: > > > > http://people.apache.org/~psteitz/commons-math/1-1-rc4/apidocs/ > > > > The RC4 jar (commons-math-1.1-RC4.jar) has also been > > deployed to the apache internal maven repository at > > cvs.apache.org/repository > > > > Votes, please. The vote will close in 72 hours. > > Thanks in advance! > > > > Phil > > --------------------------------------------------------------- > > [ ] +1 I support this release and am willing to help > > [ ] +0 I support this release but am unable to help > > [ ] -0 I do not support this release > > [ ] -1 I do not support this release, and here are my reasons > > --------------------------------------------------------------- --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
