Re: [PATCH v8] ls-files: Add eol diagnostics

From: Junio C Hamano <gitster@pobox.com>
Date: Dec 30, 2015, 21:45 UTC
Message-ID: <xmqqsi2jd309.fsf@gitster.mtv.corp.google.com>
In-Reply-To: <5683E8B4.7040005@ramsayjones.plus.com>

Ramsay Jones <ramsay@ramsayjones.plus.com> writes:

Show 17 quoted lines

>> +	Show line endings ("eolinfo") and the text/eol attributes ("texteolattr") of
>> files.
>> +	"eolinfo" is the file content identification used by Git when
>> +	the "text" attribute is "auto", or core.autocrlf != false.
>> ++
>> +"eolinfo" is either "" (when the the info is not available"), or one of "binary",
>> +"text-no-eol", "text-lf", "text-crlf" or "text-crlf-lf".
>
> <bikeshedding>
>
> Again, I think this list reads better as: binary, none, lf, crlf and mixed.
>
> If you prefer to have 'text' in there somewhere, how about:
>
>    binary, text-none, text-lf, text-crlf, text-mixed.
>
> :-D

Yup, these shorter ones, especially without "text-" prefix, read easier to humans; scripts won't mind either way, so I'd prefer that shorter version.

Thanks for reminding the thread.