[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Quilt-dev] [PATCH] Substitute sed with configured path
From: |
Kent R. Spillner |
Subject: |
Re: [Quilt-dev] [PATCH] Substitute sed with configured path |
Date: |
Wed, 25 Dec 2013 09:21:47 -0600 |
User-agent: |
Mutt/1.5.22 (2013-10-16) |
On Tue, Dec 24, 2013 at 07:35:34PM +0100, Martin Quinson wrote:
> Hello,
>
> thanks for this patch.
>
> I have one question, out of curiosity. Why do you set SED to @SED@ and
> then call $SET. Wouldn't it be more efficient to directly call @SED@ ?
I did it to save one byte each time it's used in the source... j/k ;-)
I did it because I thought that was the preferred idiom based on how
PATCH is substituted in bin/patch-wrapper.in. I don't mind changing it
if you prefer (and see my forthcoming reply to Andreas).