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

[PATCH 1/2] apply, entry: speak of submodules instead of subprojects

From
Thomas Rast <trast@inf.ethz.ch>
Date
Jul 16, 2013, 09:21 UTC
Message-ID
<8df0d41caa10a38e46783bebd3148a7b8445dd47.1373966389.git.trast@inf.ethz.ch>
In-Reply-To
<87bo631odi.fsf@hexa.v.cablecom.net>

There are only four (with some generous rounding) instances in the current source code where we speak of "subproject" instead of "submodule". They are as follows:

* one error message in git-apply and two in entry.c
* the patch format for submodule changes

The latter was introduced in 0478675 (Expose subprojects as special files to "git diff" machinery, 2007-04-15), apparently before the terminology was settled. We can of course not change the patch format.

Let's at least change the error messages to consistently call them "submodule".

Signed-off-by: Thomas Rast <trast@inf.ethz.ch>
---

This and the next one are message changes for things I found during my review.

 builtin/apply.c | 2 +-
 entry.c         | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/builtin/apply.c b/builtin/apply.c
index 0e9b631..f1d4cc9 100644
--- a/builtin/apply.c
+++ b/builtin/apply.c
@@ -3847,7 +3847,7 @@ static void add_index_file(const char *path, unsigned mode, void *buf, unsigned
 		const char *s = buf;
 
 		if (get_sha1_hex(s + strlen("Subproject commit "), ce->sha1))
-			die(_("corrupt patch for subproject %s"), path);
+			die(_("corrupt patch for submodule %s"), path);
 	} else {
 		if (!cached) {
 			if (lstat(path, &st) < 0)
diff --git a/entry.c b/entry.c
index d7c131d..6af4b6a 100644
--- a/entry.c
+++ b/entry.c
@@ -199,9 +199,9 @@ static int write_entry(struct cache_entry *ce, char *path, const struct checkout
 		break;
 	case S_IFGITLINK:
 		if (to_tempfile)
-			return error("cannot create temporary subproject %s", path);
+			return error("cannot create temporary submodule %s", path);
 		if (mkdir(path, 0777) < 0)
-			return error("cannot create subproject directory %s", path);
+			return error("cannot create submodule directory %s", path);
 		break;
 	default:
 		return error("unknown file mode for %s in index", path);
-- 
1.8.3.2.998.g1d087bc
Previous: Thomas RastNext: Thomas Rast
Message 13 of 35 in “Switch German translation from pure German to German+English”
  1. 0/3 Switch German translation from pure German to German+EnglishRalf Thielow, Jun 12, 2013
  2. 1/3 l10n: de.po: switch from pure German to German+English (part 1)Ralf Thielow, Jun 12, 2013
  3. 2/3 l10n: de.po: switch from pure German to German+English (part 2)Ralf Thielow, Jun 12, 2013
  4. 3/3 l10n: de.po: switch from pure German to German+English (part 3)Ralf Thielow, Jun 12, 2013
  5. 0/3 Switch German translation to G+ERalf Thielow, Jul 10, 2013
  6. 1/3 l10n: de.po: switch from pure German to German+English (part 1)Ralf Thielow, Jul 10, 2013
  7. 2/3 l10n: de.po: switch from pure German to German+English (part 2)Ralf Thielow, Jul 10, 2013
  8. 3/3 l10n: de.po: switch from pure German to German+English (part 3)Ralf Thielow, Jul 10, 2013
  9. Thomas RastJul 15, 2013
  10. Thomas RastJul 15, 2013
  11. Thomas RastJul 15, 2013
  12. Thomas RastJul 15, 2013
  13. 1/2 apply, entry: speak of submodules instead of subprojectsThomas Rast, Jul 16, 2013
  14. 2/2 show-branch: fix description of --date-orderThomas Rast, Jul 16, 2013
  15. Jonathan NiederJul 16, 2013
  16. Thomas RastJul 16, 2013
  17. Jonathan NiederJul 16, 2013
  18. Jens LehmannJul 16, 2013
  19. Thomas RastJul 16, 2013
  20. Junio C HamanoJul 17, 2013
  21. 0/3 Switch German translation to G+ERalf Thielow, Jul 17, 2013
  22. 1/3 l10n: de.po: switch from pure German to German+English (part 1)Ralf Thielow, Jul 17, 2013
  23. 2/3 l10n: de.po: switch from pure German to German+English (part 2)Ralf Thielow, Jul 17, 2013
  24. 3/3 l10n: de.po: switch from pure German to German+English (part 3)Ralf Thielow, Jul 17, 2013
  25. Thomas RastJul 17, 2013
  26. 0/3 Switch German translation to G+ERalf Thielow, Jul 17, 2013
  27. 1/3 l10n: de.po: switch from pure German to German+English (part 1)Ralf Thielow, Jul 17, 2013
  28. 2/3 l10n: de.po: switch from pure German to German+English (part 2)Ralf Thielow, Jul 17, 2013
  29. 3/3 l10n: de.po: switch from pure German to German+English (part 3)Ralf Thielow, Jul 17, 2013
  30. Ralf ThielowJul 19, 2013
  31. Thomas RastJul 18, 2013
  32. Junio C HamanoJul 18, 2013
  33. Jiang XinJul 21, 2013
  34. 1/2 apply, entry: speak of submodules instead of subprojectsThomas Rast, Jul 18, 2013
  35. 2/2 show-branch: fix description of --date-orderThomas Rast, Jul 18, 2013

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.