{"thread":{"id":"58722","subject":"What's cooking in git.git (Oct 2022, #09; Mon, 31)","startedAt":"2022-10-31T05:31:18Z","lastAt":"2022-11-11T14:52:09Z","messageCount":14,"participants":["Taylor Blau","Ævar Arnfjörð Bjarmason","Ramsay Jones","Johannes Schindelin"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"466078","messageId":"Y19dnb2M+yObnftj@nand.local","threadId":"58722","inReplyTo":null,"subject":"What's cooking in git.git (Oct 2022, #09; Mon, 31)","fromName":"Taylor Blau","fromEmail":"me@ttaylorr.com","sentAt":"2022-10-31T05:31:09Z","receivedAt":"2022-10-31T05:31:18Z","isPatch":false,"sender":{"key":"me@ttaylorr.com","avatar":"https://avatars.githubusercontent.com/u/301000140?v=4"},"body":"What's cooking in git.git (Oct 2022, #09; Mon, 31)\n--------------------------------------------------\n\nHere 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\nThis is the first pushout from the interim maintainer. Topics which\nwere marked as ready for 'master' have been merged, and 'next' is now\nempty. A number of topics have been marked for 'next' which will start\ngraduating in the next round.\n\nPlease point out any glitches you see along the way. If a topic you\nsent doesn't appear here, please nudge me or resend it.\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\thttps://github.com/git/git/\n\nThe following mirrors are currently out-of-date while the usual\nmaintainer is offline.\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://gitlab.com/git-vcs/git/\n\nWith all the integration branches and topics broken out:\n\n\thttps://github.com/ttaylorr/git/\n\nEven though the preformatted documentation in HTML and man format\nare not sources, they are typically published in these repositories\nfor convenience (replace \"htmldocs\" with \"manpages\" for the manual\n pages):\n\n\tgit://git.kernel.org/pub/scm/git/git-htmldocs.git/\n\thttps://github.com/gitster/git-htmldocs.git/\n\n...but these and the release tarballs below are similarly out-of-date:\n\n\thttps://www.kernel.org/pub/software/scm/git/\n\n--------------------------------------------------\n[Graduated to 'master']\n\n* ds/bundle-uri-3 (2022-10-12) 13 commits\n (merged to 'next' on 2022-10-28 at 9d9092b4cc)\n + bundle-uri: suppress stderr from remote-https\n + bundle-uri: quiet failed unbundlings\n + bundle: add flags to verify_bundle()\n + bundle-uri: fetch a list of bundles\n + bundle: properly clear all revision flags\n + bundle-uri: limit recursion depth for bundle lists\n + bundle-uri: parse bundle list in config format\n + bundle-uri: unit test \"key=value\" parsing\n + bundle-uri: create \"key=value\" line parsing\n + bundle-uri: create base key-value pair parsing\n + bundle-uri: create bundle_list struct and helpers\n + bundle-uri: use plain string in find_temp_filename()\n + Merge branch 'ds/bundle-uri-clone' into ds/bundle-uri-3\n\n Define the logical elements of a \"bundle list\", data structure to\n store them in-core, format to transfer them, and code to parse\n them.\n source: \n\n\n* en/merge-tree-sequence (2022-10-22) 2 commits\n (merged to 'next' on 2022-10-28 at 31459cd5a8)\n + merge-tree: support multiple batched merges with --stdin\n + merge-tree: update documentation for differences in -z output\n\n \"git merge-tree --stdin\" is a new way to request a series of merges\n and report the merge results.\n source: \n\n\n* en/ort-dir-rename-and-symlink-fix (2022-10-22) 1 commit\n (merged to 'next' on 2022-10-27 at 56f1e5222d)\n + merge-ort: fix bug with dir rename vs change dir to symlink\n\n Merging a branch with directory renames into a branch that changes\n the directory to a symlink was mishandled by the ort merge\n strategy, which has been corrected.\n source: \n\n\n* jc/doc-fsck-msgids (2022-10-25) 4 commits\n (merged to 'next' on 2022-10-28 at 3c00edabf8)\n + Documentation: add lint-fsck-msgids\n + fsck: document msg-id\n + fsck: remove the unused MISSING_TREE_OBJECT\n + fsck: remove the unused BAD_TAG_OBJECT\n\n Add documentation for message IDs in fsck error messages.\n source: <20221025224224.2352979-1-gitster@pobox.com>\n\n\n* jh/trace2-timers-and-counters (2022-10-24) 8 commits\n (merged to 'next' on 2022-10-26 at e4933e2658)\n + trace2: add global counter mechanism\n + trace2: add stopwatch timers\n + trace2: convert ctx.thread_name from strbuf to pointer\n + trace2: improve thread-name documentation in the thread-context\n + trace2: rename the thread_name argument to trace2_thread_start\n + api-trace2.txt: elminate section describing the public trace2 API\n + tr2tls: clarify TLS terminology\n + trace2: use size_t alloc,nr_open_regions in tr2tls_thread_ctx\n\n Two new facilities, \"timer\" and \"counter\", are introduced to the\n trace2 API.\n source: \n\n\n* jk/repack-tempfile-cleanup (2022-10-23) 6 commits\n (merged to 'next' on 2022-10-26 at e706eb120c)\n + t7700: annotate cruft-pack failure with ok=sigpipe\n + repack: drop remove_temporary_files()\n + repack: use tempfiles for signal cleanup\n + repack: expand error message for missing pack files\n + repack: populate extension bits incrementally\n + repack: convert \"names\" util bitfield to array\n\n The way \"git repack\" creared temporary files when it received a\n signal was prone to deadlocking, which has been corrected.\n source: \n\n\n* jr/embargoed-releases-doc (2022-10-24) 1 commit\n (merged to 'next' on 2022-10-27 at c000502eaa)\n + embargoed releases: also describe the git-security list and the process\n\n The role the security mailing list plays in an embargoed release\n has been documented.\n source: \n\n\n* jt/skipping-negotiator-wo-recursion (2022-10-25) 1 commit\n (merged to 'next' on 2022-10-28 at 4a2588ab9e)\n + negotiator/skipping: avoid stack overflow\n\n Rewrite a deep recursion in the skipping negotiator to use a loop\n with on-heap prio queue to avoid stack wastage.\n source: <20221025232934.1504445-1-jonathantanmy@google.com>\n\n\n* jz/patch-id (2022-10-24) 6 commits\n (merged to 'next' on 2022-10-24 at 1ac3b46fbe)\n + builtin: patch-id: remove unused diff-tree prefix\n + builtin: patch-id: add --verbatim as a command mode\n + patch-id: fix patch-id for mode changes\n + builtin: patch-id: fix patch-id with binary diffs\n + patch-id: use stable patch-id for rebases\n + patch-id: fix stable patch id for binary / header-only\n\n A new \"--include-whitespace\" option is added to \"git patch-id\", and\n existing bugs in the internal patch-id logic that did not match\n what \"git patch-id\" produces have been corrected.\n source: \n\n\n* pb/subtree-split-and-merge-after-squashing-tag-fix (2022-10-21) 9 commits\n (merged to 'next' on 2022-10-27 at 4f2134dd87)\n + subtree: fix split after annotated tag was squashed merged\n + subtree: fix squash merging after annotated tag was squashed merged\n + subtree: process 'git-subtree-split' trailer in separate function\n + subtree: use named variables instead of \"$@\" in cmd_pull\n + subtree: define a variable before its first use in 'find_latest_squash'\n + subtree: prefix die messages with 'fatal'\n + subtree: add 'die_incompatible_opt' function to reduce duplication\n + subtree: use 'git rev-parse --verify [--quiet]' for better error messages\n + test-lib-functions: mark 'test_commit' variables as 'local'\n\n A bugfix to \"git subtree\" in its split and merge features.\n source: \n\n\n* pw/rebase-keep-base-fixes (2022-10-17) 8 commits\n (merged to 'next' on 2022-10-27 at 802359afac)\n + rebase --keep-base: imply --no-fork-point\n + rebase --keep-base: imply --reapply-cherry-picks\n + rebase: factor out branch_base calculation\n + rebase: rename merge_base to branch_base\n + rebase: store orig_head as a commit\n + rebase: be stricter when reading state files containing oids\n + t3416: set $EDITOR in subshell\n + t3416: tighten two tests\n (this branch is used by pw/rebase-reflog-fixes.)\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 source: \n\n\n* pw/rebase-reflog-fixes (2022-10-17) 9 commits\n (merged to 'next' on 2022-10-27 at 60738821ef)\n + rebase: cleanup action handling\n + rebase --abort: improve reflog message\n + rebase --apply: make reflog messages match rebase --merge\n + rebase --apply: respect GIT_REFLOG_ACTION\n + rebase --merge: fix reflog message after skipping\n + rebase --merge: fix reflog when continuing\n + t3406: rework rebase reflog tests\n + rebase --apply: remove duplicated code\n + Merge branch 'pw/rebase-keep-base-fixes' into pw/rebase-reflog-fixes\n (this branch uses pw/rebase-keep-base-fixes.)\n\n Fix some bugs in the reflog messages when rebasing and changes the\n reflog messages of \"rebase --apply\" to match \"rebase --merge\" with\n the aim of making the reflog easier to parse.\n source: \n\n\n* rj/branch-copy-rename-error-codepath-cleanup (2022-10-26) 1 commit\n (merged to 'next' on 2022-10-27 at f01a4ff619)\n + branch: error copying or renaming a detached HEAD\n\n Code simplification.\n source: <0ac8cd48-08d7-9bdd-b074-c8d5ded522f6@gmail.com>\n\n\n* rj/branch-do-not-exit-with-minus-one-status (2022-10-26) 1 commit\n (merged to 'next' on 2022-10-27 at 061f63d4e2)\n + branch: error code with --edit-description\n\n \"git branch --edit-description\" can exit with status -1 which is\n not a good practice; it learned to use 1 as everybody else instead.\n source: \n\n\n* rs/absorb-git-dir-simplify (2022-10-23) 1 commit\n (merged to 'next' on 2022-10-26 at 3d23cfd399)\n + submodule: use strvec_pushf() for --super-prefix\n\n Code simplification by using strvec_pushf() instead of building an\n argument in a separate strbuf.\n source: <7a4e2fc6-3e01-5683-2be5-13b7e67c7fe5@web.de>\n\n\n* sd/doc-smtp-encryption (2022-10-12) 1 commit\n (merged to 'next' on 2022-10-26 at b984763a1b)\n + docs: git-send-email: difference between ssl and tls smtp-encryption\n\n Will merge to 'master'.\n source: <20221012150619.12877-1-sndanailov@wired4ever.net>\n\n\n* sg/stable-docdep (2022-10-21) 1 commit\n (merged to 'next' on 2022-10-26 at 68432e1b2c)\n + Documentation/build-docdep.perl: generate sorted output\n\n Make sure generated dependency file is stably sorted to help\n developers debugging their build issues.\n source: <20221021102950.539148-1-szeder.dev@gmail.com>\n\n\n* tb/cap-patch-at-1gb (2022-10-25) 1 commit\n (merged to 'next' on 2022-10-27 at f0b4f9c12a)\n + apply: reject patches larger than ~1 GiB\n\n \"git apply\" limits its input to a bit less than 1 GiB.\n source: <70f5763834dff373a5573a99ec4cdfa36cadf34c.1666722251.git.me@ttaylorr.com>\n\n\n* tb/midx-cleanup-fix (2022-10-25) 1 commit\n - midx.c: clear auxiliary MIDX files first\n\n The order in which multi-pack-index and its associated files are\n dropped has been tweaked to make it safer for concurrent users.\n\n Under discussion, but leaning to negative..\n cf. <143a588a-c98b-733b-2b23-34a87ca89431@github.com>\n source: \n\n\n* tb/shortlog-group (2022-10-24) 7 commits\n (merged to 'next' on 2022-10-26 at 76e64a6036)\n + shortlog: implement `--group=committer` in terms of `--group=`\n + shortlog: implement `--group=author` in terms of `--group=`\n + shortlog: extract `shortlog_finish_setup()`\n + shortlog: support arbitrary commit format `--group`s\n + shortlog: extract `--group` fragment for translation\n + shortlog: make trailer insertion a noop when appropriate\n + shortlog: accept `--date`-related options\n\n \"git shortlog\" learned to group by the \"format\" string.\n source: \n\n--------------------------------------------------\n[New Topics]\n\n* ab/cmake-nix-and-ci (2022-10-30) 11 commits\n - CI: add a \"linux-cmake-test\" to run cmake & ctest on linux\n - cmake: copy over git-p4.py for t983[56] perforce test\n - cmake: support GIT_TEST_OPTS, abstract away WIN32 defaults\n - Makefile + cmake: use environment, not GIT-BUILD-DIR\n - test-lib.sh: support a \"GIT_TEST_BUILD_DIR\"\n - cmake: set \"USE_LIBPCRE2\" in \"GIT-BUILD-OPTIONS\" for test-lib.sh\n - cmake & test-lib.sh: add a $GIT_SOURCE_DIR variable\n - cmake: chmod +x the bin-wrappers/* & SCRIPT_{SH,PERL} & git-p4\n - cmake: don't copy chainlint.pl to build directory\n - cmake: update instructions for portable CMakeLists.txt\n - cmake: don't \"mkdir -p\" and \"cd\" in build instructions\n\n Fix assorted issues with CTest on *nix machines.\n\n Waiting for review.\n source: \n\n\n* ab/make-bin-wrappers (2022-10-30) 3 commits\n - Makefile: simplify $(test_bindir_programs) rule by splitting it up\n - Makefile: define \"TEST_{PROGRAM,OBJS}\" variables earlier\n - Makefile: factor sed-powered '#!/bin/sh' munging into a variable\n\n Resolve issues with the bin-wrappers/% rules where \"make\n bin-wrappers/git\" would generate the script but not \"git\" itself.\n\n Waiting for review.\n source: \n\n\n* ab/misc-hook-submodule-run-command (2022-10-31) 3 commits\n - run-command tests: test stdout of run_command_parallel()\n - submodule tests: reset \"trace.out\" between \"grep\" invocations\n - hook tests: fix redirection logic error in 96e7225b310\n\n Various test updates.\n\n Waiting for review.\n source: \n\n\n* do/modernize-t7001 (2022-10-31) 1 commit\n - t7001-mv.sh:modernizing test script using function\n\n Modernize test script to avoid \"test -f\" and friends.\n\n Will merge to 'next'.\n source: \n\n\n* kz/merge-tree-merge-base (2022-10-29) 1 commit\n - merge-tree.c: add --merge-base= option\n\n \"merge-tree\" learns a new `--merge-base` option.\n\n Waiting for review.\n source: \n\n\n* mh/password-can-be-pat (2022-10-30) 1 commit\n - Mention that password could be a personal access token.\n\n Documentation update to git-credential(1).\n\n Will merge to 'next'.\n source: \n\n\n* po/pretty-hard-trunc (2022-10-30) 1 commit\n - pretty-formats: add hard truncation, without ellipsis, options\n\n Add a new pretty format which truncates without ellipsis.\n\n Missing test coverage.\n source: <20221030185614.3842-1-philipoakley@iee.email>\n\n\n* rr/long-status-advice (2022-10-31) 1 commit\n - status: long status advice adapted to recent capabilities\n\n The advice message emitted by a slow \"status\" run is amended to\n mention fsmonitor.\n\n Waiting for reviewer feedback on the updated round.\n source: \n\n\n* rs/archive-filter-error-once (2022-10-30) 1 commit\n - archive-tar: report filter start error only once\n\n \"git archive\" mistakenly complained twice about a missing executable,\n which has been corrected.\n\n Will merge to 'next'.\n source: \n\n--------------------------------------------------\n[Stalled]\n\n* cw/submodule-status-in-parallel (2022-10-20) 7 commits\n . diff-lib: parallelize run_diff_files for submodules\n . diff-lib: refactor match_stat_with_submodule\n . submodule: move status parsing into function\n . submodule: strbuf variable rename\n . run-command: add hide_output to run_processes_parallel_opts\n . run-command: add pipe_output_fn to run_processes_parallel_opts\n . Merge branch 'ab/run-hook-api-cleanup' into cw/submodule-status-in-parallel\n\n Allow the internal \"diff-files\" engine to run \"how has this\n submodule changed?\" in parallel to speed up \"git status\".\n\n Breaks winVS test?\n cf. \n source: <20221011232604.839941-1-calvinwan@google.com>\n\n\n* js/bisect-in-c (2022-08-30) 17 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--helper: make `state` optional\n . bisect--helper: calling `bisect_state()` without an argument is a bug\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 . bisect--helper: migrate to OPT_SUBCOMMAND()\n . bisect--helper: make the order consistently `argc, argv`\n . bisect--helper: make `terms` an explicit singleton\n . bisect--helper: simplify exit code computation\n . bisect--helper: really retire `--bisect-autostart`\n . bisect--helper: really retire --bisect-next-check\n . bisect--helper: retire the --no-log option\n . Merge branch 'sg/parse-options-subcommand' into js/bisect-in-c\n\n Final bits of \"git bisect.sh\" have been rewritten in C.\n\n Needs review.\n cf. \n source: \n\n\n* od/ci-use-checkout-v3-when-applicable (2022-10-10) 2 commits\n . ci(main): linux32 uses actions/checkout@v2\n . ci(main): upgrade actions/checkout to v3\n\n Attempt to update GitHub CI to use actions/checkout@v3\n\n Expecting a reroll.\n Seems to break the CI completely.\n source: \n\n\n* ed/fsmonitor-inotify (2022-10-14) 7 commits\n . fsmonitor: update doc for Linux\n . fsmonitor: test updates\n . fsmonitor: enable fsmonitor for Linux\n . fsmonitor: implement filesystem change listener for Linux\n . fsmonitor: determine if filesystem is local or remote\n . fsmonitor: prepare to share code between Mac OS and Linux\n . Merge branch 'ed/fsmonitor-on-networked-macos' into ed/fsmonitor-inotify\n\n Bundled fsmonitor for Linux using inotify API.\n\n Needs review.\n Occasional breakages of t7527.16?\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\n Needs more work.\n At the minimum, we should lose 11/14 and possibly 08/14.\n cf. \n source: <20220809185429.20098-1-alban.gruin@gmail.com>\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 by somebody more familiar with the logic\n cf. \n source: <7229p500-p2r4-on87-6802-8o90s36rr3s4@tzk.qr>\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 Under SANITIZE=address, t1006-cat-file.sh finds a breakage.\n cf. <20220728230210.2952731-1-calvinwan@google.com>\n cf. \n source: <20220728230210.2952731-1-calvinwan@google.com>\n\n--------------------------------------------------\n[Cooking]\n\n* ab/config-multi-and-nonbool (2022-10-27) 10 commits\n - for-each-repo: with bad config, don't conflate and \n - config API: add \"string\" version of *_value_multi(), fix segfaults\n - config tests: add \"NULL\" tests for *_get_value_multi()\n - config API: add and use \"lookup_value\" functions\n - builtin/gc.c: use \"unsorted_string_list_has_string()\" where appropriate\n - string-list API: make has_string() and list_lookup() \"const\"\n - string-list API: mark \"struct_string_list\" to \"for_each_string_list\" const\n - config API: mark *_multi() with RESULT_MUST_BE_USED\n - for-each-repo: error on bad --config\n - config API: have *_multi() return an \"int\" and take a \"dest\"\n\n A mixed bag of config API updates.\n\n Expecting a reroll.\n cf. <221026.86pmeebcj9.gmgdl@evledraar.gmail.com>\n source: \n\n\n* ab/sha-makefile-doc (2022-10-26) 9 commits\n - Makefile: discuss SHAttered in *_SHA{1,256} discussion\n - Makefile: document default SHA-1 backend on OSX\n - Makefile: document SHA-1 and SHA-256 default and selection order\n - Makefile: document default SHA-256 backend\n - Makefile: rephrase the discussion of *_SHA1 knobs\n - Makefile: create and use sections for \"define\" flag listing\n - Makefile: correct DC_SHA1 documentation\n - INSTALL: remove discussion of SHA-1 backends\n - Makefile: always (re)set DC_SHA1 on fallback\n\n Makefile comments updates and reordering to clarify knobs used to\n choose SHA implementations.\n\n Will merge to 'next'?\n source: \n\n\n* rs/no-more-run-command-v (2022-10-30) 12 commits\n - replace and remove run_command_v_opt()\n - replace and remove run_command_v_opt_cd_env_tr2()\n - replace and remove run_command_v_opt_tr2()\n - replace and remove run_command_v_opt_cd_env()\n - use child_process members \"args\" and \"env\" directly\n - use child_process member \"args\" instead of string array variable\n - sequencer: simplify building argument list in do_exec()\n - bisect--helper: factor out do_bisect_run()\n - bisect: simplify building \"checkout\" argument list\n - am: simplify building \"show\" argument list\n - run-command: fix return value comment\n - merge: remove always-the-same \"verbose\" arguments\n\n Simplify the run-command API.\n\n Will merge to 'next'.\n source: \n\n\n* tb/howto-using-redo-script (2022-10-26) 1 commit\n - Documentation/howto/maintain-git.txt: fix Meta/redo-jch.sh invocation\n\n Doc update.\n\n Will merge to 'next'.\n source: <4ba057094ae6b1bd5c18583f23f7f99232034c72.1666815325.git.me@ttaylorr.com>\n\n\n* ps/receive-use-only-advertised (2022-10-28) 3 commits\n - SQUASH - leakfix\n - receive-pack: use advertised reference tips to inform connectivity check\n - connected: allow supplying different view of reachable objects\n\n \"git receive-pack\" used to use all the local refs as the boundary\n for checking connectivity of the data \"git push\" sent, but now it\n uses only the refs that it advertised to the pusher. In a\n repository with the .hideRefs configuration, this reduces the\n resource needed to perform the check, and also forces the pusher to\n prove they have all objects that are necessary to complete the\n history on top of only the history available to them.\n\n Expecting a reroll.\n cf. <221028.86bkpw805n.gmgdl@evledraar.gmail.com>\n cf. \n source: \n\n\n* jc/set-gid-bit-less-aggressively (2022-10-28) 1 commit\n - adjust_shared_perm(): leave g+s alone when the group does not matter\n\n The adjust_shared_perm() helper function learned to refrain from\n setting the \"g+s\" bit on directories when it is not necessary.\n\n Will merge to 'next'.\n source: \n\n\n* gc/submodule-clone-update-with-branches (2022-10-30) 8 commits\n - clone, submodule update: create and check out branches\n - submodule--helper: remove update_data.suboid\n - submodule update: refactor update targets\n - submodule: return target of submodule symref\n - t5617: drop references to remote-tracking branches\n - submodule--helper clone: create named branch\n - repo-settings: add submodule_propagate_branches\n - clone: teach --detach option\n\n \"git clone --recurse-submodules\" and \"git submodule update\" learns\n to honor the \"propagete branches\" option.\n\n Waiting for review.\n source: \n\n\n* es/mark-gc-cruft-as-experimental (2022-10-26) 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 Will merge to 'next'.\n source: \n\n\n* pw/config-int-parse-fixes (2022-10-22) 3 commits\n - git_parse_signed(): avoid integer overflow\n - config: require at least one digit when parsing numbers\n - git_parse_unsigned: reject negative values\n\n Assorted fixes of parsing end-user input as integers.\n\n Expecting a reroll to add test coverage.\n cf. \n source: \n\n\n* sa/cat-file-mailmap (2022-10-21) 1 commit\n - doc/cat-file: allow --use-mailmap for --batch options\n\n Doc updates.\n\n Waiting for review response.\n source: <20221021103442.202759-1-siddharthasthana31@gmail.com>\n\n\n* tb/repack-expire-to (2022-10-24) 4 commits\n - builtin/repack.c: implement `--expire-to` for storing pruned objects\n - builtin/repack.c: write cruft packs to arbitrary locations\n - builtin/repack.c: pass \"cruft_expiration\" to `write_cruft_pack`\n - builtin/repack.c: pass \"out\" to `prepare_pack_objects`\n\n \"git repack\" learns to send cruft objects out of the way into\n packfiles outside the repository.\n\n Waiting for review.\n source: \n\n\n* cc/filtered-repack (2022-10-25) 2 commits\n - repack: add --filter= option\n - pack-objects: allow --filter without --stdout\n\n \"git repack\" learns to discard objects that ought to be retrievable\n again from the promissor remote.\n\n Needs review.\n source: <20221025122856.20204-1-christian.couder@gmail.com>\n\n\n* al/trace2-clearing-skip-worktree (2022-10-28) 2 commits\n - SQUASH???\n - index: add trace2 region for clear skip worktree\n\n Add trace2 counters to the region to clear skip worktree bits in a\n sparse checkout.\n\n Expecting a reroll?\n source: \n\n\n* po/glossary-around-traversal (2022-10-30) 4 commits\n - glossary: add reachability bitmap description\n - glossary: add \"commit graph\" description\n - doc: use 'object database' not ODB or abbreviation\n - doc: use \"commit-graph\" hyphenation consistently\n\n The glossary entries for \"commit-graph file\" and \"reachability\n bitmap\" have been added.\n\n Will merge to 'next'.\n source: <20221029164112.2097-1-philipoakley@iee.email>\n\n\n* mc/credential-helper-auth-headers (2022-10-21) 6 commits\n - t5556-http-auth: add test for HTTP auth hdr logic\n - http: set specific auth scheme depending on credential\n - http: move proactive auth to first slot creation\n - http: store all request headers on active_request_slot\n - credential: add WWW-Authenticate header to cred requests\n - http: read HTTP WWW-Authenticate response headers\n\n Extending credential helper protocol.\n\n Needs review.\n source: \n\n\n* hl/archive-recursive (2022-10-19) 10 commits\n - fixup! archive: add tests for git archive --recurse-submodules\n - archive: add tests for git archive --recurse-submodules\n - archive: add --recurse-submodules to git-archive command\n - archive: remove global repository from archive_args\n - archive: pass repo objects to write_archive handlers\n - tree: add repository parameter to read_tree_fn_t\n - tree: handle submodule case for read_tree_at properly\n - tree: increase test coverage for tree.c\n - tree: update cases to use repo_ tree methods\n - tree: do not use the_repository for tree traversal methods.\n\n \"git archive\" has been taught \"--recurse-submodules\" option to\n create a tarball that includes contents from submodules.\n\n Expecting a reroll.\n Seems to break win+VS test(8).\n cf. https://github.com/git/git/actions/runs/3293333066 whose only\n difference from https://github.com/git/git/actions/runs/3293553109\n is the inclusion of this topic.\n source: \n\n\n* en/sparse-checkout-design (2022-10-08) 1 commit\n - sparse-checkout.txt: new document with sparse-checkout directions\n\n Design doc.\n\n Needs review.\n source: \n\n\n* pw/test-todo (2022-10-06) 3 commits\n - test_todo: allow [verbose] test as the command\n - test_todo: allow [!] grep as the command\n - tests: add test_todo() to mark known breakages\n\n RFC for test framework improvement.\n\n Needs review.\n source: \n\n\n* ab/coccicheck-incremental (2022-10-26) 12 commits\n - spatchcache: add a ccache-alike for \"spatch\"\n - cocci: run against a generated ALL.cocci\n - cocci rules: remove 's from rules that don't need them\n - cocci: optimistically use COMPUTE_HEADER_DEPENDENCIES\n - cocci: make \"coccicheck\" rule incremental\n - cocci: split off \"--all-includes\" from SPATCH_FLAGS\n - cocci: split off include-less \"tests\" from SPATCH_FLAGS\n - Makefile: split off SPATCH_BATCH_SIZE comment from \"cocci\" heading\n - Makefile: have \"coccicheck\" re-run if flags change\n - Makefile: add ability to TAB-complete cocci *.patch rules\n - cocci rules: remove unused \"F\" metavariable from pending rule\n - Makefile + shared.mak: rename and indent $(QUIET_SPATCH_T)\n\n \"make coccicheck\" is time consuming. It has been made to run more\n incrementally.\n\n Will merge to 'next'?\n source: \n"},{"id":"466134","messageId":"221101.86r0yn5zhp.gmgdl@evledraar.gmail.com","threadId":"58722","inReplyTo":"Y19dnb2M+yObnftj@nand.local","subject":"ab/cmake-nix-and-ci (was: What's cooking in git.git (Oct 2022, #09; Mon, 31))","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2022-10-31T23:44:32Z","receivedAt":"2022-10-31T23:45:31Z","isPatch":false,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"\nOn Mon, Oct 31 2022, Taylor Blau wrote:\n\n> * ab/cmake-nix-and-ci (2022-10-30) 11 commits\n> - CI: add a \"linux-cmake-test\" to run cmake & ctest on linux\n> - cmake: copy over git-p4.py for t983[56] perforce test\n> - cmake: support GIT_TEST_OPTS, abstract away WIN32 defaults\n> - Makefile + cmake: use environment, not GIT-BUILD-DIR\n> - test-lib.sh: support a \"GIT_TEST_BUILD_DIR\"\n> - cmake: set \"USE_LIBPCRE2\" in \"GIT-BUILD-OPTIONS\" for test-lib.sh\n> - cmake & test-lib.sh: add a $GIT_SOURCE_DIR variable\n> - cmake: chmod +x the bin-wrappers/* & SCRIPT_{SH,PERL} & git-p4\n> - cmake: don't copy chainlint.pl to build directory\n> - cmake: update instructions for portable CMakeLists.txt\n> - cmake: don't \"mkdir -p\" and \"cd\" in build instructions\n>\n> Fix assorted issues with CTest on *nix machines.\n>\n> Waiting for review.\n> source: \n\nI'll send a v3 of this (currently waiting on CI to finish).\n"},{"id":"466135","messageId":"221101.86mt9b5zf3.gmgdl@evledraar.gmail.com","threadId":"58722","inReplyTo":"Y19dnb2M+yObnftj@nand.local","subject":"ab/make-bin-wrappers (was: What's cooking in git.git (Oct 2022, #09; Mon, 31))","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2022-10-31T23:45:24Z","receivedAt":"2022-10-31T23:47:01Z","isPatch":false,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"\nOn Mon, Oct 31 2022, Taylor Blau wrote:\n\n> * ab/make-bin-wrappers (2022-10-30) 3 commits\n> - Makefile: simplify $(test_bindir_programs) rule by splitting it up\n> - Makefile: define \"TEST_{PROGRAM,OBJS}\" variables earlier\n> - Makefile: factor sed-powered '#!/bin/sh' munging into a variable\n>\n> Resolve issues with the bin-wrappers/% rules where \"make\n> bin-wrappers/git\" would generate the script but not \"git\" itself.\n>\n> Waiting for review.\n> source: \n\nI've since sent a v3, hopefully addressing outstanding feedback:\nhttps://lore.kernel.org/git/cover-v3-0.4-00000000000-20221031T222249Z-avarab@gmail.com/\n"},{"id":"466139","messageId":"221101.86iljz5z7l.gmgdl@evledraar.gmail.com","threadId":"58722","inReplyTo":"Y19dnb2M+yObnftj@nand.local","subject":"ab/misc-hook-submodule-run-command (was: What's cooking in git.git (Oct 2022, #09; Mon, 31))","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2022-10-31T23:46:57Z","receivedAt":"2022-10-31T23:51:32Z","isPatch":false,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"\nOn Mon, Oct 31 2022, Taylor Blau wrote:\n\n> * ab/misc-hook-submodule-run-command (2022-10-31) 3 commits\n> - run-command tests: test stdout of run_command_parallel()\n> - submodule tests: reset \"trace.out\" between \"grep\" invocations\n> - hook tests: fix redirection logic error in 96e7225b310\n>\n> Various test updates.\n>\n> Waiting for review.\n> source: \n\nI think per\nhttps://lore.kernel.org/git/221031.86tu3k6u2d.gmgdl@evledraar.gmail.com/\n&\nhttps://lore.kernel.org/git/221031.86pme86tcg.gmgdl@evledraar.gmail.com/\nthat this is ready to advance.\n\nIf you don't agree about the 2/3 let's just drop that one. I don't\nreally care enough to start some alternate rewrite proposals for that\nrandom test which I didn't add in the first place, so I don't really\nhave the motivation to fix it. It just seemed like an easy enough\npost-cleanup for an earlier landed topic.\n"},{"id":"466143","messageId":"Y2BlK5uhCy77Ot4Z@nand.local","threadId":"58722","inReplyTo":"221101.86iljz5z7l.gmgdl@evledraar.gmail.com","subject":"Re: ab/misc-hook-submodule-run-command (was: What's cooking in git.git (Oct 2022, #09; Mon, 31))","fromName":"Taylor Blau","fromEmail":"me@ttaylorr.com","sentAt":"2022-11-01T00:15:39Z","receivedAt":"2022-11-01T00:15:48Z","isPatch":false,"sender":{"key":"me@ttaylorr.com","avatar":"https://avatars.githubusercontent.com/u/301000140?v=4"},"body":"On Tue, Nov 01, 2022 at 12:46:57AM +0100, Ævar Arnfjörð Bjarmason wrote:\n>\n> On Mon, Oct 31 2022, Taylor Blau wrote:\n>\n> > * ab/misc-hook-submodule-run-command (2022-10-31) 3 commits\n> > - run-command tests: test stdout of run_command_parallel()\n> > - submodule tests: reset \"trace.out\" between \"grep\" invocations\n> > - hook tests: fix redirection logic error in 96e7225b310\n> >\n> > Various test updates.\n> >\n> > Waiting for review.\n> > source: \n>\n> I think per\n> https://lore.kernel.org/git/221031.86tu3k6u2d.gmgdl@evledraar.gmail.com/\n> &\n> https://lore.kernel.org/git/221031.86pme86tcg.gmgdl@evledraar.gmail.com/\n> that this is ready to advance.\n\nYeah, I agree. I was mistaken that GIT_TRACE truncated its output when\nredirected to a file, but it appends. So, the series looks fine to me.\n\nI would still like to hear from Emily, but otherwise I don't have a\nproblem starting to merge this down.\n\nThanks,\nTaylor\n"},{"id":"466322","messageId":"eee42f09-32ca-a254-d362-27be1045c558@ramsayjones.plus.com","threadId":"58722","inReplyTo":"Y19dnb2M+yObnftj@nand.local","subject":"Re: What's cooking in git.git (Oct 2022, #09; Mon, 31)","fromName":"Ramsay Jones","fromEmail":"ramsay@ramsayjones.plus.com","sentAt":"2022-11-02T16:49:47Z","receivedAt":"2022-11-02T16:53:04Z","isPatch":false,"sender":{"key":"ramsay@ramsayjones.plus.com","avatar":"https://avatars.githubusercontent.com/u/33702710?v=4"},"body":"\n\nOn 31/10/2022 05:31, Taylor Blau wrote:\n> What's cooking in git.git (Oct 2022, #09; Mon, 31)\n> --------------------------------------------------\n[snip]\n> \n> The following mirrors are currently out-of-date while the usual\n> maintainer is offline.\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://gitlab.com/git-vcs/git/\n> \n\nAh, I wondered why it had gone quiet ... :)\n\nOn Linux (same on 32-bit Linux and cygwin):\n\n $ git config remote.origin.url\n git://git.kernel.org/pub/scm/git/git.git\n $ \n\n[yes, an old curmudgeon!]\n\nWell, I guess I should just enjoy the silence until\nJunio returns. :D\n\nATB,\nRamsay Jones\n\n\n\n\n"},{"id":"466323","messageId":"221102.86h6zh2rxf.gmgdl@evledraar.gmail.com","threadId":"58722","inReplyTo":"Y2BlK5uhCy77Ot4Z@nand.local","subject":"Re: ab/misc-hook-submodule-run-command (was: What's cooking in git.git (Oct 2022, #09; Mon, 31))","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2022-11-02T17:13:51Z","receivedAt":"2022-11-02T17:21:39Z","isPatch":false,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"\nOn Mon, Oct 31 2022, Taylor Blau wrote:\n\n> On Tue, Nov 01, 2022 at 12:46:57AM +0100, Ævar Arnfjörð Bjarmason wrote:\n>>\n>> On Mon, Oct 31 2022, Taylor Blau wrote:\n>>\n>> > * ab/misc-hook-submodule-run-command (2022-10-31) 3 commits\n>> > - run-command tests: test stdout of run_command_parallel()\n>> > - submodule tests: reset \"trace.out\" between \"grep\" invocations\n>> > - hook tests: fix redirection logic error in 96e7225b310\n>> >\n>> > Various test updates.\n>> >\n>> > Waiting for review.\n>> > source: \n>>\n>> I think per\n>> https://lore.kernel.org/git/221031.86tu3k6u2d.gmgdl@evledraar.gmail.com/\n>> &\n>> https://lore.kernel.org/git/221031.86pme86tcg.gmgdl@evledraar.gmail.com/\n>> that this is ready to advance.\n>\n> Yeah, I agree. I was mistaken that GIT_TRACE truncated its output when\n> redirected to a file, but it appends. So, the series looks fine to me.\n>\n> I would still like to hear from Emily, but otherwise I don't have a\n> problem starting to merge this down.\n\n(Just looking at what I can & can't unblock)\n\nIn case that's a \"let's hear from the author\" (understandable) then this\nis entirely my work, even if it's her in the \"author\" header.\n\nHer last version of tha commit was[1], but this entire test is from\nmy[2] re-roll of that topic[3].\n\n1. https://lore.kernel.org/git/20210527000856.695702-8-emilyshaffer@google.com/\n2. https://lore.kernel.org/git/patch-03.31-1ad4e69f7da-20210528T110515Z-avarab@gmail.com/\n3. https://lore.kernel.org/git/cover-00.31-00000000000-20210528T110515Z-avarab@gmail.com/\n"},{"id":"466324","messageId":"221102.86cza52rt0.gmgdl@evledraar.gmail.com","threadId":"58722","inReplyTo":"Y19dnb2M+yObnftj@nand.local","subject":"js/bisect-in-c (was: What's cooking in git.git (Oct 2022, #09; Mon, 31))","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2022-11-02T17:22:17Z","receivedAt":"2022-11-02T17:24:17Z","isPatch":false,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"\nOn Mon, Oct 31 2022, Taylor Blau wrote:\n\n> What's cooking in git.git (Oct 2022, #09; Mon, 31)\n> --------------------------------------------------\n>\n> * js/bisect-in-c (2022-08-30) 17 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--helper: make `state` optional\n> . bisect--helper: calling `bisect_state()` without an argument is a bug\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> . bisect--helper: migrate to OPT_SUBCOMMAND()\n> . bisect--helper: make the order consistently `argc, argv`\n> . bisect--helper: make `terms` an explicit singleton\n> . bisect--helper: simplify exit code computation\n> . bisect--helper: really retire `--bisect-autostart`\n> . bisect--helper: really retire --bisect-next-check\n> . bisect--helper: retire the --no-log option\n> . Merge branch 'sg/parse-options-subcommand' into js/bisect-in-c\n>\n> Final bits of \"git bisect.sh\" have been rewritten in C.\n>\n> Needs review.\n> cf. \n> source: \n\nI see this has been ejected out of \"seen\", presumably due to the\noutstanding conflicts.\n\nTo the extent that Johannes doesn't have time I'd by happy to re-roll\nthis & look at it/fix whatever outstanding issues I spot.\n\nI gave it a once over a few weeks ago, and didn't see any outstanding\nproblems with it worth rewriting in that iteration, but didn't have time\nto give it stress testing to see if there were still some lurking\nunexpected/untested behavior changes.\n\nJohannes?\n"},{"id":"466325","messageId":"Y2Kn5MVQojz33cta@nand.local","threadId":"58722","inReplyTo":"221102.86h6zh2rxf.gmgdl@evledraar.gmail.com","subject":"Re: ab/misc-hook-submodule-run-command (was: What's cooking in git.git (Oct 2022, #09; Mon, 31))","fromName":"Taylor Blau","fromEmail":"me@ttaylorr.com","sentAt":"2022-11-02T17:24:52Z","receivedAt":"2022-11-02T17:24:58Z","isPatch":false,"sender":{"key":"me@ttaylorr.com","avatar":"https://avatars.githubusercontent.com/u/301000140?v=4"},"body":"On Wed, Nov 02, 2022 at 06:13:51PM +0100, Ævar Arnfjörð Bjarmason wrote:\n> > Yeah, I agree. I was mistaken that GIT_TRACE truncated its output when\n> > redirected to a file, but it appends. So, the series looks fine to me.\n> >\n> > I would still like to hear from Emily, but otherwise I don't have a\n> > problem starting to merge this down.\n>\n> (Just looking at what I can & can't unblock)\n>\n> In case that's a \"let's hear from the author\" (understandable) then this\n> is entirely my work, even if it's her in the \"author\" header.\n>\n> Her last version of tha commit was[1], but this entire test is from\n> my[2] re-roll of that topic[3].\n\nFair enough, I didn't realize the original topic in [1] was so old.\n\nLet's merge this down.\n\nThanks,\nTaylor\n\n> 1. https://lore.kernel.org/git/20210527000856.695702-8-emilyshaffer@google.com/\n> 2. https://lore.kernel.org/git/patch-03.31-1ad4e69f7da-20210528T110515Z-avarab@gmail.com/\n> 3. https://lore.kernel.org/git/cover-00.31-00000000000-20210528T110515Z-avarab@gmail.com/\n"},{"id":"466326","messageId":"Y2KoKO5Sj71Nfa61@nand.local","threadId":"58722","inReplyTo":"221102.86cza52rt0.gmgdl@evledraar.gmail.com","subject":"Re: js/bisect-in-c (was: What's cooking in git.git (Oct 2022, #09; Mon, 31))","fromName":"Taylor Blau","fromEmail":"me@ttaylorr.com","sentAt":"2022-11-02T17:26:00Z","receivedAt":"2022-11-02T17:26:05Z","isPatch":false,"sender":{"key":"me@ttaylorr.com","avatar":"https://avatars.githubusercontent.com/u/301000140?v=4"},"body":"On Wed, Nov 02, 2022 at 06:22:17PM +0100, Ævar Arnfjörð Bjarmason wrote:\n>\n> On Mon, Oct 31 2022, Taylor Blau wrote:\n>\n> > What's cooking in git.git (Oct 2022, #09; Mon, 31)\n> > --------------------------------------------------\n> >\n> > * js/bisect-in-c (2022-08-30) 17 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--helper: make `state` optional\n> > . bisect--helper: calling `bisect_state()` without an argument is a bug\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> > . bisect--helper: migrate to OPT_SUBCOMMAND()\n> > . bisect--helper: make the order consistently `argc, argv`\n> > . bisect--helper: make `terms` an explicit singleton\n> > . bisect--helper: simplify exit code computation\n> > . bisect--helper: really retire `--bisect-autostart`\n> > . bisect--helper: really retire --bisect-next-check\n> > . bisect--helper: retire the --no-log option\n> > . Merge branch 'sg/parse-options-subcommand' into js/bisect-in-c\n> >\n> > Final bits of \"git bisect.sh\" have been rewritten in C.\n> >\n> > Needs review.\n> > cf. \n> > source: \n>\n> I see this has been ejected out of \"seen\", presumably due to the\n> outstanding conflicts.\n\nIf I recall correctly, I ejected this one due to its age and lack of\nattention. If you want to reroll it or if Johannes wants to take a look,\nI'd be appreciative.\n\nThanks,\nTaylor\n"},{"id":"467042","messageId":"s024qpqn-0roo-3rr2-nr4p-74p9296r6p02@tzk.qr","threadId":"58722","inReplyTo":"221102.86cza52rt0.gmgdl@evledraar.gmail.com","subject":"Re: js/bisect-in-c (was: What's cooking in git.git (Oct 2022, #09; Mon, 31))","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2022-11-10T13:32:15Z","receivedAt":"2022-11-10T13:32:25Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi Ævar,\n\nOn Wed, 2 Nov 2022, Ævar Arnfjörð Bjarmason wrote:\n\n> On Mon, Oct 31 2022, Taylor Blau wrote:\n>\n> > What's cooking in git.git (Oct 2022, #09; Mon, 31)\n> > --------------------------------------------------\n> >\n> > * js/bisect-in-c (2022-08-30) 17 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--helper: make `state` optional\n> > . bisect--helper: calling `bisect_state()` without an argument is a bug\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> > . bisect--helper: migrate to OPT_SUBCOMMAND()\n> > . bisect--helper: make the order consistently `argc, argv`\n> > . bisect--helper: make `terms` an explicit singleton\n> > . bisect--helper: simplify exit code computation\n> > . bisect--helper: really retire `--bisect-autostart`\n> > . bisect--helper: really retire --bisect-next-check\n> > . bisect--helper: retire the --no-log option\n> > . Merge branch 'sg/parse-options-subcommand' into js/bisect-in-c\n> >\n> > Final bits of \"git bisect.sh\" have been rewritten in C.\n> >\n> > Needs review.\n> > cf. \n> > source: \n>\n> I see this has been ejected out of \"seen\", presumably due to the\n> outstanding conflicts.\n>\n> To the extent that Johannes doesn't have time I'd by happy to re-roll\n> this & look at it/fix whatever outstanding issues I spot.\n>\n> I gave it a once over a few weeks ago, and didn't see any outstanding\n> problems with it worth rewriting in that iteration, but didn't have time\n> to give it stress testing to see if there were still some lurking\n> unexpected/untested behavior changes.\n>\n> Johannes?\n\nI saw that this was marked as \"Needs review.\" for quite a while. Seeing as\nthis patch series has been kept out of `next` for quite a few iterations\nbasically solely due to your concerns (others have said they're fine with\nit), I kind of expected you to give a thumbs-up or a thumbs-down (combined\nwith clear, concise suggestions how to get this moving again).\n\nSo now you say that you did not see any outstanding problems? That would\nproabably have been good information for Junio to use to move it along\nwhile it did not conflict.\n\nCiao,\nJohannes\n"},{"id":"467043","messageId":"8r1s24sp-8p26-sr61-3pp0-8o7r6pr72641@tzk.qr","threadId":"58722","inReplyTo":"Y2KoKO5Sj71Nfa61@nand.local","subject":"Re: js/bisect-in-c (was: What's cooking in git.git (Oct 2022, #09; Mon, 31))","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2022-11-10T13:35:26Z","receivedAt":"2022-11-10T13:35:40Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi Taylor,\n\nOn Wed, 2 Nov 2022, Taylor Blau wrote:\n\n> On Wed, Nov 02, 2022 at 06:22:17PM +0100, Ævar Arnfjörð Bjarmason wrote:\n> >\n> > On Mon, Oct 31 2022, Taylor Blau wrote:\n> >\n> > > What's cooking in git.git (Oct 2022, #09; Mon, 31)\n> > > --------------------------------------------------\n> > >\n> > > * js/bisect-in-c (2022-08-30) 17 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--helper: make `state` optional\n> > > . bisect--helper: calling `bisect_state()` without an argument is a bug\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> > > . bisect--helper: migrate to OPT_SUBCOMMAND()\n> > > . bisect--helper: make the order consistently `argc, argv`\n> > > . bisect--helper: make `terms` an explicit singleton\n> > > . bisect--helper: simplify exit code computation\n> > > . bisect--helper: really retire `--bisect-autostart`\n> > > . bisect--helper: really retire --bisect-next-check\n> > > . bisect--helper: retire the --no-log option\n> > > . Merge branch 'sg/parse-options-subcommand' into js/bisect-in-c\n> > >\n> > > Final bits of \"git bisect.sh\" have been rewritten in C.\n> > >\n> > > Needs review.\n> > > cf. \n> > > source: \n> >\n> > I see this has been ejected out of \"seen\", presumably due to the\n> > outstanding conflicts.\n>\n> If I recall correctly, I ejected this one due to its age and lack of\n> attention. If you want to reroll it or if Johannes wants to take a look,\n> I'd be appreciative.\n\nMy last information was that I sent an iteration that was designed to\naddress all outstanding concerns, including a rather major haul to put\nthis on top of the new `OPT_SUBCOMMAND` feature that wasn't even dreamed\nof when I sent v1 of `bisect-in-c`, and then I only saw \"Needs review.\"\nfor several weeks and nobody objecting but also in particular Ævar (who\nraised concerns against this patch series several times over the last 10\nmonths) not chiming in with a \"go ahead\".\n\nSo basically, I thought this was finally done and the next thing I hear is\nthat it is ejected. That's quite a frustrating experience, I must admit.\nAt least I am not a new contributor who would be very much deterred from\ncontributing any further by such an experience.\n\nCiao,\nDscho\n"},{"id":"467120","messageId":"Y23Tvud88nwDm0Va@nand.local","threadId":"58722","inReplyTo":"8r1s24sp-8p26-sr61-3pp0-8o7r6pr72641@tzk.qr","subject":"Re: js/bisect-in-c (was: What's cooking in git.git (Oct 2022, #09; Mon, 31))","fromName":"Taylor Blau","fromEmail":"me@ttaylorr.com","sentAt":"2022-11-11T04:46:54Z","receivedAt":"2022-11-11T04:49:05Z","isPatch":false,"sender":{"key":"me@ttaylorr.com","avatar":"https://avatars.githubusercontent.com/u/301000140?v=4"},"body":"On Thu, Nov 10, 2022 at 02:35:26PM +0100, Johannes Schindelin wrote:\n> So basically, I thought this was finally done and the next thing I hear is\n> that it is ejected. That's quite a frustrating experience, I must admit.\n> At least I am not a new contributor who would be very much deterred from\n> contributing any further by such an experience.\n\nI ejected the series for two reasons:\n\n - it began to conflict with newer series that folks are working on,\n and\n\n - because I hadn't seen any activity on it from either you or any of\n the previous reviewers.\n\nI don't want to drop the topic on the floor if we are getting close to\nor are at the finish line. *But*, there are a handful of other series in\nflight which seem to conflict with this one.\n\nIt would have been much nicer to graduate this earlier, and certainly I\nthink some reviewer-signaling could have helped that.\n\nBut now that other topics have appeared, it would be nice to see a\nrerolled version on top of any topic(s) with which it might conflict.\nThat, along with clear instructions of which topics it depends on will\nhelp me queue it in the right order.\n\nIf it helps, I can try and take a closer look tomorrow when rebuilding\n'seen' with the new topics. It may be possible for me to resolve the\nconflicts with minimal effort. If it appears in 'seen', then you should\nbe OK, and we can start graduating this down. But if not, then having a\nrerolled version would be very much appreciated.\n\nThanks,\nTaylor\n"},{"id":"467140","messageId":"221111.86a64xo86x.gmgdl@evledraar.gmail.com","threadId":"58722","inReplyTo":"8r1s24sp-8p26-sr61-3pp0-8o7r6pr72641@tzk.qr","subject":"Re: js/bisect-in-c (was: What's cooking in git.git (Oct 2022, #09; Mon, 31))","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2022-11-11T14:11:56Z","receivedAt":"2022-11-11T14:52:09Z","isPatch":false,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"\nOn Thu, Nov 10 2022, Johannes Schindelin wrote:\n\n> Hi Taylor,\n>\n> On Wed, 2 Nov 2022, Taylor Blau wrote:\n>\n>> On Wed, Nov 02, 2022 at 06:22:17PM +0100, Ævar Arnfjörð Bjarmason wrote:\n>> >\n>> > On Mon, Oct 31 2022, Taylor Blau wrote:\n>> >\n>> > > What's cooking in git.git (Oct 2022, #09; Mon, 31)\n>> > > --------------------------------------------------\n>> > >\n>> > > * js/bisect-in-c (2022-08-30) 17 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--helper: make `state` optional\n>> > > . bisect--helper: calling `bisect_state()` without an argument is a bug\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>> > > . bisect--helper: migrate to OPT_SUBCOMMAND()\n>> > > . bisect--helper: make the order consistently `argc, argv`\n>> > > . bisect--helper: make `terms` an explicit singleton\n>> > > . bisect--helper: simplify exit code computation\n>> > > . bisect--helper: really retire `--bisect-autostart`\n>> > > . bisect--helper: really retire --bisect-next-check\n>> > > . bisect--helper: retire the --no-log option\n>> > > . Merge branch 'sg/parse-options-subcommand' into js/bisect-in-c\n>> > >\n>> > > Final bits of \"git bisect.sh\" have been rewritten in C.\n>> > >\n>> > > Needs review.\n>> > > cf. \n>> > > source: \n>> >\n>> > I see this has been ejected out of \"seen\", presumably due to the\n>> > outstanding conflicts.\n>>\n>> If I recall correctly, I ejected this one due to its age and lack of\n>> attention. If you want to reroll it or if Johannes wants to take a look,\n>> I'd be appreciative.\n\n[Replying here instead of ,\nbut take this as a reply to that as well]\n\n> My last information was that I sent an iteration that was designed to\n> address all outstanding concerns, including a rather major haul to put\n> this on top of the new `OPT_SUBCOMMAND` feature that wasn't even dreamed\n> of when I sent v1 of `bisect-in-c`, and then I only saw \"Needs review.\"\n> for several weeks and nobody objecting but also in particular Ævar (who\n> raised concerns against this patch series several times over the last 10\n> months) not chiming in with a \"go ahead\".\n\nI think that \"go ahead\" from me was pretty unambiguous in [1], i.e. I said:\n\n\t\"I think as it stands that it's a net improvement on \"master\". I\n\twould not mind it advanced in its current state. We can fix any\n\toutstanding issues with it later, particularly due to what seem\n\tlike time constraints on your end[...]\"\n\n> So basically, I thought this was finally done and the next thing I hear is\n> that it is ejected. [...]\n\nWhat you're eliding here is that on the 4th Lukáš Doktor reported a\nregression in \"git bisect\" that pre-dated your cooking topic[2].\n\nYou didn't participate in that discussion, except now to say that you\nweren't motivated to review parts of the proposed fixes for that\nregression it[3].\n\nWhich is relevant because in your topic you spend a lot of time\nconverting various code for the supposed needs of a migration to\nOPT_SUBCOMMAND()[4]. Just the diffstat for that was:\n\n 1 file changed, 128 insertions(+), 134 deletions(-)\n\nWhich didn't get all the way to the full OPT_SUBCOMMAND(). That still\nneeded [5] and [6], which are, respectively:\n\n 1 file changed, 14 insertions(+), 15 deletions(-)\n 2 files changed, 288 insertions(+), 194 deletions(-)\n\nWhereas Đoàn's \"dd/bisect-helper-subcommand\", which is now in \"next\",\nand which was aiming for the shortest route to fixing that regression,\nand which also gave us OPT_SUBCOMMAND() is:\n\n 3 files changed, 142 insertions(+), 120 deletions(-)\n\nThis is all around 2 months after Junio's feedback on your latest\niteration was also questioning the need for that rewrite to get to the\ngoal of OPT_SUBCOMMAND()[7].\n\n> That's quite a frustrating experience, I must admit.\n> At least I am not a new contributor who would be very much deterred from\n> contributing any further by such an experience.\n\nIt's been frustrating for me too, particularly the as this isn't the\nfirst time that I feel like you're outright misrepresenting something\nI've said or done.\n\nBut I really don't think any of this translates to some general\nunfriendliness on the list that new contributors would run into.\n\nTo take just the above example: New contributors aren't sending hundreds\nof lines of patches to get to a stated goal of \"X\" by doing \"A\", and\nthen ignoring or dismissing reviews when it's pointed out to them that\nthey can get to \"X\" in a much easier manner by doing \"B\" instead.\n\nYour original topic is a daunting:\n\n 7 files changed, 406 insertions(+), 388 deletions(-)\n\nWhich, as noted above the replacement of Đoàn's\n\"dd/bisect-helper-subcommand\" is:\n\n 3 files changed, 142 insertions(+), 120 deletions(-)\n\nAnd his \"dd/git-bisect-builtin\" in seen is:\n\n 5 files changed, 77 insertions(+), 58 deletions(-)\n\nDoes that get us all the same thing sas js/bisect-in-c? No. But it does\nget us to the primary goal whith fewer changes, which I think is the\nmain reason why your series has had such a hard time advancing.\n\n1. https://lore.kernel.org/git/220819.86sfls6zhh.gmgdl@evledraar.gmail.com/\n2. https://lore.kernel.org/git/1cb1c033-0525-7e62-8c09-81019bf26060@redhat.com/\n3. https://lore.kernel.org/git/2477861r-9363-75sn-q415-o19206q70p90@tzk.qr/\n4. https://lore.kernel.org/git/92b3b116ef8f879192d9deb94d68b73e29d5dcd6.1661885419.git.gitgitgadget@gmail.com/\n5. https://lore.kernel.org/git/c9dc0281e38bf9bc0bce72de172b5dbadbcbb1f5.1661885419.git.gitgitgadget@gmail.com/\n6. https://lore.kernel.org/git/e97e187bbec93b47f35e3dd42b4831f1c1d8658d.1661885419.git.gitgitgadget@gmail.com/\n7. https://lore.kernel.org/git/xmqqmtblighr.fsf@gitster.g/\n"}]}