Re: [PATCH] contrib/git-resurrect.sh: do not write \t for HT in sed scripts

From: Junio C Hamano <gitster@pobox.com>
Date: Apr 2, 2017, 16:46 UTC
Message-ID: <xmqqzifysqol.fsf@gitster.mtv.corp.google.com>
In-Reply-To: <7028b4eb-aa2b-aaf6-764d-cffab24a9968@gmail.com>

Jakub Narębski <jnareb@gmail.com> writes:

Show 15 quoted lines

> W dniu 01.04.2017 o 06:14, Junio C Hamano pisze:
>
>> Just like we did in 0d1d6e50 ("t/t7003: replace \t with literal tab
>> in sed expression", 2010-08-12), avoid writing "\t" for HT in sed
>> scripts, which is not portable.
>
> I guess it is not possible to use HT variable (similar to LF that we
> have in t/test-lib.sh, and which is defined by some scripts) set to
> literal tab
>
>   HT="	"
>
> wouldn't work there, is it?
>
> Using this variable would make literal tab character more visible.

Patches welcome.