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

t5313-pack-bounds-checks.sh flaky?

From
Lars Schneider <larsxschneider@gmail.com>
Date
Jun 5, 2017, 10:33 UTC
Message-ID
<9A7C995F-ED4F-4AEB-B2B7-00FF05B80E84@gmail.com>
Hi Peff,

t5313.7 failed recently on master [1]. Are you aware of any flaky parts in this test?

Here is the output:
expecting success: 
	# We need two objects here, so we can plausibly require
	# an extended table (if the first object were larger than 2^31).
	do_pack "$object $(git rev-parse HEAD)" --index-version=2 &&
	# We have to make extra room for the table, so we cannot
	# just munge in place as usual.
	{
		dd if=$idx bs=1 count=$(($(ofs_table 2) + 4)) &&
		printf "\200\0\0\0" &&
		printf "\377\0\0\0\0\0\0\0" &&
		dd if=$idx bs=1 skip=$(extended_table 2)
	} >tmp &&
	mv tmp "$idx" &&
	clear_base &&
	test_must_fail git cat-file blob $object &&
	test_must_fail git index-pack --verify $pack

1084+0 records in 1084+0 records out 1084 bytes (1.1 kB) copied, 0.00119315 s, 909 kB/s 40+0 records in 40+0 records out 40 bytes (40 B) copied, 6.9825e-05 s, 573 kB/s 74 test_must_fail: command succeeded: git cat-file blob fff0a2476aa5c8e60a3ef21cfc66e0cc670920be

Cheers, Lars

[1] https://travis-ci.org/git/git/jobs/239451919
Next: Jeff King
Message 1 of 4 in “t5313-pack-bounds-checks.sh flaky?”
  1. Lars SchneiderJun 5, 2017
  2. Jeff KingJun 5, 2017
  3. t5313: make extended-table test more deterministicJeff King, Jun 5, 2017
  4. Lars SchneiderJun 6, 2017

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.