[Konsole-devel] Review Request 128416: Send EOF before sending SIGHUP when closing

Martin Tobias Holmedahl Sandsmark martin.sandsmark at kde.org
Sun Jul 10 11:09:36 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128416/
-----------------------------------------------------------

(Updated July 10, 2016, 11:09 a.m.)


Review request for Konsole, Kurt Hindenburg and Oswald Buddenhagen.


Changes
-------

Do it properly.


Bugs: 185140
    http://bugs.kde.org/show_bug.cgi?id=185140


Repository: konsole


Description
-------

According to http://permalink.gmane.org/gmane.comp.shells.bash.bugs/12602 (from https://bugs.kde.org/show_bug.cgi?id=185140), sending EOF is the appropriate way of telling e. g. bash to quit.

I wasn't able to get KPtyProcess to send that in a nice way (calling setStandardInputFile(QProcess::nullDevice(), for example), so we fetch the EOF character manually with tcgetattr and send that.

In addition KPtyProcess doesn't support waitForFinished(), so we have to spin the event loop manually for KPtyProcess to notice that the child is finished.


Diffs (updated)
-----

  src/Pty.h 0e7c478 
  src/Pty.cpp eaeda07 
  src/Session.cpp 768dc06 

Diff: https://git.reviewboard.kde.org/r/128416/diff/


Testing
-------

Running backgrounded processes (e. g. launching «kate&») and then closing the tab/window now leaves the backgrounded tasks running, which is apparently The Right Thing.


Thanks,

Martin Tobias Holmedahl Sandsmark

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20160710/5efb8742/attachment.html>


More information about the konsole-devel mailing list