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

[PATCH 1/8] t5004: skip SHA-1-only test in SHA-256 repository

From
Patrick Steinhardt <ps@pks.im>
Date
Oct 8, 2026, 10:01 UTC
Message-ID
<20261008-pks-ci-housekeeping-v1-1-baf015c589c0@pks.im>
In-Reply-To
<20261008-pks-ci-housekeeping-v1-0-baf015c589c0@pks.im>

One of the tests in t5004 extracts a ZIP file that contains some objects larger than 4GB and then double-checks whether we can read and archive such an object. That test has a bunch of prerequities: it requires a 64 bit `long`, unzip with 64-bit support and it only runs when EXPENSIVE is enabled. Consequently, not a lot of jobs even exercise this.

One of the jobs that does run it though our Fedora-based job, as it ticks all the necessary boxes. But that job was silently broken: while the intent was to run on Fedora with breaking changes enabled, they are in fact disabled due to a typo.

We're about to fix that typo in the next commit, but this will also uncover that the above test case is broken when running in SHA-256 repositories. The extracted objects are SHA-1 objects, so extracting them into a SHA-256 repository is not going to yield anything good. So once we fix the Fedora-based job to enable breaking changes, which will make tests use SHA-256 by default, the test will break.

Fix this issue by adding the SHA1 prerequisite.
Signed-off-by: Patrick Steinhardt <ps@pks.im>
---
 t/t5004-archive-corner-cases.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/t/t5004-archive-corner-cases.sh b/t/t5004-archive-corner-cases.sh
index 768b0ff85d..c9c879cc5f 100755
--- a/t/t5004-archive-corner-cases.sh
+++ b/t/t5004-archive-corner-cases.sh
@@ -185,7 +185,7 @@ test_expect_success EXPENSIVE,UNZIP,UNZIP_ZIP64_SUPPORT \
 	"$GIT_UNZIP" -t many-big.zip
 '
 
-test_expect_success EXPENSIVE,LONG_IS_64BIT,UNZIP,UNZIP_ZIP64_SUPPORT,ZIPINFO \
+test_expect_success EXPENSIVE,LONG_IS_64BIT,UNZIP,UNZIP_ZIP64_SUPPORT,ZIPINFO,SHA1 \
 	'zip archive with files bigger than 4GB' '
 	# Pack created with:
 	#   dd if=/dev/zero of=file bs=1M count=4100 && git hash-object -w file
-- 
2.56.0.406.ga2d225a756.dirty
Previous: Patrick SteinhardtNext: Junio C Hamano
Message 2 of 27 in “ci: some housekeeping and modernizations”
  1. 0/8 ci: some housekeeping and modernizationsPatrick Steinhardt, Oct 8, 2026
  2. 1/8 t5004: skip SHA-1-only test in SHA-256 repositoryPatrick Steinhardt, Oct 8, 2026
  3. Junio C HamanoOct 8, 2026
  4. Patrick SteinhardtOct 9, 2026
  5. 2/8 ci: fix "fedora-breaking-changes-meson" jobPatrick Steinhardt, Oct 8, 2026
  6. 3/8 ci: drop unused "linux-clang" logicPatrick Steinhardt, Oct 8, 2026
  7. 4/8 ci: switch away from unsupported i386/ubuntu imagePatrick Steinhardt, Oct 8, 2026
  8. Todd ZullingerOct 8, 2026
  9. Junio C HamanoOct 8, 2026
  10. Patrick SteinhardtOct 9, 2026
  11. 5/8 ci: rename linux-TEST-vars jobPatrick Steinhardt, Oct 8, 2026
  12. 6/8 ci: switch away from EOL'd Ubuntu version in linux-exoticPatrick Steinhardt, Oct 8, 2026
  13. 7/8 ci: drop now-dead Python 2 coveragePatrick Steinhardt, Oct 8, 2026
  14. 8/8 ci: drop redundant linux-reftable jobPatrick Steinhardt, Oct 8, 2026
  15. Junio C HamanoOct 8, 2026
  16. Patrick SteinhardtOct 9, 2026
  17. Junio C HamanoOct 9, 2026
  18. 0/8 ci: some housekeeping and modernizationsPatrick Steinhardt, Oct 9, 2026
  19. 1/8 t5004: skip SHA-1-only test in SHA-256 repositoryPatrick Steinhardt, Oct 9, 2026
  20. 2/8 ci: fix "fedora-breaking-changes-meson" jobPatrick Steinhardt, Oct 9, 2026
  21. 3/8 ci: drop unused "linux-clang" logicPatrick Steinhardt, Oct 9, 2026
  22. 4/8 ci: switch away from unsupported i386/ubuntu imagePatrick Steinhardt, Oct 9, 2026
  23. 5/8 ci: rename linux-TEST-vars jobPatrick Steinhardt, Oct 9, 2026
  24. SZEDER GáborOct 9, 2026
  25. 6/8 ci: switch away from EOL'd Ubuntu version in linux-exoticPatrick Steinhardt, Oct 9, 2026
  26. 7/8 ci: drop now-dead Python 2 coveragePatrick Steinhardt, Oct 9, 2026
  27. 8/8 ci: improve reftable test coveragePatrick Steinhardt, Oct 9, 2026

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.