Re: [PATCH] add a diff-files command
- From
Junio C Hamano <junkio@cox.net>
- Date
- Apr 28, 2005, 17:16 UTC
- Message-ID
- <7vk6mm9607.fsf@assigned-by-dhcp.cox.net>
- In-Reply-To
- <Pine.LNX.4.62.0504281238130.14033@localhost.localdomain>
>>>>> "NP" == Nicolas Pitre <nico@cam.org> writes:
NP> ... And yesterday I realized that the (currently unimplemented) NP> --ignore switch to show-files, combined with the exclusion pattern list, NP> whould be more logical than teaching show-diff (which I still think is a NP> misnamer in the context of the other diff tools) about files unknown to NP> the cache. The patch to show-files is also much smaller and logical.
I agree wholeheartedly with both counts. (1) Linus and I discussed briefly about renaming show-diff to diff-files but it is on hold, waiting for a big wholesale rename. (2) the logical place for the -X and -x is "show-files --ignore".