Junio C Hamano wrote:
> Thomas Rast <trast@xxxxxxxxxxxxxxx> writes:
>
> > I came up with this after sending two incomplete patches on the same
> > night, and really like it. Perhaps others might find it useful.
>
> Any patch worth discussing (on this list at least) would need a nontrivial
> commit log message that you need to really think while writing. It is
> natural to assume people would be making them with their editor, not with
> "commit -m". These two incomplete patches could have been avoided if you
> paid attention to the status output that is in the commit log message
> buffer. Perhaps we should make it even louder in some way?
Actually I tend to write the commit (and message) sometime halfway
through, and then amend the commit with fixes, docs and such, possibly
tweaking the message if I need to. That night I just forgot to amend
before format-patch, and there's no status message at that point which
could have reminded me. So the *+ display is just what I needed; it
shows the status right before I get a chance to format-patch (or
whatever else command expects a commit).
[As a side note, this kind of workflow is what will probably prevent
me from working with any other SCM in the near future. I simply
cannot imagine going back to a world without add -p, commit --amend
and rebase -i.]
That being said, I never look at that status message; so far I've been
too lazy to make my emacs add syntax highlighting there, and without
it, it's just a big chunk of text. In fact, for most commits the 1-2
file names completely drown in the big chunk of surrounding,
invariant, instructions. If it becomes even louder in the ASCII
dimension, that most likely means I'll just have to steer my eye away
from it even harder to see the commit message I'm typing. Perhaps
some colours would help, I should really try that.
Of course the real solution would be to hack less and sleep more, but
who would want to do that?
--
Thomas Rast
trast@{inf,student}.ethz.ch
Attachment:
signature.asc
Description: This is a digitally signed message part.
