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

Re: svn commit: rev 7551 - trunk/subversion/libsvn_client

From: Julian Foad <julianfoad_at_btopenworld.com>
Date: 2003-10-29 02:42:59 CET

philip@tigris.org wrote:
> Author: philip
> Date: Tue Oct 28 17:00:06 2003
> New Revision: 7551
>
> Modified:
> trunk/subversion/libsvn_client/copy.c
> trunk/subversion/libsvn_client/prop_commands.c
> Log:
> Don't refer to the --force command line argument in library error
> messages.

Why? Oh, I think I know: we shouldn't refer to anything that is specific to our command-line client, because it is not the only client.

It is useful if log messages always indicate _why_ the change was made, because although it is usually obvious to the committer at the time, it is less obvious to other people and at future times.

  grep "\".*--[a-z].*\"" subversion/lib*/*.c

says you missed one in subversion/libsvn_client/export.c.

:)

- Julian

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Oct 29 02:41:57 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.