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.

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.

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

Reply via email to