This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] Regression setting breakpoint
- From: Joel Brobecker <brobecker at adacore dot com>
- To: Sergio Durigan Junior <sergiodj at redhat dot com>
- Cc: Michael Eager <eager at eagerm dot com>, gdb-patches ml <gdb-patches at sourceware dot org>, Keith Seitz <keiths at redhat dot com>
- Date: Thu, 16 Jan 2014 09:37:02 +0400
- Subject: Re: [PATCH] Regression setting breakpoint
- Authentication-results: sourceware.org; auth=none
- References: <52A8CD31 dot 5090809 at eagerm dot com> <m361pkbiwk dot fsf at redhat dot com>
> Not really a code review, but the ChangeLog entries should be of the
> form:
>
> 2014-01-16 Your Name <you@email.com>
>
> * file.c (function): What changed.
FWIW, I have been using:
| gdb/ChangeLog:
|
| * file.c (function): What changed.
|
| gdb/testsuite/ChangeLog:
|
| * gdb.base/file.exp: Bla bla bla.
Two reasons: It makes it obvious which ChangeLog the entries refer to,
and it avoids the need to put a date in the revision log, which may
become obe due to the time it takes to get the patch in after it is
written.
The format with dates are accepted as well, of course. And I have
seen people add the directory usually on the line just ahead of
the CL entry. Eg:
| gdb/:
| 2014-01-16 Your Name <you@example.com>
--
Joel