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

Re: What's cooking in git.git (Oct 2026, #06)

From
Patrick Steinhardt <ps@pks.im>
Date
Oct 21, 2026, 05:45 UTC
Message-ID
<athRc1-tSEVgBupp@pks.im>
In-Reply-To
<20261020185101.GA6367@tb-raspi4>
On Tue, Oct 20, 2026 at 08:51:01PM +0200, Torsten Bögershausen wrote:
Show 8 quoted lines
> > * ps/pack-check-bigfile-threshold (2026-10-14) 1 commit
> >  - pack-check: fix bogus corruption report with core.bigFileThreshold
> > 
> 
> That does fail here - the read-only files was not overwritten.
> Patrick, can we get this extra line ?
> 
> diff --git a/t/t1450-fsck.sh b/t/t1450-fsck.sh                                                                                        index 16cc5d5dcb..a19bb27db7 100755                                                                                                   --- a/t/t1450-fsck.sh                                                                                                                 +++ b/t/t1450-fsck.sh                                                                                                                 @@ -544,6 +544,7 @@ test_expect_success 'rev-list --verify-objects with truncated loose blob' '                                                       cd truncated-blob &&                                                                                                                  blob=$(test-tool genrandom one 5k | git hash-object -t blob -w --stdin) &&                                                            obj=.git/objects/$(test_oid_to_path $blob) &&                                                                         +               chmod +w $obj &&                                                                                                                                                                                                                                                            # Truncate the loose blob such that its header can still be                                                                           # parsed, but reading the object data fails mid-stream.                                                                                                                  
This diff is quite mangled, let me untangle it:
diff --git a/t/t1450-fsck.sh b/t/t1450-fsck.sh
index 16cc5d5dcb..a19bb27db7 100755
--- a/t/t1450-fsck.sh
+++ b/t/t1450-fsck.sh
@@ -544,6 +544,7 @@ test_expect_success 'rev-list --verify-objects with truncated loose blob' ' 
 		cd truncated-blob &&
 		blob=$(test-tool genrandom one 5k | git hash-object -t blob -w --stdin) &&
 		obj=.git/objects/$(test_oid_to_path $blob) &&
+		chmod +w $obj &&
 		# Truncate the loose blob such that its header can still be
 		# parsed, but reading the object data fails mid-stream.

Ah, right, that makes sense. Makes me wonder why none of our tests in CI
hit this error. Weird.

Junio, as this patch series is already in "next", do you want me to send
a patch on top to fix it or shall I send a new version?

Thanks for the report!

Patrick
Previous: Torsten BögershausenNext: Junio C Hamano
Message 3 of 15 in “What's cooking in git.git (Oct 2026, #06)”
  1. Junio C HamanoOct 16, 2026
  2. Torsten BögershausenOct 20, 2026
  3. Patrick SteinhardtOct 21, 2026
  4. Junio C HamanoOct 21, 2026
  5. Torsten BögershausenOct 22, 2026
  6. Patrick SteinhardtOct 22, 2026
  7. 1/1 t1450: use mv -f when overwriting a read only filetboegi@web.de, Oct 24, 2026
  8. Tuomas AholaOct 24, 2026
  9. SZEDER GáborOct 24, 2026
  10. Junio C HamanoOct 25, 2026
  11. 1/1 t1450: use mv -f when overwriting a read only filetboegi@web.de, Oct 25, 2026
  12. Tuomas AholaOct 25, 2026
  13. Torsten BögershausenOct 25, 2026
  14. Junio C HamanoOct 25, 2026
  15. 1/1 t1450: use 'mv -f' when overwriting a read-only filetboegi@web.de, Oct 25, 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.