bug#63600: Prevent python-inferior-mode from using $PAGER

bug#63600: Prevent python-inferior-mode from using $PAGER

[Olav Fosse]

[-- Attachment #1: Type: text/plain, Size: 187 bytes --]

This patch prevents python-inferior-mode from using $PAGER.
$PAGER is typically set to an interactive program, such as 
less(1) which causes problems when ran inside of a comint
buffer.


[-- Attachment #2: 0001-Prevent-python-inferior-mode-from-using-PAGER.patch --]
[-- Type: application/octet-stream, Size: 1280 bytes --]

From d58c04d661ca415da41bc09c52bbce2bdae93cc1 Mon Sep 17 00:00:00 2001
From: Olav Fosse <post@olavfosse.no>
Date: Fri, 19 May 2023 20:33:28 +0200
Subject: [PATCH] Prevent python-inferior-mode from using $PAGER

---
 lisp/progmodes/python.el | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/lisp/progmodes/python.el b/lisp/progmodes/python.el
index 4f57eda3cfc..9304479094e 100644
--- a/lisp/progmodes/python.el
+++ b/lisp/progmodes/python.el
@@ -2563,8 +2563,15 @@ python-shell-internal-buffer-name
   :type 'string
   :safe 'stringp)
 
-(defcustom python-shell-interpreter-args "-i"
-  "Default arguments for the Python interpreter."
+(defcustom python-shell-interpreter-args "-i -c \"import pydoc;pydoc.pager = pydoc.plainpager;del pydoc\""
+  "Default arguments for the Python interpreter.
+
+-c \"import pydoc;pydoc.pager = pydoc.plainpager;del pydoc\"
+makes it so that Python will print large messages directly to the
+buffer, as opposed to using $PAGER. This is useful because $PAGER
+is typically set to an interative program, such as less(1), which
+will not work properly when ran inside `inferior-python-mode'.
+"
   :type 'string)
 
 (defcustom python-shell-interpreter-interactive-arg "-i"
-- 
2.39.2

bug#63600:

[Olav Fosse]

This might be solved for all comint-derived modes in
https://debbugs.gnu.org/cgi/bugreport.cgi?bug=62958

bug#63600:

[Eli Zaretskii]

> From: Olav Fosse <post@olavfosse.no>
> Date: Sat, 20 May 2023 13:00:11 +0200
> 
> This might be solved for all comint-derived modes in
> https://debbugs.gnu.org/cgi/bugreport.cgi?bug=62958

Can you see if bug#62958 solves this one as well?

Thanks.

bug#63600:

[Eli Zaretskii]

> Cc: 63600@debbugs.gnu.org
> Date: Fri, 26 May 2023 14:25:02 +0300
> From: Eli Zaretskii <eliz@gnu.org>
> 
> > From: Olav Fosse <post@olavfosse.no>
> > Date: Sat, 20 May 2023 13:00:11 +0200
> > 
> > This might be solved for all comint-derived modes in
> > https://debbugs.gnu.org/cgi/bugreport.cgi?bug=62958
> 
> Can you see if bug#62958 solves this one as well?
> 
> Thanks.

Ping!

Can we close this?

bug#63600:

[Eli Zaretskii]

> Cc: 63600@debbugs.gnu.org
> Date: Wed, 31 May 2023 16:19:50 +0300
> From: Eli Zaretskii <eliz@gnu.org>
> 
> > Cc: 63600@debbugs.gnu.org
> > Date: Fri, 26 May 2023 14:25:02 +0300
> > From: Eli Zaretskii <eliz@gnu.org>
> > 
> > > From: Olav Fosse <post@olavfosse.no>
> > > Date: Sat, 20 May 2023 13:00:11 +0200
> > > 
> > > This might be solved for all comint-derived modes in
> > > https://debbugs.gnu.org/cgi/bugreport.cgi?bug=62958
> > 
> > Can you see if bug#62958 solves this one as well?
> > 
> > Thanks.
> 
> Ping!
> 
> Can we close this?

> From: Olav Fosse <post@olavfosse.no>
> Date: Wed, 31 May 2023 15:52:44 +0200
> 
> Setting comint-pager to “cat” does solve this, so feel free to
> close 🎉.

Done.