8000 gh-122273: Support PyREPL history in Windows by devdanzin · Pull Request #122274 · python/cpython · GitHub
[go: up one dir, main page]
Skip to content

gh-122273: Support PyREPL history in Windows #122274

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.

Sign up for GitHub

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

Closed
wants to merge 13 commits into from
Closed

gh-122273: Support PyREPL history in Windows #122274

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
cd2d7fb
Support PyREPL history in Windows.
devdanzin Jul 25, 2024
aa3b826
Support multi-line history items in Windows.
devdanzin Jul 25, 2024
bb9904b
Silence a warning for readline.read_init_file() in Windows.
devdanzin Jul 25, 2024
4ff7328
Merge branch 'main' into pyrepl_history_windows
devdanzin Jul 25, 2024
a1d2336
📜🤖 Added by blurb_it.
blurb-it[bot] Jul 25, 2024
6c68ca4
Merge main.
devdanzin Oct 9, 2024
1c75896
Remove duplicated imports.
devdanzin Oct 9, 2024
bef3c7c
Fix importing of _pyrepl in non-Windows when CAN_USE_PYREPL is False.
devdanzin Oct 9, 2024
0a5699d
Refactor import of _pyrepl.readline so test_python_basic_repl passes.
devdanzin Oct 9, 2024
3ea8c79
Refactor a test for readline.backend == 'editline', thanks @vstinner
devdanzin Oct 9, 2024
9ad8e72
Only import _pyrepl if PYTHON_BASIC_REPL isn't set.
devdanzin Oct 12, 2024
5464efd
Merge main.
devdanzin Oct 12, 2024
746099b
Only import _pyrepl.readline as readline if real readline wasn't impo…
devdanzin Oct 12, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Only import _pyrepl if PYTHON_BASIC_REPL isn't set.
  • Loading branch information
@devdanzin
devdanzin committed Oct 12, 2024
commit 9ad8e721373bf7b53d2fb9141fd3fc4863a80e3d
16 changes: 10 additions & 6 deletions Lib/site.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -502,12 +502,13 @@ def register_readline():
import readline
real_readline = True
except ImportError:
import _pyrepl.readline as readline
readline = None
real_readline = False
import rlcompleter # noqa: F401
if PYTHON_BASIC_REPL:
CAN_USE_PYREPL = False
else:
import _pyrepl.readline as readline
from _pyrepl.main import CAN_USE_PYREPL
if real_readline:
import _pyrepl.unix_console
Expand All @@ -521,9 +522,10 @@ def register_readline():

# Reading the initialization (config) file may not be enough to set a
# completion key, so we set one first and then read the file.
if getattr(readline, "backend", None) == 'editline':
backend = getattr(readline, "backend", None)
if backend == 'editline':
readline.parse_and_bind('bind ^I rl_complete')
else:
elif backend:
readline.parse_and_bind('tab: complete')

try:
Expand All @@ -536,7 +538,7 @@ def register_readline():
# want to ignore the exception.
pass

if readline.get_current_history_length() == 0:
if readline and readline.get_current_history_length() == 0:
# If no history was loaded, default to .python_history,
# or PYTHON_HISTORY.
# The guard is necessary to avoid doubling history size at
Expand All @@ -553,13 +555,15 @@ def register_readline():
exceptions = OSError

try:
readline_module.read_history_file(history)
if readline:
readline_module.read_history_file(history)
except exceptions:
pass

def write_history():
try:
readline_module.write_history_file(history)
if readline:
readline_module.write_history_file(history)
except (FileNotFoundError, PermissionError):
# home directory does not exist or is not writable
# https://bugs.python.org/issue19891
Expand Down
0