Date: Tue, 5 Dec 2017 09:50:32 -0800 From: Cy Schubert <Cy.Schubert@komquats.com> To: Devin Teske <devin@shxd.cx> Cc: Hans Petter Selasky <hps@selasky.org>, "rgrimes@freebsd.org" <rgrimes@freebsd.org>, "cem@freebsd.org" <cem@freebsd.org>, Eitan Adler <eadler@freebsd.org>, src-committers <src-committers@freebsd.org>, "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>, "svn-src-head@freebsd.org" <svn-src-head@freebsd.org> Subject: RE: svn commit: r326554 - in head: . usr.bin/sponge usr.bin/sponge/tests usr.bin/tee Message-ID: <20171205175029.64B7135B@spqr.komquats.com>
next in thread | raw e-mail | index | archive | help
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
<Cy.Schubert@cschubert.com> or <cy@freebsd.org>
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 <Cy.Schubert@komquats.com> 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
<Cy.Schubert@cschubert.com> or <cy@freebsd.org>
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 <hps@selasky.org> 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<<EOF
$( ... )
EOF
cat > ... <&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=
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20171205175029.64B7135B>