git.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* inaccurate commit message message?
@ 2010-09-04 21:37 Ramana Kumar
  2010-09-04 21:55 ` Sverre Rabbelier
  0 siblings, 1 reply; 4+ messages in thread
From: Ramana Kumar @ 2010-09-04 21:37 UTC (permalink / raw)
  To: git

The commit message message says "Lines starting with '#' will be
ignored", but fails to say "Lines below this message will be ignored,
even if they don't start with '#'".
If the second sentence is true, I think it (or something equivalent)
should be said, especially because of the relevance during commit
--verbose.

In fact the documentation for commit --verbose explicitly says to note
that the diff lines won't start with '#'s, as if warning people that
those lines might show up in the commit message.
Yet they do not seem to show up... (and I don't think they should)

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: inaccurate commit message message?
  2010-09-04 21:37 inaccurate commit message message? Ramana Kumar
@ 2010-09-04 21:55 ` Sverre Rabbelier
  2010-09-04 22:55   ` Ramana Kumar
  0 siblings, 1 reply; 4+ messages in thread
From: Sverre Rabbelier @ 2010-09-04 21:55 UTC (permalink / raw)
  To: Ramana Kumar; +Cc: git

Heya,

On Sat, Sep 4, 2010 at 16:37, Ramana Kumar <ramana.kumar@gmail.com> wrote:
> The commit message message says "Lines starting with '#' will be
> ignored", but fails to say "Lines below this message will be ignored,
> even if they don't start with '#'".

Because they're not. Only those starting with # are ignored.

> If the second sentence is true, I think it (or something equivalent)
> should be said, especially because of the relevance during commit
> --verbose.

It is not true.

> In fact the documentation for commit --verbose explicitly says to note
> that the diff lines won't start with '#'s, as if warning people that
> those lines might show up in the commit message.
> Yet they do not seem to show up... (and I don't think they should)

Ah, you are apparently talking about 'git commit --verbose'. It works
a little different there, all lines starting with # are ignored, as
well as everything in and after the diff.

-- 
Cheers,

Sverre Rabbelier

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: inaccurate commit message message?
  2010-09-04 21:55 ` Sverre Rabbelier
@ 2010-09-04 22:55   ` Ramana Kumar
  2010-09-04 23:07     ` Sverre Rabbelier
  0 siblings, 1 reply; 4+ messages in thread
From: Ramana Kumar @ 2010-09-04 22:55 UTC (permalink / raw)
  To: Sverre Rabbelier; +Cc: git

On Sun, Sep 5, 2010 at 7:55 AM, Sverre Rabbelier <srabbelier@gmail.com> wrote:
> Heya,
>
> On Sat, Sep 4, 2010 at 16:37, Ramana Kumar <ramana.kumar@gmail.com> wrote:
>> The commit message message says "Lines starting with '#' will be
>> ignored", but fails to say "Lines below this message will be ignored,
>> even if they don't start with '#'".
>
> Because they're not. Only those starting with # are ignored.
>
>> If the second sentence is true, I think it (or something equivalent)
>> should be said, especially because of the relevance during commit
>> --verbose.
>
> It is not true.
>
>> In fact the documentation for commit --verbose explicitly says to note
>> that the diff lines won't start with '#'s, as if warning people that
>> those lines might show up in the commit message.
>> Yet they do not seem to show up... (and I don't think they should)
>
> Ah, you are apparently talking about 'git commit --verbose'. It works
> a little different there, all lines starting with # are ignored, as
> well as everything in and after the diff.

The documentation, both in man git-commit and in the commit message
message, is misleading when you do git commit --verbose.
My proposal is to make it less misleading, by saying in at least one,
and ideally both, of those places what you just told me for the first
time: when you do a git commit --verbose, the diff lines and
everything after will be ignored for the commit message.

>
> --
> Cheers,
>
> Sverre Rabbelier
>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: inaccurate commit message message?
  2010-09-04 22:55   ` Ramana Kumar
@ 2010-09-04 23:07     ` Sverre Rabbelier
  0 siblings, 0 replies; 4+ messages in thread
From: Sverre Rabbelier @ 2010-09-04 23:07 UTC (permalink / raw)
  To: Ramana Kumar; +Cc: git

Heya,

On Sat, Sep 4, 2010 at 17:55, Ramana Kumar <ramana.kumar@gmail.com> wrote:
> The documentation, both in man git-commit and in the commit message
> message, is misleading when you do git commit --verbose.

Correct.

> My proposal is to make it less misleading, by saying in at least one,
> and ideally both, of those places what you just told me for the first
> time: when you do a git commit --verbose, the diff lines and
> everything after will be ignored for the commit message.

Sounds like a good idea. See builtin/commit.c around line 660. You'll
want to check for the verbose flag and adjust the message printed
there accordingly. See Documentation/SubmittingPatches.txt on how to
submit patches.

Please do ask if you have any questions, I'd be glad to help.

-- 
Cheers,

Sverre Rabbelier

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2010-09-04 23:08 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2010-09-04 21:37 inaccurate commit message message? Ramana Kumar
2010-09-04 21:55 ` Sverre Rabbelier
2010-09-04 22:55   ` Ramana Kumar
2010-09-04 23:07     ` Sverre Rabbelier

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).