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

Re: svn commit: r18059 - trunk/subversion/libsvn_ra_dav

From: Julian Foad <julianfoad_at_btopenworld.com>
Date: 2006-01-11 06:50:17 CET

Daniel Rall wrote:
> r18065 differed from r18059 by the text "(replacing the overzealous
> r18059)". r18059 included more than was intended.

You should modify r18059's log message to briefly explain the problem,
otherwise it will confuse anybody who reads it and finds that it doesn't match
the actual changes made (unless they know about r18063/5).

- Julian

>
> On Tue, 10 Jan 2006, Karl Fogel wrote:
>
>
>>dlr@tigris.org writes:
>>
>>>Modified:
>>> trunk/subversion/libsvn_ra_dav/session.c
>>>
>>>Log:
>>>A follow-up to r18042.
>>>
>>>* subversion/libsvn_ra_dav/session.c
>>> Include ctype.h for tolower().
>>
>>? This is the same log message as for r18065. Except this is
>>r18059... again? And the log message doesn't *quite* match the
>>change.
>>
>>Help. I'm confused :-)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Jan 11 06:51:34 2006

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