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

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

From
Yann Dirson <ydirson@altern.org>
Date
Oct 3, 2010, 20:42 UTC
Message-ID
<1286138529-6780-1-git-send-email-ydirson@altern.org>

At last a reroll of that old series. The most prominent change is a split of the old --factorize-renames flag into --detect-dir-renames and --hide-dir-rename-details - hopefully following Junio's comments dated nearly 2 years ago. While doing this, a couple of testcases got added (as well as a list of more testcases to be written), and some minor optimizations were done, but no heavy surgery.

I hope we can turn the simple detection feature to something acceptable for inclusion - that is, once unified diff output shows the detected renames as annotations, and once the known major holes are plugged (both from FIXME and testcases). For the "hiding" one OTOH, a more precise description of what we want must probably be done before progressing, but I suggest we keep that for later.

Also, maybe the --detect-dir-renames would get a good alias as "-M -M".
Does all of that look reasonable ?
Yann Dirson (4):
  Introduce wholesame directory move detection in diffcore.
  Add testcases for the --detect-dir-renames diffcore flag.
  Allow hiding renames of individual files involved in a directory
    rename.
  Add testcases for the --hide-dir-rename-details diffcore flag.
 diff-lib.c                       |    6 +-
 diff.c                           |   12 +
 diff.h                           |    6 +
 diffcore-rename.c                |  316 +++++++++++++++++++++++++++-
 diffcore.h                       |    1 +
 t/t4046-diff-rename-factorize.sh |  433 ++++++++++++++++++++++++++++++++++++++
 tree-diff.c                      |    4 +-
 7 files changed, 766 insertions(+), 12 deletions(-)
 create mode 100755 t/t4046-diff-rename-factorize.sh
-- 
1.7.2.3
Next: Yann Dirson
Message 1 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.