GNU bug report logs - #49822
28.0.50; python-shell-send functions show no output

Previous Next

Package: emacs;

Reported by: dalanicolai <at> gmail.com

Date: Mon, 2 Aug 2021 14:33:01 UTC

Severity: normal

Tags: patch

Found in version 28.0.50

Fixed in version 28.1

Done: Lars Ingebrigtsen <larsi <at> gnus.org>

Bug is archived. No further changes may be made.

Full log


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

From: Augusto Stoffel <arstoffel <at> gmail.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: andrewjmoreton <at> gmail.com, Mark.Barton <at> disney.com, emacs-devel <at> gnu.org,
 schwab <at> linux-m68k.org, monnier <at> iro.umontreal.ca, 49822 <at> debbugs.gnu.org,
 michael.albinus <at> gmx.de, larsi <at> gnus.org
Subject: Re: bug#49822: master e32c7d2: Change Python eval to send directly
 instead of using temporary files
Date: Tue, 07 Sep 2021 19:59:06 +0200
On Tue,  7 Sep 2021 at 20:48, Eli Zaretskii <eliz <at> gnu.org> wrote:

>> From: Augusto Stoffel <arstoffel <at> gmail.com>
>> Date: Tue, 07 Sep 2021 19:37:41 +0200
>> Cc: Andy Moreton <andrewjmoreton <at> gmail.com>, emacs-devel <at> gnu.org,
>>  Andreas Schwab <schwab <at> linux-m68k.org>,
>>  Stefan Monnier <monnier <at> iro.umontreal.ca>, 49822 <at> debbugs.gnu.org,
>>  Michael Albinus <michael.albinus <at> gmx.de>, Lars Ingebrigtsen <larsi <at> gnus.org>
>> 
>> Okay then. Since there seem to be no better alternatives, I have
>> attached a new patch reducing the limit to a hard-coded 1024 bytes.  If
>> some day someone adds a variable specifying a more precise limit, then
>> we can change this.
>
> I thought the conclusion was that we do know the limits, but they are
> different on different OSes?  If so, why not use the correct limit for
> each OS, instead of using the most strict limit?  The underlying OS is
> available in system-type.
>
>> -    (if (<= (string-bytes code) 4096)
>> +    (if (<= (string-bytes code) 1024)
>
> In any case, IMO this should be a defconst with a suitable doc string,
> so that we'd have this stuff documented for posterity.
>
> Thanks.

Yes, I agree this should be a defconst, defined in comint.el or maybe
even in C, if there turns out to be a sensible way to query the OS.

I'm afraid I've already proved that I am not up to this particular task :-)




This bug report was last modified 3 years and 333 days ago.

Previous Next


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