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

filenames in log message (was: [PATCH] libsvn_client: start using svn_io_file_close where applicable)

From: Greg Stein <gstein_at_lyra.org>
Date: 2003-11-09 00:50:45 CET

On Fri, Nov 07, 2003 at 12:20:19AM +0000, Julian Foad wrote:
>...
> I changed the log message a bit, to indicate briefly _why_ the change is
> being made, and to omit the list of files, since the same change is made
> to each file and the list of files is recorded automatically.

The standard format lists all files changed. The theory here is that
somebody might want to search the log messages -- for either file names,
or maybe for certain functions.

It is also helpful that when you pull up a log message for <FOO> that you
see all effects during that patch.

I'd request that the file list remains in the log message.

All that said, there is an argument that 'svn log -v' will give you the
list of files, so in certain cases we shouldn't bother. But I believe that
is a specific discussion to have, and then an update of HACKING.

Cheers,
-g

-- 
Greg Stein, http://www.lyra.org/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sun Nov 9 00:52:15 2003

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.