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

Re: [RFC PATCH v4 0/4] Detection of directory renames

From
YDYann Dirson <ydirson@free.fr>
Date
Oct 6, 2010, 23:17 UTC
Message-ID
<20101006231734.GZ4983@home.lan>
In-Reply-To
<20101005014208.GE9994@burratino>
On Mon, Oct 04, 2010 at 08:42:08PM -0500, Jonathan Nieder wrote:
Show 5 quoted lines
> >From my point of view, I think it would be best to start with
> the smallest usable piece, which is the raw format.  It probably
> makes the most sense to error out when -u and --detect-dir-renames are
> used together.  Then unified diff could be reenabled in a separate
> patch series on top of this one.

Well, there's probably not much to do - in the same order of work than adding the check to error out :)

> Another nice feature might be to let the directory-move
> detection feed back into file-move detection to make it more
> accurate.  Have you thought about this?  Would it be feasible,
> and if so, would it be useful?

That was suggested by Junio at that time. It would surely be useful in some cases, but it's not clear to me how frequently it would. And I'd suspect that quite some design changes would be necessary: that would be a 2-way influence, in that a nearly-complete bulkmove could bump the score of the latest rename that would make it a full bulkmove. Probably not trivial to do right.

Previous: Jonathan Nieder
Message 25 of 25 in “Detection of directory renames”
  1. 0/4 Detection of directory renamesYann Dirson, Oct 3, 2010
  2. 1/4 Introduce wholesame directory move detection in diffcore.Yann Dirson, Oct 3, 2010
  3. 2/4 Add testcases for the --detect-dir-renames diffcore flag.Yann Dirson, Oct 3, 2010
  4. 3/4 Allow hiding renames of individual files involved in a directory rename.Yann Dirson, Oct 3, 2010
  5. 4/4 Add testcases for the --hide-dir-rename-details diffcore flag.Yann Dirson, Oct 3, 2010
  6. Sverre RabbelierOct 3, 2010
  7. Sverre RabbelierOct 3, 2010
  8. Junio C HamanoOct 3, 2010
  9. Sverre RabbelierOct 4, 2010
  10. Yann DirsonOct 4, 2010
  11. Ævar Arnfjörð BjarmasonOct 4, 2010
  12. Yann DirsonOct 4, 2010
  13. Jonathan NiederOct 4, 2010
  14. Yann DirsonOct 4, 2010
  15. Jonathan NiederOct 4, 2010
  16. Andreas EricssonOct 5, 2010
  17. Ævar Arnfjörð BjarmasonOct 4, 2010
  18. Yann DirsonOct 4, 2010
  19. Jonathan NiederOct 4, 2010
  20. Yann DirsonOct 4, 2010
  21. Jonathan NiederOct 5, 2010
  22. Yann DirsonOct 6, 2010
  23. Jonathan NiederOct 4, 2010
  24. Jonathan NiederOct 5, 2010
  25. Yann DirsonOct 6, 2010

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.