GNU bug report logs - #61552
Feature request: Add variable Man-prefer-synchronous-call

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 61552 in the body.
You can then email your comments to 61552 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: Sebastian Tennant <sdt <at> sebyte.me>
To: bug-gnu-emacs <at> gnu.org
Subject: Feature request: Add variable Man-prefer-synchronous-call
Date: Thu, 16 Feb 2023 16:12:51 +0000

Patch against master (on branch master-sebyte, rebased moments ago):

 https://gitweb.sebyte.me/cgi-bin/gitweb.cgi/emacs/commitdiff/280a56c96d4530273c308addda016d889082d058

Patch against emacs-28 (on branch emacs-28-sebyte):

 https://gitweb.sebyte.me/cgi-bin/gitweb.cgi/emacs/commitdiff/a08053f12303045905907b992cc78b84692cf26a

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

From: Basil Contovounesios <contovob <at> tcd.ie>
To: Sebastian Tennant <sdt <at> sebyte.me>
Cc: 61552 <at> debbugs.gnu.org
Subject: Re: bug#61552: Feature request: Add variable
 Man-prefer-synchronous-call
Date: Thu, 16 Feb 2023 17:09:27 +0000

Sebastian Tennant [2023-02-16 16:12 +0000] wrote:

> Patch against master (on branch master-sebyte, rebased moments ago):
>
>  https://gitweb.sebyte.me/cgi-bin/gitweb.cgi/emacs/commitdiff/280a56c96d4530273c308addda016d889082d058
>
> Patch against emacs-28 (on branch emacs-28-sebyte):
>
>  https://gitweb.sebyte.me/cgi-bin/gitweb.cgi/emacs/commitdiff/a08053f12303045905907b992cc78b84692cf26a

Please attach the patches here so that they can be tracked in this
bugtracker.

Thanks,

-- 
Basil

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

From: Sebastian Tennant <sdt <at> sebyte.me>
To: 61552 <at> debbugs.gnu.org
Subject: Re: bug#61552: Feature request: Add variable
 Man-prefer-synchronous-call
Date: Mon, 20 Feb 2023 18:11:17 +0000

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

Quoth Basil Contovounesios <contovob <at> tcd.ie>
on Thu, 16 Feb 2023 17:09:27 +0000:
> […]
> Please attach the patches here so that they can be tracked in this
> bugtracker.
>
> Thanks,

No problem Basil.  Here's a patch against master:

[0001-Add-user-customizable-variable-Man-prefer-synchronou.patch (text/x-diff, inline)]

From 179af66f885630a7bdf3d3f3146aceb8ad205ce6 Mon Sep 17 00:00:00 2001
From: Sebastian Tennant <sdt <at> sebyte.me>
Date: Thu, 10 Mar 2022 08:36:04 +0000
Subject: [PATCH] Add user-customizable variable Man-prefer-synchronous-call

* lisp/man.el (Man-getpage-in-background): Only call #'start-process
when 'make-process satisfies #'fboundp AND Man-prefer-synchronous-call
is bound to nil.
---
 lisp/man.el | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/lisp/man.el b/lisp/man.el
index 286edf9314e..fb4f8f02db7 100644
--- a/lisp/man.el
+++ b/lisp/man.el
@@ -97,6 +97,12 @@ man
   :group 'external
   :group 'help)
 
