{"thread":{"id":"58745","subject":"What's cooking in git.git (Nov 2022, #01; Thu, 3)","startedAt":"2022-11-04T01:06:07Z","lastAt":"2022-11-09T14:04:13Z","messageCount":18,"participants":["Taylor Blau","Philip Oakley","Ævar Arnfjörð Bjarmason","Jeff King","Phillip Wood"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"466468","messageId":"Y2RldUHTwNzmez73@nand.local","threadId":"58745","inReplyTo":null,"subject":"What's cooking in git.git (Nov 2022, #01; Thu, 3)","fromName":"Taylor Blau","fromEmail":"me@ttaylorr.com","sentAt":"2022-11-04T01:05:57Z","receivedAt":"2022-11-04T01:06:07Z","isPatch":false,"sender":{"key":"me@ttaylorr.com","avatar":"https://avatars.githubusercontent.com/u/301000140?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\nA large-ish handful of topics are awaiting review. I'll continue to\nhold those topics in 'seen' until they have enough attention to feel\ncomfortable merging them down.\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[New Topics]\n\n* ab/submodule-helper-prep-only (2022-11-02) 8 commits\n - submodule--helper: use OPT_SUBCOMMAND() API\n - submodule--helper: drop \"update --prefix \" for \"-C update\"\n - submodule--helper: remove --prefix from \"absorbgitdirs\"\n - submodule API & \"absorbgitdirs\": remove \"----recursive\" option\n - submodule.c: refactor recursive block out of absorb function\n - submodule tests: test for a \"foreach\" blind-spot\n - submodule tests: add tests for top-level flag output\n - submodule--helper: move \"config\" to a test-tool\n\n Preparation to remove git-submodule.sh and replace it with a builtin.\n\n Waiting for review?\n source: \n\n\n* ds/bundle-uri-4 (2022-10-31) 9 commits\n - clone: unbundle the advertised bundles\n - bundle-uri: download bundles from an advertised list\n - bundle-uri: allow relative URLs in bundle lists\n - strbuf: reintroduce strbuf_parent_directory()\n - bundle-uri client: add boolean transfer.bundleURI setting\n - bundle-uri: serve bundle.* keys from config\n - bundle-uri client: add helper for testing server\n - bundle-uri client: add minimal NOOP client\n - protocol v2: add server-side \"bundle-uri\" skeleton\n\n Bundle URIs part 4.\n\n source: \n\n\n* jk/ref-filter-parsing-bugs (2022-11-02) 2 commits\n (merged to 'next' on 2022-11-03 at 92c1a50548)\n + ref-filter: fix parsing of signatures with CRLF and no body\n + ref-filter: fix parsing of signatures without blank lines\n\n Various tests exercising the transfer.credentialsInUrl configuration\n are taught to avoid making requests which require resolving localhost\n to reduce CI-flakiness.\n\n Will merge to 'master'.\n source: \n\n\n* ma/drop-redundant-diagnostic (2022-10-31) 1 commit\n (merged to 'next' on 2022-11-03 at 196c5a337b)\n + test-lib-functions: drop redundant diagnostic print\n\n A redundant diagnostic message is dropped from test_path_is_missing().\n\n Will merge to 'master'.\n source: <20221031180048.5766-1-martin.agren@gmail.com>\n\n\n* sg/plug-line-log-leaks (2022-11-02) 3 commits\n - diff.c: use diff_free_queue()\n - line-log: free the diff queues' arrays when processing merge commits\n - line-log: free diff queue when processing non-merge commits\n\n A handful of leaks in the line-log machinery have been plugged.\n\n Expecting a reroll.\n source: <20221102220142.574890-1-szeder.dev@gmail.com>\n\n\n* tb/branch-delete-detached (2022-11-01) 1 commit\n - branch: gracefully handle '-d' on detached HEAD\n\n 'git branch -d' is taught to ignore failures to resolve HEAD when\n detached.\n\n Expecting a reroll.\n source: \n\n\n* tb/howto-maintain-git-fixes (2022-10-31) 2 commits\n - Documentation: build redo-seen.sh from jch..seen\n - Documentation: build redo-jch.sh from master..jch\n (this branch uses tb/howto-using-redo-script.)\n\n A pair of bugfixes to the Documentation/howto/maintain-git.txt guide.\n\n Will merge to 'next'.\n source: \n\n\n* tl/pack-bitmap-absolute-paths (2022-11-02) 1 commit\n - pack-bitmap.c: avoid exposing absolute paths\n\n The pack-bitmap machinery is taught to log the paths of redundant\n bitmap(s) to trace2 instead of stderr.\n\n Expecting a reroll.\n source: <87a494e5ac0cc992689944ab13600d097c51e54a.1667393419.git.dyroneteng@gmail.com>\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/cmake-nix-and-ci (2022-11-02) 12 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 - cmake: increase test timeout on Windows only\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-31) 4 commits\n - Makefile: simplify $(test_bindir_programs) rule by splitting it up\n - Makefile: rename \"test_bindir_programs\" variable, pre-declare\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 (merged to 'next' on 2022-11-03 at 0f01b25561)\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-11-01) 2 commits\n - merge-tree.c: support --merge-base in conjunction with --stdin\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-11-02) 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-11-02) 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: <20221102120853.2013-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 (merged to 'next' on 2022-11-03 at c2d1004d96)\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 'master'.\n source: \n\n\n* ab/config-multi-and-nonbool (2022-11-02) 9 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 API users: test for *_get_value_multi() segfaults\n - for-each-repo: error on bad --config\n - config API: have *_multi() return an \"int\" and take a \"dest\"\n - versioncmp.c: refactor config reading next commit\n - config tests: add \"NULL\" tests for *_get_value_multi()\n - config tests: cover blind spots in git_die_config() tests\n - for-each-repo tests: test bad --config keys\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 (merged to 'next' on 2022-11-03 at 27eb80811c)\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 'master'.\n source: \n\n\n* tb/howto-using-redo-script (2022-10-26) 1 commit\n (merged to 'next' on 2022-11-03 at 521600fd56)\n + Documentation/howto/maintain-git.txt: fix Meta/redo-jch.sh invocation\n (this branch is used by tb/howto-maintain-git-fixes.)\n\n Doc update.\n\n Will merge to 'master'.\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 (merged to 'next' on 2022-11-03 at 32bfd99ffd)\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 'master'.\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 (merged to 'next' on 2022-11-03 at 4079e1b06b)\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 'master'.\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-31) 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 (merged to 'next' on 2022-11-03 at 58b659f92b)\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 'master'.\n source: <20221029164112.2097-1-philipoakley@iee.email>\n\n\n* mc/credential-helper-auth-headers (2022-11-02) 11 commits\n - t5556: add HTTP authentication tests\n - test-http-server: add simple authentication\n - test-http-server: pass Git requests to http-backend\n - test-http-server: add HTTP request parsing\n - test-http-server: add HTTP error response function\n - test-http-server: add stub HTTP server test helper\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-11-02) 13 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 - Makefile: copy contrib/coccinelle/*.cocci to build/\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":"466566","messageId":"e1c10d50-779d-2a22-b0c6-83bc0769ea37@iee.email","threadId":"58745","inReplyTo":"Y2RldUHTwNzmez73@nand.local","subject":"Re: What's cooking in git.git (Nov 2022, #01; Thu, 3)","fromName":"Philip Oakley","fromEmail":"philipoakley@iee.email","sentAt":"2022-11-05T00:21:23Z","receivedAt":"2022-11-05T00:21:30Z","isPatch":false,"sender":{"key":"philipoakley@iee.email","avatar":"https://avatars.githubusercontent.com/u/914343?v=4"},"body":"On 04/11/2022 01:05, Taylor Blau wrote:\n> --------------------------------------------------\n> [Cooking]\n> * po/pretty-hard-trunc (2022-11-02) 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.\nIs that out of date? V3 did include tests in t4205 and t6006.\n\n> source: <20221102120853.2013-1-philipoakley@iee.email>\nThis source is the V3.\n\nPhilip\n"},{"id":"466567","messageId":"Y2Wtn0wKhbQrScGU@nand.local","threadId":"58745","inReplyTo":"e1c10d50-779d-2a22-b0c6-83bc0769ea37@iee.email","subject":"Re: What's cooking in git.git (Nov 2022, #01; Thu, 3)","fromName":"Taylor Blau","fromEmail":"me@ttaylorr.com","sentAt":"2022-11-05T00:26:07Z","receivedAt":"2022-11-05T00:26:12Z","isPatch":false,"sender":{"key":"me@ttaylorr.com","avatar":"https://avatars.githubusercontent.com/u/301000140?v=4"},"body":"On Sat, Nov 05, 2022 at 12:21:23AM +0000, Philip Oakley wrote:\n> On 04/11/2022 01:05, Taylor Blau wrote:\n> > --------------------------------------------------\n> > [Cooking]\n> > * po/pretty-hard-trunc (2022-11-02) 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> Is that out of date? V3 did include tests in t4205 and t6006.\n>\n> > source: <20221102120853.2013-1-philipoakley@iee.email>\n> This source is the V3.\n\nYes, thanks for catching. Indeed, the version that I have at:\n\n git@github.com:ttaylorr/git.git\n\non po/pretty-hard-trunc has the new tests. I think we are still waiting\nfor review on that topic, though...\n\nThanks,\nTaylor\n"},{"id":"466655","messageId":"88b30b5f-2712-4c84-331c-2ea7338053ad@iee.email","threadId":"58745","inReplyTo":"Y2Wtn0wKhbQrScGU@nand.local","subject":"Re: What's cooking in git.git (Nov 2022, #01; Thu, 3)","fromName":"Philip Oakley","fromEmail":"philipoakley@iee.email","sentAt":"2022-11-07T12:19:22Z","receivedAt":"2022-11-07T12:19:29Z","isPatch":false,"sender":{"key":"philipoakley@iee.email","avatar":"https://avatars.githubusercontent.com/u/914343?v=4"},"body":"On 05/11/2022 00:26, Taylor Blau wrote:\n> On Sat, Nov 05, 2022 at 12:21:23AM +0000, Philip Oakley wrote:\n>> On 04/11/2022 01:05, Taylor Blau wrote:\n>>> --------------------------------------------------\n>>> [Cooking]\n>>> * po/pretty-hard-trunc (2022-11-02) 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>> Is that out of date? V3 did include tests in t4205 and t6006.\n>>\n>>> source: <20221102120853.2013-1-philipoakley@iee.email>\n>> This source is the V3.\n> Yes, thanks for catching. Indeed, the version that I have at:\n>\n> git@github.com:ttaylorr/git.git\n>\n> on po/pretty-hard-trunc has the new tests. I think we are still waiting\n> for review on that topic, though...\n>\n> Thanks,\n> Taylor\nAn extra review would be useful.\n\nI wasn't certain about the use of `qz_to_tab_space` in the here-doc\nformulation of t6006-rev-list-format where some of the new tests now\nincluded trailing spaces (but others didn't).\n\nTa,\nPhilip\n"},{"id":"466662","messageId":"221107.86leomx2dg.gmgdl@evledraar.gmail.com","threadId":"58745","inReplyTo":"Y2RldUHTwNzmez73@nand.local","subject":"ab/config-multi-and-nonbool (was: What's cooking in git.git (Nov 2022, #01; Thu, 3))","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2022-11-07T14:28:14Z","receivedAt":"2022-11-07T14:33:53Z","isPatch":false,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"\nOn Thu, Nov 03 2022, Taylor Blau wrote:\n\n> * ab/config-multi-and-nonbool (2022-11-02) 9 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 API users: test for *_get_value_multi() segfaults\n> - for-each-repo: error on bad --config\n> - config API: have *_multi() return an \"int\" and take a \"dest\"\n> - versioncmp.c: refactor config reading next commit\n> - config tests: add \"NULL\" tests for *_get_value_multi()\n> - config tests: cover blind spots in git_die_config() tests\n> - for-each-repo tests: test bad --config keys\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\nThe \"mixed bag\" part of this comes from Junio's v1 summary[1], which I\nthink I addressed by narrowing the scope in the v2[2].\n\nPer [3] it looks like you were going to try to find time to take a\nsecond look, *gentle poke* :)\n\n1. https://lore.kernel.org/git/xmqqy1szef49.fsf@gitster.g/\n2. https://lore.kernel.org/git/cover-v2-0.9-00000000000-20221101T225822Z-avarab@gmail.com/\n3. https://lore.kernel.org/git/Y2G+qczur9eLVtk5@nand.local/\n"},{"id":"466663","messageId":"221107.86h6zax26k.gmgdl@evledraar.gmail.com","threadId":"58745","inReplyTo":"Y2RldUHTwNzmez73@nand.local","subject":"ab/make-bin-wrappers (was: What's cooking in git.git (Nov 2022, #01; Thu, 3))","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2022-11-07T14:33:40Z","receivedAt":"2022-11-07T14:35:41Z","isPatch":false,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"\nOn Thu, Nov 03 2022, Taylor Blau wrote:\n\n> * ab/make-bin-wrappers (2022-10-31) 4 commits\n> - Makefile: simplify $(test_bindir_programs) rule by splitting it up\n> - Makefile: rename \"test_bindir_programs\" variable, pre-declare\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\nOn my end I'm waiting to see what you and/or Jeff think about:\nhttps://lore.kernel.org/git/221101.86edun5tgn.gmgdl@evledraar.gmail.com/\n\nI.e. i can certainly re-roll with a one-line fix, but don't see how it's\nuseful to have a \"make\" target produce a wrapper that doesn't work.\n"},{"id":"466740","messageId":"221107.861qqev53o.gmgdl@evledraar.gmail.com","threadId":"58745","inReplyTo":"Y2RldUHTwNzmez73@nand.local","subject":"ab/coccicheck-incremental (was: What's cooking in git.git (Nov 2022, #01; Thu, 3))","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2022-11-07T21:14:28Z","receivedAt":"2022-11-07T21:19:08Z","isPatch":false,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"\nOn Thu, Nov 03 2022, Taylor Blau wrote:\n\n> * ab/coccicheck-incremental (2022-11-02) 13 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> - Makefile: copy contrib/coccinelle/*.cocci to build/\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\nI think it's ready for that, the v5 having hopefully solved any last\nremaining nits / issues.\n"},{"id":"466752","messageId":"221107.86wn86tq2e.gmgdl@evledraar.gmail.com","threadId":"58745","inReplyTo":"Y2RldUHTwNzmez73@nand.local","subject":"ab/sha-makefile-doc (was: What's cooking in git.git (Nov 2022, #01; Thu, 3))","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2022-11-07T21:24:04Z","receivedAt":"2022-11-07T21:26:00Z","isPatch":false,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"\nOn Thu, Nov 03 2022, Taylor Blau wrote:\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\nThere was one outstanding issue from Junio noted in the v4, I just sent\na v5 in that should address that & more:\nhttps://lore.kernel.org/git/cover-v5-00.10-00000000000-20221107T211736Z-avarab@gmail.com/\n\nSo hopefully with that it's ready for 'next'. I have a trivial patch\nqueued after it to flip the default on OSX to use\nsha1collisiondetection.\n"},{"id":"466833","messageId":"221108.86h6z9sfd9.gmgdl@evledraar.gmail.com","threadId":"58745","inReplyTo":"Y2RldUHTwNzmez73@nand.local","subject":"ab/submodule-helper-prep-only (was: What's cooking in git.git (Nov 2022, #01; Thu, 3))","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2022-11-08T14:13:23Z","receivedAt":"2022-11-08T14:14:20Z","isPatch":false,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"\nOn Thu, Nov 03 2022, Taylor Blau wrote:\n\n> * ab/submodule-helper-prep-only (2022-11-02) 8 commits\n> - submodule--helper: use OPT_SUBCOMMAND() API\n> - submodule--helper: drop \"update --prefix \" for \"-C update\"\n> - submodule--helper: remove --prefix from \"absorbgitdirs\"\n> - submodule API & \"absorbgitdirs\": remove \"----recursive\" option\n> - submodule.c: refactor recursive block out of absorb function\n> - submodule tests: test for a \"foreach\" blind-spot\n> - submodule tests: add tests for top-level flag output\n> - submodule--helper: move \"config\" to a test-tool\n>\n> Preparation to remove git-submodule.sh and replace it with a builtin.\n>\n> Waiting for review?\n> source: \n\nIt was waiting on a small re-roll I just sent in:\nhttps://lore.kernel.org/git/cover-v2-0.9-00000000000-20221108T140501Z-avarab@gmail.com/\n"},{"id":"466834","messageId":"221108.86cz9xsf1w.gmgdl@evledraar.gmail.com","threadId":"58745","inReplyTo":"Y2RldUHTwNzmez73@nand.local","subject":"ab/cmake-nix-and-ci (was: What's cooking in git.git (Nov 2022, #01; Thu, 3))","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2022-11-08T14:14:50Z","receivedAt":"2022-11-08T14:21:23Z","isPatch":false,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"\nOn Thu, Nov 03 2022, Taylor Blau wrote:\n\n> * ab/cmake-nix-and-ci (2022-11-02) 12 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> - cmake: increase test timeout on Windows only\n>\n> Fix assorted issues with CTest on *nix machines.\n>\n> Waiting for review.\n> source: \n\nI think the v4[1] you've since picked up should be ready, and I think\naddressed the outstanding comments, except as noted in \"let's pursue\nthat separately\" in [2].\n\n1. https://lore.kernel.org/git/cover-v4-00.14-00000000000-20221103T160255Z-avarab@gmail.com/\n2. https://lore.kernel.org/git/221103.86a6581129.gmgdl@evledraar.gmail.com/\n"},{"id":"466835","messageId":"221108.868rklseyd.gmgdl@evledraar.gmail.com","threadId":"58745","inReplyTo":"Y2RldUHTwNzmez73@nand.local","subject":"ab/misc-hook-submodule-run-command (was: What's cooking in git.git (Nov 2022, #01; Thu, 3))","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2022-11-08T14:22:12Z","receivedAt":"2022-11-08T14:24:33Z","isPatch":false,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"\nOn Thu, Nov 03 2022, Taylor Blau wrote:\n\n> * ab/misc-hook-submodule-run-command (2022-10-31) 3 commits\n> (merged to 'next' on 2022-11-03 at 0f01b25561)\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\nPer [1] that's a stale comment, and you'd decided on a \"let's merge this\ndown\".\n\n1. https://lore.kernel.org/git/Y2Kn5MVQojz33cta@nand.local/\n"},{"id":"466837","messageId":"Y2pnZic1mi205/MQ@coredump.intra.peff.net","threadId":"58745","inReplyTo":"221107.86h6zax26k.gmgdl@evledraar.gmail.com","subject":"Re: ab/make-bin-wrappers (was: What's cooking in git.git (Nov 2022, #01; Thu, 3))","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2022-11-08T14:27:50Z","receivedAt":"2022-11-08T14:30:07Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Mon, Nov 07, 2022 at 03:33:40PM +0100, Ævar Arnfjörð Bjarmason wrote:\n\n> \n> On Thu, Nov 03 2022, Taylor Blau wrote:\n> \n> > * ab/make-bin-wrappers (2022-10-31) 4 commits\n> > - Makefile: simplify $(test_bindir_programs) rule by splitting it up\n> > - Makefile: rename \"test_bindir_programs\" variable, pre-declare\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> On my end I'm waiting to see what you and/or Jeff think about:\n> https://lore.kernel.org/git/221101.86edun5tgn.gmgdl@evledraar.gmail.com/\n\nI don't have any strong opinion on that. I read the v2 cover letter, was\nskeptical/confused of the motivation, and didn't go much further.\n\nYour explanation in the linked email is that there are _other_ reasons\nto do this refactoring, but I don't have any knowledge there that would\nadd to the review. My gut is still that building bin-wrappers/foo\ndoesn't need to depend on foo, but if it's one line, I don't care that\nmuch either way. If it was 50 lines of complicated Makefile refactoring,\nthen would probably not be worth it.\n\n-Peff\n"},{"id":"466842","messageId":"64b91b29-bbcd-e946-1f20-c0a5be63d9b7@dunelm.org.uk","threadId":"58745","inReplyTo":"221108.86cz9xsf1w.gmgdl@evledraar.gmail.com","subject":"Re: ab/cmake-nix-and-ci (was: What's cooking in git.git (Nov 2022, #01; Thu, 3))","fromName":"Phillip Wood","fromEmail":"phillip.wood123@gmail.com","sentAt":"2022-11-08T14:49:27Z","receivedAt":"2022-11-08T14:49:33Z","isPatch":false,"sender":{"key":"phillip.wood@dunelm.org.uk","avatar":null},"body":"On 08/11/2022 14:14, Ævar Arnfjörð Bjarmason wrote:\n> \n> On Thu, Nov 03 2022, Taylor Blau wrote:\n> \n>> * ab/cmake-nix-and-ci (2022-11-02) 12 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>> - cmake: increase test timeout on Windows only\n>>\n>> Fix assorted issues with CTest on *nix machines.\n>>\n>> Waiting for review.\n>> source: \n> \n> I think the v4[1] you've since picked up should be ready, and I think\n> addressed the outstanding comments, except as noted in \"let's pursue\n> that separately\" in [2].\n\nI think we need to wait for someone with windows experience to look at \nthis as that is the primary use of the cmake build. Without that I think \n patches 4(?), 5 and 7 are still could go forward as a separate series \nbut the rest of the patches including the documentation and test changes \nneed to have buy-in from their primary audience.\n\nBest Wishes\n\nPhillip\n\n\n> 1. https://lore.kernel.org/git/cover-v4-00.14-00000000000-20221103T160255Z-avarab@gmail.com/\n> 2. https://lore.kernel.org/git/221103.86a6581129.gmgdl@evledraar.gmail.com/\n"},{"id":"466906","messageId":"Y2rPAGp96IwrLS1T@nand.local","threadId":"58745","inReplyTo":"88b30b5f-2712-4c84-331c-2ea7338053ad@iee.email","subject":"Re: What's cooking in git.git (Nov 2022, #01; Thu, 3)","fromName":"Taylor Blau","fromEmail":"me@ttaylorr.com","sentAt":"2022-11-08T21:49:52Z","receivedAt":"2022-11-08T21:49:57Z","isPatch":false,"sender":{"key":"me@ttaylorr.com","avatar":"https://avatars.githubusercontent.com/u/301000140?v=4"},"body":"On Mon, Nov 07, 2022 at 12:19:22PM +0000, Philip Oakley wrote:\n> On 05/11/2022 00:26, Taylor Blau wrote:\n> > On Sat, Nov 05, 2022 at 12:21:23AM +0000, Philip Oakley wrote:\n> >> On 04/11/2022 01:05, Taylor Blau wrote:\n> >>> --------------------------------------------------\n> >>> [Cooking]\n> >>> * po/pretty-hard-trunc (2022-11-02) 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> >> Is that out of date? V3 did include tests in t4205 and t6006.\n> >>\n> >>> source: <20221102120853.2013-1-philipoakley@iee.email>\n> >> This source is the V3.\n> > Yes, thanks for catching. Indeed, the version that I have at:\n> >\n> > git@github.com:ttaylorr/git.git\n> >\n> > on po/pretty-hard-trunc has the new tests. I think we are still waiting\n> > for review on that topic, though...\n> >\n> > Thanks,\n> > Taylor\n> An extra review would be useful.\n>\n> I wasn't certain about the use of `qz_to_tab_space` in the here-doc\n> formulation of t6006-rev-list-format where some of the new tests now\n> included trailing spaces (but others didn't).\n\nLet's wait for a little more review before we start merging this one\ndown. I'll try and take a look if others don't beat me to it shortly.\n\nThanks,\nTaylor\n"},{"id":"466907","messageId":"Y2rPrNz6BD6DlRcO@nand.local","threadId":"58745","inReplyTo":"Y2pnZic1mi205/MQ@coredump.intra.peff.net","subject":"Re: ab/make-bin-wrappers (was: What's cooking in git.git (Nov 2022, #01; Thu, 3))","fromName":"Taylor Blau","fromEmail":"me@ttaylorr.com","sentAt":"2022-11-08T21:52:44Z","receivedAt":"2022-11-08T21:52:49Z","isPatch":false,"sender":{"key":"me@ttaylorr.com","avatar":"https://avatars.githubusercontent.com/u/301000140?v=4"},"body":"On Tue, Nov 08, 2022 at 09:27:50AM -0500, Jeff King wrote:\n> On Mon, Nov 07, 2022 at 03:33:40PM +0100, Ævar Arnfjörð Bjarmason wrote:\n>\n> >\n> > On Thu, Nov 03 2022, Taylor Blau wrote:\n> >\n> > > * ab/make-bin-wrappers (2022-10-31) 4 commits\n> > > - Makefile: simplify $(test_bindir_programs) rule by splitting it up\n> > > - Makefile: rename \"test_bindir_programs\" variable, pre-declare\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> > On my end I'm waiting to see what you and/or Jeff think about:\n> > https://lore.kernel.org/git/221101.86edun5tgn.gmgdl@evledraar.gmail.com/\n>\n> I don't have any strong opinion on that. I read the v2 cover letter, was\n> skeptical/confused of the motivation, and didn't go much further.\n>\n> Your explanation in the linked email is that there are _other_ reasons\n> to do this refactoring, but I don't have any knowledge there that would\n> add to the review. My gut is still that building bin-wrappers/foo\n> doesn't need to depend on foo, but if it's one line, I don't care that\n> much either way. If it was 50 lines of complicated Makefile refactoring,\n> then would probably not be worth it.\n\nIsn't this topic exactly the latter? IOW:\n\n $ git diff --stat master...ab/make-bin-wrappers\n Makefile | 72 ++++++++++++++++++++++++++++++++++++++++++++++++++++--------------------\n 1 file changed, 52 insertions(+), 20 deletions(-)\n\n...maybe that was exactly your point ;-).\n\nTBH, I am not sold whatsoever on the premise of that series, and agree\nthat plodding through 50+ lines of complicated Makefile diff is\ndifficult to justify when the premise is hazy to me.\n\nÆvar, any strong objections against dropping this one? If there is a\nsimpler way forward, I'm all ears, but in the meantime I find it\ndifficult to justify the review time as-is.\n\nThanks,\nTaylor\n"},{"id":"466908","messageId":"Y2rP2xN2CJbeDdsl@nand.local","threadId":"58745","inReplyTo":"221107.861qqev53o.gmgdl@evledraar.gmail.com","subject":"Re: ab/coccicheck-incremental (was: What's cooking in git.git (Nov 2022, #01; Thu, 3))","fromName":"Taylor Blau","fromEmail":"me@ttaylorr.com","sentAt":"2022-11-08T21:53:31Z","receivedAt":"2022-11-08T21:53:37Z","isPatch":false,"sender":{"key":"me@ttaylorr.com","avatar":"https://avatars.githubusercontent.com/u/301000140?v=4"},"body":"On Mon, Nov 07, 2022 at 10:14:28PM +0100, Ævar Arnfjörð Bjarmason wrote:\n>\n> On Thu, Nov 03 2022, Taylor Blau wrote:\n>\n> > * ab/coccicheck-incremental (2022-11-02) 13 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> > - Makefile: copy contrib/coccinelle/*.cocci to build/\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>\n> I think it's ready for that, the v5 having hopefully solved any last\n> remaining nits / issues.\n\nYep, let's start merging this one down.\n\nThanks,\nTaylor\n"},{"id":"466909","messageId":"Y2rSVSXWBEKgcIlJ@nand.local","threadId":"58745","inReplyTo":"221108.86h6z9sfd9.gmgdl@evledraar.gmail.com","subject":"Re: ab/submodule-helper-prep-only (was: What's cooking in git.git (Nov 2022, #01; Thu, 3))","fromName":"Taylor Blau","fromEmail":"me@ttaylorr.com","sentAt":"2022-11-08T22:04:05Z","receivedAt":"2022-11-08T22:04:11Z","isPatch":false,"sender":{"key":"me@ttaylorr.com","avatar":"https://avatars.githubusercontent.com/u/301000140?v=4"},"body":"On Tue, Nov 08, 2022 at 03:13:23PM +0100, Ævar Arnfjörð Bjarmason wrote:\n>\n> On Thu, Nov 03 2022, Taylor Blau wrote:\n>\n> > * ab/submodule-helper-prep-only (2022-11-02) 8 commits\n> > - submodule--helper: use OPT_SUBCOMMAND() API\n> > - submodule--helper: drop \"update --prefix \" for \"-C update\"\n> > - submodule--helper: remove --prefix from \"absorbgitdirs\"\n> > - submodule API & \"absorbgitdirs\": remove \"----recursive\" option\n> > - submodule.c: refactor recursive block out of absorb function\n> > - submodule tests: test for a \"foreach\" blind-spot\n> > - submodule tests: add tests for top-level flag output\n> > - submodule--helper: move \"config\" to a test-tool\n> >\n> > Preparation to remove git-submodule.sh and replace it with a builtin.\n> >\n> > Waiting for review?\n> > source: \n>\n> It was waiting on a small re-roll I just sent in:\n> https://lore.kernel.org/git/cover-v2-0.9-00000000000-20221108T140501Z-avarab@gmail.com/\n\nThanks. I took the updated round and I think that this is ready to start\nmerging down per Glen's review.\n\nThanks,\nTaylor\n"},{"id":"466950","messageId":"Y2uzWbl3M235Jlu+@coredump.intra.peff.net","threadId":"58745","inReplyTo":"Y2rPrNz6BD6DlRcO@nand.local","subject":"Re: ab/make-bin-wrappers (was: What's cooking in git.git (Nov 2022, #01; Thu, 3))","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2022-11-09T14:04:09Z","receivedAt":"2022-11-09T14:04:13Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Tue, Nov 08, 2022 at 04:52:44PM -0500, Taylor Blau wrote:\n\n> > I don't have any strong opinion on that. I read the v2 cover letter, was\n> > skeptical/confused of the motivation, and didn't go much further.\n> >\n> > Your explanation in the linked email is that there are _other_ reasons\n> > to do this refactoring, but I don't have any knowledge there that would\n> > add to the review. My gut is still that building bin-wrappers/foo\n> > doesn't need to depend on foo, but if it's one line, I don't care that\n> > much either way. If it was 50 lines of complicated Makefile refactoring,\n> > then would probably not be worth it.\n> \n> Isn't this topic exactly the latter? IOW:\n> \n> $ git diff --stat master...ab/make-bin-wrappers\n> Makefile | 72 ++++++++++++++++++++++++++++++++++++++++++++++++++++--------------------\n> 1 file changed, 52 insertions(+), 20 deletions(-)\n> \n> ...maybe that was exactly your point ;-).\n\nSorry, I perhaps could have made the last sentence more clear as:\n\n If it was 50 lines of complicated Makefile refactoring _just_ to have\n \"make bin-wrappers/foo\" depend on \"foo\", it would not be worth it. But\n if it is doing something else, it might be. I don't have a strong\n opinion because I haven't (and don't want to have) looked into that\n \"something else\".\n\n-Peff\n"}]}