Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 5 Dec 2017 08:49:09 -0800
From:      Cy Schubert <Cy.Schubert@komquats.com>
To:        Cy Schubert <Cy.Schubert@komquats.com>, Devin Teske <devin@shxd.cx>,  Hans Petter Selasky <hps@selasky.org>
Cc:        "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/spongeusr.bin/sponge/tests usr.bin/tee
Message-ID:  <20171205164906.361ED247@spqr.komquats.com>

next in thread | raw e-mail | index | archive | help
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
<Cy.Schubert@cschubert.com> or <cy@freebsd.org>
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
<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?20171205164906.361ED247>