Subject: PR/33623 CVS commit: [netbsd-2] src/sbin/ping
To: None <gnats-admin@netbsd.org, netbsd-bugs@netbsd.org, chuck@ece.cmu.edu>
From: Manuel Bouyer <bouyer@netbsd.org>
List: netbsd-bugs
Date: 08/11/2007 14:10:06
The following reply was made to PR bin/33623; it has been noted by GNATS.

From: Manuel Bouyer <bouyer@netbsd.org>
To: gnats-bugs@NetBSD.org
Cc: 
Subject: PR/33623 CVS commit: [netbsd-2] src/sbin/ping
Date: Sat, 11 Aug 2007 14:08:43 +0000 (UTC)

 Module Name:	src
 Committed By:	bouyer
 Date:		Sat Aug 11 14:08:43 UTC 2007
 
 Modified Files:
 	src/sbin/ping [netbsd-2]: ping.c
 
 Log Message:
 Pull up following revision(s) (requested by ghen in ticket #11351):
 	sbin/ping/ping.c: revision 1.79, 1.80
 PR/33623: Chuck Cranor: Ping stops when ran in the background
 because it tries to set the tty not to print kerninfo. Change it
 to only only play with the tty when ping is running in the foreground
 (and will not stop when calling tcsetattr()). In my opinion, it is
 preferable to print the kerninfo line with the ping info message
 rather than to mess with the tty, but that's just me.
 Factor out the tty code and explain we prefer to stop in the ^Z bg case
 when we exit, rather than kill the tty setting.
 
 
 To generate a diff of this commit:
 cvs rdiff -r1.75 -r1.75.4.1 src/sbin/ping/ping.c
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.