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

[PATCH 3/4] submodule--helper clone: remove double path checking

From
Stefan Beller <sbeller@google.com>
Date
Mar 31, 2016, 00:17 UTC
Message-ID
<1459383457-6848-4-git-send-email-sbeller@google.com>
In-Reply-To
<1459383457-6848-1-git-send-email-sbeller@google.com>
Just a few lines after the deleted code we call
  safe_create_leading_directories_const(path + "/.git")

so the check is done twice without action in between. Remove the first check.

Signed-off-by: Stefan Beller <sbeller@google.com>
---
 builtin/submodule--helper.c | 4 ----
 1 file changed, 4 deletions(-)
diff --git a/builtin/submodule--helper.c b/builtin/submodule--helper.c
index 88002ca..914e561 100644
--- a/builtin/submodule--helper.c
+++ b/builtin/submodule--helper.c
@@ -212,11 +212,7 @@ static int module_clone(int argc, const char **argv, const char *prefix)
 	}
 
 	/* Write a .git file in the submodule to redirect to the superproject. */
-	if (safe_create_leading_directories_const(path) < 0)
-		die(_("could not create directory '%s'"), path);
-
 	strbuf_addf(&sb, "%s/.git", path);
-
 	if (safe_create_leading_directories_const(sb.buf) < 0)
 		die(_("could not create leading directories of '%s'"), sb.buf);
 	submodule_dot_git = fopen(sb.buf, "w");
-- 
2.5.0.264.g4004fdc.dirty
Previous: Eric SunshineNext: Junio C Hamano
Message 11 of 17 in “Fix relative path issues in recursive submodules.”
  1. 0/4 Fix relative path issues in recursive submodules.Stefan Beller, Mar 31, 2016
  2. 1/4 recursive submodules: test for relative pathsStefan Beller, Mar 31, 2016
  3. Junio C HamanoMar 31, 2016
  4. Stefan BellerMar 31, 2016
  5. 2/4 submodule--helper clone: simplify path checkStefan Beller, Mar 31, 2016
  6. Jacob KellerMar 31, 2016
  7. Eric SunshineMar 31, 2016
  8. Stefan BellerMar 31, 2016
  9. Junio C HamanoMar 31, 2016
  10. Eric SunshineMar 31, 2016
  11. 3/4 submodule--helper clone: remove double path checkingStefan Beller, Mar 31, 2016
  12. Junio C HamanoMar 31, 2016
  13. 4/4 submodule--helper: use relative path if possibleStefan Beller, Mar 31, 2016
  14. Junio C HamanoMar 31, 2016
  15. Stefan BellerMar 31, 2016
  16. Junio C HamanoMar 31, 2016
  17. Stefan BellerMar 31, 2016

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.