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

Re: svn commit: r1541082 - in /subversion/branches/invoke-diff-merge-feature: ./ subversion/include/ subversion/include/private/ subversion/libsvn_client/ subversion/libsvn_wc/ subversion/svn/

From: Stefan Sperling <stsp_at_elego.de>
Date: Tue, 12 Nov 2013 17:06:42 +0100

On Tue, Nov 12, 2013 at 04:37:37PM +0100, Branko Čibej wrote:
> On 12.11.2013 15:31, gbg_at_apache.org wrote:
> > Author: gbg
> > Date: Tue Nov 12 14:31:11 2013
> > New Revision: 1541082
> >
> > URL: http://svn.apache.org/r1541082
> > Log:
> > On the invoke-diff-cmd-feature branch: Checkpoint commit (ignore, trivial).
>
> Sorry, we don't do that; and this is definitely not a trivial commit.
> You'll have to edit the log message to describe the changes, as per our
> policy (that's if you ever intend to merge this branch to trunk, which I
> guess you do).

Perhaps there was a misunderstanding about the text at the bottom of
http://subversion.apache.org/docs/community-guide/conventions.html#log-messages

It says:

[[[
If you're using a branch to "checkpoint" your code, and don't feel it's
ready for review, please put some sort of notice at the top of the log
message, after the 'On the 'xxx' branch notice', such as:

   *** checkpoint commit -- please don't waste your time reviewing it ***
]]]

It doesn't imply that no full log message should be provided.
But I can imagine that it's possible to misread it like that.
Received on 2013-11-12 17:07:23 CET

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