-
-
Notifications
You must be signed in to change notification settings - Fork 504
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
core: get rid of termios2, use termios on Linux #700
Conversation
In order to simplify and to enable the build in manylinux1 it makes sense to go back to termios instead of termios2.
ea4a37c
to
e762058
Compare
Do we lose something by doing that? e.g. the ability to use non-standard baudrates? |
I don't think so. |
e762058
to
0eb13fe
Compare
Tested this on Linux and baudrate change doesn't seem to be working anymore. Needs more investigation. |
This includes the following baudrate fixes: - Checks to see if the baudrate could actually be set as expected. - Use baudrate defines instead of straight int, otherwise this doesn't work on Linux. - Add error code for the case of an unknown/unsupported baudrate.
The assumption is that we don't need to use the defines on macOS.
Tested on Linux and macOS, now works for both. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I kind of have to trust you on that, but I'm happy to get rid of termios2 :)
Seems to me like it is failing on appveyor, and only in Debug, because of an unrelated curl test. |
In order to simplify and to enable the build in manylinux1 it makes sense to go back to termios instead of termios2.