bug#3571: marked as done (23.0.94; log-edit-insert-changelog option for not stripping filename)

[Emacs bug Tracking System]

Your message dated Wed, 19 Aug 2009 15:57:03 -0400
with message-id <jwv4os38tmr.fsf-monnier+emacs-23.2@gnu.org>
and subject line Re: bug#3571: 23.0.94; log-edit-insert-changelog option for 
not stripping filename
has caused the Emacs bug report #3571,
regarding 23.0.94; log-edit-insert-changelog option for not stripping filename
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@emacsbugs.donarmstrong.com
immediately.)


-- 
3571: http://emacsbugs.donarmstrong.com/cgi-bin/bugreport.cgi?bug=3571
Emacs Bug Tracking System
Contact owner@emacsbugs.donarmstrong.com with problems
> --- Begin Message ---
>
>
>
> Subject: 
>
> 23.0.94; log-edit-insert-changelog option for not stripping filename
>
>
>
>
> Date: 
>
> Mon, 15 Jun 2009 12:41:21 +0100
>
>
>
>
> User-agent: 
>
> Gnus/5.13 (Gnus v5.13) Emacs/23.0.94 (gnu/linux)
>
>
>
>
> Severity: wishlist
> Tags: patch
>
> I have grown into the habit of writing ChangeLog entries while writing
> code, and then using them for the commit message when I commit, pressing
> C-c C-a in the log buffer.  This works great, except when I'm committing
> a single file.  In that case, log-edit strips the file name from the
> changelog entry, but as the version control systems I use are
> changeset-based rather than file-based, I would like the file name to
> stay.
>
> I've been using the attached patch for a while with good results.
>
> Magnus
>
> >From 0bccc332a7e04f68d47f2b66bd6e0d425c4389c8 Mon Sep 17 00:00:00 2001
> From: Magnus Henoch <magnus.henoch@gmail.com>
> Date: Wed, 27 May 2009 17:19:13 +0100
> Subject: [PATCH] * log-edit.el (log-edit-strip-single-file-name): New variable.
>  (log-edit-insert-changelog): Check its value.
>
> ---
>  lisp/log-edit.el |   20 +++++++++++++-------
>  1 files changed, 13 insertions(+), 7 deletions(-)
>
> diff --git a/lisp/log-edit.el b/lisp/log-edit.el
> index 4ca1372..7552e3c 100644
> --- a/lisp/log-edit.el
> +++ b/lisp/log-edit.el
> @@ -148,6 +148,11 @@ can be obtained from `log-edit-files'."
>    :type '(hook :options (log-edit-set-common-indentation
>                          log-edit-add-to-changelog)))
>  
> +(defcustom log-edit-strip-single-file-name t
> +  "If non-nil, remove file name from single-file log entries."
> +  :group 'log-edit
> +  :type 'boolean)
> +
>  (defvar cvs-changelog-full-paragraphs t)
>  (make-obsolete-variable 'cvs-changelog-full-paragraphs
>                          'log-edit-changelog-full-paragraphs
> @@ -521,13 +526,14 @@ regardless of user name or time."
>      (log-edit-insert-changelog-entries (log-edit-files)))
>    (log-edit-set-common-indentation)
>    (goto-char (point-min))
> -  (when (looking-at "\\*\\s-+")
> -    (forward-line 1)
> -    (when (not (re-search-forward "^\\*\\s-+" nil t))
> -      (goto-char (point-min))
> -      (skip-chars-forward "^():")
> -      (skip-chars-forward ": ")
> -      (delete-region (point-min) (point)))))
> +  (when log-edit-strip-single-file-name
> +    (when (looking-at "\\*\\s-+")
> +      (forward-line 1)
> +      (when (not (re-search-forward "^\\*\\s-+" nil t))
> +        (goto-char (point-min))
> +        (skip-chars-forward "^():")
> +        (skip-chars-forward ": ")
> +        (delete-region (point-min) (point))))))
>  
>  ;;;;
>  ;;;; functions for getting commit message from ChangeLog a file...
>
> -- 
> 1.6.0.2
>
>
> --- End Message ---
> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#3571: 23.0.94; log-edit-insert-changelog option for not stripping filename
>
>
>
>
> Date: 
>
> Wed, 19 Aug 2009 15:57:03 -0400
>
>
>
>
> User-agent: 
>
> Gnus/5.13 (Gnus v5.13) Emacs/23.1.50 (gnu/linux)
>
>
>
>
> > I've been using the attached patch for a while with good results.
>
> Installed,
>
>
>         Stefan
>
> --- End Message ---