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

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