From: Pete Wyckoff <pw@padd.com>
Date: Thu, 03 Jan 2013 03:08:11 GMT
Subject: Re: [PATCH V2] t9810: Do not use sed -i
Message-ID: <20130103030811.GA30979@padd.com>
In-Reply-To: <7vhamz18y3.fsf@alter.siamese.dyndns.org>

gitster@pobox.com wrote on Wed, 02 Jan 2013 15:06 -0800:
> Pete Wyckoff <pw@padd.com> writes:
> 
> > tboegi@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 <tboegi@web.de>
> >
> > One more teensy fix is needed in this hunk, following Junio's
> > comment about redirections going at the end:
> 
> I've already pushed out the previous one on 'pu' after fixing it up
> like this.
> 
> Thanks.  Anything I missed?

I should have guessed that you would just silently fix it.
Thanks!

		-- Pete