GNU bug report logs - #67388
30.0.50; Unintended man page buffer-name change?

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 67388 in the body.
You can then email your comments to 67388 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: Stephen Berman <stephen.berman <at> gmx.net>
To: bug-gnu-emacs <at> gnu.org
Subject: 30.0.50; Unintended man page buffer-name change?
Date: Wed, 22 Nov 2023 23:02:25 +0100

[Message part 1 (text/plain, inline)]

0. emacs -Q
1. M-x man RET cat RET
2. M-: (buffer-name)
=> "*Man cat *"

If you do the same in emacs-29, the result is:
"*Man cat*"

AFAICS this change is due to this commit:

commit 861ac933dd8aed1028edc4b9142400e3702874d5
Author:     Michael Albinus <michael.albinus <at> gmx.de>
Commit:     Michael Albinus <michael.albinus <at> gmx.de>
CommitDate: Wed Nov 1 16:54:31 2023 +0100

    Run `man' also on remote systems

If the change was intended, it should be called out in NEWS.  If it was
unintended, this patch restores the previous behavior, at least when
calling man locally (I currently can't test a remote call):

[Message part 2 (text/x-patch, inline)]

diff --git a/lisp/man.el b/lisp/man.el
index f18e2f50b7c..3efa29d7aad 100644
--- a/lisp/man.el
+++ b/lisp/man.el
@@ -1191,8 +1191,8 @@ Man-getpage-in-background
          (man-args topic)
 	 (bufname
           (if (file-remote-p default-directory)
-              (format "*Man %s %s *" (file-remote-p default-directory) man-args)
-            (format "*Man %s *" man-args)))
+              (format "*Man %s %s*" (file-remote-p default-directory) man-args)
+            (format "*Man %s*" man-args)))
 	 (buffer (get-buffer bufname)))
     (if buffer
 	(Man-notify-when-ready buffer)

[Message part 3 (text/plain, inline)]

In GNU Emacs 30.0.50 (build 1, x86_64-pc-linux-gnu, GTK+ Version
 3.24.38, cairo version 1.18.0) of 2023-11-22 built on strobelfs2
Repository revision: f50e54c01e84323e2689d79864dd9d65974d4429
Repository branch: master
Windowing system distributor 'The X.Org Foundation', version 11.0.12101008
System Description: Linux From Scratch r12.0-63

Configured using:
 'configure -C --with-xwidgets 'CFLAGS=-Og -g3'
 PKG_CONFIG_PATH=/opt/qt5/lib/pkgconfig'

Configured features:
ACL CAIRO DBUS FREETYPE GIF GLIB GMP GNUTLS GPM GSETTINGS HARFBUZZ JPEG
JSON LCMS2 LIBSYSTEMD LIBXML2 MODULES NATIVE_COMP NOTIFY INOTIFY PDUMPER
PNG RSVG SECCOMP SOUND SQLITE3 THREADS TIFF TOOLKIT_SCROLL_BARS WEBP X11
XDBE XIM XINPUT2 XPM XWIDGETS GTK3 ZLIB

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

From: Stephen Berman <stephen.berman <at> gmx.net>
To: Stephen Berman <stephen.berman <at> gmx.net>
Cc: 67388 <at> debbugs.gnu.org, Michael Albinus <michael.albinus <at> gmx.de>
Subject: Re: bug#67388: 30.0.50; Unintended man page buffer-name change?
Date: Wed, 29 Nov 2023 17:59:47 +0100

On Wed, 22 Nov 2023 23:02:25 +0100 Stephen Berman <stephen.berman <at> gmx.net> wrote:

> 0. emacs -Q
> 1. M-x man RET cat RET
> 2. M-: (buffer-name)
> => "*Man cat *"
>
> If you do the same in emacs-29, the result is:
> "*Man cat*"
>
> AFAICS this change is due to this commit:
>
> commit 861ac933dd8aed1028edc4b9142400e3702874d5
> Author:     Michael Albinus <michael.albinus <at> gmx.de>
> Commit:     Michael Albinus <michael.albinus <at> gmx.de>
> CommitDate: Wed Nov 1 16:54:31 2023 +0100
>
>     Run `man' also on remote systems
>
> If the change was intended, it should be called out in NEWS.  If it was
> unintended, this patch restores the previous behavior, at least when
> calling man locally (I currently can't test a remote call):
>
> diff --git a/lisp/man.el b/lisp/man.el
> index f18e2f50b7c..3efa29d7aad 100644
> --- a/lisp/man.el
> +++ b/lisp/man.el
> @@ -1191,8 +1191,8 @@ Man-getpage-in-background
>           (man-args topic)
>  	 (bufname
>            (if (file-remote-p default-directory)
> -              (format "*Man %s %s *" (file-remote-p default-directory) man-args)
> -            (format "*Man %s *" man-args)))
> +              (format "*Man %s %s*" (file-remote-p default-directory) man-args)
> +            (format "*Man %s*" man-args)))
>  	 (buffer (get-buffer bufname)))
>      (if buffer
>  	(Man-notify-when-ready buffer)

Given no reponse in a week, perhaps this fell under the radar, in
particular that of Michael Albinus (added to Cc:).  Michael, if you
confirm this was just a typo, I can install the patch to save you the
effort.  If it's not a typo, then you're in a better position than me to
decide what to do.

Steve Berman

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

From: Michael Albinus <michael.albinus <at> gmx.de>
To: Stephen Berman <stephen.berman <at> gmx.net>
Cc: 67388 <at> debbugs.gnu.org
Subject: Re: bug#67388: 30.0.50; Unintended man page buffer-name change?
Date: Wed, 29 Nov 2023 18:21:30 +0100

Stephen Berman <stephen.berman <at> gmx.net> writes:

Hi Stephen,

> Given no reponse in a week, perhaps this fell under the radar, in
> particular that of Michael Albinus (added to Cc:).

I'm very sorry, I've overlooked this, indeed.

> Michael, if you
> confirm this was just a typo, I can install the patch to save you the
> effort.  If it's not a typo, then you're in a better position than me to
> decide what to do.

Yep, it's a typo. Please install your patch. And thanks for catching this!

> Steve Berman

Best regards, M ichael.

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

From: Stephen Berman <stephen.berman <at> gmx.net>
To: Michael Albinus <michael.albinus <at> gmx.de>
Cc: 67388-done <at> debbugs.gnu.org
Subject: Re: bug#67388: 30.0.50; Unintended man page buffer-name change?
Date: Wed, 29 Nov 2023 18:47:19 +0100

On Wed, 29 Nov 2023 18:21:30 +0100 Michael Albinus <michael.albinus <at> gmx.de> wrote:

> Stephen Berman <stephen.berman <at> gmx.net> writes:
>
> Hi Stephen,
>
>> Given no reponse in a week, perhaps this fell under the radar, in
>> particular that of Michael Albinus (added to Cc:).
>
> I'm very sorry, I've overlooked this, indeed.
>
>> Michael, if you
>> confirm this was just a typo, I can install the patch to save you the
>> effort.  If it's not a typo, then you're in a better position than me to
>> decide what to do.
>
> Yep, it's a typo. Please install your patch. And thanks for catching this!

Done in commit e87644baa32 on master and closing the bug.  Thanks.

Steve Berman

Previous Next

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