git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH] Add -O<orderfile> option to diff-* brothers.

From
Junio C Hamano <junkio@cox.net>
Date
May 28, 2005, 16:14 UTC
Message-ID
<7vu0knuy36.fsf@assigned-by-dhcp.cox.net>
In-Reply-To
<20050528111238.GA1036@pasky.ji.cz>
>>>>> "PB" == Petr Baudis <pasky@ucw.cz> writes:

PB> Hmm, is this really useful in practice? diff itself doesn't appear to PB> have it either, and I haven't seen something like this before. So is it PB> worth the code?

CVS nor SVN does not do pickaxe as far as I know either, but I think some people in GIT community are already having fun with pickaxe.

The amount of the code is small, and it does not impact if you do not use the option. I am more concerned by (1) the fact that each new diffcore filters would carve out its own niche from the scarse command option namespace; (2) you have to duplicate the option parsing in all three diff-* brothers; and (3) diff-tree is sometimes an odd-man-out among those three and it is not so simple to unify the option parsing. A help in that area is very much appreciated.

Previous: Junio C HamanoNext: Junio C Hamano
Message 9 of 13 in “Add -O<orderfile> option to diff-* brothers.”
  1. Add -O<orderfile> option to diff-* brothers.Junio C Hamano, May 28, 2005
  2. Thomas GlanzmannMay 28, 2005
  3. Junio C HamanoMay 28, 2005
  4. Thomas GlanzmannMay 28, 2005
  5. Junio C HamanoMay 30, 2005
  6. Petr BaudisMay 28, 2005
  7. Thomas GlanzmannMay 28, 2005
  8. Junio C HamanoMay 28, 2005
  9. Junio C HamanoMay 28, 2005
  10. Fix leak in diffcore-order.cJunio C Hamano, May 28, 2005
  11. Linus TorvaldsMay 29, 2005
  12. Junio C HamanoMay 29, 2005
  13. Junio C HamanoMay 30, 2005

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.