From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: jemarch@gnu.org (Jose E. Marchesi) Newsgroups: gmane.emacs.devel Subject: Re: Referring to revisions in the git future. Date: Wed, 29 Oct 2014 14:00:38 +0100 Message-ID: <87wq7jxc7d.fsf@gnu.org> References: NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain X-Trace: ger.gmane.org 1414587365 31726 80.91.229.3 (29 Oct 2014 12:56:05 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Wed, 29 Oct 2014 12:56:05 +0000 (UTC) Cc: "Eric S. Raymond" , David Kastrup , emacs-devel@gnu.org To: Stefan Monnier Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Wed Oct 29 13:55:57 2014 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1XjSn6-0000AY-QQ for ged-emacs-devel@m.gmane.org; Wed, 29 Oct 2014 13:55:56 +0100 Original-Received: from localhost ([::1]:45644 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XjSn6-000515-Cy for ged-emacs-devel@m.gmane.org; Wed, 29 Oct 2014 08:55:56 -0400 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:50134) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XjSmo-0004zw-Tl for emacs-devel@gnu.org; Wed, 29 Oct 2014 08:55:44 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XjSmj-0007uO-RO for emacs-devel@gnu.org; Wed, 29 Oct 2014 08:55:38 -0400 Original-Received: from fencepost.gnu.org ([2001:4830:134:3::e]:41606) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XjSmj-0007uK-LM for emacs-devel@gnu.org; Wed, 29 Oct 2014 08:55:33 -0400 Original-Received: from ukc1-proxy-mwg07-o.oracle.com ([144.24.20.232]:26210 helo=termi.gnu.org) by fencepost.gnu.org with esmtpsa (TLS1.0:DHE_RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1XjSmi-0006tJ-SJ; Wed, 29 Oct 2014 08:55:33 -0400 In-Reply-To: (Stefan Monnier's message of "Wed, 29 Oct 2014 08:35:33 -0400") User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3.50 (gnu/linux) X-detected-operating-system: by eggs.gnu.org: Error: Malformed IPv6 address (bad octet value). X-Received-From: 2001:4830:134:3::e X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:175980 Archived-At: > About summary lines, a reminder: Please don't write the traditional > GNUish run-on change comment with a semi-infinite number of bulleted > items in it any more. That's your opinion, but the convention we still use here (and don't just recommend but *request* people to follow) is the GNU ChangeLog format. You can have both. In other GNU projects using git (gdb, binutils) we include a summary line for the benefit of `git log --oneline', followed by an extended description in a separated paragraph(s) and finally the ChangeLog entries. This way `git log --oneline' will show you something like this: e44528a New commands `enable probe' and `disable probe'. While `git log' will give you the full description, including the ChangeLog entries: commit e44528a65592707466a9434434ed272dd3b13d9a Author: Jose E. Marchesi Date: Tue Oct 28 14:35:24 2014 +0100 New commands `enable probe' and `disable probe'. This patch adds the above-mentioned commands to the generic probe abstraction implemented in probe.[ch]. The effects associated to enabling or disabling a probe depend on the type of probe being handled, and is triggered by invoking two back-end hooks in `probe_ops'. In case some particular probe type does not support the notion of enabling and/or disabling, the corresponding fields on `probe_ops' can be initialized to NULL. This is the case of SystemTap probes. gdb/ChangeLog: 2014-10-28 Jose E. Marchesi * stap-probe.c (stap_probe_ops): Add NULLs in the static stap_probe_ops for `enable_probe' and `disable_probe'. * probe.c (enable_probes_command): New function. (disable_probes_command): Likewise. (_initialize_probe): Define the cli commands `enable probe' and `disable probe'. (parse_probe_linespec): New function. (info_probes_for_ops): Use parse_probe_linespec. * probe.h (probe_ops): New hooks `enable_probe' and `disable_probe'. gdb/doc/ChangeLog: 2014-10-28 Jose E. Marchesi * gdb.texinfo (Static Probe Points): Cover the `enable probe' and `disable probe' commands.