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

[PATCH 1/4] t5510: refactor bundle->pack conversion

From
Thomas Rast <trast@student.ethz.ch>
Date
Mar 1, 2012, 21:40 UTC
Message-ID
<c8e641fe6f33a4ed795da8d40f5be743a2e5c45d.1330637923.git.trast@student.ethz.ch>
In-Reply-To
<cover.1330637923.git.trast@student.ethz.ch>

It's not so much a conversion as a "strip everything up to and including the first blank line", but it will come in handy again.

Signed-off-by: Thomas Rast <trast@student.ethz.ch>
---
 t/t5510-fetch.sh |   24 ++++++++++--------------
 1 file changed, 10 insertions(+), 14 deletions(-)
diff --git a/t/t5510-fetch.sh b/t/t5510-fetch.sh
index 79ee913..6df24b2 100755
--- a/t/t5510-fetch.sh
+++ b/t/t5510-fetch.sh
@@ -14,6 +14,14 @@ test_bundle_object_count () {
 	test "$2" = $(grep '^[0-9a-f]\{40\} ' verify.out | wc -l)
 }
 
+convert_bundle_to_pack () {
+	while read x && test -n "$x"
+	do
+		:;
+	done
+	cat
+}
+
 test_expect_success setup '
 	echo >file original &&
 	git add file &&
@@ -206,13 +214,7 @@ test_expect_success 'unbundle 1' '
 
 test_expect_success 'bundle 1 has only 3 files ' '
 	cd "$D" &&
-	(
-		while read x && test -n "$x"
-		do
-			:;
-		done
-		cat
-	) <bundle1 >bundle.pack &&
+	convert_bundle_to_pack <bundle1 >bundle.pack &&
 	git index-pack bundle.pack &&
 	test_bundle_object_count bundle.pack 3
 '
@@ -229,13 +231,7 @@ test_expect_success 'bundle does not prerequisite objects' '
 	git add file2 &&
 	git commit -m add.file2 file2 &&
 	git bundle create bundle3 -1 HEAD &&
-	(
-		while read x && test -n "$x"
-		do
-			:;
-		done
-		cat
-	) <bundle3 >bundle.pack &&
+	convert_bundle_to_pack <bundle3 >bundle.pack &&
 	git index-pack bundle.pack &&
 	test_bundle_object_count bundle.pack 3
 '
-- 
1.7.9.2.467.g7fee4
Previous: Thomas RastNext: Thomas Rast
Message 2 of 11 in “Another bundle fix: reading freed memory”
  1. 0/4 Another bundle fix: reading freed memoryThomas Rast, Mar 1, 2012
  2. 1/4 t5510: refactor bundle->pack conversionThomas Rast, Mar 1, 2012
  3. 2/4 t5510: fix indent with spacesThomas Rast, Mar 1, 2012
  4. 3/4 t5510: ensure we stay in the toplevel test dirThomas Rast, Mar 1, 2012
  5. Junio C HamanoMar 1, 2012
  6. Thomas RastMar 1, 2012
  7. Junio C HamanoMar 1, 2012
  8. Junio C HamanoMar 1, 2012
  9. 4/4 bundle: keep around names passed to add_pending_object()Thomas Rast, Mar 1, 2012
  10. Junio C HamanoMar 1, 2012
  11. Thomas RastMar 1, 2012

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.