Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 5 Dec 2017 09:17:24 -0800
From:      Cy Schubert <Cy.Schubert@komquats.com>
To:        Pedro Giffuni <pfg@FreeBSD.org>, 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:  <20171205171721.73E1C2D8@spqr.komquats.com>

next in thread | raw e-mail | index | archive | help
What side effects? Can you give one or two examples, please?

---
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: Pedro Giffuni
Sent: 05/12/2017 08:40
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
Subject: Re: svn commit: r326554 - in head: . usr.bin/spongeusr.bin/sponge/=
tests usr.bin/tee




On 05/12/2017 11:29, Cy Schubert wrote:




Why not update sed to create the backup file only if the suffix is given to=
 -i, like gnu sed does.



 No, no .. there have been several failed attempts at that that cause nasty=
 side effects.
 It is also a rather non-standardish thing to do.

 Pedro.




---
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 ) &&
 =EF=BF=BD=EF=BF=BD=EF=BF=BD=EF=BF=BD=EF=BF=BD=EF=BF=BD=EF=BF=BD echo "$dat=
a" > 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 hel=
p sh(1)/similar when help is unneeded.

 Why buffer data into memory via fork-exec-pipe to sponge when you can buff=
er 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?20171205171721.73E1C2D8>