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

2015-11-24 Thread Ramsay Jones
On 23/11/15 17:05, Torsten Bögershausen wrote: > On 22.11.15 09:20, Sebastian Schuberth wrote: >> On 21.11.2015 08:36, Torsten Bögershausen wrote: >> >>> git ls-files --eol gives an output like this: >>> >>> i/text-no-eol w/text-no-eol attr/text=auto t/t5100/empty >> >> I'm sorry if this has

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

2015-11-23 Thread Eric Sunshine
On Mon, Nov 23, 2015 at 2:45 PM, Sebastian Schuberth wrote: > On Mon, Nov 23, 2015 at 6:05 PM, Torsten Bögershausen wrote: >> git ls-files -o --eol >> i/ w/binaryattr/ zlib.o > > I see, somewhat convincing I have to agree. > > On another note, how about making the pr

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

2015-11-23 Thread Sebastian Schuberth
On Mon, Nov 23, 2015 at 6:05 PM, Torsten Bögershausen wrote: > or, as another example: > git ls-files -o --eol > i/ w/binaryattr/ zlib.o I see, somewhat convincing I have to agree. On another note, how about making the prefix either all just one letter (i.e. "attr/

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

2015-11-23 Thread Eric Sunshine
On Sat, Nov 21, 2015 at 2:36 AM, Torsten Bögershausen wrote: > When working in a cross-platform environment, a user wants to > check if text files are stored normalized in the repository and if > .gitattributes are set appropriately. > > Make it possible to let Git show the line endings in the ind

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

2015-11-23 Thread Torsten Bögershausen
On 22.11.15 09:20, Sebastian Schuberth wrote: > On 21.11.2015 08:36, Torsten Bögershausen wrote: > >> git ls-files --eol gives an output like this: >> >> i/text-no-eol w/text-no-eol attr/text=auto t/t5100/empty > > I'm sorry if this has been discussed before, but hav you considered to use a

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

2015-11-22 Thread Sebastian Schuberth
On 21.11.2015 08:36, Torsten Bögershausen wrote: > git ls-files --eol gives an output like this: > > i/text-no-eol w/text-no-eol attr/text=auto t/t5100/empty I'm sorry if this has been discussed before, but hav you considered to use a header line and omit the prefixed from the columns inste