This is the mail archive of the
gdb@sourceware.org
mailing list for the GDB project.
Re: ChangeLogs in commit messages
- From: Doug Evans <dje at google dot com>
- To: Sergio Durigan Junior <sergiodj at redhat dot com>
- Cc: Gary Benson <gbenson at redhat dot com>, Andreas Schwab <schwab at linux-m68k dot org>, Joel Brobecker <brobecker at adacore dot com>, gdb <gdb at sourceware dot org>, Andreas Arnez <arnez at linux dot vnet dot ibm dot com>
- Date: Thu, 4 Sep 2014 09:08:43 -0700
- Subject: Re: ChangeLogs in commit messages
- Authentication-results: sourceware.org; auth=none
- References: <20140814083231 dot GA6283 at blade dot nx> <20140814125224 dot GF4924 at adacore dot com> <8761h4fmu4 dot fsf at redhat dot com> <87mwage6x2 dot fsf at redhat dot com> <871trsuz55 dot fsf at igel dot home> <20140904090616 dot GA23758 at blade dot nx> <87y4tzcvny dot fsf at redhat dot com>
On Thu, Sep 4, 2014 at 7:22 AM, Sergio Durigan Junior
<sergiodj@redhat.com> wrote:
> On Thursday, September 04 2014, Gary Benson wrote:
>
>>> There is no such thing as a "push date". What you see is the author
>>> date and the committer date. But both are set during the local
>>> commit, and are unrelated to the point of time of pushing the
>>> commits to the remote repository.
>>
>> Yeah, in git no file (object) is modified when you push or pull
>> commits, they're just copied from one place to another.
>
> Yeah, sorry for not being fluent in git's parlance.
>
>> The committer date on the commit you mentioned is likely the time
>> I updated the ChangeLog prior to pushing, so that date will be the
>> "push date", but that's not guaranteed to be the case for all
>> commits.
>
> Right. Anyway, I still consider it is valid to include this info
> (author and date) in the commit message.
For reference sake, what started this for me
is noticing git blame shows the author date by default.
Not picking on Gary of course, his patch is just
why I noticed this.
If I do a git blame of server.c I see patch 860789c7 with a date
of 2014-08-08. That's three weeks before it was pushed upstream.
Bleah. I'd really like to be able to do a git blame and have what
I see be useful, including the date.
The author date is basically useless to me.
[I realize neither author nor commit dates are the so-called push date,
but I'm guessing the commit date is generally reasonably
close to the date the patch is pushed upstream.
If not ... yikes.]
I realize any changes to what goes into the commit log
won't help solve this of course. But that's what
started the discussion for me.