+(defcustom Man-prefer-synchronous-call nil
+  "Non-nil means always call the Un*x man command synchronously,
+rather than asynchronously, which is the default behaviour."
+  :type 'boolean
+  :group 'man)
+
 (defcustom Man-filter-list nil
   "Manpage cleaning filter command phrases.
 This variable contains a list of the following form:
@@ -1118,7 +1124,8 @@ Man-getpage-in-background
 					"[cleaning...]")
 				      'face 'mode-line-emphasis)))
 	(Man-start-calling
-	 (if (fboundp 'make-process)
+	 (if (and (fboundp 'make-process)
+                  (not Man-prefer-synchronous-call))
 	     (let ((proc (start-process
 			  manual-program buffer
 			  (if (memq system-type '(cygwin windows-nt))
-- 
2.30.2

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

From: Stefan Kangas <stefankangas <at> gmail.com>
To: Sebastian Tennant <sdt <at> sebyte.me>, 61552 <at> debbugs.gnu.org
Subject: Re: bug#61552: Feature request: Add variable
 Man-prefer-synchronous-call
Date: Tue, 21 Feb 2023 01:53:18 -0800

Sebastian Tennant <sdt <at> sebyte.me> writes:

> Patch against master (on branch master-sebyte, rebased moments ago):
>
>  https://gitweb.sebyte.me/cgi-bin/gitweb.cgi/emacs/commitdiff/280a56c96d4530273c308addda016d889082d058
>
> Patch against emacs-28 (on branch emacs-28-sebyte):
>
>  https://gitweb.sebyte.me/cgi-bin/gitweb.cgi/emacs/commitdiff/a08053f12303045905907b992cc78b84692cf26a

Thanks.  Could you explain the use case for this?

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

From: Robert Pluim <rpluim <at> gmail.com>
To: Sebastian Tennant <sdt <at> sebyte.me>
Cc: 61552 <at> debbugs.gnu.org
Subject: Re: bug#61552: Feature request: Add variable
 Man-prefer-synchronous-call
Date: Tue, 21 Feb 2023 11:44:14 +0100

>>>>> On Mon, 20 Feb 2023 18:11:17 +0000, Sebastian Tennant <sdt <at> sebyte.me> said:

    Sebastian> Quoth Basil Contovounesios <contovob <at> tcd.ie>
    Sebastian> on Thu, 16 Feb 2023 17:09:27 +0000:
    >> […]
    >> Please attach the patches here so that they can be tracked in this
    >> bugtracker.
    >> 
    >> Thanks,

    Sebastian> No problem Basil.  Here's a patch against master:

    Sebastian> From 179af66f885630a7bdf3d3f3146aceb8ad205ce6 Mon Sep 17 00:00:00 2001
    Sebastian> From: Sebastian Tennant <sdt <at> sebyte.me>
    Sebastian> Date: Thu, 10 Mar 2022 08:36:04 +0000
    Sebastian> Subject: [PATCH] Add user-customizable variable Man-prefer-synchronous-call

    Sebastian> * lisp/man.el (Man-getpage-in-background): Only call #'start-process
    Sebastian> when 'make-process satisfies #'fboundp AND Man-prefer-synchronous-call
    Sebastian> is bound to nil.

Your commit message should indicate that youʼve added a new defcustom,
and you should announce it in etc/NEWS (and add a description to the
manual, if that makes sense)

    Sebastian> ---
    Sebastian>  lisp/man.el | 9 ++++++++-
    Sebastian>  1 file changed, 8 insertions(+), 1 deletion(-)

    Sebastian> diff --git a/lisp/man.el b/lisp/man.el
    Sebastian> index 286edf9314e..fb4f8f02db7 100644
    Sebastian> --- a/lisp/man.el
    Sebastian> +++ b/lisp/man.el
    Sebastian> @@ -97,6 +97,12 @@ man
    Sebastian>    :group 'external
    Sebastian>    :group 'help)
 
    Sebastian> +(defcustom Man-prefer-synchronous-call nil
    Sebastian> +  "Non-nil means always call the Un*x man command synchronously,
    Sebastian> +rather than asynchronously, which is the default
    Sebastian> behaviour."

The first line should be a complete sentence. Something like

"Whether to call the Un*x 'manʼ command synchronously."

    Sebastian> +  :type 'boolean
    Sebastian> +  :group 'man)

This needs a :version tag

    Sebastian> +
    Sebastian>  (defcustom Man-filter-list nil
    Sebastian>    "Manpage cleaning filter command phrases.
    Sebastian>  This variable contains a list of the following form:
    Sebastian> @@ -1118,7 +1124,8 @@ Man-getpage-in-background
    Sebastian>  					"[cleaning...]")
    Sebastian>  				      'face 'mode-line-emphasis)))
    Sebastian>  	(Man-start-calling
    Sebastian> -	 (if (fboundp 'make-process)
    Sebastian> +	 (if (and (fboundp 'make-process)
    Sebastian> +                  (not Man-prefer-synchronous-call))
    Sebastian>  	     (let ((proc (start-process
    Sebastian>  			  manual-program buffer
    Sebastian>  			  (if (memq system-type '(cygwin windows-nt))
    Sebastian> -- 
    Sebastian> 2.30.2


Robert
-- 

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

From: Sebastian Tennant <sdt <at> sebyte.me>
To: 61552 <at> debbugs.gnu.org
Subject: Re: bug#61552: Feature request: Add variable
 Man-prefer-synchronous-call
Date: Tue, 21 Feb 2023 12:16:00 +0000

Hello Stefan,

Quoth Stefan Kangas <stefankangas <at> gmail.com>
on Tue, 21 Feb 2023 01:53:18 -0800:
> […]
> Thanks.  Could you explain the use case for this?

A function I've written (called #’man-spot) allows me to make notes in
commentary which look like this:

;;; (man-spot "openssl" "^ +s_client")

for example.

By evaluating this form, I'm able to jump to the specific point in the
openssl manpage which describes the s_client command.

The second argument to #'man-spot is a regular expression which is
passed to #'re-search-backward.  This only works if #'manual-entry has
finished putting the manpage into a buffer.  Hence the preference for
a synchronous call to #'Man-getpage-in-background.

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

From: Sebastian Tennant <sdt <at> sebyte.me>
To: 61552 <at> debbugs.gnu.org
Subject: Re: bug#61552: Feature request: Add variable
 Man-prefer-synchronous-call
Date: Tue, 21 Feb 2023 12:28:15 +0000

> Hence the preference for a synchronous call to
> #'Man-getpage-in-background.

Correction.

Hence the preference for THE synchronous call IN
#'Man-getpage-in-background.

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

From: Sebastian Tennant <sdt <at> sebyte.me>
To: 61552 <at> debbugs.gnu.org
Subject: Re: bug#61552: Feature request: Add variable
 Man-prefer-synchronous-call
Date: Tue, 21 Feb 2023 14:08:35 +0000

Hello Robert,

Quoth Robert Pluim <rpluim <at> gmail.com>
on Tue, 21 Feb 2023 11:44:14 +0100:
> […]
>> * lisp/man.el (Man-getpage-in-background): Only call
>> #'start-process when 'make-process satisfies #'fboundp AND
>> Man-prefer-synchronous-call is bound to nil
>
> Your commit message should indicate that youʼve added a new
> defcustom

How about this:

 * lisp/man.el (Man-getpage-in-background): Add new defcustom
 Man-prefer-synchronous-call and only call #'start-process when
 'make-process satisfies #'fboundp AND Man-prefer-synchronous-call is
 bound to nil.

> and you should announce it in etc/NEWS

Rightio.  Will do.

> (and add a description to the manual, if that makes sense)

Sure.  I didn't think reading manpages would have an entry in the
manual but, sure enough, it does:

 (info "(emacs) Man Page")

I shall edit doc/emacs/programs.texi accordingly.

> […]
>> (defcustom Man-prefer-synchronous-call nil
>>   "Non-nil means always call the Un*x man command synchronously,
>>     rather than asynchronously, which is the default behaviour."
>
> The first line should be a complete sentence. Something like
>
>  "Whether to call the Un*x 'manʼ command synchronously."

How about this:

 "Control whether the Un*x 'man' command is called synchronously or
  asynchronously.  (Asynchronous is the default)."

> […]
>> :type 'boolean
>> :group 'man)
>
> This needs a :version tag

Noted.

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

From: Robert Pluim <rpluim <at> gmail.com>
To: Sebastian Tennant <sdt <at> sebyte.me>
Cc: 61552 <at> debbugs.gnu.org
Subject: Re: bug#61552: Feature request: Add variable
 Man-prefer-synchronous-call
Date: Tue, 21 Feb 2023 15:47:11 +0100

>>>>> On Tue, 21 Feb 2023 14:08:35 +0000, Sebastian Tennant <sdt <at> sebyte.me> said:

    >> (and add a description to the manual, if that makes sense)

    Sebastian> Sure.  I didn't think reading manpages would have an entry in the
    Sebastian> manual but, sure enough, it does:

Everything in Emacs should be in the manual :-)

    Sebastian>  (info "(emacs) Man Page")

    Sebastian> I shall edit doc/emacs/programs.texi accordingly.

    >> […]
    >>> (defcustom Man-prefer-synchronous-call nil
    >>> "Non-nil means always call the Un*x man command synchronously,
    >>> rather than asynchronously, which is the default behaviour."
    >> 
    >> The first line should be a complete sentence. Something like
    >> 
    >> "Whether to call the Un*x 'manʼ command synchronously."

    Sebastian> How about this:

    Sebastian>  "Control whether the Un*x 'man' command is called synchronously or
    Sebastian>   asynchronously.  (Asynchronous is the default)."

Sorry, I meant "a complete sentence, and only taking up one screen
line, not exceeding `emacs-lisp-docstring-fill-column'", ie

    Whether to call the Un*x 'man' command synchronously.
    When this is non-nil call 'man' synchronously instead of the default
    asynchronous behaviour.

since `apropos' displays only the first line

Robert
-- 

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

From: Sebastian Tennant <sdt <at> sebyte.me>
To: 61552 <at> debbugs.gnu.org
Subject: Re: bug#61552: Feature request: Add variable
 Man-prefer-synchronous-call
Date: Wed, 22 Feb 2023 17:16:24 +0000

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

Quoth Robert Pluim <rpluim <at> gmail.com>
on Tue, 21 Feb 2023 15:47:11 +0100:
> […]
> Everything in Emacs should be in the manual :-)

Indeed.

> […]
> Sorry, I meant "a complete sentence, and only taking up one screen
> line, not exceeding `emacs-lisp-docstring-fill-column'", ie
>
>     Whether to call the Un*x 'man' command synchronously.
>     When this is non-nil call 'man' synchronously instead of the default
>     asynchronous behaviour.
>
> since `apropos' displays only the first line

Understood.  Hopefully this updated patch (against master) is starting
to look acceptable:

[0001-Add-new-user-option-Man-prefer-synchronous-call.patch (text/x-diff, inline)]

From 80fab7ad842bd9f41a98e18bb7a09a9e07866707 Mon Sep 17 00:00:00 2001
From: Sebastian Tennant <sdt <at> sebyte.me>
Date: Thu, 10 Mar 2022 08:36:04 +0000
Subject: [PATCH] Add new user option Man-prefer-synchronous-call

* lisp/man.el (Man-getpage-in-background): Add new defcustom
Man-prefer-synchronous-call and modify #'Man-getpage-in-background.
Only call #'start-process when 'make-process satisfies #'fboundp AND
Man-prefer-synchronous-call is bound to nil.
---
 doc/emacs/programs.texi |  6 ++++++
 etc/NEWS                |  7 +++++++
 lisp/man.el             | 11 ++++++++++-
 3 files changed, 23 insertions(+), 1 deletion(-)

diff --git a/doc/emacs/programs.texi b/doc/emacs/programs.texi
index d97bd56be21..c6c67047c43 100644
--- a/doc/emacs/programs.texi
+++ b/doc/emacs/programs.texi
@@ -1387,6 +1387,12 @@ Man Page
 @kbd{M-p} to switch between man pages in different sections.  The mode
 line shows how many manual pages are available.
 
+@vindex Man-prefer-synchronous-call
+  By default, @kbd{M-x man} calls the @code{man} program
+asynchronously.  If you would prefer it if @kbd{M-x man} called the
+@code{man} program synchronously, you may set variable
+@code{Man-prefer-synchronous-calls} to a non-@code{nil} value.
+
 @findex woman
 @cindex manual pages, on MS-DOS/MS-Windows
   An alternative way of reading manual pages is the @kbd{M-x woman}
diff --git a/etc/NEWS b/etc/NEWS
index 4b0e4e6bd46..51526079313 100644
--- a/etc/NEWS
+++ b/etc/NEWS
@@ -192,6 +192,13 @@ This command adds a docstring comment to the current defun.  If a
 comment already exists, point is only moved to the comment.  It is
 bound to 'C-c C-d' in 'go-ts-mode'.
 
+** Man-mode
+
++++
+*** New user option 'Man-prefer-synchronous-call'.
+When this is non-nil, call the 'man' program synchronously rather than
+asynchronously, which is the default behaviour.
+
 
 * New Modes and Packages in Emacs 30.1
 
diff --git a/lisp/man.el b/lisp/man.el
index 9f75c07c791..5e5c10aac36 100644
--- a/lisp/man.el
+++ b/lisp/man.el
@@ -97,6 +97,14 @@ man
   :group 'external
   :group 'help)
 
+(defcustom Man-prefer-synchronous-call nil
+  "Whether to call the Un*x 'man' program synchronously.
+When this is non-nil, call the 'man' program synchronously
+(rather than asynchronously, which is the default behaviour)."
+  :type 'boolean
+  :group 'man
+  :version 30.1)
+
 (defcustom Man-filter-list nil
   "Manpage cleaning filter command phrases.
 This variable contains a list of the following form:
@@ -1118,7 +1126,8 @@ Man-getpage-in-background
 					"[cleaning...]")
 				      'face 'mode-line-emphasis)))
 	(Man-start-calling
-	 (if (fboundp 'make-process)
+	 (if (and (fboundp 'make-process)
+                  (not Man-prefer-synchronous-call))
 	     (let ((proc (start-process
 			  manual-program buffer
 			  (if (memq system-type '(cygwin windows-nt))
-- 
2.30.2

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

From: Eli Zaretskii <eliz <at> gnu.org>
To: Sebastian Tennant <sdt <at> sebyte.me>
Cc: 61552-done <at> debbugs.gnu.org
Subject: Re: bug#61552: Feature request: Add variable
 Man-prefer-synchronous-call
Date: Thu, 02 Mar 2023 13:30:46 +0200

> From: Sebastian Tennant <sdt <at> sebyte.me>
> Date: Wed, 22 Feb 2023 17:16:24 +0000
> 
> Quoth Robert Pluim <rpluim <at> gmail.com>
> on Tue, 21 Feb 2023 15:47:11 +0100:
> > […]
> > Everything in Emacs should be in the manual :-)
> 
> Indeed.
> 
> > […]
> > Sorry, I meant "a complete sentence, and only taking up one screen
> > line, not exceeding `emacs-lisp-docstring-fill-column'", ie
> >
> >     Whether to call the Un*x 'man' command synchronously.
> >     When this is non-nil call 'man' synchronously instead of the default
> >     asynchronous behaviour.
> >
> > since `apropos' displays only the first line
> 
> Understood.  Hopefully this updated patch (against master) is starting
> to look acceptable:

Thanks, installed on the master branch, and closing the bug.

Assuming that you want to be able to submit contributions to Emacs in
the future, would you like to start the legal paperwork of assigning
the copyright for your changes to the FSF?  If yes, I will send you
the form to fill to start the paperwork.

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

From: Sebastian Tennant <sdt <at> sebyte.me>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 61552-done <at> debbugs.gnu.org
Subject: Re: bug#61552: Feature request: Add variable
 Man-prefer-synchronous-call
Date: Fri, 03 Mar 2023 08:51:36 +0000

Hello Eli,

Quoth Eli Zaretskii <eliz <at> gnu.org>
on Thu, 02 Mar 2023 13:30:46 +0200:
>> From: Sebastian Tennant <sdt <at> sebyte.me>
>> Date: Wed, 22 Feb 2023 17:16:24 +0000
>> […]
>> Understood.  Hopefully this updated patch (against master) is
>> starting to look acceptable:
>
> Thanks, installed on the master branch, and closing the bug.

Great!  Thank you.

> Assuming that you want to be able to submit contributions to Emacs
> in the future, would you like to start the legal paperwork of
> assigning the copyright for your changes to the FSF?

Yes, I would.

> If yes, I will send you the form to fill to start the paperwork.

Thank you again.

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

From: Eli Zaretskii <eliz <at> gnu.org>
To: Sebastian Tennant <sdt <at> sebyte.me>
Cc: 61552-done <at> debbugs.gnu.org
Subject: Re: bug#61552: Feature request: Add variable
 Man-prefer-synchronous-call
Date: Fri, 03 Mar 2023 13:23:40 +0200

> From: Sebastian Tennant <sdt <at> sebyte.me>
> Cc: 61552-done <at> debbugs.gnu.org
> Date: Fri, 03 Mar 2023 08:51:36 +0000
> 
> > Assuming that you want to be able to submit contributions to Emacs
> > in the future, would you like to start the legal paperwork of
> > assigning the copyright for your changes to the FSF?
> 
> Yes, I would.
> 
> > If yes, I will send you the form to fill to start the paperwork.
> 
> Thank you again.

Thanks, form sent off-list.

Previous Next

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