Re: [PATCH 00/11] nd/icase updates
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Jun 23, 2016, 20:32 UTC
- Message-ID
- <xmqqpor7hchx.fsf@gitster.mtv.corp.google.com>
- In-Reply-To
- <20160623162907.23295-1-pclouds@gmail.com>
Nguyễn Thái Ngọc Duy <pclouds@gmail.com> writes:
Show 5 quoted lines
> This update drops 1/12, which is an unnecessary change, and changes a > couple of echo/printf to test_write_lines. One of those echo uses > backlashes and causes problems with Debian dash. > > Interdiff therefore is not really interesting
Neither Debian dash (that understands XSI echo) nor other shells (that do not do XSI echo) was at fault, though. It was "one of thoese echo was written in an unportable way with backslashes" ;-)
In any case, I re-read the whole thing, and except for a small nit I sent separately, all looked sensible.
Thanks, will replace what has been queued on 'pu'.