Re: [PATCH 00/19] Add git-list-files

2014-12-02 Thread Jeff King
On Tue, Dec 02, 2014 at 06:45:52PM +0700, Duy Nguyen wrote: > > As a side note, I wonder if it would be sensible to whitelist some > > commands as porcelain, and allow aliases to override them (either > > entirely, or just to add-in some options). > > Agreed. Maybe not all porcelain (some like gi

Re: [PATCH 00/19] Add git-list-files

2014-12-02 Thread Michael J Gruber
Jeff King schrieb am 02.12.2014 um 06:42: > On Sun, Nov 30, 2014 at 03:55:48PM +0700, Nguyễn Thái Ngọc Duy wrote: > >> This is something else that's been sitting in my tree for a while now. >> It adds "git list-files", intended to be aliased as "ls" with your >> favourite display options. > > Whe

Re: [PATCH 00/19] Add git-list-files

2014-12-02 Thread Duy Nguyen
On Tue, Dec 2, 2014 at 12:42 PM, Jeff King wrote: > On Sun, Nov 30, 2014 at 03:55:48PM +0700, Nguyễn Thái Ngọc Duy wrote: > >> This is something else that's been sitting in my tree for a while now. >> It adds "git list-files", intended to be aliased as "ls" with your >> favourite display options.

Re: [PATCH 00/19] Add git-list-files

2014-12-02 Thread Duy Nguyen
On Tue, Dec 2, 2014 at 3:02 AM, Junio C Hamano wrote: > Does this contain a lot of borrowed code or something? The style > violation in the patches are unusually high, even compared with your > other series. The first one is from coreutils, but I reformatted (and trimmed) to fit Git. I recall yo

Re: [PATCH 00/19] Add git-list-files

2014-12-01 Thread Jeff King
On Sun, Nov 30, 2014 at 03:55:48PM +0700, Nguyễn Thái Ngọc Duy wrote: > This is something else that's been sitting in my tree for a while now. > It adds "git list-files", intended to be aliased as "ls" with your > favourite display options. When I read the subject, I thought "why isn't this calle

Re: [PATCH 00/19] Add git-list-files

2014-12-01 Thread Junio C Hamano
Does this contain a lot of borrowed code or something? The style violation in the patches are unusually high, even compared with your other series. I've tried to fix it up and will push out the result on 'pu' if things work OK, but this does not even have tests, so it is unlikely that it would br

[PATCH 00/19] Add git-list-files

2014-11-30 Thread Nguyễn Thái Ngọc Duy
This is something else that's been sitting in my tree for a while now. It adds "git list-files", intended to be aliased as "ls" with your favourite display options. As you can guess it's a friendlier version (and pretty close to GNU ls) of ls-files. On one hand, I think this is a nice addition. On