Hello Oilver, thanks for your review and the suggestions.
2013/12/22 Oliver Heger <oliver.he...@oliver-heger.de> > Hi Benedikt, > > the release notes mention that Lang 3.0 and onwards requires Java 5, > while the 3.2 release actually needs Java 6. I think this has to be > explicitly stated. > Oh, yes! Would probably be better if this was more explicit. But if there are no other urgent problems with the RC, I'd like to spare myself the trouble of cutting an new release just for this. > > Otherwise, everything looks good. Build runs successful with Java 1.7 on > Windows 8.1. Artifacts and site look good; the release notes give a good > explanation for the errors reported by Clirr. > > One very minor point: The built-by header in the generated manifest > files does not contain your Apache ID. You can achieve this by adding > the property -Duser.name=<apacheID> when building the distributions with > Maven. > Didn't know that! I've updated our releasing manual with this information in SVN (for some reason build bot doesn't publish it at the moment, I'll try again later today). What is your vote for this RC? Do I get your +1 or would you rather see a RC2? Benedikt > > Oliver > > Am 22.12.2013 12:11, schrieb Benedikt Ritter: > > Hello All, > > > > We have fixed quite a few bugs and added some significant enhancements > > since Commons Lang 3.1 was released, so I would like to release Commons > > Lang 3.2. > > > > Commons Lang 3.2 RC1 is available for review here: > > https://dist.apache.org/repos/dist/dev/commons/lang/ (svn revision > 3964) > > > > Maven artifacts are here: > > > > > https://repository.apache.org/content/repositories/orgapachecommons-014/org/apache/commons/commons-lang3/3.2/ > > > > Details of changes since 3.1 are in the release notes: > > > https://dist.apache.org/repos/dist/dev/commons/lang/RELEASE-NOTES.txt > > > > > http://people.apache.org/~britter/commons-lang3/3.2-RC1/site/changes-report.html > > > > The tag is here: > > > http://svn.apache.org/repos/asf/commons/proper/lang/tags/LANG_3_2_RC1/(svn > > 1552978) > > > > Site: > > http://people.apache.org/~britter/commons-lang3/3.2-RC1/site/ > > (note some *relative* links are broken and the 1.2 directories are not > > yet created - these will be OK once the site is deployed. > > Also note, that PMD shows a lot of warnings due to useless parenthesis. > > We have decided to leave these in the code for readability. > > Findbugs has a few warnings, none of which we consider to be dangerous, > > but as always: patches are welcome ;-) > > > > Clirr Report (compared to 3.1): > > > > > http://people.apache.org/~britter/commons-lang3/3.2-RC1/site/clirr-report.html > > (note that the Clirr report show 3 errors. This changes are considered > to > > be save for client code. More Information about this can be found in the > > release notes) > > > > RAT Report: > > > > > http://people.apache.org/~britter/commons-lang3/3.2-RC1/site/rat-report.html > > > > KEYS: > > http://www.apache.org/dist/commons/KEYS > > > > Please review the release candidate and vote. > > This vote will close no sooner that 72 hours from now, i.e. after > > December 25 2013 at 12:30 CET > > > > [ ] +1 Release these artifacts > > [ ] +0 OK, but... > > [ ] -0 OK, but really should fix... > > [ ] -1 I oppose this release because... > > > > Thanks! > > > > Benedikt > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org > For additional commands, e-mail: dev-h...@commons.apache.org > > -- http://people.apache.org/~britter/ http://www.systemoutprintln.de/ http://twitter.com/BenediktRitter http://github.com/britter
