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

[PATCH 6/6] builtin/repo: add object disk size info to structure table

From
Justin Tobler <jltobler@gmail.com>
Date
Dec 9, 2025, 22:58 UTC
Message-ID
<20251209225820.2861276-7-jltobler@gmail.com>
In-Reply-To
<20251209225820.2861276-1-jltobler@gmail.com>

Similar to a prior commit, update the table output format for the git-repo(1) structure commdn to display the total object disk usage by object type.

Since disk size may vary between platforms, tests do not validate actual values and only check that size info is printed in an empty repository.

Signed-off-by: Justin Tobler <jltobler@gmail.com>
---
 builtin/repo.c            | 13 +++++++++++++
 t/t1901-repo-structure.sh | 19 ++++++++++++++++++-
 2 files changed, 31 insertions(+), 1 deletion(-)
diff --git a/builtin/repo.c b/builtin/repo.c
index 8ea7c9b24f..8ddefd523e 100644
--- a/builtin/repo.c
+++ b/builtin/repo.c
@@ -371,6 +371,7 @@ static void stats_table_setup_structure(struct stats_table *table,
 	struct ref_stats *refs = &stats->refs;
 	size_t inflated_object_total;
 	size_t object_count_total;
+	size_t disk_object_total;
 	size_t ref_total;
 
 	ref_total = get_total_reference_count(refs);
@@ -405,6 +406,18 @@ static void stats_table_setup_structure(struct stats_table *table,
 			      "    * %s", _("Blobs"));
 	stats_table_size_addf(table, objects->inflated_sizes.tags,
 			      "    * %s", _("Tags"));
+
+	disk_object_total = get_total_object_values(&objects->disk_sizes);
+	stats_table_size_addf(table, disk_object_total,
+			      "  * %s", _("Disk size"));
+	stats_table_size_addf(table, objects->disk_sizes.commits,
+			      "    * %s", _("Commits"));
+	stats_table_size_addf(table, objects->disk_sizes.trees,
+			      "    * %s", _("Trees"));
+	stats_table_size_addf(table, objects->disk_sizes.blobs,
+			      "    * %s", _("Blobs"));
+	stats_table_size_addf(table, objects->disk_sizes.tags,
+			      "    * %s", _("Tags"));
 }
 
 static void stats_table_print_structure(const struct stats_table *table)
diff --git a/t/t1901-repo-structure.sh b/t/t1901-repo-structure.sh
index a98c651f1d..51820cc3f6 100755
--- a/t/t1901-repo-structure.sh
+++ b/t/t1901-repo-structure.sh
@@ -4,6 +4,15 @@ test_description='test git repo structure'
 
 . ./test-lib.sh
 
+strip_object_disk_usage() {
+	awk '
+		/^\|   \* Disk size/ { skip=1; next }
+		skip && /^\|     \* / { next }
+		skip && !/^\|     \* / { skip=0 }
+		{ print }
+	' $1
+}
+
 test_expect_success 'empty repository' '
 	test_when_finished "rm -rf repo" &&
 	git init repo &&
@@ -30,6 +39,11 @@ test_expect_success 'empty repository' '
 		|     * Trees          |    0 B |
 		|     * Blobs          |    0 B |
 		|     * Tags           |    0 B |
+		|   * Disk size        |    0 B |
+		|     * Commits        |    0 B |
+		|     * Trees          |    0 B |
+		|     * Blobs          |    0 B |
+		|     * Tags           |    0 B |
 		EOF
 
 		git repo structure >out 2>err &&
@@ -107,7 +121,10 @@ test_expect_success SHA1 'repository with references and objects' '
 		|     * Tags           |    132 B   |
 		EOF
 
-		git repo structure >out 2>err &&
+		git repo structure >out.raw 2>err &&
+
+		# Skip object disk sizes due to platform variance.
+		strip_object_disk_usage out.raw >out &&
 
 		test_cmp expect out &&
 		test_line_count = 0 err
