Skip to content
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.

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

RFC2217 Remote Serial connection fails #373

Closed
gernoteger opened this issue Nov 11, 2018 · 4 comments
Closed

RFC2217 Remote Serial connection fails #373

gernoteger opened this issue Nov 11, 2018 · 4 comments

Comments

@gernoteger
Copy link

gernoteger commented Nov 11, 2018

When starting esptool over a RFC2217 remote connection it fails.

This commit introduced write timeout which isn't implemented in rfc2217.py

Full output from esptool.py (please copy and paste all lines of output)

Flashing binaries to serial port rfc2217://rasp3:3333 (app at offset 0x10000 )...
esptool.py v2.6-beta1
Serial port rfc2217://rasp3:3333
Traceback (most recent call last):
  File "/mnt/c/Users/gernot/dev/geger.at/ESP32-toolchain/tools/esp-idf/components/esptool_py/esptool/esptool.py", line 2865, in <module>
    _main()
  File "/mnt/c/Users/gernot/dev/geger.at/ESP32-toolchain/tools/esp-idf/components/esptool_py/esptool/esptool.py", line 2858, in _main
    main()
  File "/mnt/c/Users/gernot/dev/geger.at/ESP32-toolchain/tools/esp-idf/components/esptool_py/esptool/esptool.py", line 2565, in main
    esp = chip_class(each_port, initial_baud, args.trace)
  File "/mnt/c/Users/gernot/dev/geger.at/ESP32-toolchain/tools/esp-idf/components/esptool_py/esptool/esptool.py", line 224, in __init__
    self._port.write_timeout = DEFAULT_SERIAL_WRITE_TIMEOUT
  File "/home/gernot/.local/lib/python2.7/site-packages/serial/serialutil.py", line 388, in write_timeout
    self._reconfigure_port()
  File "/home/gernot/.local/lib/python2.7/site-packages/serial/rfc2217.py", line 506, in _reconfigure_port
    raise NotImplementedError('write_timeout is currently not supported')
NotImplementedError: write_timeout is currently not supported

Do you have any other information from investigating this?

This seems to be starting with commit 47302cf first visible in version 2.5.0.

@gernoteger
Copy link
Author

Workaround: Revert to 2.4.1

@snahmad
Copy link

snahmad commented Nov 20, 2018

How we can revert to 2.4.1

@me21
Copy link
Contributor

me21 commented Nov 20, 2018

Clone repo locally and check out the corresponding tag. You'll see it in git log.

@ghost
Copy link

ghost commented Nov 30, 2018

Ran into the same issue here using the esp32-idf, I guess I can do something with git modules to change the version there?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants