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

[PATCH 3/4] fix output for deleted submodules in git diff --submodule-summary

From
Jens Lehmann <jens.lehmann@web.de>
Date
Oct 14, 2009, 18:31 UTC
Message-ID
<4AD6190C.4050804@web.de>
In-Reply-To
<4AD61880.4040600@web.de>

When a submodule has been deleted, add_submodule_odb() returns false because the directory of the submodule is gone. So we have to test the second sha for null before we call add_submodule_odb() to get the correct output.

Signed-off-by: Jens Lehmann <Jens.Lehmann@web.de>
---
 submodule.c |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/submodule.c b/submodule.c
index 5076113..d5fce7a 100644
--- a/submodule.c
+++ b/submodule.c
@@ -45,12 +45,12 @@ void show_submodule_summary(FILE *f, const char *path,
 	static const char *format = "  %m %s";
 	int fast_forward = 0, fast_backward = 0;

-	if (add_submodule_odb(path))
+	if (is_null_sha1(two))
+		message = "(submodule deleted)";
+	else if (add_submodule_odb(path))
 		message = "(not checked out)";
 	else if (is_null_sha1(one))
 		message = "(new submodule)";
-	else if (is_null_sha1(two))
-		message = "(submodule deleted)";
 	else if (!(left = lookup_commit_reference(one)) ||
 		 !(right = lookup_commit_reference(two)))
 		message = "(commits not present)";
-- 
1.6.5.4.g707c
Previous: Jens LehmannNext: Jens Lehmann
Message 9 of 14 in “What's cooking in git.git (Oct 2009, #02; Sun, 11)”
  1. Junio C HamanoOct 12, 2009
  2. Jeff KingOct 12, 2009
  3. Junio C HamanoOct 12, 2009
  4. Jeff KingOct 14, 2009
  5. Sverre RabbelierOct 12, 2009
  6. Jens LehmannOct 14, 2009
  7. 1/4 Add the --submodule-summary option to the diff option familyJens Lehmann, Oct 14, 2009
  8. 2/4 fix indentation depth for git diff --submodule-summaryJens Lehmann, Oct 14, 2009
  9. 3/4 fix output for deleted submodules in git diff --submodule-summaryJens Lehmann, Oct 14, 2009
  10. 4/4 add tests for git diff --submodule-summaryJens Lehmann, Oct 14, 2009
  11. submodule-summaryJunio C Hamano, Oct 14, 2009
  12. Jens LehmannOct 14, 2009
  13. Junio C HamanoOct 14, 2009
  14. Jens LehmannOct 15, 2009

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.