GNU bug report logs - #17884
24.3.92; `vc-next-action' discards commit message

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 17884 in the body.
You can then email your comments to 17884 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Richard Copley <rcopley <at> gmail.com>
To: "bug-gnu-emacs <at> gnu.org" <bug-gnu-emacs <at> gnu.org>
Subject: 24.3.92; `vc-next-action' discards commit message
Date: Mon, 30 Jun 2014 16:23:25 +0100

It used to be that if you typed "C-v v" from a file buffer for which you
had already started to compose a commit message, you would be taken to
your message so that you could continue composing it. In recent builds
the message gets clobbered by the (unwanted!) text "Summary: ^J".
That's not very nice.


In GNU Emacs 24.3.92.1 (x86_64-w64-mingw32)
 of 2014-06-30 on 80461UHB
Repository revision: 117326
schwab <at> linux-m68k.org-20140629204855-6i3r3pi3ls4e4c6q
Windowing system distributor `Microsoft Corp.', version 6.1.7601
Configured using:
 `configure --prefix c:/emacs/emacs-117326 --disable-dependency-tracking
 --enable-locallisppath=%emacs_dir%/../site-lisp --with-wide-int
 --build=x86_64-w64-mingw32 'CPPFLAGS=-I G:/usr/include -I
 C:/GnuWin32/include' 'LDFLAGS=-L G:/usr/lib -L C:/GnuWin32/lib''

Important settings:
  value of $LANG: ENG
  locale-coding-system: cp1252

Major mode: Log-Edit

Minor modes in effect:
  tooltip-mode: t
  electric-indent-mode: t
  mouse-wheel-mode: t
  tool-bar-mode: t
  menu-bar-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  font-lock-mode: t
  blink-cursor-mode: t
  auto-composition-mode: t
  auto-encryption-mode: t
  auto-compression-mode: t
  line-number-mode: t
  transient-mark-mode: t

Recent input:
C-x C-f \ t e m p \ t e s t \ t e s t . t x t <return>
C-x v v T h i s SPC t o o k SPC m e SPC a g e s . C-x
v v O h ! M-x r - e - b <return>

Recent messages:
For information about GNU Emacs and the GNU system, type C-h C-a.
(New file)
Mark set
Press C-c C-c when you are done editing.
Enter a change comment.  Type C-c C-c when done
Mark set
Press C-c C-c when you are done editing.
Enter a change comment.  Type C-c C-c when done

Load-path shadows:
None found.

Features:
(shadow sort gnus-util mail-extr emacsbug sendmail log-edit easy-mmode
message format-spec rfc822 mml easymenu mml-sec mm-decode mm-bodies
mm-encode mail-parse rfc2231 rfc2047 rfc2045 ietf-drums mm-util help-fns
mail-prsvr mailabbrev mail-utils gmm-utils mailheader ring pcvs-util
add-log vc vc-dispatcher vc-rcs time-date tooltip electric uniquify
ediff-hook vc-hooks lisp-float-type mwheel dos-w32 ls-lisp
w32-common-fns disp-table w32-win w32-vars tool-bar dnd fontset image
regexp-opt fringe tabulated-list newcomment lisp-mode prog-mode register
page menu-bar rfn-eshadow timer select scroll-bar mouse jit-lock
font-lock syntax facemenu font-core frame cham georgian utf-8-lang
misc-lang vietnamese tibetan thai tai-viet lao korean japanese hebrew
greek romanian slovak czech european ethiopic indian cyrillic chinese
case-table epa-hook jka-cmpr-hook help simple abbrev minibuffer nadvice
loaddefs button faces cus-face macroexp files text-properties overlay
sha1 md5 base64 format env code-pages mule custom widget
hashtable-print-readable backquote make-network-process w32notify w32
multi-tty emacs)

Memory information:
((conses 16 79005 8573)
 (symbols 56 18045 0)
 (miscs 48 49 159)
 (strings 32 12796 4781)
 (string-bytes 1 329463)
 (vectors 16 9811)
 (vector-slots 8 383011 9074)
 (floats 8 62 207)
 (intervals 56 246 0)
 (buffers 960 16))

Message #8 received at 17884 <at> debbugs.gnu.org (full text, mbox):

From: Glenn Morris <rgm <at> gnu.org>
To: Richard Copley <rcopley <at> gmail.com>
Cc: 17884 <at> debbugs.gnu.org
Subject: Re: bug#17884: 24.3.92; `vc-next-action' discards commit message
Date: Mon, 30 Jun 2014 11:39:00 -0400

Richard Copley wrote:

> It used to be that if you typed "C-v v" from a file buffer for which you
> had already started to compose a commit message, you would be taken to
> your message so that you could continue composing it. In recent builds
> the message gets clobbered by the (unwanted!) text "Summary: ^J".
> That's not very nice.

No, it's not; but as a workaround I find M-x undo brings the text back.

> In GNU Emacs 24.3.92.1 (x86_64-w64-mingw32)

Message #11 received at 17884 <at> debbugs.gnu.org (full text, mbox):

From: Richard Copley <rcopley <at> gmail.com>
To: Glenn Morris <rgm <at> gnu.org>
Cc: 17884 <at> debbugs.gnu.org
Subject: Re: bug#17884: 24.3.92; `vc-next-action' discards commit message
Date: Mon, 30 Jun 2014 18:35:33 +0100

On 30 June 2014 16:39, Glenn Morris <rgm <at> gnu.org> wrote:
>> That's not very nice.
>
> No, it's not; but as a workaround I find M-x undo brings the text back.
>
That's good to know. Thanks!

