Re: [PATCH] check-ignore: --non-matching without --verbose

From: Junio C Hamano <gitster@pobox.com>
Date: May 24, 2022, 19:18 UTC
Message-ID: <xmqqwneahh8t.fsf@gitster.g>
In-Reply-To: <hl3lllnkPVswb8IZoaMCS62N42epy97_8rLH8qrvSGCbKoovP5_qjtq4nrot5-h-q_7KOu-JpPX4nC0f74BXm6uWdtxosa6z5niHqAJKRUw=@protonmail.com>

Carl Smedstad <carl.smedstad@protonmail.com> writes:

Show 14 quoted lines

>> Not commenting on the patch itself, but ...
>>
>> > find . |
>> > git check-ignore --verbose --non-matching |
>> > grep $'^::\t' |
>> > sed -e 's/.*\t//'
>>
>> ... feeding grep output to sed caught my eyes. Isn't that equivalent to
>>
>> git ... |
>> sed -n '/^::\t/s/.*\t//p'
>
> That is indeed equivalent. TIL about regular-expression addresses in
> sed, thanks!

I however suspect that these "\t"'s should be spelled out as an actual TAB in the final production version for portability.

Thanks.