[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r984926 - in /subversion/trunk: ./ subversion/libsvn_subr/dirent_uri.c

From: Lieven Govaerts <svnlgo_at_mobsol.be>
Date: Thu, 12 Aug 2010 22:19:48 +0200

On Thu, Aug 12, 2010 at 10:08 PM, Lieven Govaerts <svnlgo_at_mobsol.be> wrote:
> On Thu, Aug 12, 2010 at 9:59 PM, Daniel Shahaf <d.s_at_daniel.shahaf.name> wrote:
>> stefan2_at_apache.org wrote on Thu, Aug 12, 2010 at 19:29:24 -0000:
>>> Author: stefan2
>>> Date: Thu Aug 12 19:29:23 2010
>>> New Revision: 984926
>>>
>>> URL: http://svn.apache.org/viewvc?rev=984926&view=rev
>>> Log:
>>> Merge r983764 from branches/performance. Approved by: danielsh
>>> http://svn.haxx.se/dev/archive-2010-08/0217.shtml
>>
>> Usually, the "Approved by:" appears on a separate line, and an accompanying URL
>> is not necessary.  (See [1].)
>>
>> I won't be bothered if these log messages aren't fixed, but please be aware
>> of the convention for next time.
>
> These conventions are there to make the contribulyzer work no?
>
> http://www.red-bean.com/svnproject/contribulyzer/
>
> Not following these conventions means we have to pay attention to your
> work instead of having an automated tool do that for us :-).
>

Slightly off topic remark, but it would be nice if we could run
stefan's branch on the buildbots from time to time, would be
interesting to see the impact on the test run times.

Lieven
Received on 2010-08-12 22:20:44 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.