From: Augusto Stoffel <arstoffel@gmail.com>
To: Michael Albinus <michael.albinus@gmx.de>
Cc: 49822@debbugs.gnu.org, Lars Ingebrigtsen <larsi@gnus.org>,
Andreas Schwab <schwab@linux-m68k.org>,
Stefan Monnier <monnier@iro.umontreal.ca>,
emacs-devel@gnu.org
Subject: Re: bug#49822: master e32c7d2: Change Python eval to send directly instead of using temporary files
Date: Mon, 06 Sep 2021 18:08:43 +0200 [thread overview]
Message-ID: <87lf49wut0.fsf@gmail.com> (raw)
In-Reply-To: <874kaxgbgs.fsf@gmx.de> (Michael Albinus's message of "Mon, 06 Sep 2021 14:00:51 +0200")
On Mon, 6 Sep 2021 at 14:00, Michael Albinus <michael.albinus@gmx.de> wrote:
> Andreas Schwab <schwab@linux-m68k.org> writes:
>
>> Neither has PIPE_BUF.
>
> Great that you always say what's not working. Perhaps you have a better
> answer to Augusto?
>
>> Andreas.
>
> Best regards, Michael.
Thanks for the ideas. If there's no simple and platform-independent
way to obtain the line length limit in a PTY, then I think there's no
reason to attempt anything fancy in python.el (or any derived comint
mode, for that matter).
So the options seem to be: either hard-code a limit of 1024 chars, or
create a defcustom with a platform-dependent default. But such a
defcustom should probably be part of comint.el, and should be added by
someone who (unlike me) actually understands this stuff.
Working around a limit of less than 1024 characters is too awkward, so I
hope this is a safe assumption. The juggling of temp files that existed
in python.el for many years already required lines of ~350 chars,
depending on how the OS names temp files.
Sorry for introducing a bug in master. I think I got it fixed here, and
I'll submit the patch when I'm confident about it.
next prev parent reply other threads:[~2021-09-06 16:08 UTC|newest]
Thread overview: 68+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20210903122828.16890.65271@vcs0.savannah.gnu.org>
[not found] ` <20210903122829.EAAC220B71@vcs0.savannah.gnu.org>
2021-09-03 23:04 ` master e32c7d2: Change Python eval to send directly instead of using temporary files Stefan Monnier
2021-09-04 9:49 ` bug#49822: " Augusto Stoffel
2021-09-04 13:52 ` Stefan Monnier
2021-09-04 13:52 ` bug#49822: " Stefan Monnier via Bug reports for GNU Emacs, the Swiss army knife of text editors
2021-09-05 6:13 ` Barton, Mark
2021-09-05 6:13 ` Barton, Mark
2021-09-05 7:46 ` bug#49822: " Augusto Stoffel
2021-09-05 8:10 ` Augusto Stoffel
2021-09-05 8:10 ` Augusto Stoffel
2021-09-05 17:18 ` bug#49822: " Mark Barton
2021-09-05 17:33 ` Mark Barton
2021-09-05 17:46 ` Augusto Stoffel
2021-09-05 17:46 ` Augusto Stoffel
2021-09-05 7:41 ` bug#49822: " Lars Ingebrigtsen
2021-09-05 7:41 ` Lars Ingebrigtsen
2021-09-05 7:41 ` bug#32042: " Lars Ingebrigtsen
2021-09-05 16:36 ` Andreas Schwab
2021-09-05 18:40 ` Augusto Stoffel
2021-09-06 7:43 ` Michael Albinus
2021-09-06 8:40 ` Andreas Schwab
2021-09-06 8:40 ` Andreas Schwab
2021-09-06 11:23 ` Michael Albinus
2021-09-06 11:53 ` Andreas Schwab
2021-08-02 14:32 ` bug#49822: 28.0.50; python-shell-send functions show no output dalanicolai
2021-08-02 15:11 ` bug#49822: forgot link (and mentioned wrong user) dalanicolai
2021-08-02 15:17 ` bug#49822: 28.0.50; python-shell-send functions show no output Andrea Corallo via Bug reports for GNU Emacs, the Swiss army knife of text editors
2021-08-04 7:18 ` Lars Ingebrigtsen
2021-08-05 10:03 ` dalanicolai
2021-08-12 14:57 ` Augusto Stoffel
2021-08-13 11:10 ` Lars Ingebrigtsen
2021-08-13 16:44 ` Andreas Röhler
2021-08-14 9:29 ` Augusto Stoffel
2021-08-15 6:41 ` Andreas Röhler
2021-08-26 8:27 ` Augusto Stoffel
2021-08-26 14:09 ` Lars Ingebrigtsen
2021-08-28 9:28 ` bug#49822: [RFC PATCH] " Augusto Stoffel
2021-08-28 15:41 ` Lars Ingebrigtsen
2021-09-03 8:02 ` bug#49822: [PATCH] " Augusto Stoffel
2021-09-03 12:28 ` Lars Ingebrigtsen
2021-09-06 22:15 ` bug#49822: master e32c7d2: Change Python eval to send directly instead of using temporary files Andy Moreton
2021-09-07 7:18 ` Andreas Schwab
2021-09-06 12:00 ` Michael Albinus
2021-09-06 16:08 ` Augusto Stoffel
2021-09-06 16:08 ` Augusto Stoffel [this message]
2021-09-07 17:37 ` Augusto Stoffel
2021-09-07 17:48 ` Eli Zaretskii
2021-09-07 17:48 ` Eli Zaretskii
2021-09-07 17:59 ` Augusto Stoffel
2021-09-07 18:19 ` Eli Zaretskii
2021-09-07 17:59 ` Augusto Stoffel
2021-09-07 18:13 ` Augusto Stoffel
2021-09-07 18:13 ` Augusto Stoffel
2021-09-07 18:31 ` Eli Zaretskii
2021-09-07 19:00 ` Augusto Stoffel
2021-09-07 19:16 ` Eli Zaretskii
2021-09-08 7:02 ` Michael Albinus
2021-09-08 7:02 ` Michael Albinus
2021-09-07 19:16 ` Eli Zaretskii
2021-09-07 18:31 ` Eli Zaretskii
2021-09-08 3:17 ` Barton, Mark
2021-09-08 5:09 ` Augusto Stoffel
2021-09-08 7:50 ` Lars Ingebrigtsen
2021-09-08 14:05 ` Augusto Stoffel
2021-09-08 14:05 ` Augusto Stoffel
2021-09-09 13:48 ` Lars Ingebrigtsen
2021-09-08 7:50 ` Lars Ingebrigtsen
2021-09-06 11:23 ` Michael Albinus
2021-09-06 7:43 ` Michael Albinus
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
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87lf49wut0.fsf@gmail.com \
--to=arstoffel@gmail.com \
--cc=49822@debbugs.gnu.org \
--cc=emacs-devel@gnu.org \
--cc=larsi@gnus.org \
--cc=michael.albinus@gmx.de \
--cc=monnier@iro.umontreal.ca \
--cc=schwab@linux-m68k.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 external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.