On Mon, May 14, 2012 at 9:34 PM, Kevan Miller <[email protected]>wrote:
> checksum/signature, build, rat, source, license/notice -- all look fine. > > Here's my +1, but I have a few comments/questions. > > 1. velocity.log need not be in svn. best that we remove… > velocity.log is not in the svn, and I noticed it's in the source zip only, and think it might be auto-introduced during release process with maven release plugin. > 2. why does geronimo need a bundle release of bval? any reason why our > requirements have not/cannot be pushed into BVAL? > Geronimo needs some integration code in ValidationParser.java. And the 0.4 release of bval itself is not a bundle either. > 3. why is there a CDDL license and note in the NOTICE file? > I have no idea why there is a CDDL license resource there, anyone else know the history? > > --kevan > > On May 14, 2012, at 12:06 AM, Forrest Xia wrote: > > > This is a version upgrade release, from 0.3_incubating to 0.4, for bval > bundled in Geronimo, raised via GERONIMO-6345, it's to fix the tck failures > when upgrading BV TCK to 1.0.6.GA level. > > > > The artifacts up for vote are: > > > https://repository.apache.org/content/repositories/orgapachegeronimo-086/org/apache/geronimo/bundles/bval-jsr303/0.4_1/bval-jsr303-0.4_1-source-release.tar.gz > > > https://repository.apache.org/content/repositories/orgapachegeronimo-086/org/apache/geronimo/bundles/bval-jsr303/0.4_1/bval-jsr303-0.4_1-source-release.zip > > > > > > The tag is: > > http://svn.apache.org/repos/asf/geronimo/bundles/tags/bval-jsr303-0.4_1/ > > > > Vote will be open for 72 hours. > > > > [ ] +1 approve > > [ ] +0 no opinion > > [ ] -1 disapprove (and reason why) > > > > -- > > Thanks! > > > > Regards, Forrest > > > > -- Thanks! Regards, Forrest
