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

[PATCH 0/1] Be nicer to the user on tracked/untracked merge conflicts

From
JJonathan <git.jonathan.bressat@gmail.com>
Date
Apr 12, 2022, 19:15 UTC
Message-ID
<20220412191556.21135-1-Jonathan.bressat@etu.univ-lyon1.fr>
In-Reply-To
<CAA0Qn1sBF=PAduCQCXbYkeu4cphw7O+AnvwFNMWijuKYskaT8g@mail.gmail.com>

When doing a merge while there is untracked files with the same name as merged files, git refuses to proceed. This patch make git overwrite files if their content are the same.

We added a statement to check_ok_to_remove() (unpack-trees.c) with ie_modified() (read-cache.c) to test if the untracked file has the same content as the merged one. It seems to work well with all three o->result, o->dst_index and o->src_index, We are not sure of what is the usage of those three, did we used it properly?

Our tests need some improvement, for example using test_commit, and testing more possibilities, it's not a real patch, just to comfirm if we are on the right track.

The next idea is when it's a fastforward, attempt to merge the untracked file and the upstream version (like if the file had just been committed, but without introducing an extra commit).

you can see this idea here: https://git.wiki.kernel.org/index.php/SmallProjectsIdeas#Be_nicer_to_the_user_on_tracked.2Funtracked_merge_conflicts

Questions: The old behaviour was here for technical reasons? The new behavior that we introduce here become the default one? If the old behavior was important for some people or for some reasons, we can set a global variable to switch between the old and the new one. And if we define a global variable, should we print a warning to let users know that there is a new behavior when a merge is called and that he can switch between the old and new one. For some reason, test_commit make the merge not working like if it's the old behaviour of merge, I dont understand why ?

Jonathan (1):
  Merge with untracked file that are the same without failure and test
 t/t7615-merge-untracked.sh | 79 ++++++++++++++++++++++++++++++++++++++
 unpack-trees.c             |  4 ++
 2 files changed, 83 insertions(+)
 create mode 100755 t/t7615-merge-untracked.sh
-- 
2.35.1.7.gc8609858e0.dirty
Previous: Guillaume CogoniNext: Jonathan
Message 2 of 27 in “[WIP]: make merge nicer to the user”
  1. Guillaume CogoniMar 27, 2022
  2. 0/1 Be nicer to the user on tracked/untracked merge conflictsJonathan, Apr 12, 2022
  3. 1/1 Merge with untracked file that are the same without failure and testJonathan, Apr 12, 2022
  4. Ævar Arnfjörð BjarmasonApr 12, 2022
  5. Junio C HamanoApr 13, 2022
  6. 0/2 Be nicer to the user on tracked/untracked merge conflictsJonathan, Apr 25, 2022
  7. 1/2 t7615: test how merge behave when there is untracked fileJonathan, Apr 25, 2022
  8. 2/2 merge with untracked file that are the same without failureJonathan, Apr 25, 2022
  9. Junio C HamanoApr 25, 2022
  10. Guillaume CogoniApr 25, 2022
  11. Junio C HamanoApr 25, 2022
  12. Ævar Arnfjörð BjarmasonApr 12, 2022
  13. Jonathan BressatApr 14, 2022
  14. Matthieu MoyApr 26, 2022
  15. Junio C HamanoApr 26, 2022
  16. Jonathan BressatApr 28, 2022
  17. 0/4 Be nicer to the user on tracked/untracked merge conflictsJonathan Bressat, May 27, 2022
  18. 1/4 t6436: tests how merge behave when there is untracked file with the same contentJonathan Bressat, May 27, 2022
  19. 2/4 merge with untracked file that are the same without failureJonathan Bressat, May 27, 2022
  20. 3/4 add configuration variable corresponding to --overwrite-same-contentJonathan Bressat, May 27, 2022
  21. 4/4 error message now advice to use the new optionJonathan Bressat, May 27, 2022
  22. Matthieu MoyApr 26, 2022
  23. Matthieu MoyJun 4, 2022
  24. Guillaume CogoniJun 10, 2022
  25. Matthieu MoyJun 4, 2022
  26. Matthieu MoyJun 4, 2022
  27. Matthieu MoyJun 4, 2022

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.