Re: [PATCH] fixup-builtins: remove unused cruft

2013-07-03 Thread Ramkumar Ramachandra
Junio C Hamano wrote: It is not nobody is working on, but it has already been used to help the transision and served its purpose, and is no longer is very useful for follow-up work because the majority of remaining matches it finds are false positives. Please amend the commit message

Re: [PATCH] fixup-builtins: remove unused cruft

2013-06-28 Thread Jeff King
On Fri, Jun 28, 2013 at 09:16:19PM +0530, Ramkumar Ramachandra wrote: The fixup-builtins script is only used by an unused remove-dashes target in the Makefile: remove that along with the script. I am not sure of this justification. If you read the commit message from 36e5e70, which introduced

Re: [PATCH] fixup-builtins: remove unused cruft

2013-06-28 Thread Ramkumar Ramachandra
Jeff King wrote: This script was added in 36e5e70 (Start deprecating git-command in favor of git command, 2007-06-30) with the intent of aiding the transition away from dashed forms. However, nobody is really working on that transition, and even if they did, this tool will probably

Re: [PATCH] fixup-builtins: remove unused cruft

2013-06-28 Thread Junio C Hamano
Jeff King p...@peff.net writes: So I think it is probably a good idea to remove it, but the justification is not this is unused cruft, but more like: This script was added in 36e5e70 (Start deprecating git-command in favor of git command, 2007-06-30) with the intent of aiding the