commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From sebb <seb...@gmail.com>
Subject Re: [VOTE] Release Commons Configuration 1.7 based on RC2
Date Thu, 01 Sep 2011 18:25:52 GMT
On 30 August 2011 21:00, Oliver Heger <oliver.heger@oliver-heger.de> wrote:
> This is a vote to release Apache Commons Configuration 1.7 based on the 2nd
> RC.
>
> There have been the following changes since RC1:
> * The ant build file has been improved (many thanks to Phil!)
> * The release notes mention that the optional dependency to Commons VFS
> requires Java 1.5+. The page listing runtime dependencies has been updated
> correspondingly.
> * Minor improvements of the building page.
>
> Tag:
> http://svn.apache.org/repos/asf/commons/proper/configuration/tags/CONFIGURATION_1_7RC2/

There are quite a few files without AL headers.
Some are generated Java files, which is OK, but there are some scripts
and other files which could and should have AL headers (assuming that
they are ASF files).

For example:

conf/digesterRules.xml

If the file is very short (e.g. one-line css file) there is no need to
add the header.

Not a blocker, but it does make checking releases harder: a lot of the
source files have

@version $Revision: nnn $, $Date: xxx $

The Date field is Locale-dependent, so my checkout of the SVN tag does
not agree with the checkout you used to create the source archive.

If you really want to see a date, use $Id: $; otherwise keep the
$Revision: $ and drop the $Date: $
Thanks.

> Distributions: http://people.apache.org/~oheger/configuration-1.7rc2/
>
> Maven artifacts:
> http://people.apache.org/~oheger/configuration-1.7rc2/maven/
>
> Site: http://people.apache.org/~oheger/configuration-1.7rc2/site/

Not a blocker, but the description uses Commons Configuration as a
noun; for trademark purposes it needs to be used as an adjective, at
least initially.

For example:

The Commons Configuration software library provides a generic configuration ...

> [ ] +1 release it
> [ ] +0 go ahead I don't care
> [X] -1 no, do not release it because...

Missing AL headers.

Release otherwise looks OK, and builds/tests using Java 1.4 with Maven 2.2.1.

> Vote will remain open for 72 hours.
>
> Oliver
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>
>

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


Mime
View raw message