Re: [PATCH V2] t9810: Do not use sed -i

2013-01-02 Thread Pete Wyckoff
gits...@pobox.com wrote on Wed, 02 Jan 2013 15:06 -0800: > Pete Wyckoff writes: > > > tbo...@web.de wrote on Wed, 02 Jan 2013 00:20 +0100: > >> sed -i is not portable on all systems. > >> Use sed with different input and output files. > >> Utilize a tmp file whenever needed > >> > >> Added missi

Re: [PATCH V2] t9810: Do not use sed -i

2013-01-02 Thread Junio C Hamano
Pete Wyckoff writes: > tbo...@web.de wrote on Wed, 02 Jan 2013 00:20 +0100: >> sed -i is not portable on all systems. >> Use sed with different input and output files. >> Utilize a tmp file whenever needed >> >> Added missing && at 2 places >> >> Signed-off-by: Torsten Bögershausen > > One mor

Re: [PATCH V2] t9810: Do not use sed -i

2013-01-02 Thread Pete Wyckoff
tbo...@web.de wrote on Wed, 02 Jan 2013 00:20 +0100: > sed -i is not portable on all systems. > Use sed with different input and output files. > Utilize a tmp file whenever needed > > Added missing && at 2 places > > Signed-off-by: Torsten Bögershausen One more teensy fix is needed in this hunk