{"thread":{"id":"59969","subject":"What's cooking in git.git (Jul 2023, #02; Mon, 10)","startedAt":"2023-07-11T01:05:12Z","lastAt":"2023-07-11T17:37:41Z","messageCount":2,"participants":["Junio C Hamano","Taylor Blau"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"479371","messageId":"xmqqmt032r7m.fsf@gitster.g","threadId":"59969","inReplyTo":null,"subject":"What's cooking in git.git (Jul 2023, #02; Mon, 10)","fromName":"Junio C Hamano","fromEmail":"junio@pobox.com","sentAt":"2023-07-11T01:05:01Z","receivedAt":"2023-07-11T01:05:12Z","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-vcs/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[New Topics]\n\n* ah/advise-force-pushing (2023-07-07) 1 commit\n - push: advise about force-pushing as an alternative to reconciliation\n\n Help newbies by suggesting that there are cases where force-pushing\n is a valid and sensible thing to update a branch at a remote\n repository, rather than reconciling with merge/rebase.\n\n Will merge to 'next'?\n Perhaps we should go ahead with only the \"git push\" half (queued here)\n without touching \"git checkout\" and \"git status\" half (not queued here).\n source: <20230707054257.3366355-3-alexhenrie24@gmail.com>\n\n\n* jc/pathspec-match-with-common-prefix (2023-07-08) 2 commits\n (merged to 'next' on 2023-07-10 at 4146f391ed)\n + dir: match \"attr\" pathspec magic with correct paths\n + t6135: attr magic with path pattern\n\n \"git ls-files '(attr:X)D/'\" that triggers the common prefix\n optimization codepath failed to read from \"D/.gitattributes\",\n which has been corrected.\n\n Will merge to 'master'.\n source: <20230707220457.3655121-1-gitster@pobox.com>\n\n\n* jc/tree-walk-drop-base-offset (2023-07-07) 2 commits\n - tree-walk: drop unused base_offset from do_match()\n - tree-walk: lose base_offset that is never used in tree_entry_interesting\n\n Code simplification.\n\n Needs review.\n source: <20230707222116.4129415-1-gitster@pobox.com>\n\n\n* rs/ls-tree-prefix-simplify (2023-07-07) 1 commit\n (merged to 'next' on 2023-07-08 at 3e4826525f)\n + ls-tree: simplify prefix handling\n\n Code simplification.\n\n Will merge to 'master'.\n source: \n\n\n* rs/pretty-format-double-negation-fix (2023-07-07) 1 commit\n (merged to 'next' on 2023-07-08 at 6674b562ff)\n + pretty: avoid double negative in format_commit_item()\n\n Code clarification.\n\n Will merge to 'master'.\n source: <8d4fcad1-5f71-4790-17ee-e9ba0a3118dc@web.de>\n\n\n* rs/userformat-find-requirements-simplify (2023-07-07) 1 commit\n (merged to 'next' on 2023-07-08 at 6cdc328a08)\n + pretty: use strchr(3) in userformat_find_requirements()\n\n Code simplification.\n\n Will merge to 'master'.\n source: <108ab1a2-28f4-dd62-1f90-64f37708aa8c@web.de>\n\n\n* mh/doc-credential-helpers (2023-07-10) 1 commit\n - doc: gitcredentials: link to helper list\n\n Doc update.\n\n Will merge to 'next'.\n source: \n\n\n* tb/fsck-no-progress (2023-07-10) 6 commits\n - commit-graph.c: avoid duplicated progress output during `verify`\n - commit-graph.c: pass progress to `verify_one_commit_graph()`\n - commit-graph.c: iteratively verify commit-graph chains\n - commit-graph.c: extract `verify_one_commit_graph()`\n - fsck: suppress MIDX output with `--no-progress`\n - fsck: suppress commit-graph output with `--no-progress`\n\n \"git fsck --no-progress\" still spewed noise from the commit-graph\n subsystem, which has been corrected.\n\n Will merge to 'next'.\n source: \n\n\n* tb/repack-cleanup (2023-07-10) 2 commits\n - builtin/repack.c: avoid dir traversal in `collect_pack_filenames()`\n - builtin/repack.c: only repack `.pack`s that exist\n\n The recent change to \"git repack\" made it react less nicely when a\n leftover .idx file that no longer has the corresponding .pack file\n in the repository, which has been corrected.\n\n Will merge to 'next'.\n source: \n\n--------------------------------------------------\n[Graduated to 'master']\n\n* cw/strbuf-cleanup (2023-06-12) 7 commits\n (merged to 'next' on 2023-06-28 at c158b004a8)\n + strbuf: remove global variable\n + path: move related function to path\n + object-name: move related functions to object-name\n + credential-store: move related functions to credential-store file\n + abspath: move related functions to abspath\n + strbuf: clarify dependency\n + strbuf: clarify API boundary\n\n Move functions that are not about pure string manipulation out of\n strbuf.[ch]\n source: <20230606194720.2053551-1-calvinwan@google.com>\n\n\n* gc/config-context (2023-06-28) 11 commits\n (merged to 'next' on 2023-06-28 at 38632f3daf)\n + config: pass source to config_parser_event_fn_t\n + config: add kvi.path, use it to evaluate includes\n + config.c: remove config_reader from configsets\n + config: pass kvi to die_bad_number()\n + trace2: plumb config kvi\n + config.c: pass ctx with CLI config\n + config: pass ctx with config files\n + config.c: pass ctx in configsets\n + config: add ctx arg to config_fn_t\n + urlmatch.h: use config_fn_t type\n + config: inline git_color_default_config\n\n Reduce reliance on a global state in the config reading API.\n source: \n\n\n* gc/config-partial-submodule-kvi-fix (2023-06-26) 1 commit\n (merged to 'next' on 2023-06-28 at 1fb30a34e3)\n + config: don't BUG when both kvi and source are set\n\n Partially revert a sanity check that the rest of the config code\n was not ready, to avoid triggering it in a corner case.\n source: \n\n\n* jc/doc-hash-object-types (2023-06-28) 1 commit\n (merged to 'next' on 2023-06-28 at adff1700c2)\n + docs: add git hash-object -t option's possible values\n\n Doc update.\n source: \n\n\n* jk/cherry-pick-revert-status (2023-06-27) 1 commit\n (merged to 'next' on 2023-06-28 at 4a0d88ef7a)\n + fix cherry-pick/revert status when doing multiple commits\n\n During a cherry-pick or revert session that works on multiple\n commits, \"git status\" did not give correct information, which has\n been corrected.\n source: <20230627224230.1951135-1-jacob.e.keller@intel.com>\n\n\n* jk/fsck-indices-in-worktrees (2023-06-29) 1 commit\n (merged to 'next' on 2023-06-29 at 9d245b3681)\n + fsck: avoid misleading variable name\n\n Code clarification.\n source: <20230629181333.87465-1-ericsunshine@charter.net>\n\n\n* js/empty-index-fixes (2023-06-29) 3 commits\n (merged to 'next' on 2023-06-29 at cdfd131900)\n + commit -a -m: allow the top-level tree to become empty again\n + split-index: accept that a base index can be empty\n + do_read_index(): always mark index as initialized unless erroring out\n\n A few places failed to differenciate the case where the index is\n truly empty (nothing added) and we haven't yet read from the\n on-disk index file, which have been corrected.\n source: \n\n\n* ks/t4205-test-describe-with-abbrev-fix (2023-06-29) 1 commit\n (merged to 'next' on 2023-06-29 at 5fc309dc75)\n + t4205: correctly test %(describe:abbrev=...)\n\n Test update.\n source: <20230629133841.18784-2-five231003@gmail.com>\n\n\n* pb/complete-diff-options (2023-06-26) 24 commits\n (merged to 'next' on 2023-06-28 at ccff93557d)\n + diff.c: mention completion above add_diff_options\n + completion: complete --remerge-diff\n + completion: complete --diff-merges, its options and --no-diff-merges\n + completion: move --pickaxe-{all,regex} to __git_diff_common_options\n + completion: complete --ws-error-highlight\n + completion: complete --unified\n + completion: complete --output-indicator-{context,new,old}\n + completion: complete --output\n + completion: complete --no-stat\n + completion: complete --no-relative\n + completion: complete --line-prefix\n + completion: complete --ita-invisible-in-index and --ita-visible-in-index\n + completion: complete --irreversible-delete\n + completion: complete --ignore-matching-lines\n + completion: complete --function-context\n + completion: complete --find-renames\n + completion: complete --find-object\n + completion: complete --find-copies\n + completion: complete --default-prefix\n + completion: complete --compact-summary\n + completion: complete --combined-all-paths\n + completion: complete --cc\n + completion: complete --break-rewrites\n + completion: add comments describing __git_diff_* globals\n\n Completion updates.\n source: \n\n\n* pw/apply-too-large (2023-06-26) 1 commit\n (merged to 'next' on 2023-06-28 at c0460d682c)\n + apply: improve error messages when reading patch\n\n \"git apply\" punts when it is fed too large a patch input; the error\n message it gives when it happens has been clarified.\n source: \n\n\n* rs/strbuf-expand-step (2023-06-18) 5 commits\n (merged to 'next' on 2023-06-28 at 1918915b71)\n + strbuf: simplify strbuf_expand_literal_cb()\n + replace strbuf_expand() with strbuf_expand_step()\n + replace strbuf_expand_dict_cb() with strbuf_expand_step()\n + strbuf: factor out strbuf_expand_step()\n + pretty: factor out expand_separator()\n\n Code clean-up around strbuf_expand() API.\n source: <767baa64-20a6-daf2-d34b-d81f72363749@web.de>\n\n\n* tl/notes-separator (2023-06-21) 7 commits\n (merged to 'next' on 2023-06-28 at 74fa459f4d)\n + notes: introduce \"--no-separator\" option\n + notes.c: introduce \"--[no-]stripspace\" option\n + notes.c: append separator instead of insert by pos\n + notes.c: introduce '--separator=' option\n + t3321: add test cases about the notes stripspace behavior\n + notes.c: use designated initializers for clarity\n + notes.c: cleanup 'strbuf_grow' call in 'append_edit'\n\n 'git notes append' was taught '--separator' to specify string to insert\n between paragraphs.\n source: \n\n--------------------------------------------------\n[Stalled]\n\n* ed/fsmonitor-windows-named-pipe (2023-03-24) 1 commit\n - fsmonitor: handle differences between Windows named pipe functions\n\n Fix fsmonitor on Windows when the filesystem path contains certain\n characters.\n\n Expecting a reroll.\n cf. \n source: \n\n\n* rn/sparse-diff-index (2023-04-10) 1 commit\n - diff-index: enable sparse index\n\n \"git diff-index\" command has been taught to work better with the\n sparse index.\n\n Expecting a reroll.\n cf. <62821012-4fc3-5ad8-695c-70f7ab14a8c9@github.com>\n source: <20230408112342.404318-1-nanth.raghul@gmail.com>\n\n\n* es/recurse-submodules-option-is-a-bool (2023-04-10) 1 commit\n - usage: clarify --recurse-submodules as a boolean\n\n The \"--[no-]recurse-submodules\" option of \"git checkout\" and others\n supported an undocumented syntax --recurse-submodules= where\n the value can spell a Boolean in various ways. The support for the\n syntax is being dropped.\n\n Expecting a reroll.\n cf. \n source: \n\n\n* cb/checkout-same-branch-twice (2023-03-22) 2 commits\n - SQUASH??? the test marked to expect failure passes from day one\n - checkout/switch: disallow checking out same branch in multiple worktrees\n\n \"git checkout -B $branch\" failed to protect against checking out\n a branch that is checked out elsewhere, unlike \"git branch -f\" did.\n\n Expecting a hopefully minor and final reroll.\n cf. \n source: <20230120113553.24655-1-carenas@gmail.com>\n\n\n* tk/pull-conflict-suggest-rebase-merge-not-rebase-true (2023-02-13) 1 commit\n - pull: conflict hint pull.rebase suggestion should offer \"merges\" vs \"true\"\n\n In an advice message after failed non-ff pull, we used to suggest\n setting pull.rebase=true, but these days pull.rebase=merges may be\n more inline with the original spirit of \"rebuild your side on top\n of theirs\".\n\n May want to discard.\n This is too much of a departure from the existing practice.\n cf. \n cf. \n source: \n\n\n* ab/tag-object-type-errors (2023-05-10) 4 commits\n - tag: don't emit potentially incorrect \"object is a X, not a Y\"\n - tag: don't misreport type of tagged objects in errors\n - object tests: add test for unexpected objects in tags\n - Merge branch 'jk/parse-object-type-mismatch' into ab/tag-object-type-errors\n\n Hardening checks around mismatched object types when one of those\n objects is a tag.\n source: \n\n\n* ad/test-record-count-when-harness-is-in-use (2022-12-25) 1 commit\n - test-lib: allow storing counts with test harnesses\n\n Allow summary results from tests to be written to t/test-results\n directory even when a test harness like 'prove' is in use.\n\n Expecting a reroll.\n cf. \n source: <20221224225200.1027806-1-adam@dinwoodie.org>\n\n\n* so/diff-merges-more (2022-12-18) 5 commits\n - diff-merges: improve --diff-merges documentation\n - diff-merges: issue warning on lone '-m' option\n - diff-merges: support list of values for --diff-merges\n - diff-merges: implement log.diffMerges-m-imply-p config\n - diff-merges: implement [no-]hide option and log.diffMergesHide config\n\n Assorted updates to \"--diff-merges=X\" option.\n\n May want to discard.\n Breaking compatibility does not seem worth it.\n source: <20221217132955.108542-1-sorganov@gmail.com>\n\n\n* ab/imap-send-requires-curl (2023-02-02) 6 commits\n - imap-send: correctly report \"host\" when using \"tunnel\"\n - imap-send: remove old --no-curl codepath\n - imap-send: make --curl no-optional\n - imap-send: replace auto-probe libcurl with hard dependency\n - imap-send doc: the imap.sslVerify is used with imap.tunnel\n - imap-send: note \"auth_method\", not \"host\" on auth method failure\n\n Give a hard dependency on cURL library to build \"git imap-send\",\n and remove the code to interact with IMAP server without using cURL.\n\n Expecting a reroll.\n The 'tunnel' part is still iffy.\n cf. <230203.86bkmabfjr.gmgdl@evledraar.gmail.com>\n source: \n\n\n* cw/submodule-status-in-parallel (2023-03-02) 6 commits\n - diff-lib: parallelize run_diff_files for submodules\n - diff-lib: refactor out diff_change logic\n - submodule: refactor is_submodule_modified()\n - submodule: move status parsing into function\n - submodule: rename strbuf variable\n - run-command: add on_stderr_output_fn to run_processes_parallel_opts\n\n \"git submodule status\" learned to run the comparison in submodule\n repositories in parallel.\n\n Expecting a reroll.\n cf. \n cf. <230307.865ybc273g.gmgdl@evledraar.gmail.com>\n source: <20230302215237.1473444-1-calvinwan@google.com>\n\n--------------------------------------------------\n[Cooking]\n\n* cw/compat-util-header-cleanup (2023-07-05) 6 commits\n (merged to 'next' on 2023-07-08 at c244b65c4f)\n + git-compat-util: move alloc macros to git-compat-util.h\n + treewide: remove unnecessary includes for wrapper.h\n + kwset: move translation table from ctype\n + sane-ctype.h: create header for sane-ctype macros\n + git-compat-util: move wrapper.c funcs to its header\n + git-compat-util: move strbuf.c funcs to its header\n\n Further shuffling of declarations across header files to streamline\n file dependencies.\n\n Will merge to 'master'.\n source: <20230705170812.3833103-1-calvinwan@google.com>\n\n\n* jk/imap-send-unused-variable-cleanup (2023-07-05) 3 commits\n (merged to 'next' on 2023-07-06 at d2798ee0fe)\n + imap-send: drop unused fields from imap_cmd_cb\n + imap-send: drop unused parameter from imap_cmd_cb callback\n + imap-send: use server conf argument in setup_curl()\n\n \"imap-send\" codepaths got cleaned up to get rid of unused\n parameters.\n\n Will merge to 'master'.\n source: <20230703063247.GA3524302@coredump.intra.peff.net>\n\n\n* ma/t0091-fixup (2023-07-05) 1 commit\n (merged to 'next' on 2023-07-06 at 3f82b468b4)\n + t0091-bugreport.sh: actually verify some content of report\n\n \"git bugreport\" tests did not test what it wanted to test, which\n has been corrected.\n\n Will merge to 'master'.\n source: <20230705183532.3057433-1-martin.agren@gmail.com>\n\n\n* rs/packet-length-simplify (2023-07-07) 1 commit\n (merged to 'next' on 2023-07-08 at 7f4b52e75b)\n + pkt-line: add size parameter to packet_length()\n\n Code simplification.\n\n Will merge to 'master'.\n source: <4d2bb49b-9c3c-ea58-274b-d9e9586620c1@web.de>\n\n\n* vd/adjust-mfow-doc-to-updated-headers (2023-07-04) 1 commit\n (merged to 'next' on 2023-07-08 at 38103c697d)\n + docs: add necessary headers to Documentation/MFOW.txt\n\n Code snippets in a tutorial document no longer compiled after\n recent header shuffling, which have been corrected.\n\n Will merge to 'master'.\n source: <20230702151457.3227-1-vinayakdev.sci@gmail.com>\n\n\n* mh/credential-erase-improvements-more (2023-06-24) 2 commits\n - credential/wincred: erase matching creds only\n - credential/libsecret: erase matching creds only\n\n Needs review.\n source: \n\n\n* pw/diff-no-index-from-named-pipes (2023-07-05) 4 commits\n (merged to 'next' on 2023-07-08 at 225230a441)\n + diff --no-index: support reading from named pipes\n + t4054: test diff --no-index with stdin\n + diff --no-index: die on error reading stdin\n + diff --no-index: refuse to compare stdin to a directory\n\n \"git diff --no-index\" learned to read from named pipes as if they\n were regular files, to allow \"git diff <(process) <(substitution)\"\n some shells support.\n\n Will merge to 'master'.\n source: \n\n\n* cc/repack-sift-filtered-objects-to-separate-pack (2023-07-05) 8 commits\n - gc: add `gc.repackFilterTo` config option\n - repack: implement `--filter-to` for storing filtered out objects\n - gc: add `gc.repackFilter` config option\n - repack: add `--filter=` option\n - repack: refactor finding pack prefix\n - repack: refactor finishing pack-objects command\n - t/helper: add 'find-pack' test-tool\n - pack-objects: allow `--filter` without `--stdout`\n\n \"git repack\" machinery learns to pay attention to the \"--filter=\"\n option.\n\n Comments?\n cf. \n source: <20230705060812.2865188-1-christian.couder@gmail.com>\n\n\n* js/doc-unit-tests (2023-06-30) 1 commit\n - unit tests: Add a project plan document\n\n Process to add some form of low-level unit tests has started.\n\n Still filling in blanks.\n source: <0169ce6fb9ccafc089b74ae406db0d1a8ff8ac65.1688165272.git.steadmon@google.com>\n\n\n* mh/mingw-case-sensitive-build (2023-06-12) 1 commit\n - mingw: use lowercase includes for some Windows headers\n\n Names of MinGW header files are spelled in mixed case in some\n source files, but the build host can be using case sensitive\n filesystem with header files with their name spelled in all\n lowercase.\n\n Needs review.\n source: <20230604211934.1365289-1-mh@glandium.org>\n\n\n* ks/ref-filter-signature (2023-06-06) 2 commits\n (merged to 'next' on 2023-07-06 at 1748d2bb93)\n + ref-filter: add new \"signature\" atom\n + t/lib-gpg: introduce new prereq GPG2\n\n The \"git for-each-ref\" family of commands learned placeholders\n related to GPG signature verification.\n\n Will merge to 'master'.\n cf. \n source: <20230604185815.15761-1-five231003@gmail.com>\n\n\n* jt/path-filter-fix (2023-06-13) 4 commits\n - commit-graph: new filter ver. that fixes murmur3\n - repo-settings: introduce commitgraph.changedPathsVersion\n - t4216: test changed path filters with high bit paths\n - gitformat-commit-graph: describe version 2 of BDAT\n\n The Bloom filter used for path limited history traversal was broken\n on systems whose \"char\" is unsigned; update the implementation and\n bump the format version to 2.\n\n Expecting a reroll.\n cf. \n source: \n\n\n* tk/cherry-pick-sequence-requires-clean-worktree (2023-06-01) 1 commit\n - cherry-pick: refuse cherry-pick sequence if index is dirty\n\n \"git cherry-pick A\" that replays a single commit stopped before\n clobbering local modification, but \"git cherry-pick A..B\" did not,\n which has been corrected.\n\n Expecting a reroll.\n cf. <999f12b2-38d6-f446-e763-4985116ad37d@gmail.com>\n source: \n\n\n* mh/credential-libsecret-attrs (2023-06-16) 1 commit\n - credential/libsecret: store new attributes\n\n The way authentication related data other than passwords (e.g.\n oath token and password expiration data) are stored in libsecret\n keyrings has been rethought.\n\n Needs review.\n source: \n\n\n* tb/refs-exclusion-and-packed-refs (2023-07-10) 16 commits\n - ls-refs.c: avoid enumerating hidden refs where possible\n - upload-pack.c: avoid enumerating hidden refs where possible\n - builtin/receive-pack.c: avoid enumerating hidden references\n - refs.h: implement `hidden_refs_to_excludes()`\n - refs.h: let `for_each_namespaced_ref()` take excluded patterns\n - revision.h: store hidden refs in a `strvec`\n - refs/packed-backend.c: add trace2 counters for jump list\n - refs/packed-backend.c: implement jump lists to avoid excluded pattern(s)\n - refs/packed-backend.c: refactor `find_reference_location()`\n - refs: plumb `exclude_patterns` argument throughout\n - builtin/for-each-ref.c: add `--exclude` option\n - ref-filter.c: parameterize match functions over patterns\n - ref-filter: add `ref_filter_clear()`\n - ref-filter: clear reachable list pointers after freeing\n - ref-filter.h: provide `REF_FILTER_INIT`\n - refs.c: rename `ref_filter`\n\n Enumerating refs in the packed-refs file, while excluding refs that\n match certain patterns, has been optimized.\n\n Under discussion.\n source: \n\n\n* cc/git-replay (2023-06-03) 15 commits\n - replay: stop assuming replayed branches do not diverge\n - replay: add --contained to rebase contained branches\n - replay: add --advance or 'cherry-pick' mode\n - replay: disallow revision specific options and pathspecs\n - replay: use standard revision ranges\n - replay: make it a minimal server side command\n - replay: remove HEAD related sanity check\n - replay: remove progress and info output\n - replay: add an important FIXME comment about gpg signing\n - replay: don't simplify history\n - replay: introduce pick_regular_commit()\n - replay: die() instead of failing assert()\n - replay: start using parse_options API\n - replay: introduce new builtin\n - t6429: remove switching aspects of fast-rebase\n\n source: <20230602102533.876905-1-christian.couder@gmail.com>\n\n\n* ob/revert-of-revert (2023-05-05) 1 commit\n - sequencer: beautify subject of reverts of reverts\n\n Instead of \"Revert \"Revert \"original\"\", give \"Reapply \"original\"\"\n as the title for a revert of a revert.\n\n Expecting a hopefully final reroll.\n Looking much better, except for minor cosmetic issues.\n source: <20230428083528.1699221-1-oswald.buddenhagen@gmx.de>\n\n\n* pw/rebase-i-after-failure (2023-04-21) 6 commits\n - rebase -i: fix adding failed command to the todo list\n - rebase: fix rewritten list for failed pick\n - rebase --continue: refuse to commit after failed command\n - sequencer: factor out part of pick_commits()\n - rebase -i: remove patch file after conflict resolution\n - rebase -i: move unlink() calls\n\n Various fixes to the behaviour of \"rebase -i\" when the command got\n interrupted by conflicting changes.\n\n Expecting a reroll.\n cf. \n cf. <1fd54422-b66a-c2e4-7cd7-934ea01190ad@gmail.com>\n source: \n"},{"id":"479389","messageId":"ZK2TWdME96AODU4+@nand.local","threadId":"59969","inReplyTo":"xmqqmt032r7m.fsf@gitster.g","subject":"Re: What's cooking in git.git (Jul 2023, #02; Mon, 10)","fromName":"Taylor Blau","fromEmail":"me@ttaylorr.com","sentAt":"2023-07-11T17:37:29Z","receivedAt":"2023-07-11T17:37:41Z","isPatch":false,"sender":{"key":"me@ttaylorr.com","avatar":"https://avatars.githubusercontent.com/u/301000140?v=4"},"body":"On Mon, Jul 10, 2023 at 06:05:01PM -0700, Junio C Hamano wrote:\n> * tb/fsck-no-progress (2023-07-10) 6 commits\n> - commit-graph.c: avoid duplicated progress output during `verify`\n> - commit-graph.c: pass progress to `verify_one_commit_graph()`\n> - commit-graph.c: iteratively verify commit-graph chains\n> - commit-graph.c: extract `verify_one_commit_graph()`\n> - fsck: suppress MIDX output with `--no-progress`\n> - fsck: suppress commit-graph output with `--no-progress`\n>\n> \"git fsck --no-progress\" still spewed noise from the commit-graph\n> subsystem, which has been corrected.\n>\n> Will merge to 'next'.\n> source: \n\nThanks, this one is fairly innocuous and shouldn't cause any problems.\n\n> * tb/repack-cleanup (2023-07-10) 2 commits\n> - builtin/repack.c: avoid dir traversal in `collect_pack_filenames()`\n> - builtin/repack.c: only repack `.pack`s that exist\n>\n> The recent change to \"git repack\" made it react less nicely when a\n> leftover .idx file that no longer has the corresponding .pack file\n> in the repository, which has been corrected.\n>\n> Will merge to 'next'.\n> source: \n\nThis one is definitely a higher risk change, but I am as confident as I\ncan be in terms of its correctness. I sent a new version[1] to remove the\nunnecessary 'ls'-to-'find' conversion, but feel free to queue either.\n\n> * cc/repack-sift-filtered-objects-to-separate-pack (2023-07-05) 8 commits\n> - gc: add `gc.repackFilterTo` config option\n> - repack: implement `--filter-to` for storing filtered out objects\n> - gc: add `gc.repackFilter` config option\n> - repack: add `--filter=` option\n> - repack: refactor finding pack prefix\n> - repack: refactor finishing pack-objects command\n> - t/helper: add 'find-pack' test-tool\n> - pack-objects: allow `--filter` without `--stdout`\n>\n> \"git repack\" machinery learns to pay attention to the \"--filter=\"\n> option.\n>\n> Comments?\n> cf. \n> source: <20230705060812.2865188-1-christian.couder@gmail.com>\n\nThis is on my review queue, sorry for the delay in responding to the\nrest of the newer version.\n\n> * jt/path-filter-fix (2023-06-13) 4 commits\n> - commit-graph: new filter ver. that fixes murmur3\n> - repo-settings: introduce commitgraph.changedPathsVersion\n> - t4216: test changed path filters with high bit paths\n> - gitformat-commit-graph: describe version 2 of BDAT\n>\n> The Bloom filter used for path limited history traversal was broken\n> on systems whose \"char\" is unsigned; update the implementation and\n> bump the format version to 2.\n>\n> Expecting a reroll.\n> cf. \n> source: \n\nThis is also on my review queue as soon as the reroll appears on the\nlist.\n\n> * tb/refs-exclusion-and-packed-refs (2023-07-10) 16 commits\n> - ls-refs.c: avoid enumerating hidden refs where possible\n> - upload-pack.c: avoid enumerating hidden refs where possible\n> - builtin/receive-pack.c: avoid enumerating hidden references\n> - refs.h: implement `hidden_refs_to_excludes()`\n> - refs.h: let `for_each_namespaced_ref()` take excluded patterns\n> - revision.h: store hidden refs in a `strvec`\n> - refs/packed-backend.c: add trace2 counters for jump list\n> - refs/packed-backend.c: implement jump lists to avoid excluded pattern(s)\n> - refs/packed-backend.c: refactor `find_reference_location()`\n> - refs: plumb `exclude_patterns` argument throughout\n> - builtin/for-each-ref.c: add `--exclude` option\n> - ref-filter.c: parameterize match functions over patterns\n> - ref-filter: add `ref_filter_clear()`\n> - ref-filter: clear reachable list pointers after freeing\n> - ref-filter.h: provide `REF_FILTER_INIT`\n> - refs.c: rename `ref_filter`\n>\n> Enumerating refs in the packed-refs file, while excluding refs that\n> match certain patterns, has been optimized.\n>\n> Under discussion.\n> source: \n\nPatrick took a look[2] at this (which you responded to), and gave an\nack, so this one should be ready to go down to 'next', too.\n\nThanks,\nTaylor\n\n[1]: https://lore.kernel.org/git/cover.1689096750.git.me@ttaylorr.com/\n[2]: https://lore.kernel.org/git/eddehgbfqnmkhvhkacbvnqiiripbn6jvjawpy76ysfnpohsygt@a43fbutqg64z/\n"}]}