search

tolitius / jemma

jemma & her ai agents that build software
93 stars 10 forks source link

removed readline #3

Closed ShreehariA closed 5 months ago

ShreehariA commented 5 months ago

It is said pyreadline3 can work on both Mac and Windows; I hope it does.

tolitius commented 5 months ago

mac and linux don't like this :(

seems like "pyreadline" needs 'windll'

Traceback (most recent call last):
  File "/opt/venvs/foobar/lib/python3.11/site-packages/pyreadline3/keysyms/__init__.py", line 15, in <module>
    from .keysyms import *
  File "/opt/venvs/foobar/lib/python3.11/site-packages/pyreadline3/keysyms/keysyms.py", line 13, in <module>
    from ctypes import windll
ImportError: cannot import name 'windll' from 'ctypes' (/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/ctypes/__init__.py)
ShreehariA commented 5 months ago

I just built it and tried it with the new version, and it works:

Screenshot 2024-04-11 230546
tolitius commented 5 months ago

the PR did not work as is (at least on mac & linux) I added windows support, it should be in the latest jemma

let me know if it works on your windows laptop

tolitius commented 5 months ago

@ShreehariA this can be closed if it works on windows can you confirm?

ShreehariA commented 5 months ago

It works fine in windows. Closing the PR, Thanks for the support.