From owner-cvs-src-old@FreeBSD.ORG Fri May 14 12:34:23 2010 Return-Path: Delivered-To: cvs-src-old@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 4516D1065678 for ; Fri, 14 May 2010 12:34:23 +0000 (UTC) (envelope-from ed@FreeBSD.org) Received: from repoman.freebsd.org (unknown [IPv6:2001:4f8:fff6::29]) by mx1.freebsd.org (Postfix) with ESMTP id 346708FC1E for ; Fri, 14 May 2010 12:34:23 +0000 (UTC) Received: from repoman.freebsd.org (localhost [127.0.0.1]) by repoman.freebsd.org (8.14.3/8.14.3) with ESMTP id o4ECYNTZ091572 for ; Fri, 14 May 2010 12:34:23 GMT (envelope-from ed@repoman.freebsd.org) Received: (from svn2cvs@localhost) by repoman.freebsd.org (8.14.3/8.14.3/Submit) id o4ECYN0l091571 for cvs-src-old@freebsd.org; Fri, 14 May 2010 12:34:23 GMT (envelope-from ed@repoman.freebsd.org) Message-Id: <201005141234.o4ECYN0l091571@repoman.freebsd.org> X-Authentication-Warning: repoman.freebsd.org: svn2cvs set sender to ed@repoman.freebsd.org using -f From: Ed Schouten Date: Fri, 14 May 2010 12:34:06 +0000 (UTC) To: cvs-src-old@freebsd.org X-FreeBSD-CVS-Branch: RELENG_8 Subject: cvs commit: src/usr.bin/script script.c X-BeenThere: cvs-src-old@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: **OBSOLETE** CVS commit messages for the src tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 14 May 2010 12:34:23 -0000 ed 2010-05-14 12:34:06 UTC FreeBSD src repository Modified files: (Branch: RELENG_8) usr.bin/script script.c Log: SVN rev 208070 on 2010-05-14 12:34:06Z by ed MFC r207453: Remove WNOHANG flag from wait3(). Because script(1) now reliably terminates when the TTY is closed, it may be the case that the call to wait3() occurs just before the child process exits. This causes error codes to be ignored. Just change script(1) to use waitpid() instead of wait3(). This makes it more portable and prevents the need for a loop, since waitpid() only returns a specified process. PR: bin/146189 Tested by: amdmi3@, older version Revision Changes Path 1.24.30.3 +9 -15 src/usr.bin/script/script.c