-
-
Notifications
You must be signed in to change notification settings - Fork 31.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Warning on autocomplete messes up the terminal #119205
Comments
@JelleZijlstra Are you handling this or do you mind if I take a look into this? 🙂 |
I'm not, feel free to take a look! |
fwiw, this looks to be an issue with any output:
I ran into a similar issue when bringing up pdb:
|
Just found that this happens on mac, but not ubuntu. New repl doesn't have a windows version with autocomplete. |
mac uses
|
I re-build with GNU readline lib. But, The bug reproduce :( brew install readline
export LDFLAGS="-L/opt/homebrew/opt/readline/lib"
export CPPFLAGS="-I/opt/homebrew/opt/readline/include"
./configure --with-pydebug --with-readline=readline
...
dyld_info Modules/readline.cpython-314d-darwin.so
Modules/readline.cpython-314d-darwin.so [arm64]:
-platform:
platform minOS sdk
macOS 14.4 14.4
-segments:
load-offset segment section sect-size seg-size perm
0x00000000 __TEXT 32KB r.x
0x00003E10 __text 9928
0x000064D8 __stubs 1008
0x000068C8 __stub_helper 1032
0x00006CD0 __cstring 1214
0x00007190 __const 3456
0x00007F10 __unwind_info 232
0x00008000 __DATA_CONST 16KB rw.
0x00008000 __got 240
0x0000C000 __DATA 16KB rw.
0x0000C000 __la_symbol_ptr 672
0x0000C2A0 __data 1016
0x0000C698 __bss 40
-dependents:
attributes load path
/opt/homebrew/opt/readline/lib/libreadline.8.dylib
/usr/lib/libSystem.B.dylib
|
I tried this with
But I'm not sure pypy has that deprecation warning in this codepath so I'll check that. |
Hmm, it seems that similar behaviour exists on the old terminal as well though. EDIT: Similar issue for the old REPL |
@danielhollas We should think about the direction of the solution. |
@koxudaxi Would this also resolve this issue? https://github.com/python/cpython/pull/113979/files |
@eugenetriguba |
@koxudaxi, sign the CLA and I think we will still take your PR as it includes a test. |
Co-authored-by: Łukasz Langa <lukasz@langa.pl>
(cherry picked from commit 506b1a3) Co-authored-by: Koudai Aono <koxudaxi@gmail.com> Co-authored-by: Łukasz Langa <lukasz@langa.pl>
Thanks all for the input and @koxudaxi for the PR. |
Co-authored-by: Łukasz Langa <lukasz@langa.pl>
Bug report
Bug description:
./python.exe
import sys; code = sys._getframe(1).f_code
code.co_
and hit tabCPython versions tested on:
CPython main branch
Operating systems tested on:
macOS
Linked PRs
The text was updated successfully, but these errors were encountered: