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: Gary Benson <gbenson at redhat dot com>
- Cc: Joel Brobecker <brobecker at adacore dot com>, Pedro Alves <palves at redhat dot com>, gdb <gdb at sourceware dot org>, Andreas Arnez <arnez at linux dot vnet dot ibm dot com>
- Date: Tue, 16 Sep 2014 08:50:03 -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> <54102ED8 dot 7060307 at redhat dot com> <CADPb22Q5cr2aFrZgF4LKXCgQDddSDBwUeA3UJ2V=D93SqETmGQ at mail dot gmail dot com> <20140910162853 dot GT13931 at adacore dot com> <20140915102949 dot GC13503 at blade dot nx> <CADPb22QFA7k6OH86ouy2wep1fpFbvBOAdfCd753r_fscMDNZ5g at mail dot gmail dot com> <20140915160545 dot GN4962 at adacore dot com> <CADPb22Tf2cMz79-Wg50p=31KinJ7iPOFNmdfkJQcaJ8A2SeO_g at mail dot gmail dot com> <20140916093120 dot GA32511 at blade dot nx>
On Tue, Sep 16, 2014 at 2:31 AM, Gary Benson <gbenson@redhat.com> wrote:
> Doug Evans wrote:
>> On Mon, Sep 15, 2014 at 9:05 AM, Joel Brobecker <brobecker@adacore.com> wrote:
>> > > There's still something missing (IIUC).
>> > > One of the problems that needs to be solved is documenting the
>> > > author in the patch submission (the email that goes to the
>> > > list). The above convention allows for a default where the
>> > > absence of a name means author == committer, but we're still not
>> > > specifying an absolute requirement that the patch author appears
>> > > in the email sent to the list.
>> > > Am I missing something?
>> >
>> > I am not sure I understand what you are trying to say: You are now
>> > establishing that there are 3 users, not 2. author, submitter, and
>> > committer. Why do we need the submitter's name in the revision
>> > log?
>>
>> We don't need the submitter's name in the revision log. I was
>> referring to the patch author appearing in the email sent to the
>> list.
>>
>> Pedro wrote "I think author info must be explicit in patch submissions
>> somehow."
>> ref: https://sourceware.org/ml/gdb/2014-09/msg00038.html
>> Read the full text of 00038 for more context.
>>
>> I agree.
>>
>> The changes specified in
>> https://sourceware.org/ml/gdb/2014-09/msg00052.html
>> do not address this, yet in the text of that email (00052)
>> Pedro's comment is included (again, ref: 00052).
>
> We inline the ChangeLog entries in the patch submission emails, eg
> https://sourceware.org/ml/gdb-patches/2014-09/msg00286.html
> so the optional author lines would be inlined right there.
I don't understand.
The ChangeLog diff is not included in the "patch" in 00286 and unless
things have changed recently we still discourage that. So for the
nonce I'm going to assume you're not talking about adding ChangeLog
diffs to patches (though that would solve the problem of making the
author explicit - I'm not suggesting this as a solution though).
I guess I still don't see how this proposed change makes author
specification *explicit* in patch submissions, unless you are also
saying that the absence of author info means the patch submitter is
the author.
This might be ok, but that's not a definition of "explicit" that I've
been employing here.
And it is a departure from how patches have been submitted in the past
where the full changelog entry is always cut-n-pasted to the top of
the patch and it includes author info.
Has your definition of the word "explicit" been different than mine?