-
Notifications
You must be signed in to change notification settings - Fork 27
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
lxqt-sudo: allow bidirectional communication with launched process #210
Conversation
Given that lxqt-sudo should not regress in terms of functionality compared to its CLI counterparts, and given that the code looks fairly sane, +1. Friendly reminder that I'm a fan of +2s prior to merging. |
It definitely needs @palinek's approval. |
Also for better readability: - add comments - use one constant instead of plain number here-and-there
..and optimize splitting of output (for large outputs it may have been a penalty to split the whole batch into lines).
Pr 210 enhancements
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.
GTM
lxqt-sudo: allow bidirectional communication with launched process
Landed in e4a321e Thank you. |
Fixes #209. I tested the changes pretty well, but definitely someone who knows more about pseudoterminals should look closely at this since this is unfamiliar territory for me.