{"thread":{"id":"64061","subject":"What's cooking in git.git (Aug 2025, #12; Fri, 29)","startedAt":"2025-08-29T21:37:22Z","lastAt":"2025-09-02T12:39:37Z","messageCount":3,"participants":["Junio C Hamano","René Scharfe","Jeff King"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"525245","messageId":"xmqq8qj1n71s.fsf@gitster.g","threadId":"64061","inReplyTo":null,"subject":"What's cooking in git.git (Aug 2025, #12; Fri, 29)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2025-08-29T21:37:19Z","receivedAt":"2025-08-29T21:37:22Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Here are the topics that have been cooking in my tree. Commits\nprefixed with '+' are in 'next' (being in 'next' is a sign that a\ntopic is stable enough to be used and are candidate to be in a\nfuture release). Commits prefixed with '-' are only in 'seen', and\naren't considered \"accepted\" at all and may be annotated with an URL\nto a message that raises issues but they are no means exhaustive. A\ntopic without enough support may be discarded after a long period of\nno activity (of course they can be resubmit when new interests\narise).\n\nCopies of the source code to Git live in many repositories, and the\nfollowing is a list of the ones I push into or their mirrors. Some\nrepositories have only a subset of branches.\n\nWith maint, master, next, seen, todo:\n\n\tgit://git.kernel.org/pub/scm/git/git.git/\n\tgit://repo.or.cz/alt-git.git/\n\thttps://kernel.googlesource.com/pub/scm/git/git/\n\thttps://github.com/git/git/\n\thttps://gitlab.com/git-scm/git/\n\nWith all the integration branches and topics broken out:\n\n\thttps://github.com/gitster/git/\n\nEven though the preformatted documentation in HTML and man format\nare not sources, they are published in these repositories for\nconvenience (replace \"htmldocs\" with \"manpages\" for the manual\npages):\n\n\tgit://git.kernel.org/pub/scm/git/git-htmldocs.git/\n\thttps://github.com/gitster/git-htmldocs.git/\n\nRelease tarballs are available at:\n\n\thttps://www.kernel.org/pub/software/scm/git/\n\n--------------------------------------------------\n[Graduated to 'master']\n\n* ad/t1517-short-help-tests-fix (2025-08-19) 1 commit\n (merged to 'next' on 2025-08-21 at f686ad352a)\n + t/t1517: mark tests that fail with GIT_TEST_INSTALLED\n\n Test fix.\n source: <20250819074631.3303-1-adam@dinwoodie.org>\n\n\n* bc/doc-compat-object-format-not-working (2025-08-25) 1 commit\n (merged to 'next' on 2025-08-26 at 5224444f11)\n + docs: note that extensions.compatobjectformat is incomplete\n\n The compatObjectFormat extension is used to hide an incomplete\n feature that is not yet usable for any purpose other than\n developing the feature further. Document it as such to discourage\n its use by mere mortals.\n source: <20250825221101.611876-1-sandals@crustytoothpaste.net>\n\n\n* ds/doc-community-discord (2025-08-20) 1 commit\n (merged to 'next' on 2025-08-21 at 7f9aa8da4e)\n + doc: add discord to ways of getting help\n\n Discord has been added to the first contribution documentation as\n another way to ask for help.\n source: \n\n\n* je/doc-rebase (2025-08-22) 5 commits\n (merged to 'next' on 2025-08-26 at 0d9a1d75d5)\n + doc: git-rebase: update discussion of internals\n + doc: git-rebase: move --onto explanation down\n + doc: git rebase: clarify arguments syntax\n + doc: git rebase: dedup merge conflict discussion\n + doc: git-rebase: start with an example\n\n Documentation for \"git rebase\" has been updated.\n source: \n\n\n* jk/describe-blob (2025-08-18) 5 commits\n (merged to 'next' on 2025-08-21 at 671998ff24)\n + describe: pass commit to describe_commit()\n + describe: handle blob traversal with no commits\n + describe: catch unborn branch in describe_blob()\n + describe: error if blob not found\n + describe: pass oid struct by const pointer\n\n \"git describe \" misbehaves and/or crashes in some corner\n cases, which has been taught to exit with failure gracefully.\n source: <20250818205812.GA1018043@coredump.intra.peff.net>\n\n\n* jk/fetch-check-graph-objects-fix (2025-08-23) 1 commit\n (merged to 'next' on 2025-08-26 at 51a561116f)\n + fetch-pack: re-scan when double-checking graph objects\n\n Under a race against another process that is repacking the\n repository, especially a partially cloned one, \"git fetch\" may\n mistakenly think some objects we do have are missing, which has\n been corrected.\n source: <20250824050040.GA228050@coredump.intra.peff.net>\n\n\n* jk/no-clobber-dangling-symref-with-fetch (2025-08-19) 4 commits\n (merged to 'next' on 2025-08-21 at 29b96663c0)\n + refs: do not clobber dangling symrefs\n + t5510: prefer \"git -C\" to subshell for followRemoteHEAD tests\n + t5510: stop changing top-level working directory\n + t5510: make confusing config cleanup more explicit\n\n \"git fetch\" can clobber a symref that is dangling when the\n remote-tracking HEAD is set to auto update, which has been\n corrected.\n source: <20250819192004.GA1058857@coredump.intra.peff.net>\n\n\n* js/doc-gitk-history (2025-08-19) 1 commit\n (merged to 'next' on 2025-08-21 at e7e1a08f82)\n + doc/gitk: update reference to the external project\n\n Manual page for \"gitk\" is updated with the current maintainer's\n name.\n source: <249056e7-1332-4e6f-8d07-16c80fd4913e@kdbg.org>\n\n\n* js/progress-delay-fix (2025-08-25) 1 commit\n (merged to 'next' on 2025-08-26 at 4e832eb295)\n + progress: pay attention to (customized) delay time\n\n The start_delayed_progress() function in the progress eye-candy API\n did not clear its internal state, making an initial delay value\n larger than 1 second ineffective, which has been corrected.\n source: <7b848623-ce64-4679-9b5e-9d91d947b269@kdbg.org>\n\n\n* ps/reftable-libgit2-cleanup (2025-08-12) 8 commits\n (merged to 'next' on 2025-08-21 at ff82e3fa11)\n + refs/reftable: always reload stacks when creating lock\n + reftable: don't second-guess errors from flock interface\n + reftable/stack: handle outdated stacks when compacting\n + reftable/stack: allow passing flags to `reftable_stack_add()`\n + reftable/stack: fix compiler warning due to missing braces\n + reftable/stack: reorder code to avoid forward declarations\n + reftable/writer: drop Git-specific `QSORT()` macro\n + reftable/writer: fix type used for number of records\n\n Code clean-ups.\n source: <20250812-pks-reftable-fixes-for-libgit2-v3-0-cf3b2267867e@pks.im>\n\n\n* sg/line-log-merge-optim (2025-08-24) 4 commits\n (merged to 'next' on 2025-08-26 at e110ef974a)\n + line-log: simplify condition checking for merge commits\n + line-log: initialize diff queue in process_ranges_ordinary_commit()\n + line-log: get rid of the parents array in process_ranges_merge_commit()\n + line-log: avoid unnecessary tree diffs when processing merge commits\n\n \"git log -L...\" compared trees of multiple parents with the tree of the\n merge result in an unnecessarily inefficient way.\n source: <20250824190644.2573279-1-szeder.dev@gmail.com>\n\n--------------------------------------------------\n[New Topics]\n\n* ag/doc-sendmail-gmail-example-update (2025-08-26) 1 commit\n - docs: update sendmail docs to use more secure SMTP server for Gmail\n\n Doc update.\n\n Will merge to 'next'?\n source: <20250826150919.5239-1-gargaditya08@live.com>\n\n\n* da/cargo-serialize (2025-08-26) 1 commit\n - Makefile: build libgit-rs and libgit-sys serially\n\n Makefile tried to run multiple \"cargo build\" which would not work\n very well; serialize their execution to work it around.\n\n Will merge to 'next'.\n source: <20250826233525.2635432-1-davvid@gmail.com>\n\n\n* en/rust-xdiff (2025-08-22) 6 commits\n - ivec: create a vector type that is interoperable between C and Rust\n - github workflows: upload Cargo.lock\n - win+Meson: do allow linking with the Rust-built xdiff\n - github workflows: install rust\n - xdiff: introduce rust\n - doc: add a policy for using Rust\n\n Rust!\n\n Comments?\n source: \n\n\n* je/doc-checkout (2025-08-29) 5 commits\n - doc: git-checkout: clarify restoring files section\n - doc: git-checkout: deduplicate --detach explanation\n - doc: git-checkout: don't use \"reset\"\n - doc: git-checkout: clarify `git checkout `\n - doc: git-checkout: clarify intro\n\n Doc updates.\n source: \n\n\n* kn/clang-format-bitfields (2025-08-26) 1 commit\n - Documentation: note styling for bit fields\n\n CodingGuidelines now spells out how bitfields are to be written.\n\n Will merge to 'next'?\n source: <20250826121928.22317-1-karthik.188@gmail.com>\n\n\n* jk/curl-global-trace-components (2025-08-27) 1 commit\n - curl: add support for curl_global_trace() components\n\n Adjust to the way newer versions of cURL selectivel enables tracing\n options, so that our tests can continue to work.\n\n Will merge to 'next'?\n source: <20250827080702.GA3572995@coredump.intra.peff.net>\n\n\n* kh/you-still-use-whatchanged-fix (2025-08-29) 4 commits\n - BreakingChanges: remove claim about whatchanged reports\n - whatchanged: remove not-even-shorter clause\n - whatchanged: tell users the git-log(1) equivalent\n - you-still-use-that??: help the user help themselves\n\n Update \"do you still use it?\" message given by a command that is\n deeply deprecated and allow us to suggest alternatives.\n\n Will merge to 'next'?\n source: \n\n\n* ds/ls-files-lazy-unsparse (2025-08-15) 1 commit\n - ls-files: conditionally leave index sparse\n\n \"git ls-files ...\" should not necessarily have to expand\n the index fully if a sparsified directory is excluded by the\n pathspec; the code is taught to expand the index on demand to avoid\n this.\n\n Will merge to 'next'.\n source: \n\n\n* pc/range-diff-memory-limit (2025-08-29) 1 commit\n - range-diff: add configurable memory limit for cost matrix\n\n \"git range-diff\" learned a way to limit the memory consumed by\n O(N*N) cost matrix.\n\n Will merge to 'next'?\n source: \n\n\n* ne/alloc-free-and-null (2025-08-29) 1 commit\n - alloc: fix dangling pointer in alloc_state cleanup\n\n The clear_alloc_state() API function was not fully clearing the\n structure for reuse, but since nobody reuses it, replace it with a\n variant that frees the structure as well, making the callers simpler.\n\n Will merge to 'next'?\n source: \n\n--------------------------------------------------\n[Cooking]\n\n* ds/doc-ggg-pr-fork-clarify (2025-08-23) 1 commit\n (merged to 'next' on 2025-08-25 at 71aea64ee4)\n + doc: clarify which remotes can be used with GitGitGadget\n\n Update the instruction to use of GGG in the MyFirstContribution\n document to say that a GitHub PR could be made against `git/git`\n instead of `gitgitgadget/git`.\n\n Will merge to 'master'.\n cf. \n source: \n\n\n* js/doc-sending-patch-via-thunderbird (2025-08-22) 1 commit\n (merged to 'next' on 2025-08-25 at 1e63ebff8c)\n + doc/format-patch: adjust Thunderbird MUA hint to new add-on\n\n Doc update.\n\n Will merge to 'master'.\n source: <6ec34bbc-6811-41fa-aa06-1d54fd2acb0c@kdbg.org>\n\n\n* kh/doc-config-typofix (2025-08-24) 1 commit\n (merged to 'next' on 2025-08-25 at f82e0a4c9b)\n + doc: config: replace backtick with apostrophe for possessive\n\n Documentation typofix.\n\n Will merge to 'master'.\n cf. \n source: <3ec6a00e3046166c7adb593f38c4099921d8ada3.1756064760.git.code@khaugsbakk.name>\n\n\n* rs/describe-with-lazy-queue-and-khash (2025-08-24) 1 commit\n - describe: use khash in finish_depth_computation()\n\n Instead of scanning for the remaining items to see if there are\n still commits to be explored in the queue, use khash to remember\n which items are still on the queue (an unacceptable alternative is\n to reserve one object flag bits).\n\n Will merge to 'next'?\n source: <9110f085-aec0-42e9-9774-b153ece6284f@web.de>\n\n\n* lo/repo-info-step-2 (2025-08-26) 3 commits\n - repo: add the field objects.format\n - repo: add the flag -z as an alias for --format=nul\n - Merge branch 'lo/repo-info' into lo/repo-info-step-2\n\n \"repo info\" learns a short-hand option \"-z\" that is the same as\n \"--format=nul\", and learns to report the objects format used in the\n repository.\n\n Waiting for review responses.\n cf. \n source: <20250826183205.19566-1-lucasseikioshiro@gmail.com>\n\n\n* jc/doc-includeif-hasconfig-remote-url-fix (2025-08-21) 1 commit\n - config: document includeIf conditions consistently\n\n Doc mark-up fix.\n\n Will merge to 'next'?\n source: \n\n\n* jk/add-i-color (2025-08-21) 4 commits\n - contrib/diff-highlight: mention interactive.diffFilter\n - add-interactive: manually fall back color config to color.ui\n - add-interactive: respect color.diff for diff coloring\n - stash: pass --no-color to diff-tree child processes\n\n Some among \"git add -p\" and friends ignored color.diff and/or\n color.ui configuration variables, which is an old regression, which\n has been corrected.\n\n Will merge to 'next'?\n source: <20250821070740.GA3356411@coredump.intra.peff.net>\n\n\n* jt/de-global-bulk-checkin (2025-08-22) 4 commits\n - bulk-checkin: use repository variable from transaction\n - bulk-checkin: require transaction for index_blob_bulk_checkin()\n - bulk-checkin: remove global transaction state\n - bulk-checkin: introduce object database transaction structure\n\n The bulk-checkin code used to depend on a file-scope static\n singleton variable, which has been updated to pass an instance\n throughout the callchain.\n\n Will merge to 'next'?\n source: <20250822213500.1488064-1-jltobler@gmail.com>\n\n\n* kh/doc-interpret-trailers-markup-fix (2025-08-22) 1 commit\n (merged to 'next' on 2025-08-25 at 9bee54a64d)\n + doc: interpret-trailers: close all pairs of single quotes\n\n Fix missing single-quote pairs in a documentation page.\n\n Will merge to 'master'.\n source: <4eac944102a846695a9f61ead39a5a86361a0532.1755875970.git.code@khaugsbakk.name>\n\n\n* ar/submodule-gitdir-tweak (2025-08-18) 10 commits\n . fixup! t: add gitdir encoding tests\n . t: add gitdir encoding tests\n . t: move nested gitdir tests to proper location\n . submodule: remove validate_submodule_git_dir()\n . submodule: encode gitdir paths to avoid conflicts\n . strbuf: bring back is_rfc3986_unreserved\n . t: submodules: add basic mixed gitdir path tests\n . submodule: add gitdir path config override\n . submodule: create new gitdirs under submodules path\n . submodule--helper: use submodule_name_to_gitdir in add_submodule\n\n Avoid local submodule repository directory paths overlapping with\n each other by encoding submodule names before using them as path\n components.\n\n Expecting a reroll.\n cf. <87sehk7r66.fsf@collabora.com>\n source: <20250816213642.3517822-1-adrian.ratiu@collabora.com>\n\n\n* je/doc-add (2025-08-19) 2 commits\n (merged to 'next' on 2025-08-25 at 0c84501ed2)\n + doc: git-add: simplify discussion of ignored files\n + doc: git-add: clarify intro & add an example\n\n Documentation for \"git add\" has been updated.\n\n Will merge to 'master'.\n source: \n\n\n* ds/path-walk-repack-fix (2025-08-25) 2 commits\n - path-walk: create initializer for path lists\n - path-walk: fix setup of pending objects\n\n \"git repack --path-walk\" lost objects in some corner cases, which\n has been corrected.\n\n Will merge to 'next'.\n cf. \n source: \n\n\n* sg/line-log-boundary-fixes (2025-08-18) 2 commits\n - line-log: show all line ranges touched by the same diff range\n - line-log: fix assertion error\n\n Fix for a corner case bug in \"git log -L...\".\n\n Will merge to 'next'?\n source: <20250818111310.1283932-1-szeder.dev@gmail.com>\n\n\n* ja/asciidoc-doctor-verbatim-fixes (2025-08-20) 1 commit\n (merged to 'next' on 2025-08-22 at 61fb953314)\n + doc: fix asciidoc format compatibility in pretty-formats.adoc\n\n Doc mark-up fix.\n\n Will merge to 'master'.\n source: <20250820212319.41044-1-jn.avila@free.fr>\n\n\n* jc/longer-disambiguation-fix (2025-08-14) 1 commit\n - abbrev: allow extending beyond 32 chars to disambiguate\n\n \"git rev-parse --short\" and friends failed to disambiguate two\n objects with object names that share common prefix longer than 32\n characters.\n\n Will merge to 'next'?\n source: \n\n\n* tc/t0450-harden (2025-08-12) 3 commits\n - fixup! t0450: add allowlist for builtins with missing .adoc\n - t0450: add allowlist for builtins with missing .adoc\n - t0450: fix test for out-of-tree builds\n\n Test updates.\n\n Expecting a reroll.\n cf. \n source: <20250804073002.1586332-1-toon@iotcl.com>\n\n\n* lc/rebase-trailer (2025-08-03) 2 commits\n - rebase: support --trailer\n - trailer: append trailers in-process and drop the fork to `interpret-trailers`\n\n Expecting a reroll.\n cf. <198826af571.62b85cb31711042.2415806544948206668@linux.beauty>\n cf. \n source: <20250803150059.402017-1-me@linux.beauty>\n\n\n* ps/object-store-midx-dedup-info (2025-08-11) 11 commits\n - midx: compute paths via their source\n - midx: stop duplicating info redundant with its owning source\n - midx: write multi-pack indices via their source\n - midx: load multi-pack indices via their source\n - midx: drop redundant `struct repository` parameter\n - odb: simplify calling `link_alt_odb_entry()`\n - odb: return newly created in-memory sources\n - odb: consistently use \"dir\" to refer to alternate's directory\n - odb: allow `odb_find_source()` to fail\n - odb: store locality in object database sources\n - Merge branch 'ps/object-store-midx' into ps/object-store-midx-dedup-info\n\n Further code clean-up for multi-pack-index code paths.\n\n Will merge to 'next'?\n cf. \n cf. \n source: <20250811-b4-pks-midx-deduplicate-source-info-v3-0-e442bdf2b4ad@pks.im>\n\n\n* am/xdiff-hash-tweak (2025-07-28) 2 commits\n - xdiff: optimize xdl_hash_record_verbatim\n - xdiff: refactor xdl_hash_record()\n\n Inspired by Ezekiel's recent effort to showcase Rust interface, the\n hash function implementation used to hash lines have been updated\n to the one used for ELF symbol lookup by Glibc.\n\n Will merge to 'next'.\n cf. \n source: <20250728190520.10962-1-amonakov@ispras.ru>\n\n\n* ag/send-email-imap-sent (2025-08-11) 2 commits\n - send-email: enable copying emails to an IMAP folder without actually sending them\n - send-email: add ability to send a copy of sent emails to an IMAP folder\n\n \"git send-email\" learned to drive \"git imap-send\" to store already\n sent e-mails in an IMAP folder.\n\n Will merge to 'next'?\n source: \n\n\n* ds/sparse-checkout-clean (2025-07-16) 9 commits\n - sparse-checkout: make 'clean' clear more files\n - t: expand tests around sparse merges and clean\n - sparse-index: point users to new 'clean' action\n - sparse-checkout: add --verbose option to 'clean'\n - dir: add generic \"walk all files\" helper\n - sparse-checkout: match some 'clean' behavior\n - sparse-checkout: add basics of 'clean' command\n - sparse-checkout: remove use of the_repository\n - Merge branch 'ac/deglobal-sparse-variables' into ds/sparse-checkout-clean\n (this branch uses ac/deglobal-sparse-variables.)\n\n \"git sparse-checkout\" subcommand learned a new \"clean\" action to\n prune otherwise unused working-tree files that are outside the\n areas of interest.\n\n Needs to wait for the base topic to solidify.\n Waiting for review responses.\n cf. \n source: \n\n\n* pw/3.0-commentchar-auto-deprecation (2025-08-26) 4 commits\n - commit: print advice when core.commentString=auto\n - config: warn on core.commentString=auto\n - breaking-changes: deprecate support for core.commentString=auto\n - Merge branch 'ps/config-wo-the-repository' into pw/3.0-commentchar-auto-deprecation\n\n Proposes to deprecate \"core.commentChar=auto\" that attempts to\n dynamically pick a suitable comment character, as it is too much\n trouble to support for little benefit.\n\n Will merge to 'next'?\n source: \n\n\n* tc/last-modified (2025-08-05) 3 commits\n - last-modified: use Bloom filters when available\n - t/perf: add last-modified perf script\n - last-modified: new subcommand to show when files were last modified\n\n A new command \"git last-modified\" is proposed to show the closest\n ancestor commit that touched each path.\n\n Will merge to 'next'.\n source: <20250730175510.987383-1-toon@iotcl.com>\n\n\n* ac/deglobal-sparse-variables (2025-07-18) 3 commits\n - environment: remove the global variable 'sparse_expect_files_outside_of_patterns'\n - environment: move access to \"core.sparsecheckoutcone\" into repo_settings\n - environment: move access to \"core.sparsecheckout\" into repo_settings\n (this branch is used by ds/sparse-checkout-clean.)\n\n Two global variables related to sparse checkout have been moved to\n the repository settings structure.\n\n Expecting a reroll.\n cf. \n source: \n\n\n* tb/prepare-midx-pack-cleanup (2025-05-28) 5 commits\n - midx: return a `packed_git` pointer from `prepare_midx_pack()`\n - midx-write.c: extract inner loop from fill_packs_from_midx()\n - midx-write.c: guard against incremental MIDXs in want_included_pack()\n - midx: access pack names through `nth_midxed_pack_name()`\n - Merge branch 'ps/midx-negative-packfile-cache' into tb/prepare-midx-pack-cleanup\n\n Improvement on Multi-pack-index API.\n\n Expecting a reroll.\n cf. <20250530065034.GC1321283@coredump.intra.peff.net>\n source: \n\n\n* cc/promisor-remote-capability (2025-07-31) 5 commits\n - promisor-remote: use string constants for 'name' and 'url' too\n - promisor-remote: allow a client to check fields\n - promisor-remote: refactor how we parse advertised fields\n - promisor-remote: allow a server to advertise more fields\n - promisor-remote: refactor to get rid of 'struct strvec'\n\n The \"promisor-remote\" capability mechanism has been updated to\n allow the \"partialCloneFilter\" settings and the \"token\" value to be\n communicated from the server side.\n\n Expecting a reroll.\n cf. \n source: <20250731072401.3817074-1-christian.couder@gmail.com>\n"},{"id":"525263","messageId":"fdee7ee5-21ef-4c5e-92e7-eb92fe07251e@web.de","threadId":"64061","inReplyTo":"xmqq8qj1n71s.fsf@gitster.g","subject":"Re: What's cooking in git.git (Aug 2025, #12; Fri, 29)","fromName":"René Scharfe","fromEmail":"l.s.r@web.de","sentAt":"2025-08-31T17:29:30Z","receivedAt":"2025-08-31T17:29:35Z","isPatch":false,"sender":{"key":"l.s.r@web.de","avatar":"https://avatars.githubusercontent.com/u/26122331?v=4"},"body":"On 8/29/25 11:37 PM, Junio C Hamano wrote:\n> * rs/describe-with-lazy-queue-and-khash (2025-08-24) 1 commit\n> - describe: use khash in finish_depth_computation()\n> \n> Instead of scanning for the remaining items to see if there are\n> still commits to be explored in the queue, use khash to remember\n> which items are still on the queue (an unacceptable alternative is\n> to reserve one object flag bits).\n> \n> Will merge to 'next'?\n> source: <9110f085-aec0-42e9-9774-b153ece6284f@web.de>\nPlease keep it in seen for now -- Peff's proposal to use an oidset\ninstead looks promising. I'll do some more measurements over the next\nfew days and report back.\n\nRené\n\n"},{"id":"525339","messageId":"20250902123936.GB711442@coredump.intra.peff.net","threadId":"64061","inReplyTo":"fdee7ee5-21ef-4c5e-92e7-eb92fe07251e@web.de","subject":"Re: What's cooking in git.git (Aug 2025, #12; Fri, 29)","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2025-09-02T12:39:36Z","receivedAt":"2025-09-02T12:39:37Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Sun, Aug 31, 2025 at 07:29:30PM +0200, René Scharfe wrote:\n\n> On 8/29/25 11:37 PM, Junio C Hamano wrote:\n> > * rs/describe-with-lazy-queue-and-khash (2025-08-24) 1 commit\n> > - describe: use khash in finish_depth_computation()\n> > \n> > Instead of scanning for the remaining items to see if there are\n> > still commits to be explored in the queue, use khash to remember\n> > which items are still on the queue (an unacceptable alternative is\n> > to reserve one object flag bits).\n> > \n> > Will merge to 'next'?\n> > source: <9110f085-aec0-42e9-9774-b153ece6284f@web.de>\n> Please keep it in seen for now -- Peff's proposal to use an oidset\n> instead looks promising. I'll do some more measurements over the next\n> few days and report back.\n\nThanks for digging into it further. I am not terribly opposed to what\nyou sent earlier, and most of my responses were veering pretty far\noff-topic. But if we can get away with using oidset here for the same\nperformance, I think the simplicity is worth it.\n\n-Peff\n"}]}