Re: [PATCH next v2] git-notes: add test case for multi-line notes

From: Boyd Stephen Smith Jr. <bss@iguanasuicide.net>
Date: Jan 14, 2009, 17:09 UTC
Message-ID: <200901141109.56580.bss@iguanasuicide.net>
In-Reply-To: <20090114165633.GC15758@coredump.intra.peff.net>

On Wednesday 2009 January 14 10:56:33 Jeff King wrote:

Show 6 quoted lines

>On Wed, Jan 14, 2009 at 05:28:11PM +0100, Tor Arne Vestbø wrote:
>> +MSG=${MSG//%/}
>> +printf "$MSG" > "$1"
>> +printf "$MSG" >& 2
>
>Substitution parameter expansion is a bash-ism, IIRC. How about just

MSG=$(printf '%s\n' "$MSG" | sed -e 's/%/%%/g') printf "$MSG" > "$1" printf "$MSG" >& 2

Is my best attempt at portable and "safe".  It's a few extra processes though.

>  printf %s "$MSG" ?

On my box $ printf '%s\n' '\n' \n $

He wants '\n' in $MSG to be expanded, and what you gave doesn't do that.

-- 
Boyd Stephen Smith Jr.                     ,= ,-_-. =. 
bss@iguanasuicide.net                     ((_/)o o(\_))
ICQ: 514984 YM/AIM: DaTwinkDaddy           `-'(. .)`-' 
http://iguanasuicide.net/                      \_/