summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKyle King <KyleKing@users.noreply.github.com>2021-01-27 19:16:43 -0500
committerKyle King <KyleKing@users.noreply.github.com>2021-01-27 19:16:43 -0500
commita96c0f53b2a62a94582624c76c7d990012891204 (patch)
tree7cd0010127ae50250e0843cab31540ec05e5806d
parent44eb9d4e1946a3cc9a10fe85ccb617b6076f627c (diff)
downloadcmd2-git-a96c0f53b2a62a94582624c76c7d990012891204.tar.gz
Update conditional pyreadline3 dependency for Win
-rw-r--r--Pipfile3
-rw-r--r--cmd2/rl_utils.py4
-rw-r--r--docs/overview/integrating.rst9
-rwxr-xr-xsetup.py5
4 files changed, 13 insertions, 8 deletions
diff --git a/Pipfile b/Pipfile
index b384709c..bfa222b6 100644
--- a/Pipfile
+++ b/Pipfile
@@ -22,7 +22,8 @@ ipython = "*"
isort = "*"
mock = {version = "*",markers = "python_version < '3.6'"}
plumbum = "*"
-pyreadline = {version = "*",sys_platform = "== 'win32'"}
+pyreadline = {version = "*",sys_platform = "== 'win32'",markers = "python_version < '3.8'"}
+pyreadline3 = {version = "*",sys_platform = "== 'win32'",markers = "python_version >= '3.8'"}
pytest = "*"
pytest-cov = "*"
pytest-mock = "*"
diff --git a/cmd2/rl_utils.py b/cmd2/rl_utils.py
index e435c3f5..ca75fd8a 100644
--- a/cmd2/rl_utils.py
+++ b/cmd2/rl_utils.py
@@ -37,8 +37,8 @@ vt100_support = False
# Explanation for why readline wasn't loaded
_rl_warn_reason = ''
-# The order of this check matters since importing pyreadline will also show readline in the modules list
-if 'pyreadline' in sys.modules:
+# The order of this check matters since importing pyreadline/pyreadline3 will also show readline in the modules list
+if 'pyreadline' in sys.modules or 'pyreadline3' in sys.modules:
rl_type = RlType.PYREADLINE
from ctypes import byref
diff --git a/docs/overview/integrating.rst b/docs/overview/integrating.rst
index db5cb206..041083bc 100644
--- a/docs/overview/integrating.rst
+++ b/docs/overview/integrating.rst
@@ -26,10 +26,13 @@ Windows Considerations
If you would like to use :ref:`features/completion:Completion`, and you want
your application to run on Windows, you will need to ensure you install the
-``pyreadline`` package. Make sure to include the following in your
-``setup.py``::
+``pyreadline3`` or ``pyreadline`` package. Make sure to include the following
+in your ``setup.py``::
install_requires=[
'cmd2>=1,<2',
- ":sys_platform=='win32'": ['pyreadline'],
+ ":sys_platform=='win32'": [
+ "pyreadline ; python_version<'3.8'",
+ "pyreadline3 ; python_version>='3.8'", # pyreadline3 is a drop-in replacement for Python 3.8 and above
+ ],
]
diff --git a/setup.py b/setup.py
index 5e2995a9..6b438b13 100755
--- a/setup.py
+++ b/setup.py
@@ -44,8 +44,9 @@ INSTALL_REQUIRES = [
]
EXTRAS_REQUIRE = {
- # Windows also requires pyreadline to ensure tab completion works
- ":sys_platform=='win32'": ['pyreadline'],
+ # Windows also requires pyreadline or the replacement, pyreadline3, to ensure tab completion works
+ ":sys_platform=='win32' and python_version<'3.8'": ["pyreadline"],
+ ":sys_platform=='win32' and python_version>='3.8'": ["pyreadline3"],
# Extra dependencies for running unit tests
'test': [
"gnureadline; sys_platform=='darwin'", # include gnureadline on macOS to ensure it is available in nox env