Forgot to mention about checkstyle: no idea. If you built the Digester using the provided pom, there shouldn't be ambiguity... any hint? TIA!!! Simo
http://people.apache.org/~simonetripodi/ http://simonetripodi.livejournal.com/ http://twitter.com/simonetripodi http://www.99soft.org/ On Fri, Oct 21, 2011 at 11:14 PM, Simone Tripodi <simonetrip...@apache.org> wrote: > Hi Oliver, > thanks for the deep review!! > > I think that under a legal point of view - NOTICE file is included in > the artifact - it would be better give another RC and take advantage > to fix also PMD violations. > > Let's wait anyway for the end of the vote and see how thing go, in the > meanwhile I fix what you noticed. > > Thanks a lot, alles gute! > Simo > > http://people.apache.org/~simonetripodi/ > http://simonetripodi.livejournal.com/ > http://twitter.com/simonetripodi > http://www.99soft.org/ > > > > On Fri, Oct 21, 2011 at 10:08 PM, Oliver Heger > <oliver.he...@oliver-heger.de> wrote: >> Build works fine with Java 1.5 on Windows 7. Artifacts look good. >> >> I found the following problems: >> - The Copyright in NOTICE.txt says still 2010. >> - The release notes claim to be for version 3.0 in the header. >> - There are some PMD errors which could be addressed (not blocking). >> - A bit strange: When I build the site locally I get 75 Checkstyle errors. >> Maybe this depends on the version of the checkstyle plug-in? >> >> Oliver >> >> Am 19.10.2011 23:38, schrieb Simone Tripodi: >>> >>> Hi all guys, >>> I'm writing to call for a vote to release apache commons-digester-3.1 >>> based on RC1. >>> >>> This is the 3.0 release with the new async feature - not very much but >>> enough to justify a new release. >>> Please take in consideration that broken 3.1 links will be fixed once >>> the site will be deployed. >>> >>> The vote will stay open for 72 hours anc closes on Saturday 22th, at >>> 9:40pm CET. >>> >>> Many thanks in advance for reviewing, have a nice day! >>> All the best, >>> Simo >>> >>> Release notes: >>> >>> >>> http://people.apache.org/builds/commons/digester/3.1/RC1/RELEASE-NOTES.txt >>> >>> Tag: >>> >>> >>> https://svn.apache.org/repos/asf/commons/proper/digester/tags/DIGESTER3_3_1_RC1/ >>> >>> Site: >>> >>> http://people.apache.org/builds/commons/digester/3.1/RC1/site/ >>> >>> Binaries: >>> >>> http://people.apache.org/builds/commons/digester/3.1/RC1/binaries/ >>> >>> Maven Artifacts (staged on Nexus) >>> >>> >>> https://repository.apache.org/content/repositories/orgapachecommons-080/org/apache/commons/commons-digester3/ >>> >>> [ ] +1 release it >>> [ ] +0 go ahead I don't care >>> [ ] -1 no, do not release it because... (please explain why) >>> >>> http://people.apache.org/~simonetripodi/ >>> http://simonetripodi.livejournal.com/ >>> http://twitter.com/simonetripodi >>> http://www.99soft.org/ >>> >>> --------------------------------------------------------------------- >>> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org >>> For additional commands, e-mail: dev-h...@commons.apache.org >>> >> >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org >> For additional commands, e-mail: dev-h...@commons.apache.org >> >> > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org