-- 
2.52.0.209.ge85ae279b0
Previous: Junio C HamanoNext: Patrick Steinhardt
Message 22 of 80 in “builtin/repo: add object size info to structure output”
  1. 0/6 builtin/repo: add object size info to structure outputJustin Tobler, Dec 9, 2025
  2. 1/6 builtin/repo: group per-type object values into structJustin Tobler, Dec 9, 2025
  3. 2/6 builtin/repo: humanise count values in structure outputJustin Tobler, Dec 9, 2025
  4. Patrick SteinhardtDec 10, 2025
  5. Justin ToblerDec 10, 2025
  6. Junio C HamanoDec 11, 2025
  7. Justin ToblerDec 12, 2025
  8. 3/6 builtin/repo: add inflated object info to keyvalue structure outputJustin Tobler, Dec 9, 2025
  9. 4/6 builtin/repo: add inflated object info to structure tableJustin Tobler, Dec 9, 2025
  10. Patrick SteinhardtDec 10, 2025
  11. Justin ToblerDec 10, 2025
  12. 5/6 builtin/repo: add disk size info to keyvalue stucture outputJustin Tobler, Dec 9, 2025
  13. Patrick SteinhardtDec 10, 2025
  14. Justin ToblerDec 10, 2025
  15. Justin ToblerDec 12, 2025
  16. Patrick SteinhardtDec 15, 2025
  17. Justin ToblerDec 15, 2025
  18. Junio C HamanoDec 10, 2025
  19. Lucas Seiki OshiroDec 10, 2025
  20. Justin ToblerDec 12, 2025
  21. Junio C HamanoDec 12, 2025
  22. 6/6 builtin/repo: add object disk size info to structure tableJustin Tobler, Dec 9, 2025
  23. Patrick SteinhardtDec 10, 2025
  24. Justin ToblerDec 10, 2025
  25. 0/7 builtin/repo: add object size info to structure outputJustin Tobler, Dec 12, 2025
  26. 1/7 builtin/repo: group per-type object values into structJustin Tobler, Dec 12, 2025
  27. 2/7 strbuf: split out logic to humanise byte valuesJustin Tobler, Dec 12, 2025
  28. Patrick SteinhardtDec 15, 2025
  29. Justin ToblerDec 15, 2025
  30. Junio C HamanoDec 15, 2025
  31. Justin ToblerDec 15, 2025
  32. Jiang XinDec 16, 2025
  33. Junio C HamanoDec 16, 2025
  34. Jiang XinDec 16, 2025
  35. Justin ToblerDec 16, 2025
  36. 3/7 builtin/repo: humanise count values in structure outputJustin Tobler, Dec 12, 2025
  37. Patrick SteinhardtDec 15, 2025
  38. 4/7 builtin/repo: add inflated object info to keyvalue structure outputJustin Tobler, Dec 12, 2025
  39. Patrick SteinhardtDec 15, 2025
  40. Justin ToblerDec 15, 2025
  41. 5/7 builtin/repo: add inflated object info to structure tableJustin Tobler, Dec 12, 2025
  42. 6/7 builtin/repo: add disk size info to keyvalue stucture outputJustin Tobler, Dec 12, 2025
  43. Patrick SteinhardtDec 15, 2025
  44. 7/7 builtin/repo: add object disk size info to structure tableJustin Tobler, Dec 12, 2025
  45. 0/7 builtin/repo: add object size info to structure outputJustin Tobler, Dec 15, 2025
  46. 1/7 builtin/repo: group per-type object values into structJustin Tobler, Dec 15, 2025
  47. 2/7 strbuf: split out logic to humanise byte valuesJustin Tobler, Dec 15, 2025
  48. Junio C HamanoDec 16, 2025
  49. Justin ToblerDec 16, 2025
  50. 3/7 builtin/repo: humanise count values in structure outputJustin Tobler, Dec 15, 2025
  51. Patrick SteinhardtDec 16, 2025
  52. 4/7 builtin/repo: add inflated object info to keyvalue structure outputJustin Tobler, Dec 15, 2025
  53. 5/7 builtin/repo: add inflated object info to structure tableJustin Tobler, Dec 15, 2025
  54. 6/7 builtin/repo: add disk size info to keyvalue stucture outputJustin Tobler, Dec 15, 2025
  55. 7/7 builtin/repo: add object disk size info to structure tableJustin Tobler, Dec 15, 2025
  56. Patrick SteinhardtDec 16, 2025
  57. Justin ToblerDec 16, 2025
  58. 0/7 builtin/repo: add object size info to structure outputJustin Tobler, Dec 16, 2025
  59. 1/7 builtin/repo: group per-type object values into structJustin Tobler, Dec 16, 2025
  60. 2/7 strbuf: split out logic to humanise byte valuesJustin Tobler, Dec 16, 2025
  61. Junio C HamanoDec 16, 2025
  62. Justin ToblerDec 16, 2025
  63. 4/7 builtin/repo: add inflated object info to keyvalue structure outputJustin Tobler, Dec 16, 2025
  64. Patrick SteinhardtDec 17, 2025
  65. Justin ToblerDec 17, 2025
  66. 3/7 builtin/repo: humanise count values in structure outputJustin Tobler, Dec 16, 2025
  67. 5/7 builtin/repo: add inflated object info to structure tableJustin Tobler, Dec 16, 2025
  68. 6/7 builtin/repo: add disk size info to keyvalue stucture outputJustin Tobler, Dec 16, 2025
  69. 7/7 builtin/repo: add object disk size info to structure tableJustin Tobler, Dec 16, 2025
  70. Patrick SteinhardtDec 17, 2025
  71. Justin ToblerDec 17, 2025
  72. 0/7 builtin/repo: add object size info to structure outputJustin Tobler, Dec 17, 2025
  73. 1/7 builtin/repo: group per-type object values into structJustin Tobler, Dec 17, 2025
  74. 2/7 strbuf: split out logic to humanise byte valuesJustin Tobler, Dec 17, 2025
  75. 4/7 builtin/repo: add inflated object info to keyvalue structure outputJustin Tobler, Dec 17, 2025
  76. 3/7 builtin/repo: humanise count values in structure outputJustin Tobler, Dec 17, 2025
  77. 5/7 builtin/repo: add inflated object info to structure tableJustin Tobler, Dec 17, 2025
  78. 6/7 builtin/repo: add disk size info to keyvalue stucture outputJustin Tobler, Dec 17, 2025
  79. 7/7 builtin/repo: add object disk size info to structure tableJustin Tobler, Dec 17, 2025
  80. Patrick SteinhardtDec 18, 2025

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.