Re: [PATCH 0/5] some shell portability fixes

* Junio C Hamano wrote on Wed, Nov 07, 2007 at 09:42:41PM CET:

"Nguyen Thai Ngoc Duy" [off-list ref] writes:

quoted

Argh, should have made it clear, busybox sed is good enough.

Thanks.  And you can also happy grok Ralf's rewritten construct,
right?

That is, existing

        $ sed -e 's/foo/\n/' file

will be rewritten by the patch [2/5] to

        $ sed -e 's/foo/\
        /' file

The original was something like
  sed 's/[|]/\n/g'

Using instead
  tr '|' '\n'

should work for the original construct, and AFAIK only /usr/ucb/tr on
Solaris fails to understand \n correctly.  Would that be better for you?
Or even
  tr '|' '\012'

which fails only on EBCDIC, which I don't think git targets.

I'll resend the patches tonight.

Cheers,
Ralf