On 30 August 2011 21:00, Oliver Heger <oliver.he...@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-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
