search

pexpect / ptyprocess

Run a subprocess in a pseudo terminal
https://ptyprocess.readthedocs.io/en/latest/
Other
222 stars 71 forks source link

A minor issue found via static analysis #35

Closed tzickel closed 6 years ago

tzickel commented 7 years ago

ExceptionPexpect is not defined. https://github.com/pexpect/ptyprocess/blob/8f1ba73b82a4b4fbe342fef1498cd35362f63982/ptyprocess/_fork_pty.py#L66

Found using cython and https://gist.github.com/tzickel/ea4541b12c1298a8092c3ec5043ef77e

pip install -I --root temp_ptyprocess ptyprocess
python checkcython.py temp_ptyprocess
takluyver commented 7 years ago

Thanks! #36 should fix it.