polyfile icon indicating copy to clipboard operation
polyfile copied to clipboard

Readline missing from setup.py?

Open ross-spencer opened this issue 3 years ago • 2 comments

Trying to install this fresh on a Windows box today.

Traceback (most recent call last):
  File "C:\Users\Spencer\Apps\python\lib\runpy.py", line 197, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "C:\Users\Spencer\Apps\python\lib\runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "C:\Users\Spencer\Apps\python\Scripts\polyfile.exe\__main__.py", line 4, in <module>
  File "C:\Users\Spencer\Apps\python\lib\site-packages\polyfile\__init__.py", line 2, in <module>
    from .__main__ import main
  File "C:\Users\Spencer\Apps\python\lib\site-packages\polyfile\__main__.py", line 18, in <module>
    from .debugger import Debugger
  File "C:\Users\Spencer\Apps\python\lib\site-packages\polyfile\debugger.py", line 12, in <module>
    from .repl import ANSIColor, ANSIWriter, arg_completer, command, ExitREPL, log, REPL, SetCompleter
  File "C:\Users\Spencer\Apps\python\lib\site-packages\polyfile\repl.py", line 7, in <module>
    import readline
ModuleNotFoundError: No module named 'readline'

It looks like readline is a new requirement and isn't listed in the requirements: https://github.com/trailofbits/polyfile/blob/9c2d20b0013c670e36e8051cfbb3199f9f36d762/setup.py#L123-L134

Pip version: pip 21.3.1 from C:\Users\Spencer\Apps\python\lib\site-packages\pip (python 3.9)

ross-spencer avatar Feb 14 '22 09:02 ross-spencer

Curious, readline has been bundled with Python since at least version 3.6, and the documentation doesn't mention any restrictions with OS 🤔. We haven't had any issues on macOS or Linux, so I suspect this is a Windows issue. I'm going to try adding support for Windows via https://github.com/pyreadline3/pyreadline3

ESultanik avatar Feb 14 '22 13:02 ESultanik

PolyFile v0.4.1 should now support Windows. @ross-spencer are you able to confirm that this resolved your issues?

ESultanik avatar Apr 07 '22 19:04 ESultanik