Am 23.12.2013 10:26, schrieb Benedikt Ritter:
> 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.
As this is a significant change compared to the previous version, I
would prefer having this mentioned in the release notes. Also on the
site, under "Documentation" it is stated "The current stable release 3.2
[Java 5.0+]".

> 
> 
>>
>> 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).
Great. Thanks.

> 
> What is your vote for this RC? Do I get your +1 or would you rather see a
> RC2?
Well, I am more -0 because of the release notes. But I think you have
enough +1s.

Oliver

> 
> 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
>>
>>
> 
> 


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org

Reply via email to