bug#62958: closed ([PATCH] Set PAGER=cat in comint.el)

[GNU bug Tracking System]

Your message dated Fri, 26 May 2023 14:31:58 +0300
with message-id <83wn0vjpld.fsf@gnu.org>
and subject line Re: bug#62958: [PATCH] Set PAGER=cat in comint.el
has caused the debbugs.gnu.org bug report #62958,
regarding [PATCH] Set PAGER=cat in comint.el
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


-- 
62958: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=62958
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems

--- Begin Message --- Subject: [PATCH] Set PAGER=cat in comint.el Date: Wed, 19 Apr 2023 17:57:38 -0400

Tags: patch


Paging is useless and annoying in comint-derived commands such as
async-shell-command and M-x shell.  It is a frequent footgun for new
Emacs users when they try to run commands which start a pager in such
modes.  I know a number of users who set PAGER=cat in their Emacs to
avoid it.

Simply adding (setenv "PAGER" "cat") globally is not correct, since
that will break modes like term, which support paging quite well.
It's only and exactly the comint-derived modes which don't need
paging.

Changing the default to "cat" in this way might be a bit
controversial...

In GNU Emacs 29.0.60 (build 3, x86_64-pc-linux-gnu, X toolkit, cairo
 version 1.15.12, Xaw scroll bars) of 2023-03-13 built on
 igm-qws-u22796a
Repository revision: e759905d2e0828eac4c8164b09113b40f6899656
Repository branch: emacs-29
Windowing system distributor 'The X.Org Foundation', version 11.0.12011000
System Description: CentOS Linux 7 (Core)

Configured using:
 'configure --with-x-toolkit=lucid --with-modules
 --with-gif=ifavailable'

0001-Set-PAGER-cat-in-comint.el.patch
Description: Text Data

--- End Message ---

> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#62958: [PATCH] Set PAGER=cat in comint.el
>
>
>
>
> Date: 
>
> Fri, 26 May 2023 14:31:58 +0300
>
>
>
>
> > Cc: sbaugh@catern.com, 62958@debbugs.gnu.org
> > Date: Fri, 19 May 2023 09:09:54 +0300
> > From: Eli Zaretskii <eliz@gnu.org>
> > 
> > > From: Spencer Baugh <sbaugh@janestreet.com>
> > > Cc: sbaugh@catern.com,  62958@debbugs.gnu.org
> > > Date: Wed, 17 May 2023 10:55:16 -0400
> > > 
> > > Updated patch.
> > 
> > LGTM, thanks.
>
> No further comment, so I've now installed this on the master branch,
> and I'm therefore closing this bug.
>
> Thanks.
>
>
> --- End Message ---