From: Sebastian Tennant <sdt@sebyte.me>
To: 61552@debbugs.gnu.org
Subject: bug#61552: Feature request: Add variable Man-prefer-synchronous-call
Date: Wed, 22 Feb 2023 17:16:24 +0000 [thread overview]
Message-ID: <87zg95lixj.fsf@sebyte.me> (raw)
In-Reply-To: <87h6vlpp18.fsf@sebyte.me>
[-- Attachment #1: Type: text/plain, Size: 610 bytes --]
Quoth Robert Pluim <rpluim@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:
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-Add-new-user-option-Man-prefer-synchronous-call.patch --]
[-- Type: text/x-diff, Size: 2857 bytes --]
From 80fab7ad842bd9f41a98e18bb7a09a9e07866707 Mon Sep 17 00:00:00 2001
From: Sebastian Tennant <sdt@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.
+
\f
* 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
next prev parent reply other threads:[~2023-02-22 17:16 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-02-16 16:12 bug#61552: Feature request: Add variable Man-prefer-synchronous-call Sebastian Tennant
2023-02-16 17:09 ` Basil Contovounesios via Bug reports for GNU Emacs, the Swiss army knife of text editors
2023-02-20 18:11 ` Sebastian Tennant
2023-02-21 10:44 ` Robert Pluim
2023-02-21 14:08 ` Sebastian Tennant
2023-02-21 14:47 ` Robert Pluim
2023-02-22 17:16 ` Sebastian Tennant [this message]
2023-03-02 11:30 ` Eli Zaretskii
2023-03-03 8:51 ` Sebastian Tennant
2023-03-03 11:23 ` Eli Zaretskii
2023-02-21 9:53 ` Stefan Kangas
2023-02-21 12:16 ` Sebastian Tennant
2023-02-21 12:28 ` Sebastian Tennant
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87zg95lixj.fsf@sebyte.me \
--to=sdt@sebyte.me \
--cc=61552@debbugs.gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).