What's cooking (2024/08 #04)

Junio C Hamano committed Aug 12, 2024 at 15:35 UTC de4670cb38eda48a77b3ba249a5d54d491039d22
1 file changed +141 -173
whats-cooking.txt
+141 -173
@@ -1,10 +1,10 @@
1 To: git@vger.kernel.org
2 -Subject: What's cooking in git.git (Aug 2024, #03; Thu, 8)
2 +Subject: What's cooking in git.git (Aug 2024, #04; Mon, 12)
3 X-master-at: 25673b1c476756ec0587fb0596ab3c22b96dc52a
4 -X-next-at: 6999bdac580e8ef7d27526f50530785bce6e1b4d
4 +X-next-at: ee80dbc4f68248653629bdd1361376c366b51623
5 Bcc: lwn@lwn.net, gitster@pobox.com
6
7 -What's cooking in git.git (Aug 2024, #03; Thu, 8)
7 +What's cooking in git.git (Aug 2024, #04; Mon, 12)
8 --------------------------------------------------
9
10 Here are the topics that have been cooking in my tree. Commits
@@ -46,97 +46,71 @@ Release tarballs are available at:
46 https://www.kernel.org/pub/software/scm/git/
47
48 --------------------------------------------------
49 -[Graduated to 'master']
50 -
51 -* dd/notes-empty-no-edit-by-default (2024-07-29) 1 commit
52 - (merged to 'next' on 2024-08-01 at fe75c53260)
53 - + notes: do not trigger editor when adding an empty note
54 -
55 - "git notes add -m '' --allow-empty" and friends that take prepared
56 - data to create notes should not invoke an editor, but it started
57 - doing so since Git 2.42, which has been corrected.
58 - source: <20240729151639.19192-2-ddiss@suse.de>
59 -
49 +[New Topics]
50
61 -* dh/encoding-trace-optim (2024-07-31) 1 commit
62 - (merged to 'next' on 2024-08-01 at c5e023eeaf)
63 - + convert: return early when not tracing
51 +* jc/tests-no-useless-tee (2024-08-08) 1 commit
52 + (merged to 'next' on 2024-08-12 at 188e777e16)
53 + + tests: drop use of 'tee' that hides exit status
54
65 - An expensive operation to prepare tracing was done in re-encoding
66 - code path even when the tracing was not requested, which has been
67 - corrected.
68 - source: <pull.1744.v6.git.git.1722432839473.gitgitgadget@gmail.com>
55 + Test fixes.
56
57 + Will merge to 'master'.
58 + source: <xmqq4j7uhfvm.fsf@gitster.g>
59
71 -* es/shell-check-updates (2024-07-26) 5 commits
72 - (merged to 'next' on 2024-08-01 at cf06aced6d)
73 - + check-non-portable-shell: improve `VAR=val shell-func` detection
74 - + check-non-portable-shell: suggest alternative for `VAR=val shell-func`
75 - + check-non-portable-shell: loosen one-shot assignment error message
76 - + t4034: fix use of one-shot variable assignment with shell function
77 - + t3430: drop unnecessary one-shot "VAR=val shell-func" invocation
60
79 - Test script linter has been updated to catch an attempt to use
80 - one-shot export construct "VAR=VAL func" for shell functions (which
81 - does not work for some shells) better.
82 - source: <20240727053509.34339-1-ericsunshine@charter.net>
61 +* js/libgit-rust (2024-08-09) 5 commits
62 + - cgit: add higher-level cgit crate
63 + - config: add git_configset_alloc() and git_configset_clear_and_free()
64 + - cgit-sys: add repo initialization and config access
65 + - cgit-sys: introduce Rust wrapper for libgit.a
66 + - common-main: split init and exit code into new files
67
68 + An rust binding to libgit.a functions has been introduced.
69 + source: <cover.1723242556.git.steadmon@google.com>
70
85 -* ks/unit-test-comment-typofix (2024-07-29) 1 commit
86 - (merged to 'next' on 2024-08-01 at 35c44187b1)
87 - + unit-tests/test-lib: fix typo in check_pointer_eq() description
71
89 - Typofix.
90 - source: <20240729043303.3486-1-five231003@gmail.com>
72 +* xx/diff-tree-remerge-diff-fix (2024-08-09) 1 commit
73 + (merged to 'next' on 2024-08-09 at cabe67c0d1)
74 + + diff-tree: fix crash when used with --remerge-diff
75 + (this branch is used by jc/range-diff-lazy-setup.)
76
77 + "git rev-list ... | git diff-tree -p --remerge-diff --stdin" should
78 + behave more or less like "git log -p --remerge-diff" but instead it
79 + crashed, forgetting to prepare a temporary object store needed.
80
93 -* ps/doc-more-c-coding-guidelines (2024-07-30) 6 commits
94 - (merged to 'next' on 2024-08-01 at 8f8ac5d6ca)
95 - + Documentation: consistently use spaces inside initializers
96 - + Documentation: document idiomatic function names
97 - + Documentation: document naming schema for structs and their functions
98 - + Documentation: clarify indentation style for C preprocessor directives
99 - + clang-format: fix indentation width for preprocessor directives
100 - + Merge branch 'kn/ci-clang-format' into ps/doc-more-c-coding-guidelines
81 + Will merge to 'master'.
82 + source: <pull.1771.v2.git.1723188292498.gitgitgadget@gmail.com>
83
102 - Some project conventions have been added to CodingGuidelines.
103 - source: <cover.1722323818.git.ps@pks.im>
84
85 +* jc/grammo-fixes (2024-08-09) 2 commits
86 + - doc: grammofix in git-diff-tree
87 + - tutorial: grammofix
88
106 -* ps/p4-tests-updates (2024-07-31) 3 commits
107 - (merged to 'next' on 2024-08-01 at e559c4bf1a)
108 - + t98xx: mark Perforce tests as memory-leak free
109 - + ci: update Perforce version to r23.2
110 - + t98xx: fix Perforce tests with p4d r23 and newer
89 + source: <xmqqed6xehzv.fsf@gitster.g>
90
112 - Perforce tests have been updated.
113 - cf. <na5mwletzpnacietbc7pzqcgb622mvrwgrkjgjosysz3gvjcso@gzxxi7d7icr7>
114 - source: <cover.1722421911.git.ps@pks.im>
91
92 +* jc/range-diff-lazy-setup (2024-08-09) 2 commits
93 + - remerge-diff: clean up temporary objdir at a central place
94 + - remerge-diff: lazily prepare temporary objdir on demand
95 + (this branch uses xx/diff-tree-remerge-diff-fix.)
96
117 -* rj/add-p-pager (2024-07-25) 4 commits
118 - (merged to 'next' on 2024-08-01 at 6f66ace52d)
119 - + add-patch: render hunks through the pager
120 - + pager: introduce wait_for_pager
121 - + pager: do not close fd 2 unnecessarily
122 - + add-patch: test for 'p' command
97 + source: <pull.1771.git.1723123250958.gitgitgadget@gmail.com>
98
124 - A 'P' command to "git add -p" that passes the patch hunk to the
125 - pager has been added.
126 - source: <76936fb1-446d-455f-b4e7-6e24dda3c17d@gmail.com>
99 +--------------------------------------------------
100 +[Stalled]
101
102 +* pp/add-parse-range-unit-test (2024-05-27) 1 commit
103 + - apply: add unit tests for parse_range
104
129 -* rs/grep-omit-blank-lines-after-function-at-eof (2024-07-30) 1 commit
130 - (merged to 'next' on 2024-08-01 at 0c6e2b21f9)
131 - + grep: -W: skip trailing empty lines at EOF, too
105 + A unit test for code that parses the hunk offset and length from a
106 + patch fragment header as been added.
107
133 - "git grep -W" omits blank lines that follow the found function at
134 - the end of the file, just like it omits blank lines before the next
135 - function.
136 - source: <8b90e0ec-251f-46b3-8777-96efd58e227b@web.de>
108 + Expecting a reroll.
109 + cf. <b7eca313-9ea8-4132-ba1d-ed9236e07095@gmail.com>
110 + source: <pull.1677.v2.git.git.1716710073910.gitgitgadget@gmail.com>
111
112 --------------------------------------------------
139 -[New Topics]
113 +[Cooking]
114
115 * jc/t3206-test-when-finished-fix (2024-08-06) 1 commit
116 (merged to 'next' on 2024-08-08 at 46461db3a2)
@@ -192,15 +166,14 @@ Release tarballs are available at:
166 source: <cover.1723121979.git.ps@pks.im>
167
168
195 -* cp/unit-test-reftable-readwrite (2024-08-07) 5 commits
196 - - t-reftable-readwrite: add tests for print functions
169 +* cp/unit-test-reftable-readwrite (2024-08-09) 4 commits
170 - t-reftable-readwrite: add test for known error
171 - t-reftable-readwrite: use 'for' in place of infinite 'while' loops
172 - t-reftable-readwrite: use free_names() instead of a for loop
173 - t: move reftable/readwrite_test.c to the unit testing framework
174
175 Needs review.
203 - source: <20240807141608.4524-1-chandrapratap3519@gmail.com>
176 + source: <20240809111312.4401-1-chandrapratap3519@gmail.com>
177
178
179 * ps/config-wo-the-repository (2024-08-07) 21 commits
@@ -236,18 +209,19 @@ Release tarballs are available at:
209
210
211 * ps/submodule-ref-format (2024-08-08) 8 commits
239 - - object: fix leaking packfiles when closing object store
240 - - submodule: fix leaking seen submodule names
241 - - submodule: fix leaking fetch tasks
242 - - builtin/submodule: allow "add" to use different ref storage format
243 - - refs: fix ref storage format for submodule ref stores
244 - - builtin/clone: propagate ref storage format to submodules
245 - - builtin/submodule: allow cloning with different ref storage format
246 - - git-submodule.sh: break overly long command lines
212 + (merged to 'next' on 2024-08-09 at 2b17964809)
213 + + object: fix leaking packfiles when closing object store
214 + + submodule: fix leaking seen submodule names
215 + + submodule: fix leaking fetch tasks
216 + + builtin/submodule: allow "add" to use different ref storage format
217 + + refs: fix ref storage format for submodule ref stores
218 + + builtin/clone: propagate ref storage format to submodules
219 + + builtin/submodule: allow cloning with different ref storage format
220 + + git-submodule.sh: break overly long command lines
221
222 Support to specify ref backend for submodules has been enhanced.
223
250 - Will merge to 'next'.
224 + Will merge to 'master'.
225 source: <cover.1723102259.git.ps@pks.im>
226
227
@@ -263,47 +237,34 @@ Release tarballs are available at:
237
238
239 * ps/transport-leakfix-test-updates (2024-08-08) 1 commit
266 - - transport: mark more tests leak-free
240 + (merged to 'next' on 2024-08-12 at ee80dbc4f6)
241 + + transport: mark more tests leak-free
242 (this branch uses jc/transport-leakfix.)
243
244 Test updates.
245
271 - On hold.
246 + Will merge to 'master'.
247 source: <ZrRPD0ggZapZym7E@tanuki>
248
274 ---------------------------------------------------
275 -[Stalled]
276 -
277 -* pp/add-parse-range-unit-test (2024-05-27) 1 commit
278 - - apply: add unit tests for parse_range
279 -
280 - A unit test for code that parses the hunk offset and length from a
281 - patch fragment header as been added.
282 -
283 - Expecting a reroll.
284 - cf. <b7eca313-9ea8-4132-ba1d-ed9236e07095@gmail.com>
285 - source: <pull.1677.v2.git.git.1716710073910.gitgitgadget@gmail.com>
286 -
287 ---------------------------------------------------
288 -[Cooking]
249
250 * ag/t7004-modernize (2024-08-08) 8 commits
291 - - t7004: make use of write_script
292 - - t7004: use single quotes instead of double quotes
293 - - t7004: begin the test body on the same line as test_expect_success
294 - - t7004: description on the same line as test_expect_success
295 - - t7004: do not prepare things outside test_expect_success
296 - - t7004: use indented here-doc
297 - - t7004: one command per line
298 - - t7004: remove space after redirect operators
251 + (merged to 'next' on 2024-08-09 at 7bce577df4)
252 + + t7004: make use of write_script
253 + + t7004: use single quotes instead of double quotes
254 + + t7004: begin the test body on the same line as test_expect_success
255 + + t7004: description on the same line as test_expect_success
256 + + t7004: do not prepare things outside test_expect_success
257 + + t7004: use indented here-doc
258 + + t7004: one command per line
259 + + t7004: remove space after redirect operators
260
261 Coding style fixes to a test script.
262
302 - Will merge to 'next'.
263 + Will merge to 'master'.
264 source: <20240808163302.17521-1-abdobngad@gmail.com>
265
266
306 -* ds/for-each-ref-is-base (2024-08-02) 3 commits
267 +* ds/for-each-ref-is-base (2024-08-12) 3 commits
268 - p1500: add is-base performance tests
269 - for-each-ref: add 'is-base' token
270 - commit-reach: add get_branch_base_for_tip
@@ -312,8 +273,8 @@ Release tarballs are available at:
273 that the history leading to a given commit "%(is-base:<commit>)" is
274 likely based on.
275
315 - Expecting a reroll.
316 - source: <pull.1768.git.1722550226.gitgitgadget@gmail.com>
276 + Comments?
277 + source: <pull.1768.v2.git.1723397687.gitgitgadget@gmail.com>
278
279
280 * jk/apply-patch-mode-check-fix (2024-08-05) 1 commit
@@ -393,20 +354,21 @@ Release tarballs are available at:
354
355
356 * sj/ref-fsck (2024-08-08) 9 commits
396 - - fsck: add ref name check for files backend
397 - - files-backend: add unified interface for refs scanning
398 - - builtin/refs: add verify subcommand
399 - - refs: set up ref consistency check infrastructure
400 - - fsck: add refs report function
401 - - fsck: add a unified interface for reporting fsck messages
402 - - fsck: make "fsck_error" callback generic
403 - - fsck: rename objects-related fsck error functions
404 - - fsck: rename "skiplist" to "skip_oids"
357 + (merged to 'next' on 2024-08-09 at 3bde10da94)
358 + + fsck: add ref name check for files backend
359 + + files-backend: add unified interface for refs scanning
360 + + builtin/refs: add verify subcommand
361 + + refs: set up ref consistency check infrastructure
362 + + fsck: add refs report function
363 + + fsck: add a unified interface for reporting fsck messages
364 + + fsck: make "fsck_error" callback generic
365 + + fsck: rename objects-related fsck error functions
366 + + fsck: rename "skiplist" to "skip_oids"
367
368 "git fsck" infrastructure has been taught to also check the sanity
369 of the ref database, in addition to the object database.
370
409 - Will merge to 'next'.
371 + Will merge to 'master'.
372 source: <ZrSqMmD-quQ18a9F@ArchLinux.localdomain>
373
374
@@ -454,21 +416,22 @@ Release tarballs are available at:
416
417
418 * ps/reftable-stack-compaction (2024-08-08) 9 commits
457 - - reftable/stack: handle locked tables during auto-compaction
458 - - reftable/stack: fix corruption on concurrent compaction
459 - - reftable/stack: use lock_file when adding table to "tables.list"
460 - - reftable/stack: do not die when fsyncing lock file files
461 - - reftable/stack: simplify tracking of table locks
462 - - reftable/stack: update stats on failed full compaction
463 - - reftable/stack: test compaction with already-locked tables
464 - - reftable/stack: extract function to setup stack with N tables
465 - - reftable/stack: refactor function to gather table sizes
419 + (merged to 'next' on 2024-08-09 at d7875bf14b)
420 + + reftable/stack: handle locked tables during auto-compaction
421 + + reftable/stack: fix corruption on concurrent compaction
422 + + reftable/stack: use lock_file when adding table to "tables.list"
423 + + reftable/stack: do not die when fsyncing lock file files
424 + + reftable/stack: simplify tracking of table locks
425 + + reftable/stack: update stats on failed full compaction
426 + + reftable/stack: test compaction with already-locked tables
427 + + reftable/stack: extract function to setup stack with N tables
428 + + reftable/stack: refactor function to gather table sizes
429
430 The code paths to compact multiple reftable files have been updated
431 to correctly deal with multiple compaction triggering at the same
432 time.
433
471 - Will merge to 'next'.
434 + Will merge to 'master'.
435 source: <cover.1723123606.git.ps@pks.im>
436
437
@@ -485,17 +448,18 @@ Release tarballs are available at:
448 source: <pull.1767.v5.git.1722576007398.gitgitgadget@gmail.com>
449
450
488 -* jc/refs-symref-referent (2024-08-07) 3 commits
489 - - ref-filter: populate symref from iterator
490 - - refs: add referent to each_ref_fn
491 - - refs: keep track of unresolved reference value in iterators
451 +* jc/refs-symref-referent (2024-08-09) 3 commits
452 + (merged to 'next' on 2024-08-09 at 3183f3d05b)
453 + + ref-filter: populate symref from iterator
454 + + refs: add referent to each_ref_fn
455 + + refs: keep track of unresolved reference value in iterators
456
457 The refs API has been taught to give symref target information to
458 the users of ref iterators, allowing for-each-ref and friends to
459 avoid an extra ref_resolve_* API call per a symbolic ref.
460
497 - Waiting for review response.
498 - source: <pull.1712.v3.git.git.1723059768.gitgitgadget@gmail.com>
461 + Will merge to 'master'.
462 + source: <pull.1712.v4.git.git.1723217871.gitgitgadget@gmail.com>
463
464
465 * jk/osxkeychain-username-is-nul-terminated (2024-08-01) 1 commit
@@ -550,33 +514,34 @@ Release tarballs are available at:
514
515
516 * tb/incremental-midx-part-1 (2024-08-06) 19 commits
553 - - midx: implement support for writing incremental MIDX chains
554 - - t/t5313-pack-bounds-checks.sh: prepare for sub-directories
555 - - t: retire 'GIT_TEST_MULTI_PACK_INDEX_WRITE_BITMAP'
556 - - midx: implement verification support for incremental MIDXs
557 - - midx: support reading incremental MIDX chains
558 - - midx: teach `midx_fanout_add_midx_fanout()` about incremental MIDXs
559 - - midx: teach `midx_preferred_pack()` about incremental MIDXs
560 - - midx: teach `midx_contains_pack()` about incremental MIDXs
561 - - midx: remove unused `midx_locate_pack()`
562 - - midx: teach `fill_midx_entry()` about incremental MIDXs
563 - - midx: teach `nth_midxed_offset()` about incremental MIDXs
564 - - midx: teach `bsearch_midx()` about incremental MIDXs
565 - - midx: introduce `bsearch_one_midx()`
566 - - midx: teach `nth_bitmapped_pack()` about incremental MIDXs
567 - - midx: teach `nth_midxed_object_oid()` about incremental MIDXs
568 - - midx: teach `prepare_midx_pack()` about incremental MIDXs
569 - - midx: teach `nth_midxed_pack_int_id()` about incremental MIDXs
570 - - midx: add new fields for incremental MIDX chains
571 - - Documentation: describe incremental MIDX format
517 + (merged to 'next' on 2024-08-12 at 92ec55fd49)
518 + + midx: implement support for writing incremental MIDX chains
519 + + t/t5313-pack-bounds-checks.sh: prepare for sub-directories
520 + + t: retire 'GIT_TEST_MULTI_PACK_INDEX_WRITE_BITMAP'
521 + + midx: implement verification support for incremental MIDXs
522 + + midx: support reading incremental MIDX chains
523 + + midx: teach `midx_fanout_add_midx_fanout()` about incremental MIDXs
524 + + midx: teach `midx_preferred_pack()` about incremental MIDXs
525 + + midx: teach `midx_contains_pack()` about incremental MIDXs
526 + + midx: remove unused `midx_locate_pack()`
527 + + midx: teach `fill_midx_entry()` about incremental MIDXs
528 + + midx: teach `nth_midxed_offset()` about incremental MIDXs
529 + + midx: teach `bsearch_midx()` about incremental MIDXs
530 + + midx: introduce `bsearch_one_midx()`
531 + + midx: teach `nth_bitmapped_pack()` about incremental MIDXs
532 + + midx: teach `nth_midxed_object_oid()` about incremental MIDXs
533 + + midx: teach `prepare_midx_pack()` about incremental MIDXs
534 + + midx: teach `nth_midxed_pack_int_id()` about incremental MIDXs
535 + + midx: add new fields for incremental MIDX chains
536 + + Documentation: describe incremental MIDX format
537
538 Incremental updates of multi-pack index files.
539
575 - Will merge to 'next'?
540 + Will merge to 'master'.
541 source: <cover.1722958595.git.me@ttaylorr.com>
542
543
579 -* ja/doc-synopsis-markup (2024-07-24) 3 commits
544 +* ja/doc-synopsis-markup (2024-08-12) 3 commits
545 - doc: apply synopsis simplification on git-clone and git-init
546 - doc: update the guidelines to reflect the current formatting rules
547 - doc: introduce a synopsis custom paragraph attribute
@@ -585,8 +550,8 @@ Release tarballs are available at:
550 been revamped. The sources, at least for the simple cases, got
551 vastly pleasant to work with.
552
588 - Inviting further discussion.
589 - source: <pull.1766.v2.git.1721855179.gitgitgadget@gmail.com>
553 + Comments?
554 + source: <pull.1766.v3.git.1723389612.gitgitgadget@gmail.com>
555
556
557 * tc/fetch-bundle-uri (2024-07-24) 3 commits
@@ -748,39 +713,42 @@ Release tarballs are available at:
713
714
715 * es/doc-platform-support-policy (2024-08-02) 1 commit
751 - - Documentation: add platform support policy
716 + (merged to 'next' on 2024-08-09 at 110c94ea82)
717 + + Documentation: add platform support policy
718
719 A policy document that describes platform support levels and
720 expectation on platform stakeholders has been introduced.
721
756 - Will merge to 'next'?
722 + Will merge to 'master'.
723 source: <20240802221948.2367124-1-emilyshaffer@google.com>
724
725
726 * gt/unit-test-hashmap (2024-08-06) 1 commit
761 - - t: port helper/test-hashmap.c to unit-tests/t-hashmap.c
727 + (merged to 'next' on 2024-08-09 at e62febfb4e)
728 + + t: port helper/test-hashmap.c to unit-tests/t-hashmap.c
729
730 An existing test of hashmap API has been rewritten with the
731 unit-test framework.
732
766 - Getting there?
733 + Will merge to 'master'.
734 cf. <CAP8UFD04uQbWhY0QDp+nC8VyBvpUs=tBDm=w4CzKK2GcwFZdGQ@mail.gmail.com>
735 source: <20240803133517.73308-2-shyamthakkar001@gmail.com>
736
737
738 * rs/unit-tests-test-run (2024-07-30) 6 commits
772 - - t-strvec: use if_test
773 - - t-reftable-basics: use if_test
774 - - t-ctype: use if_test
775 - - unit-tests: add if_test
776 - - unit-tests: show location of checks outside of tests
777 - - t0080: use here-doc test body
739 + (merged to 'next' on 2024-08-12 at 9df4336877)
740 + + t-strvec: use if_test
741 + + t-reftable-basics: use if_test
742 + + t-ctype: use if_test
743 + + unit-tests: add if_test
744 + + unit-tests: show location of checks outside of tests
745 + + t0080: use here-doc test body
746
747 Unit-test framework has learned a simple control structure to allow
748 embedding test statements in-line instead of having to create a new
749 function to contain them.
750
783 - Will merge to 'next'?
751 + Will merge to 'master'.
752 source: <077a178e-eb30-45ff-b653-a514bfd33077@web.de>
753
754