{"thread":{"id":"58317","subject":"What's cooking in git.git (Aug 2022, #06; Wed, 17)","startedAt":"2022-08-18T02:23:36Z","lastAt":"2022-08-19T14:37:23Z","messageCount":11,"participants":["Junio C Hamano","Ævar Arnfjörð Bjarmason","Derrick Stolee","Victoria Dye","Jonathan Tan","Johannes Schindelin"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"461393","messageId":"xmqq7d36s31e.fsf@gitster.g","threadId":"58317","inReplyTo":null,"subject":"What's cooking in git.git (Aug 2022, #06; Wed, 17)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2022-08-18T02:23:25Z","receivedAt":"2022-08-18T02:23:36Z","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 future\nrelease). Commits prefixed with '-' are only in 'seen', and aren't\nconsidered \"accepted\" at all. A topic without enough support may be\ndiscarded after a long period of no activity.\n\nWe are in the middle of the week #6 of a 12-week cycle (cf.\nhttps://tinyurl.com/gitCal).\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* vd/scalar-enables-fsmonitor (2022-08-16) 5 commits\n - scalar: update technical doc roadmap with FSMonitor support\n - scalar unregister: stop FSMonitor daemon\n - scalar: enable built-in FSMonitor on `register`\n - scalar-[un]register: clearly indicate source of error\n - scalar-unregister: handle error codes greater than 0\n\n \"scalar\" now enables built-in fsmonitor on enlisted repositories,\n when able.\n\n Will merge to 'next'?\n source: \n\n\n* en/ancestry-path-in-a-range (2022-08-17) 2 commits\n - revision: allow --ancestry-path to take an argument\n - rev-list-options.txt: fix simple typo\n\n \"git rev-list --ancestry-path=C A..B\" is a natural extension of\n \"git rev-list A..B\"; instead of choosing a subset of A..B to those\n that have ancestry relationship with A, it lets a subset with\n ancestry relationship with C.\n\n Will merge to 'next'?\n source: \n\n\n* en/submodule-merge-messages-fixes (2022-08-17) 3 commits\n - merge-ort: provide helpful submodule update message when possible\n - merge-ort: add comment to avoid surprise with new sub_flag variable\n - merge-ort: remove translator lego in new \"submodule conflict suggestion\"\n (this branch uses cw/submodule-merge-messages.)\n\n Further update the help messages given while merging submodules.\n\n Will merge to 'next'?\n source: \n\n--------------------------------------------------\n[Cooking]\n\n* ll/disk-usage-humanise (2022-08-11) 1 commit\n (merged to 'next' on 2022-08-14 at 3873a83f90)\n + rev-list: support human-readable output for `--disk-usage`\n\n \"git rev-list --disk-usage\" learned to take an optional value\n \"human\" to show the reported value in human-readable format, like\n \"3.40MiB\".\n\n Will merge to 'master'.\n source: \n\n\n* ed/fsmonitor-on-network-disk (2022-08-11) 1 commit\n (merged to 'next' on 2022-08-14 at 637d458d9c)\n + fsmonitor: option to allow fsmonitor to run against network-mounted repos\n\n The built-in fsmonitor refuses to work on a network mounted\n repositories; a configuration knob for users to override this has\n been introduced.\n\n Will merge to 'master'.\n source: \n\n\n* jk/is-promisor-object-keep-tree-in-use (2022-08-14) 1 commit\n (merged to 'next' on 2022-08-17 at 96033ff88d)\n + is_promisor_object(): fix use-after-free of tree buffer\n\n An earlier optimization discarded a tree-object buffer that is\n still in use, which has been corrected.\n\n Will merge to 'master'.\n source: \n\n\n* js/fetch-negotiation-trace (2022-08-15) 1 commit\n (merged to 'next' on 2022-08-17 at 2c9e07b617)\n + fetch-pack: add tracing for negotiation rounds\n\n The common ancestor negotiation exchange during a \"git fetch\"\n session now leaves trace log.\n\n Will merge to 'master'.\n source: <4390677ec75d51487142adf7c2ab821cbd24a53e.1659477669.git.steadmon@google.com>\n\n\n* pw/rebase-keep-base-fixes (2022-08-15) 5 commits\n - rebase --keep-base: imply --no-fork-point\n - rebase --keep-base: imply --reapply-cherry-picks\n - rebase: factor out merge_base calculation\n - rebase: store orig_head as a commit\n - t3416: set $EDITOR in subshell\n\n \"git rebase --keep-base\" used to discard the commits that are\n already cherry-picked to the upstream, even when \"keep-base\" meant\n that the base, on top of which the history is being rebuilt, does\n not yet include these cherry-picked commits. The --keep-base\n option now implies --reapply-cherry-picks and --no-fork-point\n options.\n\n Needs review.\n source: \n\n\n* fc/vimdiff-layout-vimdiff3-fix (2022-08-10) 7 commits\n (merged to 'next' on 2022-08-11 at a14fec292f)\n + mergetools: vimdiff: simplify tabfirst\n + mergetools: vimdiff: fix single window layouts\n + mergetools: vimdiff: rework tab logic\n + mergetools: vimdiff: fix for diffopt\n + mergetools: vimdiff: silence annoying messages\n + mergetools: vimdiff: make vimdiff3 actually work\n + mergetools: vimdiff: fix comment\n\n \"vimdiff3\" regression fix.\n\n Will merge to 'master'.\n source: <20220810154618.307275-1-felipe.contreras@gmail.com>\n\n\n* jk/fsck-tree-mode-bits-fix (2022-08-10) 3 commits\n (merged to 'next' on 2022-08-11 at 219fe53025)\n + fsck: downgrade tree badFilemode to \"info\"\n + fsck: actually detect bad file modes in trees\n + tree-walk: add a mechanism for getting non-canonicalized modes\n\n \"git fsck\" reads mode from tree objects but canonicalizes the mode\n before passing it to the logic to check object sanity, which has\n hid broken tree objects from the checking logic. This has been\n corrected, but to help exiting projects with broken tree objects\n that they cannot fix retroactively, the severity of anomalies this\n code detects has been demoted to \"info\" for now.\n\n Will merge to 'master'.\n source: \n\n\n* ag/merge-strategies-in-c (2022-08-10) 14 commits\n - sequencer: use the \"octopus\" strategy without forking\n - sequencer: use the \"resolve\" strategy without forking\n - merge: use the \"octopus\" strategy without forking\n - merge: use the \"resolve\" strategy without forking\n - merge-octopus: rewrite in C\n - merge-recursive: move better_branch_name() to merge.c\n - merge-resolve: rewrite in C\n - merge-one-file: rewrite in C\n - update-index: move add_cacheinfo() to read-cache.c\n - merge-index: add a new way to invoke `git-merge-one-file'\n - merge-index: drop the index\n - merge-index: libify merge_one_path() and merge_all()\n - t6060: add tests for removed files\n - t6060: modify multiple files to expose a possible issue with merge-index\n\n An attempt to rewrite remaining merge strategies from shell to C.\n source: <20220809185429.20098-1-alban.gruin@gmail.com>\n\n\n* sy/mv-out-of-cone (2022-08-10) 9 commits\n (merged to 'next' on 2022-08-17 at 2316d9ce4d)\n + mv: check overwrite for in-to-out move\n + advice.h: add advise_on_moving_dirty_path()\n + mv: cleanup empty WORKING_DIRECTORY\n + mv: from in-cone to out-of-cone\n + mv: remove BOTH from enum update_mode\n + mv: check if is a SKIP_WORKTREE_DIR\n + mv: free the with_slash in check_dir_in_index()\n + mv: rename check_dir_in_index() to empty_dir_has_sparse_contents()\n + t7002: add tests for moving from in-cone to out-of-cone\n\n \"git mv A B\" in a sparsely populated working tree can be asked to\n move a path from a directory that is \"in cone\" to another directory\n that is \"out of cone\". Handling of such a case has been improved.\n\n Will merge to 'master'.\n source: <20220809120910.2021413-1-shaoxuan.yuan02@gmail.com>\n\n\n* sy/sparse-rm (2022-08-08) 5 commits\n (merged to 'next' on 2022-08-12 at 5bf10965fb)\n + rm: integrate with sparse-index\n + rm: expand the index only when necessary\n + pathspec.h: move pathspec_needs_expanded_index() from reset.c to here\n + t1092: add tests for `git-rm`\n + Merge branch 'vd/sparse-reset-checkout-fixes' into sy/sparse-rm\n (this branch uses vd/sparse-reset-checkout-fixes.)\n\n \"git rm\" has become more aware of the sparse-index feature.\n\n Will merge to 'master'.\n source: <20220807041335.1790658-1-shaoxuan.yuan02@gmail.com>\n\n\n* vd/scalar-generalize-diagnose (2022-08-12) 11 commits\n - scalar: update technical doc roadmap\n - scalar-diagnose: use 'git diagnose --mode=all'\n - builtin/bugreport.c: create '--diagnose' option\n - builtin/diagnose.c: add '--mode' option\n - builtin/diagnose.c: create 'git diagnose' builtin\n - diagnose.c: add option to configure archive contents\n - scalar-diagnose: move functionality to common location\n - scalar-diagnose: move 'get_disk_info()' to 'compat/'\n - scalar-diagnose: add directory to archiver more gently\n - scalar-diagnose: avoid 32-bit overflow of size_t\n - scalar-diagnose: use \"$GIT_UNZIP\" in test\n\n The \"diagnose\" feature to create a zip archive for diagnostic\n material has been lifted from \"scalar\" and made into a feature of\n \"git bugreport\".\n\n Will merge to 'next'?\n source: \n\n\n* jk/pipe-command-nonblock (2022-08-17) 6 commits\n (merged to 'next' on 2022-08-17 at 91fe8e6354)\n + pipe_command(): mark stdin descriptor as non-blocking\n + pipe_command(): handle ENOSPC when writing to a pipe\n + pipe_command(): avoid xwrite() for writing to pipe\n + git-compat-util: make MAX_IO_SIZE define globally available\n + nonblock: support Windows\n + compat: add function to enable nonblocking pipes\n\n Fix deadlocks between main Git process and subprocess spawned via\n the pipe_command() API, that can kill \"git add -p\" that was\n reimplemented in C recently.\n\n Will merge to 'master'.\n source: \n\n\n* es/mark-gc-cruft-as-experimental (2022-08-03) 2 commits\n - config: let feature.experimental imply gc.cruftPacks=true\n - gc: add tests for --cruft and friends\n\n Enable gc.cruftpacks by default for those who opt into\n feature.experimental setting.\n\n Expecting a reroll.\n cf. <220804.86a68ke9d5.gmgdl@evledraar.gmail.com>\n cf. <6803b725-526e-a1c8-f15c-a9ed4a144d4c@github.com>\n source: <20220803205721.3686361-1-emilyshaffer@google.com>\n\n\n* vd/sparse-reset-checkout-fixes (2022-08-08) 4 commits\n (merged to 'next' on 2022-08-12 at 755d6ecdb8)\n + unpack-trees: unpack new trees as sparse directories\n + cache.h: create 'index_name_pos_sparse()'\n + oneway_diff: handle removed sparse directories\n + checkout: fix nested sparse directory diff in sparse index\n (this branch is used by sy/sparse-rm.)\n\n Fixes to sparse index compatibility work for \"reset\" and \"checkout\"\n commands.\n\n Will merge to 'master'.\n source: \n\n\n* sg/parse-options-subcommand (2022-07-25) 20 commits\n - builtin/worktree.c: let parse-options parse subcommands\n - builtin/stash.c: let parse-options parse subcommands\n - builtin/sparse-checkout.c: let parse-options parse subcommands\n - builtin/remote.c: let parse-options parse subcommands\n - builtin/reflog.c: let parse-options parse subcommands\n - builtin/notes.c: let parse-options parse subcommands\n - builtin/multi-pack-index.c: let parse-options parse subcommands\n - builtin/hook.c: let parse-option parse subcommands\n - builtin/gc.c: let parse-options parse 'git maintenance's subcommands\n - builtin/commit-graph.c: let parse-options parse subcommands\n - builtin/bundle.c: let parse-options parse subcommands\n - parse-options: add support for parsing subcommands\n - parse-options: drop leading space from '--git-completion-helper' output\n - parse-options: clarify the limitations of PARSE_OPT_NODASH\n - parse-options: PARSE_OPT_KEEP_UNKNOWN only applies to --options\n - api-parse-options.txt: fix description of OPT_CMDMODE\n - t0040-parse-options: test parse_options() with various 'parse_opt_flags'\n - t5505-remote.sh: check the behavior without a subcommand\n - t3301-notes.sh: check that default operation mode doesn't take arguments\n - git.c: update NO_PARSEOPT markings\n\n Introduce the \"subcommand\" mode to parse-options API and update the\n command line parser of Git commands with subcommands.\n\n Expecting a reroll.\n cf. <20220812152837.GC3790@szeder.dev>\n source: <20220725123857.2773963-1-szeder.dev@gmail.com>\n\n\n* ds/bundle-uri-clone (2022-08-10) 5 commits\n - clone: --bundle-uri cannot be combined with --depth\n - bundle-uri: add support for http(s):// and file://\n - clone: add --bundle-uri option\n - bundle-uri: create basic file-copy logic\n - remote-curl: add 'get' capability\n\n Implement \"git clone --bundle-uri\".\n source: \n\n\n* ds/decorate-filter-tweak (2022-08-05) 11 commits\n - fetch: use ref_namespaces during prefetch\n - maintenance: stop writing log.excludeDecoration\n - log: create log.initialDecorationSet=all\n - log: add --clear-decorations option\n - log: add default decoration filter\n - log-tree: use ref_namespaces instead of if/else-if\n - refs: use ref_namespaces for replace refs base\n - refs: add array of ref namespaces\n - t4207: test coloring of grafted decorations\n - t4207: modernize test\n - refs: allow \"HEAD\" as decoration filter\n\n The namespaces used by \"log --decorate\" from \"refs/\" hierarchy by\n default has been tightened.\n\n Will merge to 'next'?\n source: \n\n\n* es/doc-creation-factor-fix (2022-07-28) 2 commits\n - range-diff: clarify --creation-factor=\n - format-patch: clarify --creation-factor=\n\n Expecting a reroll.\n source: <7229p500-p2r4-on87-6802-8o90s36rr3s4@tzk.qr>\n\n\n* ab/submodule-helper-prep (2022-08-03) 28 commits\n - submodule--helper: fix bad config API usage\n - submodule--helper: libify \"must_die_on_failure\" code paths (for die)\n - submodule--helper: libify \"must_die_on_failure\" code paths\n - submodule--helper: libify determine_submodule_update_strategy()\n - submodule--helper: don't exit() on failure, return\n - submodule--helper: use \"code\" in run_update_command()\n - submodule--helper: move submodule_strategy_to_string() to only user\n - submodule--helper: don't call submodule_strategy_to_string() in BUG()\n - submodule--helper: add missing braces to \"else\" arm\n - submodule--helper: return \"ret\", not \"1\" from update_submodule()\n - submodule--helper: rename \"int res\" to \"int ret\"\n - submodule--helper: don't redundantly check \"else if (res)\"\n - submodule--helper: refactor \"errmsg_str\" to be a \"struct strbuf\"\n - submodule--helper: add \"const\" to copy of \"update_data\"\n - submodule--helper: pass a \"const struct module_clone_data\" to clone_submodule()\n - submodule--helper: move \"sb\" in clone_submodule() to its own scope\n - submodule--helper: use xstrfmt() in clone_submodule()\n - submodule--helper: replace memset() with { 0 }-initialization\n - submodule--helper style: add \\n\\n after variable declarations\n - submodule--helper style: don't separate declared variables with \\n\\n\n - submodule--helper: move \"resolve-relative-url-test\" to a test-tool\n - submodule--helper: move \"check-name\" to a test-tool\n - submodule--helper: move \"is-active\" to a test-tool\n - test-tool submodule-config: remove unused \"--url\" handling\n - submodule--helper: remove unused \"list\" helper\n - submodule--helper: remove unused \"name\" helper\n - submodule tests: test for \"add \"\n - submodule tests: test usage behavior\n (this branch is used by ab/submodule-helper-leakfix.)\n\n Code clean-up of \"git submodule--helper\".\n\n Expecting a (hopefully final?) reroll.\n cf. <220803.86h72tfpcc.gmgdl@evledraar.gmail.com>\n source: \n\n\n* ab/dedup-config-and-command-docs (2022-07-29) 9 commits\n - docs: add CONFIGURATION sections that fuzzy map to built-ins\n - docs: add CONFIGURATION sections that map to a built-in\n - log docs: de-duplicate configuration sections\n - difftool docs: de-duplicate configuration sections\n - notes docs: de-duplicate configuration sections\n - apply docs: de-duplicate configuration sections\n - send-email docs: de-duplicate configuration sections\n - grep docs: de-duplicate configuration sections\n - docs: add and use include template for config/* includes\n\n Share the text used to explain configuration variables used by \"git\n \" in \"git help \" with the text from \"git help config\".\n\n Expecting a reroll.\n cf. \n cf. \n cf. \n source: \n\n\n* cw/remote-object-info (2022-08-13) 7 commits\n - SQUASH???\n - cat-file: add remote-object-info to batch-command\n - transport: add client support for object-info\n - serve: advertise object-info feature\n - protocol-caps: initialization bug fix\n - fetch-pack: move fetch initialization\n - fetch-pack: refactor packet writing\n\n A client component to talk with the object-info endpoint.\n\n Expecting a reroll.\n cf. <20220728230210.2952731-1-calvinwan@google.com>\n cf. \n source: <20220728230210.2952731-1-calvinwan@google.com>\n\n\n* mt/rot13-in-c (2022-08-14) 4 commits\n - tests: use the new C rot13-filter helper to avoid PERL prereq\n - t0021: implementation the rot13-filter.pl script in C\n - t0021: avoid grepping for a Perl-specific string at filter output\n - Merge branch 'mt/checkout-count-fix' into mt/rot13-in-c\n\n Test portability improvements.\n\n Will merge to 'next'?\n source: \n\n\n* tl/trace2-config-scope (2022-08-11) 2 commits\n - tr2: shows scope unconditionally in addition to key-value pair\n - api-trace2.txt: print config key-value pair\n\n Tweak trace2 output about configuration variables.\n source: \n\n\n* ab/submodule-helper-leakfix (2022-08-03) 18 commits\n - submodule--helper: fix a configure_added_submodule() leak\n - submodule--helper: free rest of \"displaypath\" in \"struct update_data\"\n - submodule--helper: free some \"displaypath\" in \"struct update_data\"\n - submodule--helper: fix a memory leak in print_status()\n - submodule--helper: fix a leak in module_add()\n - submodule--helper: fix obscure leak in module_add()\n - submodule--helper: fix \"reference\" leak\n - submodule--helper: fix a memory leak in get_default_remote_submodule()\n - submodule--helper: fix a leak with repo_clear()\n - submodule--helper: fix \"sm_path\" and other \"module_cb_list\" leaks\n - submodule--helper: fix \"errmsg_str\" memory leak\n - submodule--helper: add and use *_release() functions\n - submodule--helper: don't leak {run,capture}_command() cp.dir argument\n - submodule--helper: \"struct pathspec\" memory leak in module_update()\n - submodule--helper: fix most \"struct pathspec\" memory leaks\n - submodule--helper: fix trivial get_default_remote_submodule() leak\n - submodule--helper: fix a leak in \"clone_submodule\"\n - Merge branch 'ab/submodule-helper-prep' into ab/submodule-helper-leakfix\n (this branch uses ab/submodule-helper-prep.)\n\n Plugging leaks in submodule--helper.\n\n Waiting for the \"prep\" step.\n source: \n\n\n* cw/submodule-merge-messages (2022-08-04) 1 commit\n (merged to 'next' on 2022-08-12 at ede0890319)\n + submodule merge: update conflict error message\n (this branch is used by en/submodule-merge-messages-fixes.)\n\n Update the message given when \"git merge\" sees conflicts at a path\n with a submodule while merging a superproject.\n\n Will wait for en/submodule-merge-messages-fixes to mature.\n source: <20220804195105.1303455-1-calvinwan@google.com>\n\n\n* po/doc-add-renormalize (2022-08-10) 1 commit\n (merged to 'next' on 2022-08-11 at 53851663eb)\n + doc add: renormalize is not idempotent for CRCRLF\n\n Documentation for \"git add --renormalize\" has been improved.\n\n Will merge to 'master'.\n source: <20220810144450.470-2-philipoakley@iee.email>\n\n\n* po/glossary-around-traversal (2022-07-09) 3 commits\n - glossary: add reachability bitmap description\n - glossary: add commit graph description\n - glossary: add Object DataBase (ODB) abbreviation\n\n The glossary entries for \"commit-graph file\" and \"reachability\n bitmap\" have been added.\n\n Expecting a reroll.\n cf. \n source: \n\n\n* ac/bitmap-lookup-table (2022-08-14) 6 commits\n - bitmap-lookup-table: add performance tests for lookup table\n - pack-bitmap: prepare to read lookup table extension\n - pack-bitmap-write: learn pack.writeBitmapLookupTable and add tests\n - pack-bitmap-write.c: write lookup table extension\n - bitmap: move `get commit positions` code to `bitmap_writer_finish`\n - Documentation/technical: describe bitmap lookup table extension\n\n The pack bitmap file gained a bitmap-lookup table to speed up\n locating the necessary bitmap for a given commit.\n source: \n\n\n* tb/show-ref-count (2022-06-06) 2 commits\n - builtin/show-ref.c: limit output with `--count`\n - builtin/show-ref.c: rename `found_match` to `matches_nr`\n\n \"git show-ref\" learned to stop after emitting N refs with the new\n \"--count=N\" option.\n\n Expecting a reroll.\n cf. \n source: \n\n\n* ds/bundle-uri-more (2022-08-10) 2 commits\n (merged to 'next' on 2022-08-12 at 4f445a058d)\n + bundle-uri: add example bundle organization\n + docs: document bundle URI standard\n\n The \"bundle URI\" design gets documented.\n\n Will merge to 'master'.\n source: \n\n\n* js/bisect-in-c (2022-06-27) 16 commits\n - bisect: no longer try to clean up left-over `.git/head-name` files\n - bisect: remove Cogito-related code\n - Turn `git bisect` into a full built-in\n - bisect: move even the command-line parsing to `bisect--helper`\n - bisect: teach the `bisect--helper` command to show the correct usage strings\n - bisect--helper: return only correct exit codes in `cmd_*()`\n - bisect--helper: move the `BISECT_STATE` case to the end\n - bisect--helper: make `--bisect-state` optional\n - bisect--helper: align the sub-command order with git-bisect.sh\n - bisect--helper: using `--bisect-state` without an argument is a bug\n - bisect--helper: really retire `--bisect-autostart`\n - bisect--helper: really retire --bisect-next-check\n - bisect--helper: retire the --no-log option\n - bisect: avoid double-quoting when printing the failed command\n - bisect run: fix the error message\n - bisect: verify that a bogus option won't try to start a bisection\n\n Final bits of \"git bisect.sh\" have been rewritten in C.\n\n Expecting a (hopefully final) reroll.\n cf. <20627.86ilolhnnn.gmgdl@evledraar.gmail.com>\n source: \n\n--------------------------------------------------\n[Discarded]\n\n* mb/doc-rerere-autoupdate (2022-07-15) 1 commit\n . cherry-pick doc: clarify no-rerere-autoupdate still allows rerere\n\n Clarifies that the \"--no-rerere-autoupdate\" option does not disable\n the \"rerere\" mechanism (nor does \"--rerere-autoupdate\" enable it).\n\n Superseded by jc/rerere-autoupdate-doc\n source: <20220715092527.1567837-1-mail@beyermatthias.de>\n\n\n* fr/vimdiff-layout-colors-fix (2022-08-07) 3 commits\n . mergetools: vimdiff: update unit tests\n . mergetools: vimdiff: fix single tab mode, single window mode and colors\n . mergetools: vimdiff: fix comment\n\n \"vimdiff3\" regression fix.\n\n Superseded by fc/vimdiff-layout-vimdiff3-fix\n source: <20220808053459.184367-1-greenfoo@u92.eu>\n\n\n* tk/apply-case-insensitive (2022-06-21) 3 commits\n . apply: support case-only renames in case-insensitive filesystems\n . reset: new failing test for reset of case-insensitive duplicate in index\n . t4141: test \"git apply\" with core.ignorecase\n\n \"git apply\" barfed on a patch that makes a case-only rename on a\n case-insensitive filesystem.\n\n In stalled state for too long.\n source: \n\n\n* bc/stash-export (2022-04-08) 4 commits\n . builtin/stash: provide a way to import stashes from a ref\n . builtin/stash: provide a way to export stashes to a ref\n . builtin/stash: factor out revision parsing into a function\n . object-name: make get_oid quietly return an error\n\n A mechanism to export and import stash entries to and from a normal\n commit to transfer it across repositories has been introduced.\n\n In stalled state for too long.\n cf. \n source: <20220407215352.3491567-1-sandals@crustytoothpaste.net>\n\n\n* jt/connected-show-missing-from-which-side (2022-06-10) 1 commit\n . fetch,fetch-pack: clarify connectivity check error\n\n We may find an object missing after a \"git fetch\" stores the\n objects it obtained from the other side, but it is not necessarily\n because the remote failed to send necessary objects. Reword the\n messages in an attempt to help users explore other possibilities\n when they hit this error.\n\n In stalled state for too long.\n Also it seems to break t5326 with sha256.\n source: <20220610195247.1177549-1-jonathantanmy@google.com>\n"},{"id":"461412","messageId":"220818.86czcxafew.gmgdl@evledraar.gmail.com","threadId":"58317","inReplyTo":"xmqq7d36s31e.fsf@gitster.g","subject":"js/bisect-in-c (was: What's cooking in git.git (Aug 2022, #06; Wed, 17))","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2022-08-18T12:42:53Z","receivedAt":"2022-08-18T12:45:51Z","isPatch":false,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"\nOn Wed, Aug 17 2022, Junio C Hamano wrote:\n\n> * js/bisect-in-c (2022-06-27) 16 commits\n> - bisect: no longer try to clean up left-over `.git/head-name` files\n> - bisect: remove Cogito-related code\n> - Turn `git bisect` into a full built-in\n> - bisect: move even the command-line parsing to `bisect--helper`\n> - bisect: teach the `bisect--helper` command to show the correct usage strings\n> - bisect--helper: return only correct exit codes in `cmd_*()`\n> - bisect--helper: move the `BISECT_STATE` case to the end\n> - bisect--helper: make `--bisect-state` optional\n> - bisect--helper: align the sub-command order with git-bisect.sh\n> - bisect--helper: using `--bisect-state` without an argument is a bug\n> - bisect--helper: really retire `--bisect-autostart`\n> - bisect--helper: really retire --bisect-next-check\n> - bisect--helper: retire the --no-log option\n> - bisect: avoid double-quoting when printing the failed command\n> - bisect run: fix the error message\n> - bisect: verify that a bogus option won't try to start a bisection\n>\n> Final bits of \"git bisect.sh\" have been rewritten in C.\n>\n> Expecting a (hopefully final) reroll.\n> cf. <20627.86ilolhnnn.gmgdl@evledraar.gmail.com>\n\nThe msgid here is typo'd, the correct one is\nhttps://lore.kernel.org/git/220627.86ilolhnnn.gmgdl@evledraar.gmail.com/\n(i.e. a \"2\" is missing from the front).\n\nThe more recent\nhttps://lore.kernel.org/git/220817.86pmgzabv0.gmgdl@evledraar.gmail.com/\nalso has some more follow-up, and a proposed fix-up for one of the\nissues I noted with the series (the completion helper interaction with\nthe new \"bisect\").\n"},{"id":"461418","messageId":"864680d7-69e0-c392-db39-28f15bce3935@github.com","threadId":"58317","inReplyTo":"xmqq7d36s31e.fsf@gitster.g","subject":"Re: What's cooking in git.git (Aug 2022, #06; Wed, 17)","fromName":"Derrick Stolee","fromEmail":"derrickstolee@github.com","sentAt":"2022-08-18T13:18:29Z","receivedAt":"2022-08-18T13:18:37Z","isPatch":false,"sender":{"key":"stolee@gmail.com","avatar":"https://avatars.githubusercontent.com/u/570044?v=4"},"body":"On 8/17/2022 7:47 PM, Victoria Dye wrote:\n\n> (warning: long-winded tangent mostly unrelated to FSMonitor)\n> \n> In the process of testing FSMonitor behavior, I think found other issues\n> with Scalar registration. Specifically, the test I wrote attempted to\n> 'scalar register' a bare repo, since bare directories are incompatible with\n> FSMonitor. After seeing that FSMonitor was *not* incompatible with the\n> repository, I found that Scalar was 1) ignoring the bare repository and, as\n\nThis is interesting, that Scalar doesn't recognize a bare repo. There are\ndefinitely some config settings that it recommends that don't make sense\nin a bare repo, but it's interesting that it completely ignores it. Good\nfind.\n\nI'm not sure there is anything to 'fix' except maybe error out when the\ndiscovered Git repository is bare. Add a warning, at minimum.\n\n> a result, 2) identifying my Git clone (way above GIT_CEILING_DIRECTORIES) as\n> the \"enlistment root\". I think 1) might be fine as-is - uniformly ignoring\n> bare repos seems like a reasonable choice - but 2) seems like more of a\n> problem. \n\n...\n\n> The current usage of GIT_CEILING_DIRECTORIES relies on the fact that, when\n> invoking a normal 'git' command, 'setup_git_directory()' only searches\n> upwards from the current working directory to find the repo root; it's a\n> clear \"yes\" or \"no\" as to whether that search passes a ceiling directory.\n> Scalar isn't as clear, since it searches for the repo root both \"downwards\"\n> into '/src' *and* upwards through the parents of ''. It's not\n> totally clear to me what the \"right\" behavior for Scalar is, but my current\n> thought is to follow the same rules as 'setup_git_directory()', but for the\n> *enlistment* root rather than the repository root. It's more restrictive\n> than GIT_CEILING_DIRECTORIES on a normal git repo, e.g.:\n> \n> 1. 'GIT_CEILING_DIRECTORIES=somedir/enlistment git -C somedir/enlistment/src status' \n> is valid.\n> 2. 'GIT_CEILING_DIRECTORIES=somedir/enlistment scalar register somedir/enlistment/src'\n> is not valid.\n\nThis is interesting, that we can't recognize the ceiling as the root.\n\n> but since Scalar works on the entire enlistment (not just the repo inside of\n> it), I think it makes sense to prevent it from crossing a ceiling directory\n> boundary.\n\nI think the enlistment root was something that was inherited from VFS for\nGit, and we can mostly abandon it. The things we need to do are all based\non the Git repository itself, not the parent. The only thing we need to\nkeep is to allow a user to specify the repo by pointing to the directory\nimmediately above the 'src' directory.\n\n> 'scalar register somedir/enlistment/test/data' will search:\n> \n> * somedir/enlistment/test/data/src\n> * somedir/enlistment/test/data\n> * somedir/enlistment/test/src\n> * somedir/enlistment/test\n> * somedir/enlistment/src\n\nInstead, we could do the following on a specified :\n\n * If /src exists, find the Git directory by finding the first Git\n repository containing /src.\n * Otherwise, find the first Git repository containing .\n\nIs there an easy way to discover a Git repository at a specific directory?\nOr, do we do something simpler, like changing directories then calling\nsetup_git_directory()? I think simplifying the logic that way should\nrespect GIT_CEILING_DIRECTORIES correctly.\n\nThanks,\n-Stolee\n"},{"id":"461419","messageId":"d1c1ec96-8062-1eaf-c0c5-466369e7b874@github.com","threadId":"58317","inReplyTo":"864680d7-69e0-c392-db39-28f15bce3935@github.com","subject":"Re: What's cooking in git.git (Aug 2022, #06; Wed, 17)","fromName":"Derrick Stolee","fromEmail":"derrickstolee@github.com","sentAt":"2022-08-18T13:19:18Z","receivedAt":"2022-08-18T13:19:23Z","isPatch":false,"sender":{"key":"stolee@gmail.com","avatar":"https://avatars.githubusercontent.com/u/570044?v=4"},"body":"On 8/18/2022 9:18 AM, Derrick Stolee wrote:\n\nApologies, all. I sent this reply to the wrong thread.\n\n-Stolee\n"},{"id":"461429","messageId":"fbcb87f4-00fa-3371-e108-a104be7ae501@github.com","threadId":"58317","inReplyTo":"xmqq7d36s31e.fsf@gitster.g","subject":"ds/* (was Re: What's cooking in git.git (Aug 2022, #06; Wed, 17))","fromName":"Derrick Stolee","fromEmail":"derrickstolee@github.com","sentAt":"2022-08-18T15:00:18Z","receivedAt":"2022-08-18T15:00:54Z","isPatch":false,"sender":{"key":"stolee@gmail.com","avatar":"https://avatars.githubusercontent.com/u/570044?v=4"},"body":"On 8/17/2022 10:23 PM, Junio C Hamano wrote:\n\n> * ds/bundle-uri-clone (2022-08-10) 5 commits\n> - clone: --bundle-uri cannot be combined with --depth\n> - bundle-uri: add support for http(s):// and file://\n> - clone: add --bundle-uri option\n> - bundle-uri: create basic file-copy logic\n> - remote-curl: add 'get' capability\n> \n> Implement \"git clone --bundle-uri\".\n> source: \n\nThis series has been stable for a while. Please consider merging it\nto 'next' when you are ready.\n\nI have part 3 ready to go when this is merged. It will cover the\nparsing of bundle lists through the existing 'git clone --bundle-uri'\noption.\n\n> * ds/decorate-filter-tweak (2022-08-05) 11 commits\n> - fetch: use ref_namespaces during prefetch\n> - maintenance: stop writing log.excludeDecoration\n> - log: create log.initialDecorationSet=all\n> - log: add --clear-decorations option\n> - log: add default decoration filter\n> - log-tree: use ref_namespaces instead of if/else-if\n> - refs: use ref_namespaces for replace refs base\n> - refs: add array of ref namespaces\n> - t4207: test coloring of grafted decorations\n> - t4207: modernize test\n> - refs: allow \"HEAD\" as decoration filter\n> \n> The namespaces used by \"log --decorate\" from \"refs/\" hierarchy by\n> default has been tightened.\n> \n> Will merge to 'next'?\n> source: \n\nThis has been stable for a few What's Cooking mails. Should be\nready to go.\n\nThanks,\n-Stolee\n"},{"id":"461450","messageId":"xmqqilmppi2g.fsf@gitster.g","threadId":"58317","inReplyTo":"fbcb87f4-00fa-3371-e108-a104be7ae501@github.com","subject":"Re: ds/* (was Re: What's cooking in git.git (Aug 2022, #06; Wed, 17))","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2022-08-18T17:39:19Z","receivedAt":"2022-08-18T17:39:29Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Derrick Stolee writes:\n\n> On 8/17/2022 10:23 PM, Junio C Hamano wrote:\n>\n>> * ds/bundle-uri-clone (2022-08-10) 5 commits\n>> - clone: --bundle-uri cannot be combined with --depth\n>> - bundle-uri: add support for http(s):// and file://\n>> - clone: add --bundle-uri option\n>> - bundle-uri: create basic file-copy logic\n>> - remote-curl: add 'get' capability\n>> \n>> Implement \"git clone --bundle-uri\".\n>> source: \n>\n> This series has been stable for a while. Please consider merging it\n> to 'next' when you are ready.\n\nI think earlier rounds of this series were fairly well reviewed and\nthe interdiff in the cover letter has nothing questionable.\n\n>> * ds/decorate-filter-tweak (2022-08-05) 11 commits\n>> - fetch: use ref_namespaces during prefetch\n>> - maintenance: stop writing log.excludeDecoration\n>> - log: create log.initialDecorationSet=all\n>> - log: add --clear-decorations option\n>> - log: add default decoration filter\n>> - log-tree: use ref_namespaces instead of if/else-if\n>> - refs: use ref_namespaces for replace refs base\n>> - refs: add array of ref namespaces\n>> - t4207: test coloring of grafted decorations\n>> - t4207: modernize test\n>> - refs: allow \"HEAD\" as decoration filter\n>> \n>> The namespaces used by \"log --decorate\" from \"refs/\" hierarchy by\n>> default has been tightened.\n>> \n>> Will merge to 'next'?\n>> source: \n>\n> This has been stable for a few What's Cooking mails. Should be\n> ready to go.\n\nThis one I am not as confident as the other one, though. I actually\nwas thinking that this may be getting stale, not stable, without any\nfurther \"I read it once again and this round looks good\" X-<. Let\nme see if I can find time to read it over before I decide.\n\nThanks.\n\n\n\n"},{"id":"461454","messageId":"491d822c-56f0-71df-8815-91ab863ebc9d@github.com","threadId":"58317","inReplyTo":"xmqq7d36s31e.fsf@gitster.g","subject":"vd/* (was: Re: What's cooking in git.git (Aug 2022, #06; Wed, 17))","fromName":"Victoria Dye","fromEmail":"vdye@github.com","sentAt":"2022-08-18T18:12:00Z","receivedAt":"2022-08-18T18:12:07Z","isPatch":false,"sender":{"key":"vdye@github.com","avatar":"https://avatars.githubusercontent.com/u/3619353?v=4"},"body":"Junio C Hamano wrote:\n> * vd/scalar-enables-fsmonitor (2022-08-16) 5 commits\n> - scalar: update technical doc roadmap with FSMonitor support\n> - scalar unregister: stop FSMonitor daemon\n> - scalar: enable built-in FSMonitor on `register`\n> - scalar-[un]register: clearly indicate source of error\n> - scalar-unregister: handle error codes greater than 0\n> \n> \"scalar\" now enables built-in fsmonitor on enlisted repositories,\n> when able.\n> \n> Will merge to 'next'?\n> source: \n\nThis should probably be \"Expecting a reroll.\"; I have a couple of changes to\nmake in response to [1] and [2] and will send the new version later today.\n\n[1] https://lore.kernel.org/git/f5388e4d-7eb7-9333-6a8e-86ce449aced0@github.com/\n[2] https://lore.kernel.org/git/82716e5b-3522-68f5-7479-1b39811e0cb2@github.com/\n\n> * vd/scalar-generalize-diagnose (2022-08-12) 11 commits\n> - scalar: update technical doc roadmap\n> - scalar-diagnose: use 'git diagnose --mode=all'\n> - builtin/bugreport.c: create '--diagnose' option\n> - builtin/diagnose.c: add '--mode' option\n> - builtin/diagnose.c: create 'git diagnose' builtin\n> - diagnose.c: add option to configure archive contents\n> - scalar-diagnose: move functionality to common location\n> - scalar-diagnose: move 'get_disk_info()' to 'compat/'\n> - scalar-diagnose: add directory to archiver more gently\n> - scalar-diagnose: avoid 32-bit overflow of size_t\n> - scalar-diagnose: use \"$GIT_UNZIP\" in test\n> \n> The \"diagnose\" feature to create a zip archive for diagnostic\n> material has been lifted from \"scalar\" and made into a feature of\n> \"git bugreport\".\n> \n> Will merge to 'next'?\n> source: \n\nI think the review on this has stabilized (and, personally, I'm happy with\nthe current version). Assuming no late-stage reviews come in, I think this\nis ready for 'next'.\n"},{"id":"461460","messageId":"xmqq5yippdwv.fsf@gitster.g","threadId":"58317","inReplyTo":"491d822c-56f0-71df-8815-91ab863ebc9d@github.com","subject":"Re: vd/*","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2022-08-18T19:09:04Z","receivedAt":"2022-08-18T19:09:21Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Victoria Dye writes:\n\n> Junio C Hamano wrote:\n>> * vd/scalar-enables-fsmonitor (2022-08-16) 5 commits\n>> - scalar: update technical doc roadmap with FSMonitor support\n>> - scalar unregister: stop FSMonitor daemon\n>> - scalar: enable built-in FSMonitor on `register`\n>> - scalar-[un]register: clearly indicate source of error\n>> - scalar-unregister: handle error codes greater than 0\n>> \n>> \"scalar\" now enables built-in fsmonitor on enlisted repositories,\n>> when able.\n>> \n>> Will merge to 'next'?\n>> source: \n>\n> This should probably be \"Expecting a reroll.\"; I have a couple of changes to\n> make in response to [1] and [2] and will send the new version later today.\n>\n> [1] https://lore.kernel.org/git/f5388e4d-7eb7-9333-6a8e-86ce449aced0@github.com/\n> [2] https://lore.kernel.org/git/82716e5b-3522-68f5-7479-1b39811e0cb2@github.com/\n>\n>> * vd/scalar-generalize-diagnose (2022-08-12) 11 commits\n>> - scalar: update technical doc roadmap\n>> - scalar-diagnose: use 'git diagnose --mode=all'\n>> - builtin/bugreport.c: create '--diagnose' option\n>> - builtin/diagnose.c: add '--mode' option\n>> - builtin/diagnose.c: create 'git diagnose' builtin\n>> - diagnose.c: add option to configure archive contents\n>> - scalar-diagnose: move functionality to common location\n>> - scalar-diagnose: move 'get_disk_info()' to 'compat/'\n>> - scalar-diagnose: add directory to archiver more gently\n>> - scalar-diagnose: avoid 32-bit overflow of size_t\n>> - scalar-diagnose: use \"$GIT_UNZIP\" in test\n>> \n>> The \"diagnose\" feature to create a zip archive for diagnostic\n>> material has been lifted from \"scalar\" and made into a feature of\n>> \"git bugreport\".\n>> \n>> Will merge to 'next'?\n>> source: \n>\n> I think the review on this has stabilized (and, personally, I'm happy with\n> the current version). Assuming no late-stage reviews come in, I think this\n> is ready for 'next'.\n\nThanks.\n"},{"id":"461483","messageId":"20220818224222.3569397-1-jonathantanmy@google.com","threadId":"58317","inReplyTo":"xmqq7d36s31e.fsf@gitster.g","subject":"Re: What's cooking in git.git (Aug 2022, #06; Wed, 17)","fromName":"Jonathan Tan","fromEmail":"jonathantanmy@google.com","sentAt":"2022-08-18T22:42:22Z","receivedAt":"2022-08-18T22:42:29Z","isPatch":false,"sender":{"key":"jonathantanmy@fastmail.com","avatar":null},"body":"Junio C Hamano writes:\n> * en/ancestry-path-in-a-range (2022-08-17) 2 commits\n> - revision: allow --ancestry-path to take an argument\n> - rev-list-options.txt: fix simple typo\n> \n> \"git rev-list --ancestry-path=C A..B\" is a natural extension of\n> \"git rev-list A..B\"; instead of choosing a subset of A..B to those\n> that have ancestry relationship with A, it lets a subset with\n> ancestry relationship with C.\n> \n> Will merge to 'next'?\n> source: \n\nI had some minor comments [1] but besides that, overall I think this is\ngood.\n\n[1] https://lore.kernel.org/git/20220818222416.3567602-1-jonathantanmy@google.com/\n\n> * en/submodule-merge-messages-fixes (2022-08-17) 3 commits\n> - merge-ort: provide helpful submodule update message when possible\n> - merge-ort: add comment to avoid surprise with new sub_flag variable\n> - merge-ort: remove translator lego in new \"submodule conflict suggestion\"\n> (this branch uses cw/submodule-merge-messages.)\n> \n> Further update the help messages given while merging submodules.\n> \n> Will merge to 'next'?\n> source: \n\n(I reviewed a newer version than what's currently in\nhttps://github.com/gitster/git.)\n\nThis looks good. I would have rewritten the last sentence of patch 2's\ncommit message as \"Initialize it instead to the generic\nCONFLICT_SUBMODULE_FAILED_TO_MERGE value. There is no change to\nuser-visible behavior, as the only time this variable is read is when\nprinting submodule conflict information, which special-cases\nCONFLICT_SUBMODULE_NOT_INITIALIZED and\nCONFLICT_SUBMODULE_HISTORY_NOT_AVAILABLE, and treats everything else\nidentically.\", but the existing commit message is fine too.\n"},{"id":"461542","messageId":"285nq583-p3p0-q9q3-90s9-q2po503rsp0n@tzk.qr","threadId":"58317","inReplyTo":"220818.86czcxafew.gmgdl@evledraar.gmail.com","subject":"Re: js/bisect-in-c (was: What's cooking in git.git (Aug 2022, #06; Wed, 17))","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2022-08-19T11:06:14Z","receivedAt":"2022-08-19T11:07:01Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi Ævar,\n\nOn Thu, 18 Aug 2022, Ævar Arnfjörð Bjarmason wrote:\n\n> On Wed, Aug 17 2022, Junio C Hamano wrote:\n>\n> > * js/bisect-in-c (2022-06-27) 16 commits\n> > - bisect: no longer try to clean up left-over `.git/head-name` files\n> > - bisect: remove Cogito-related code\n> > - Turn `git bisect` into a full built-in\n> > - bisect: move even the command-line parsing to `bisect--helper`\n> > - bisect: teach the `bisect--helper` command to show the correct usage strings\n> > - bisect--helper: return only correct exit codes in `cmd_*()`\n> > - bisect--helper: move the `BISECT_STATE` case to the end\n> > - bisect--helper: make `--bisect-state` optional\n> > - bisect--helper: align the sub-command order with git-bisect.sh\n> > - bisect--helper: using `--bisect-state` without an argument is a bug\n> > - bisect--helper: really retire `--bisect-autostart`\n> > - bisect--helper: really retire --bisect-next-check\n> > - bisect--helper: retire the --no-log option\n> > - bisect: avoid double-quoting when printing the failed command\n> > - bisect run: fix the error message\n> > - bisect: verify that a bogus option won't try to start a bisection\n> >\n> > Final bits of \"git bisect.sh\" have been rewritten in C.\n> >\n> > Expecting a (hopefully final) reroll.\n> > cf. <20627.86ilolhnnn.gmgdl@evledraar.gmail.com>\n>\n> The msgid here is typo'd, the correct one is\n> https://lore.kernel.org/git/220627.86ilolhnnn.gmgdl@evledraar.gmail.com/\n> (i.e. a \"2\" is missing from the front).\n>\n> The more recent\n> https://lore.kernel.org/git/220817.86pmgzabv0.gmgdl@evledraar.gmail.com/\n> also has some more follow-up, and a proposed fix-up for one of the\n> issues I noted with the series (the completion helper interaction with\n> the new \"bisect\").\n\nSo now we should also include changes for the completion into this patch\nseries? Are you serious?\n\nCiao,\nJohannes\n"},{"id":"461553","messageId":"220819.86wnb4710k.gmgdl@evledraar.gmail.com","threadId":"58317","inReplyTo":"285nq583-p3p0-q9q3-90s9-q2po503rsp0n@tzk.qr","subject":"Re: js/bisect-in-c (was: What's cooking in git.git (Aug 2022, #06; Wed, 17))","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2022-08-19T14:28:03Z","receivedAt":"2022-08-19T14:37:23Z","isPatch":false,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"\nOn Fri, Aug 19 2022, Johannes Schindelin wrote:\n\n> On Thu, 18 Aug 2022, Ævar Arnfjörð Bjarmason wrote:\n>\n>> On Wed, Aug 17 2022, Junio C Hamano wrote:\n>>\n>> > * js/bisect-in-c (2022-06-27) 16 commits\n>> > - bisect: no longer try to clean up left-over `.git/head-name` files\n>> > - bisect: remove Cogito-related code\n>> > - Turn `git bisect` into a full built-in\n>> > - bisect: move even the command-line parsing to `bisect--helper`\n>> > - bisect: teach the `bisect--helper` command to show the correct usage strings\n>> > - bisect--helper: return only correct exit codes in `cmd_*()`\n>> > - bisect--helper: move the `BISECT_STATE` case to the end\n>> > - bisect--helper: make `--bisect-state` optional\n>> > - bisect--helper: align the sub-command order with git-bisect.sh\n>> > - bisect--helper: using `--bisect-state` without an argument is a bug\n>> > - bisect--helper: really retire `--bisect-autostart`\n>> > - bisect--helper: really retire --bisect-next-check\n>> > - bisect--helper: retire the --no-log option\n>> > - bisect: avoid double-quoting when printing the failed command\n>> > - bisect run: fix the error message\n>> > - bisect: verify that a bogus option won't try to start a bisection\n>> >\n>> > Final bits of \"git bisect.sh\" have been rewritten in C.\n>> >\n>> > Expecting a (hopefully final) reroll.\n>> > cf. <20627.86ilolhnnn.gmgdl@evledraar.gmail.com>\n>>\n>> The msgid here is typo'd, the correct one is\n>> https://lore.kernel.org/git/220627.86ilolhnnn.gmgdl@evledraar.gmail.com/\n>> (i.e. a \"2\" is missing from the front).\n>>\n>> The more recent\n>> https://lore.kernel.org/git/220817.86pmgzabv0.gmgdl@evledraar.gmail.com/\n>> also has some more follow-up, and a proposed fix-up for one of the\n>> issues I noted with the series (the completion helper interaction with\n>> the new \"bisect\").\n>\n> So now we should also include changes for the completion into this patch\n> series? Are you serious?\n\nThe proposed fix-up is for a part of git.c that only exists with your\nseries applied.\n\nBefore your series \"git bisect--helper\" supported\n\"--git-completion-helper\" (although not to do anything useful in\npractice).\n\nWith your series we have a \"bisect\" built-in. Built-ins by default claim\nto support \"--git-completion-helper\", but as your series also migrated\nthe code away from parse_options() it doesn't anymore.\n\nSo it makes sense for the relevant patch(es) to update the corresponding\nflag as that's changed, see also the in-flight 68494127d6c (git.c:\nupdate NO_PARSEOPT markings, 2022-07-25).\n"}]}