Review Request 128791: Only send EOF if the process that was started was a normal shell

Martin Tobias Holmedahl Sandsmark martin.sandsmark at kde.org
Sun Aug 28 22:14:14 UTC 2016



> On Aug. 28, 2016, 10:04 p.m., Kurt Hindenburg wrote:
> > Is this still needed w/ 392ee33cf8ffec54d41d342d7cba8fe82a13adc0 ?  or is this another layer to verify the bug is fixed?
> > 
> > Also, perhaps change the comment in the middle  s/default/known/

Yeah, this is just part of the solution. Now if only the users shell process is running, without any other processes in the foreground, we send EOF so that the shell can exit cleanly, otherwise we send the SIGHUP.


- Martin Tobias Holmedahl


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


On Aug. 28, 2016, 10:13 p.m., Martin Tobias Holmedahl Sandsmark wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128791/
> -----------------------------------------------------------
> 
> (Updated Aug. 28, 2016, 10:13 p.m.)
> 
> 
> Review request for Konsole.
> 
> 
> Bugs: 367746
>     http://bugs.kde.org/show_bug.cgi?id=367746
> 
> 
> Repository: konsole
> 
> 
> Description
> -------
> 
> The only processes that are "safe" or require us to send an EOF to terminate cleanly are shells, so verify that people didn't run with -e tmux or similar.
> 
> 
> Diffs
> -----
> 
>   src/Session.cpp 9178eb2 
> 
> Diff: https://git.reviewboard.kde.org/r/128791/diff/
> 
> 
> Testing
> -------
> 
> Normal bash gets EOF, -e tmux gets SIGHUP.
> 
> 
> Thanks,
> 
> Martin Tobias Holmedahl Sandsmark
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/konsole-devel/attachments/20160828/2a25c0a7/attachment.html>


More information about the konsole-devel mailing list