From owner-svn-src-head@freebsd.org Tue Dec 5 17:50:35 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 5F5F4E725A6; Tue, 5 Dec 2017 17:50:35 +0000 (UTC) (envelope-from cy.schubert@komquats.com) Received: from smtp-out-no.shaw.ca (smtp-out-no.shaw.ca [64.59.134.13]) (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 00D286535F; Tue, 5 Dec 2017 17:50:34 +0000 (UTC) (envelope-from cy.schubert@komquats.com) Received: from spqr.komquats.com ([96.50.22.10]) by shaw.ca with SMTP id MHMUePEM9RDG7MHMVet0lD; Tue, 05 Dec 2017 10:50:33 -0700 X-Authority-Analysis: v=2.2 cv=b+PC2pOx c=1 sm=1 tr=0 a=jvE2nwUzI0ECrNeyr98KWA==:117 a=jvE2nwUzI0ECrNeyr98KWA==:17 a=ocR9PWop10UA:10 a=YxBL1-UpAAAA:8 a=6I5d2MoRAAAA:8 a=BWvPGDcYAAAA:8 a=ndaoGXS1AAAA:8 a=IDPPhiR9RvHgwgAsPNAA:9 a=cExSKycrkz02Yz1S:21 a=pwaUFreBph5h-PuX:21 a=QEXdDO2ut3YA:10 a=VVDKKMnT1RUTkQuetWUA:9 a=a3gq0iXbTZ7bkWuq:21 a=l1k_JcxzDZWNceyM:21 a=UPq-Zfme56oUIyW2:21 a=_W_S_7VecoQA:10 a=Ia-lj3WSrqcvXOmTRaiG:22 a=IjZwj45LgO3ly-622nXo:22 a=pxhY87DP9d2VeQe4joPk:22 a=mFeOnlTyF09QQMGr2mMI:22 Received: from [25.81.45.55] (S0106d4ca6d8943b0.gv.shawcable.net [24.68.134.59]) by spqr.komquats.com (Postfix) with ESMTPSA id 64B7135B; Tue, 5 Dec 2017 09:50:29 -0800 (PST) MIME-Version: 1.0 From: Cy Schubert Subject: RE: svn commit: r326554 - in head: . usr.bin/sponge usr.bin/sponge/tests usr.bin/tee Date: Tue, 5 Dec 2017 09:50:32 -0800 To: Devin Teske CC: Hans Petter Selasky , "rgrimes@freebsd.org" , "cem@freebsd.org" , Eitan Adler , src-committers , "svn-src-all@freebsd.org" , "svn-src-head@freebsd.org" Message-Id: <20171205175029.64B7135B@spqr.komquats.com> X-CMAE-Envelope: MS4wfNPfAajGB81cFuEtK32rMBm6u3Iohv19CR2cSVaqJRgoVLhSNwwkFTujeeGDdbQDaAk5hYhtgJq7Y9WxBMAVLrSOJ9Rx4PvNU1JJVUvYsrxDokBtzplO Yu12+cPPhTypOdC/Z+7fyVCdUDqiAPwX753pVPQBKSxQlWDPbJ9aMkj1/C51CjzkMHDyUbYR8xIwqQ27kXTSvbCkGQ8fH3WNTrqopF988R10YkkbODMDOaLE fsKNlA1QL7E/WHFNiYxbBQyzE7eqXMrJtF2qxq6R5ieoxMn7yOyVuFQS1kphA3IzNAK0yOxszj58+2Z2pIL7KavRMVVp0NbF2dX60GnsjuEuQgbJZGIa2YU3 ZemDYzHFeKsaioBYhz6Wdeh++fT5n/4ZDzRx3gs+sup5Xqnp9l8= Content-Type: text/plain; charset="utf-8" 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 17:50:35 -0000 Our sed already creates backup files "optionally" just as gsed does. Only t= he syntax is different. No need to sponge sed, just sed -i '' (whereas gsed= is -i without the argument). --- 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: Devin Teske Sent: 05/12/2017 09:23 To: Cy Schubert Cc: Hans Petter Selasky; 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 8:29 AM, Cy Schubert wrote: Why not update sed to create the backup file only if the suffix is given to= -i, like gnu sed does. I suspect that would break countless scripts that test uname to determine h= ow to use the -i flag of sed. --=20 Devin --- 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=