2012/2/9 Sébastien Brisard <sebastien.bris...@m4x.org>:
> 2012/2/9 sebb <seb...@gmail.com>:
>> 2012/2/9 Sébastien Brisard <sebastien.bris...@m4x.org>:
>>> Hi,
>>>
>>>>
>>>>    svn:keywords = Id
>>>>
>>>> Propchange: 
>>>> commons/proper/math/trunk/src/main/java/org/apache/commons/math/filter/DefaultProcessModel.java
>>>> ------------------------------------------------------------------------------
>>>>    svn:keywords = Id
>>>>
>>>> Propchange: 
>>>> commons/proper/math/trunk/src/main/java/org/apache/commons/math/filter/KalmanFilter.java
>>>> ------------------------------------------------------------------------------
>>>>    svn:keywords = Id
>>>>
>>>> Propchange: 
>>>> commons/proper/math/trunk/src/main/java/org/apache/commons/math/filter/MeasurementModel.java
>>>> ------------------------------------------------------------------------------
>>>>    svn:keywords = Id
>>>>
>>>> Propchange: 
>>>> commons/proper/math/trunk/src/main/java/org/apache/commons/math/filter/ProcessModel.java
>>>> ------------------------------------------------------------------------------
>>>>    svn:keywords = Id
>>>>
>>>>
>>>
>>> I think the property should read
>>> Author Date Id Revision
>>
>> Please don't use Date - it is locale-dependent and causes SVN tags to
>> disagree with source archives.
>>
>> Also @Author tags are strongly discouraged.
>> We  use changes.xml and contributors in the POM instead.
>>
> OK, my mistake.
> I was conforming with what I saw in existing files. So should the
> svn:keywords property read "Id", or "Id Revision"?

Both Id and Revision are fine, and can both be useful.

There's no harm in having the keywords set - it's only a problem if
they are *used*.
But having them set might encourage their use.

> Thanks for pointing this out,
> Sébastien
>
>
> ---------------------------------------------------------------------
> 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