Maven build works fine on Windows 8.1 with JDK 1.7. Artifacts look good. Like Gary I have still problems with understanding the background and purpose of this component based on the documentation on the site - although it already has improved. However, I agree with others that this is not blocking for the release and can be changed later - especially as Bruno has offered his help in this area.
One point I noted is that the md5 files are missing in the dist area. I assume they are required, aren't they? Oliver Am 12.03.2014 00:33, schrieb Matt Benson: > Apache Commons Weaver 1.0 RC3 is available for review at: > https://dist.apache.org/repos/dist/dev/commons/weaver/ (r4624). > > Maven artifacts are at: > https://repository.apache.org/content/repositories/orgapachecommons-1018/ > . > > Tested with Oracle JDKs 6, 7 and 8. > > The Subversion tag is: > http://svn.apache.org/repos/asf/commons/proper/weaver/tags/1.0_RC3/ > (r1576397). > > Site: > http://people.apache.org/~mbenson/commons-weaver-1.0-rc3/index.html > > RAT Report: > http://people.apache.org/~mbenson/commons-weaver-1.0-rc3/rat-report.html > > Keys: https://dist.apache.org/repos/dist/release/commons/KEYS > > Changes since RC2: > - Configured project to run a single all-inclusive RAT report > - Added missing license headers > - Included issueManagement in POM (and thus in generated site) > - Documented resulting need for expanded permgen when generating site > > Changes since RC1: > - Removed -dist suffix from distribution files > - Added release notes to binary distribution > - Included project-specific LICENSE.txt and NOTICE.txt in jar artifacts > - Restricted checkstyle to run only for relevant modules > - Fixed remaining checkstyle errors > - Improved (somewhat) overview documentation > - Removed license header from release notes > - Added FindBugs and PMD reporting > > Please review the release candidate and vote. > This vote will close no sooner that 72 hours from now, i.e. after 0000UTC > 15-March 2014 > > [ ] +1 Release these artifacts > [ ] +0 OK, but... > [ ] -0 OK, but really should fix... > [ ] -1 I oppose this release because... > > Thanks! > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org