bugGNU Screen - Bugs: bug #47493, [PATCH] Make delay after child...

 
 

bug #47493: [PATCH] Make delay after child program terminates configurable

Submitted by:  Mark Schreiber <mark7b>
Submitted on:  Tue 22 Mar 2016 07:06:07 AM UTC  
 
Category: Program LogicSeverity: 3 - Normal
Priority: 5 - NormalStatus: None
Privacy: PublicAssigned to: Amadeusz Sławiński <amade>
Open/Closed: OpenRelease: None
Fixed Release: NonePlanned Release: None
Work Required: None

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

Tue 22 Mar 2016 07:06:07 AM UTC, original submission:

The patch for Bug #27061 introduced a ten second delay after a child program running in a window terminated -- this was to address a race between SIGCHLD and any output written by that program being delivered.

That behavior makes sense if the process group is expected to be empty after the child terminates. However, some users, like myself, do things like run a bash process in a screen window and have that bash process run processes nohup in the background. With the ten second delay, this causes each screen window to linger for ten seconds when one tries closing it -- annoying, to say the least.

I don't think that there is a fantastic way to resolve this; Linux's documentation states that a delay may legitimately exist:

PTY(7):

Data flow between master and slave is handled asynchronously, much like data flow with a physical terminal. Data written to the slave will be available at the master promptly, but may not be available immediately. Similarly, there may be a small processing delay between a write to the master, and the effect being visible at the slave.

END MAN PAGE EXCERPT

I'm perfectly fine with some output potentially not showing up at the end (the window is dying anyway); however, a ten second delay is a problem for me.

A patch is attached to make the delay configurable.

Mark Schreiber <mark7b>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach File(s):
   
   
Comment:
   

Attached Files
file #36719:  childdelay.diff added by mark7b (3KiB - text/x-patch)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by gedge
  • -unavailable- added by amade (Updated the item)
  • -unavailable- added by mark7b (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 3 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Sat 26 Mar 2016 02:15:39 PM UTCgedgeCarbon-Copy-=>Added gedge
    Thu 24 Mar 2016 05:12:56 PM UTCamadeAssigned toNone=>amade
    Tue 22 Mar 2016 07:06:07 AM UTCmark7bAttached File-=>Added childdelay.diff, #36719

    Back to the top


    Powered by Savane 3.1-cleanup1