-
Notifications
You must be signed in to change notification settings - Fork 1
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
Constants TCIOFLUSH, TCIFLUSH, TCOFLUSH incorrect on Linux #6
Comments
Huh, that’s strange. I thought I had put in an if else for all the values that were different for Linux. Can you submit a PR? If not, I’ll make the change tomorrow. |
Hi @kdheepak, I won't be able to submit a PR in the next week or so due to competing priorities. |
Hi @kdheepak, you may be interested in this as an alternative to hand-coding constants: https://github.com/notinaboat/CInclude.jl |
Yeah that looks like a good package! I’ll have to refactor and experiment with it. I’m also juggling tasks so I’m not sure when I’ll get to this though. |
See
https://github.com/lattera/glibc/blob/895ef79e04a953cac1493863bcae29ad85657ee1/sysdeps/unix/sysv/linux/bits/termios.h#L208-L210
vs
TERMIOS.jl/src/TERMIOS.jl
Lines 421 to 428 in 3c32ffd
The text was updated successfully, but these errors were encountered: