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

From: Junio C Hamano <[email-masked]>
Date: Wed, 30 Dec 2015 13:45:42 -0800

Ramsay Jones <[email-masked]> writes:

>> +	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.