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

Re: [PATCH] Fix warning message in merge test

From: Hyrum K. Wright <hyrum_wright_at_mail.utexas.edu>
Date: 2007-08-13 16:06:38 CEST

Senthil Kumaran S wrote:
> Senthil Kumaran S wrote:
>> I am attaching a patch to fix the doc string of
>> "merge_to_out_of_date_target" function in merge_tests.py, which has
>> some capitalized characters in it.
>
> Sorry, I forgot to attach the patch :( Doing it now.

Hi Senthil,
Thanks for your recent patches. I've noticed, though, that you haven't
included any log messages with your patches, per our patch submission
guidelines, found here:
http://subversion.tigris.org/hacking.html#patches A good log message
helps reviewers understand the purpose of your patch, and increases the
likelihood that it will be applied. Instructions for writing log
messages can be found here:
http://subversion.tigris.org/hacking.html#log-messages

Could you resend your outstanding patches, with a log message included
for each?

Thanks,
-Hyrum

Received on Mon Aug 13 16:05:10 2007

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.