GNU bug report logs -
#59457
29.0.50; Incorrect vc-parent-buffer-name on vc-change-log revert
Previous Next
Reported by: Gabriel <gabriel376 <at> hotmail.com>
Date: Mon, 21 Nov 2022 20:39:02 UTC
Severity: normal
Found in version 29.0.50
Done: Sean Whitton <spwhitton <at> spwhitton.name>
Bug is archived. No further changes may be made.
To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 59457 in the body.
You can then email your comments to 59457 AT debbugs.gnu.org in the normal way.
Toggle the display of automated, internal messages from the tracker.
Report forwarded
to
bug-gnu-emacs <at> gnu.org
:
bug#59457
; Package
emacs
.
(Mon, 21 Nov 2022 20:39:02 GMT)
Full text and
rfc822 format available.
Acknowledgement sent
to
Gabriel <gabriel376 <at> hotmail.com>
:
New bug report received and forwarded. Copy sent to
bug-gnu-emacs <at> gnu.org
.
(Mon, 21 Nov 2022 20:39:02 GMT)
Full text and
rfc822 format available.
Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):
Description:
The *vc-change-log* buffer gets an incorrect value of
`vc-parent-buffer-name' on `revert-buffer'.
Steps:
1) emacs -Q (master "f176a36f4629b56c9fd9e3fc15aebd04a168c4f5")
2) Open some VC controlled file, e.g.: C-x C-f ~/git/emacs/README RET
3) Annotate the buffer: C-x v g
4) Show log revision at line: l
Result: *vc-change-log* buffer is displayed; the value of
`vc-parent-buffer-name', displayed on the right side of mode-line (see
`mode-line-format' and `mode-line-modes'), is correct (e.g.: " from
*Annotate README (rev f176a36f4629b56c9fd9e3fc15aebd04a168c4f5)*").
5) Revert the buffer: g
Result: the buffer is refreshed, but the value of
`vc-parent-buffer-name' is incorrectly set as " from *vc-change-log*".
---
Gabriel
Information forwarded
to
bug-gnu-emacs <at> gnu.org
:
bug#59457
; Package
emacs
.
(Tue, 22 Nov 2022 18:01:02 GMT)
Full text and
rfc822 format available.
Message #8 received at submit <at> debbugs.gnu.org (full text, mbox):
The bug can also be reproduced with the steps below:
1) emacs -Q
2) Open some VC controlled file, e.g.: C-x C-f ~/git/emacs/README RET
3) List the root revision history: C-x v L
4) Revert the buffer: g
---
Gabriel
Reply sent
to
Sean Whitton <spwhitton <at> spwhitton.name>
:
You have taken responsibility.
(Mon, 10 Mar 2025 09:19:03 GMT)
Full text and
rfc822 format available.
Notification sent
to
Gabriel <gabriel376 <at> hotmail.com>
:
bug acknowledged by developer.
(Mon, 10 Mar 2025 09:19:03 GMT)
Full text and
rfc822 format available.
Message #13 received at 59457-done <at> debbugs.gnu.org (full text, mbox):
Hello,
On Sat 08 Mar 2025 at 05:11pm +08, Sean Whitton wrote:
> Hello,
>
> On Sat 08 Mar 2025 at 08:07am GMT, Stefan Kangas wrote:
>
>> I think you can use `kill-all-local-variables` for this:
>>
>> (kill-all-local-variables 'kill-permanent)
>
> Oh, thanks, I never knew it could take an argument.
> I think the thing to do here then is add that to vc-log-internal-common.
> We're a long way from releasing Emacs 31 so if this does break something
> else, we'll find out.
>
> Installed the change and closing the bug.
While preparing a fix for #59457 I discovered that invoking vc-print-log
already makes a call to kill-all-local-variables, in vc-setup-buffer.
So calling it in vc-log-internal-common too isn't right.
In other words, the place to fix this is in vc-setup-buffer, which
already has special handling for default-directory.
I've replaced the fix with one in vc-setup-buffer, and also fixed #59457.
--
Sean Whitton
bug archived.
Request was from
Debbugs Internal Request <help-debbugs <at> gnu.org>
to
internal_control <at> debbugs.gnu.org
.
(Mon, 07 Apr 2025 11:24:06 GMT)
Full text and
rfc822 format available.
This bug report was last modified 59 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.