From owner-svn-src-all@FreeBSD.ORG Sun Nov 21 14:15:41 2010 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 985301065670; Sun, 21 Nov 2010 14:15:41 +0000 (UTC) (envelope-from yanegomi@gmail.com) Received: from mail-wy0-f182.google.com (mail-wy0-f182.google.com [74.125.82.182]) by mx1.freebsd.org (Postfix) with ESMTP id AB2338FC0C; Sun, 21 Nov 2010 14:15:40 +0000 (UTC) Received: by wyb35 with SMTP id 35so5325854wyb.13 for ; Sun, 21 Nov 2010 06:15:39 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:received:sender:received :in-reply-to:references:date:x-google-sender-auth:message-id:subject :from:to:cc:content-type; bh=sq5+AOEuwbtOXjkTzKFaj4baok4WrIH/mw/SJ0JWuSs=; b=GrZaiwJe0ArsIYDGqO4rmMMUquo7eWRhVnMI4Q8QFk9o7z5KMo7HlEIUhAaTFjLLMn sX/p5We3XkMb7Dcbt1cKHtEpAStg5z3jNpyIY+W23gpgrCqctFwyM/Lwm/NeIROeg225 wRnQuWb4yqZ5u6i9FGMZ5zVOZ597bh/XIU5C0= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:cc:content-type; b=A1qWAQGZEC10n/Zl9cvNFVtWbAmKexPnUQtEvznD2Ke5noQb0f21f6nLLlkqQyShYY QE4RCtVwgHccvfqHrOniY5gyK4Akn+WnhNtBwTToGw2g9X5Idwh5aFSVriB0AZpuMgrO R6ZCmkbXhp6HzpMHOO5HzFLUtza/WvAISl7ZQ= MIME-Version: 1.0 Received: by 10.216.175.18 with SMTP id y18mr4514927wel.30.1290348934603; Sun, 21 Nov 2010 06:15:34 -0800 (PST) Sender: yanegomi@gmail.com Received: by 10.216.198.27 with HTTP; Sun, 21 Nov 2010 06:15:34 -0800 (PST) In-Reply-To: <201011211055.oALAtG0q052310@svn.freebsd.org> References: <201011211055.oALAtG0q052310@svn.freebsd.org> Date: Sun, 21 Nov 2010 06:15:34 -0800 X-Google-Sender-Auth: FKG4iTJJi9sX4hmRRnJKIneSq2c Message-ID: From: Garrett Cooper To: Stephen McKay Content-Type: multipart/mixed; boundary=0016e6567d28e22592049590c565 Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r215615 - head/usr.bin/xargs X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 21 Nov 2010 14:15:41 -0000 --0016e6567d28e22592049590c565 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable On Sun, Nov 21, 2010 at 2:55 AM, Stephen McKay wrote: > Author: mckay > Date: Sun Nov 21 10:55:16 2010 > New Revision: 215615 > URL: http://svn.freebsd.org/changeset/base/215615 > > Log: > =A0xargs can be fooled by exiting children that it did not start, causing > =A0it to kick off a new command before the previous has finished, resulti= ng > =A0in corrupted (interleaved) output. =A0It is also fooled by non-exiting > =A0children it did not start, failing to exit until all extraneous childr= en > =A0have exited. > > =A0This patch makes xargs keep track of children it starts, ignoring > =A0pre-existing ones. Unfortunately this broke tinderbox and compile on HEAD. Could you please commit the following patch that fixes the compile and a handful of style(9) violations in the new code? Thanks! -Garrett --0016e6567d28e22592049590c565 Content-Type: text/x-patch; charset=US-ASCII; name="xargs-fix-style.patch" Content-Disposition: attachment; filename="xargs-fix-style.patch" Content-Transfer-Encoding: base64 X-Attachment-Id: f_ggs0fa5d0 SW5kZXg6IHVzci5iaW4veGFyZ3MveGFyZ3MuYwo9PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSB1c3IuYmluL3hhcmdz L3hhcmdzLmMJKHJldmlzaW9uIDIxNTYzNikKKysrIHVzci5iaW4veGFyZ3MveGFyZ3MuYwkod29y a2luZyBjb3B5KQpAQCAtNTg2LDE0ICs1ODYsMTQgQEAKIAogCWlmIChwaWRzX2VtcHR5KCkpIHsK IAkJZXJybm8gPSBFQ0hJTEQ7Ci0JCXJldHVybiAtMTsKKwkJcmV0dXJuICgtMSk7CiAJfQogCiAJ d2hpbGUgKChwaWQgPSB3YWl0cGlkKC0xLCBzdGF0dXMsIGJsb2NrID8gMCA6IFdOT0hBTkcpKSA+ IDApCiAJCWlmIChwaWRzX3JlbW92ZShwaWQpKQogCQkJYnJlYWs7CiAKLQlyZXR1cm4gcGlkOwor CXJldHVybiAocGlkKTsKIH0KIAogc3RhdGljIHZvaWQKQEAgLTYyNSwyNyArNjI1LDI3IEBACiAj ZGVmaW5lCU5PUElECSgwKQogCiBzdGF0aWMgdm9pZAotcGlkc19pbml0KCkKK3BpZHNfaW5pdCh2 b2lkKQogewogCWludCBpOwogCiAJaWYgKChjaGlsZHBpZHMgPSBtYWxsb2MobWF4cHJvY3MgKiBz aXplb2YoKmNoaWxkcGlkcykpKSA9PSBOVUxMKQotCSAgICBlcnJ4KDEsICJtYWxsb2MgZmFpbGVk Iik7CisJCWVycngoMSwgIm1hbGxvYyBmYWlsZWQiKTsKIAogCWZvciAoaSA9IDA7IGkgPCBtYXhw cm9jczsgaSsrKQogCQljbGVhcnNsb3QoaSk7CiB9CiAKIHN0YXRpYyBpbnQKLXBpZHNfZW1wdHko KQorcGlkc19lbXB0eSh2b2lkKQogewotCXJldHVybiBjdXJwcm9jcyA9PSAwOworCXJldHVybiAo Y3VycHJvY3MgPT0gMCk7CiB9CiAKIHN0YXRpYyBpbnQKLXBpZHNfZnVsbCgpCitwaWRzX2Z1bGwo dm9pZCkKIHsKLQlyZXR1cm4gY3VycHJvY3MgPj0gbWF4cHJvY3M7CisJcmV0dXJuIChjdXJwcm9j cyA+PSBtYXhwcm9jcyk7CiB9CiAKIHN0YXRpYyB2b2lkCkBAIC02NjQsMjIgKzY2NCwyMiBAQAog CWludCBzbG90OwogCiAJaWYgKChzbG90ID0gZmluZHNsb3QocGlkKSkgPCAwKQotCSAgICByZXR1 cm4gMDsKKwkJcmV0dXJuICgwKTsKIAogCWNsZWFyc2xvdChzbG90KTsKIAljdXJwcm9jcy0tOwot CXJldHVybiAxOworCXJldHVybiAoMSk7CiB9CiAKIHN0YXRpYyBpbnQKLWZpbmRmcmVlc2xvdCgp CitmaW5kZnJlZXNsb3Qodm9pZCkKIHsKIAlpbnQgc2xvdDsKIAogCWlmICgoc2xvdCA9IGZpbmRz bG90KE5PUElEKSkgPCAwKQogCQllcnJ4KDEsICJpbnRlcm5hbCBlcnJvcjogbm8gZnJlZSBwaWQg c2xvdCIpOwogCi0JcmV0dXJuIHNsb3Q7CisJcmV0dXJuIChzbG90KTsKIH0KIAogc3RhdGljIGlu dApAQCAtNjg5LDkgKzY4OSw5IEBACiAKIAlmb3IgKHNsb3QgPSAwOyBzbG90IDwgbWF4cHJvY3M7 IHNsb3QrKykKIAkJaWYgKGNoaWxkcGlkc1tzbG90XSA9PSBwaWQpCi0JCQlyZXR1cm4gc2xvdDsK KwkJCXJldHVybiAoc2xvdCk7CiAKLQlyZXR1cm4gLTE7CisJcmV0dXJuICgtMSk7CiB9CiAKIHN0 YXRpYyB2b2lkCg== --0016e6567d28e22592049590c565--