[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Monotone-devel] Empty change log
From: |
Timothy Brownawell |
Subject: |
Re: [Monotone-devel] Empty change log |
Date: |
Fri, 28 Apr 2006 01:20:07 -0500 |
On Thu, 2006-04-27 at 14:56 -0700, Justin Patrin wrote:
> On 4/27/06, Nathaniel Smith <address@hidden> wrote:
> > On Thu, Apr 27, 2006 at 01:57:21PM -0500, Timothy Brownawell wrote:
> > > > But when doing it
> > > > form the command line I can only assume that you folks don't like empty
> > > > commit messages. If that is the case I think that the second example
> > > > should
> > > > fail too (only white space in commit message).
> > >
> > > It's the default version of the validate_commit_message hook. I don't
> > > think it's a very good default, so would anyone object to removing it?
> >
> > Ah-hah, is that what it is! I had been wondering, since we pretty
> > much hashed this out before:
> > http://thread.gmane.org/gmane.comp.version-control.monotone.devel/4365
> > and I thought we had decided that -m "" should just work, if someone
> > was going to bother doing it. I guess we made that work, and then the
> > validate_commit_message change changed it again?
> >
> > IMHO the validate_commit_message hook should default to accepting
> > _all_ messages, while keeping the current "empty commit message
> > in an editor means cancel this commit" behavior.
> >
>
> ::shrug:: that works too. As long as en empty commit message from an
> editor means abort I'll be happy.
This has been implemented.
Empty (or whitespace-only) messages from an editor abort the commit. No
other commit-message checking is performed by default.
Tim