Re: [PATCH] t6023-merge-file: Work around non-portable sed usage

From: Arjen Laarhoven <arjen@yaph.org>
Date: Sep 8, 2008, 19:23 UTC
Message-ID: <20080908192313.GB4148@regex.yaph.org>
In-Reply-To: <cNVNi0DglMtk8yH2LYJQdUZ7rfXlu4pff2TkbJj4KU6hnx-n_IQ3nw@cipher.nrlssc.navy.mil>

On Mon, Sep 08, 2008 at 02:06:02PM -0500, Brandon Casey wrote: [...]

Show 11 quoted lines

> I was just encountering this myself.
> 
> sed can be fixed without the use of tr by replacing '\n' with an explicit newline like:
> 
> sed -e 's/deerit./&\
> \
> \
> \
> /' -e "s/locavit,/locavit;/" < new6.txt > new8.txt
> 
> Of course it doesn't fit on one line though.

I don't think replacing 2 lines with 10 is a big win (not counting a possible comment explaining why it's necessary). I'd rather replaced the thing with a Perl one-liner though, but that seems a bit frowned upon, correct?

Arjen

-- 
Arjen Laarhoven

The presence of those seeking the truth is infinitely to be preferred to 
those who think they've found it.
                                -- Terry Pratchett, "Monstrous Regiment"