From owner-svn-src-head@freebsd.org Tue Dec 5 16:49:12 2017 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id A7A3DE707DC; Tue, 5 Dec 2017 16:49:12 +0000 (UTC) (envelope-from cy.schubert@komquats.com) Received: from smtp-out-so.shaw.ca (smtp-out-so.shaw.ca [64.59.136.138]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "Client", Issuer "CA" (not verified)) by mx1.freebsd.org (Postfix) with ESMTPS id 45D082EAA; Tue, 5 Dec 2017 16:49:11 +0000 (UTC) (envelope-from cy.schubert@komquats.com) Received: from spqr.komquats.com ([96.50.22.10]) by shaw.ca with SMTP id MGP5eVjmnGvLHMGP6ejt0V; Tue, 05 Dec 2017 09:49:10 -0700 X-Authority-Analysis: v=2.2 cv=a9pAzQaF c=1 sm=1 tr=0 a=jvE2nwUzI0ECrNeyr98KWA==:117 a=jvE2nwUzI0ECrNeyr98KWA==:17 a=ocR9PWop10UA:10 a=YxBL1-UpAAAA:8 a=6I5d2MoRAAAA:8 a=ndaoGXS1AAAA:8 a=z0teBqPy1X18kmprVqoA:9 a=vwzrjUem2fATOGXU:21 a=M3fsWT2IC9t0Ut-9:21 a=CjuIK1q_8ugA:10 a=a11oI3MFgE0DJICV_lUA:9 a=mtsX-dKVlAp7E83b:21 a=DKgg1OFqWCy208_p:21 a=JYNM_zmkhQQQPQg_:21 a=_W_S_7VecoQA:10 a=Ia-lj3WSrqcvXOmTRaiG:22 a=IjZwj45LgO3ly-622nXo:22 a=mFeOnlTyF09QQMGr2mMI:22 Received: from [25.81.45.55] (unknown [72.143.226.210]) by spqr.komquats.com (Postfix) with ESMTPSA id 361ED247; Tue, 5 Dec 2017 08:49:06 -0800 (PST) MIME-Version: 1.0 From: Cy Schubert Subject: RE: svn commit: r326554 - in head: . usr.bin/spongeusr.bin/sponge/tests usr.bin/tee Date: Tue, 5 Dec 2017 08:49:09 -0800 To: Cy Schubert , Devin Teske , Hans Petter Selasky CC: "rgrimes@freebsd.org" , "cem@freebsd.org" , Eitan Adler , src-committers , "svn-src-all@freebsd.org" , "svn-src-head@freebsd.org" Message-Id: <20171205164906.361ED247@spqr.komquats.com> X-CMAE-Envelope: MS4wfMjjpN6L5iei1/N8/srgfayvuD49cxiLZ8QQXmh+d1p0Gzp/HVygADQNCFOiatCK+Ad70DnXb9CKkZOMPtIV+Q5tDUK+eWqotabNnY+OKhW1EnPSm8SZ OR3CBh2LrBzSofsFZebvUqwgqd1RhTw4T9QUOlAJ7NkK/k9RqT9+pm5U2zL0h6vHizMjQN+Fyeyu2tkQkMM+8vbYvhEyy2UoYE0YIA3TuT/ebl92uaXkjhc9 JK+qwx1KkrcrJ+SMj/Nz0oUMPLPZ4BIslvwsCZSBEIlRC58ciR3P36UA0ZMoTkuzqwMAOzHUHCN4nMfMCHPF1/pWU+dCBO7znToyNqzEw47jUU9zXazMo5Od zN8Khfqnq4v3rwAiPrN9l0VJRadpIHV8gum2R8peCkkFjcxveDE= Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable X-Content-Filtered-By: Mailman/MimeDel 2.1.25 X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.25 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 05 Dec 2017 16:49:12 -0000 I'll post a phab review at openhack tonight. --- Sent using a tiny phone keyboard. Apologies for any typos and autocorrect. This old phone only supports top post. Apologies. Cy Schubert or The need of the many outweighs the greed of the few. --- -----Original Message----- From: Cy Schubert Sent: 05/12/2017 08:30 To: Devin Teske; Hans Petter Selasky Cc: rgrimes@freebsd.org; cem@freebsd.org; Eitan Adler; src-committers; svn-= src-all@freebsd.org; svn-src-head@freebsd.org Subject: RE: svn commit: r326554 - in head: . usr.bin/spongeusr.bin/sponge/= tests usr.bin/tee Why not update sed to create the backup file only if the suffix is given to= -i, like gnu sed does. --- Sent using a tiny phone keyboard. Apologies for any typos and autocorrect. This old phone only supports top post. Apologies. Cy Schubert or The need of the many outweighs the greed of the few. --- From: Devin Teske Sent: 05/12/2017 07:35 To: Hans Petter Selasky Cc: rgrimes@freebsd.org; cem@freebsd.org; Eitan Adler; src-committers; svn-= src-all@freebsd.org; svn-src-head@freebsd.org Subject: Re: svn commit: r326554 - in head: . usr.bin/sponge usr.bin/sponge= /tests usr.bin/tee > On Dec 5, 2017, at 5:00 AM, Hans Petter Selasky wrote: >=20 >> On 12/05/17 13:58, Rodney W. Grimes wrote: >> Further more, why does freebsd need this in base? >=20 > Hi, >=20 > I think this is useful. It could replace the "-i " (intermediate) option = for "sed" for example. It avoids creating temporary files when filtering fi= les, right? >=20 > --HPS >=20 Wth is wrong with: data=3D$( sed -e '...' somefile ) && echo "$data" > somefile or set -e data=3D... echo "$data" > ... or exec 3< ... <&3 or (I digress) Infinite variations, but the gist is that sponge looks to be trying to help= sh(1)/similar when help is unneeded. Why buffer data into memory via fork-exec-pipe to sponge when you can buffe= r to native namespace without pipe to sponge? Am I missing something? Why do we need sponge(1)? --=20 Devin=