monotone-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Monotone-devel] Empty change log


From: Nathaniel Smith
Subject: Re: [Monotone-devel] Empty change log
Date: Thu, 27 Apr 2006 14:51:16 -0700
User-agent: Mutt/1.5.11

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.

-- Nathaniel

-- 
In mathematics, it's not enough to read the words
you have to hear the music




reply via email to

[Prev in Thread] Current Thread [Next in Thread]