Message #14 received at 17884 <at> debbugs.gnu.org (full text, mbox):

From: Glenn Morris <rgm <at> gnu.org>
To: 17884 <at> debbugs.gnu.org
Cc: Richard Copley <rcopley <at> gmail.com>
Subject: Re: bug#17884: 24.3.92; `vc-next-action' discards commit message
Date: Mon, 30 Jun 2014 15:33:28 -0400

I think this is due to:

2013-11-28  Stefan Monnier  <monnier <at> iro.umontreal.ca>

   * vc/vc-dispatcher.el (vc-log-edit): Setup the Summary&Author headers.

log-edit with non-nil SETUP first erases the log buffer, thus defeating
the purpose of log-edit-empty-buffer-p.

Message #19 received at 17884-done <at> debbugs.gnu.org (full text, mbox):

From: Paul Eggert <eggert <at> cs.ucla.edu>
To: Richard Copley <rcopley <at> gmail.com>
Cc: 17884-done <at> debbugs.gnu.org
Subject: Re: 24.3.92; `vc-next-action' discards commit message
Date: Fri, 11 Jul 2014 19:25:32 -0700

Thanks for reporting that.  I installed a fix in emacs-24 bzr 117371.

Message #22 received at 17884 <at> debbugs.gnu.org (full text, mbox):

From: Glenn Morris <rgm <at> gnu.org>
To: 17884 <at> debbugs.gnu.org
Cc: rcopley <at> gmail.com, eggert <at> cs.ucla.edu
Subject: Re: bug#17884: 24.3.92; `vc-next-action' discards commit message
Date: Fri, 11 Jul 2014 22:56:01 -0400

Paul Eggert wrote:

> I installed a fix in emacs-24 bzr 117371.

I'm not sure this is 100% right.
If the *vc-log* buffer that exists corresponds to a different set of
files than the ones being committed now, then the old contents should be
replaced, even if the buffer is not empty.

Or if the buffer contains just the header template
(log-edit-empty-buffer-p), then the contents should always be replaced,
in case a proper log is available now.

Message #27 received at 17884 <at> debbugs.gnu.org (full text, mbox):

From: Paul Eggert <eggert <at> cs.ucla.edu>
To: Glenn Morris <rgm <at> gnu.org>, 17884 <at> debbugs.gnu.org
Cc: rcopley <at> gmail.com
Subject: Re: bug#17884: 24.3.92; `vc-next-action' discards commit message
Date: Fri, 11 Jul 2014 20:09:04 -0700

Glenn Morris wrote:
> I'm not sure this is 100% right.

No doubt you're right, so I'm reopening the bug.  I think emacs-24 bzr 
117371 is a step forward though, as it handles the use case in the 
original bug report, and it's easier for a novice to delete text than to 
recreate deleted text.

bzr 117371 didn't use log-edit-empty-buffer-p, as its module might not 
be loaded and I didn't want to introduce an unnecessary dependency.  But 
perhaps the dependency is necessary after all.

Message #30 received at 17884 <at> debbugs.gnu.org (full text, mbox):

From: Glenn Morris <rgm <at> gnu.org>
To: Paul Eggert <eggert <at> cs.ucla.edu>
Cc: rcopley <at> gmail.com, 17884 <at> debbugs.gnu.org
Subject: Re: bug#17884: 24.3.92; `vc-next-action' discards commit message
Date: Sat, 12 Jul 2014 16:25:20 -0400

Paul Eggert wrote:

> bzr 117371 didn't use log-edit-empty-buffer-p, as its module might not
> be loaded and I didn't want to introduce an unnecessary dependency.
> But perhaps the dependency is necessary after all.

I don't see the issue.
We are talking about vc-dispatcher calling log-edit, which is defined in
log-edit.el, which is also where log-edit-empty-buffer-p is defined.
So just make vc-log-edit explicitly require log-edit?

For the other thing I mentioned, I think checking whether there is an
existing local value of vc-log-fileset that matches the new one should
cover it.

Message #33 received at 17884 <at> debbugs.gnu.org (full text, mbox):

From: Glenn Morris <rgm <at> gnu.org>
To: Paul Eggert <eggert <at> cs.ucla.edu>
Cc: rcopley <at> gmail.com, 17884 <at> debbugs.gnu.org
Subject: Re: bug#17884: 24.3.92; `vc-next-action' discards commit message
Date: Mon, 14 Jul 2014 15:31:12 -0400

I applied this, which hopefully covers it.


--- a/lisp/vc/vc-dispatcher.el	2014-07-12 02:24:02 +0000
+++ b/lisp/vc/vc-dispatcher.el	2014-07-14 19:29:29 +0000
@@ -591,12 +591,19 @@
 
 ;; Set up key bindings for use while editing log messages
 
+(declare-function log-edit-empty-buffer-p "log-edit" ())
+
 (defun vc-log-edit (fileset mode backend)
   "Set up `log-edit' for use on FILE."
   (setq default-directory
 	(buffer-local-value 'default-directory vc-parent-buffer))
+  (require 'log-edit)
   (log-edit 'vc-finish-logentry
-	    (= (point-min) (point-max))
+	    ;; Setup a new log message if the log buffer is "empty",
+	    ;; or was previously used for a different set of files.
+	    (or (log-edit-empty-buffer-p)
+		(and (local-variable-p 'vc-log-fileset)
+		     (not (equal vc-log-fileset fileset))))
 	    `((log-edit-listfun . (lambda ()
                                     ;; FIXME: Should expand the list
                                     ;; for directories.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.