{"thread":{"id":"56034","subject":"What's cooking in git.git (Jun 2021, #07; Wed, 30)","startedAt":"2021-06-30T22:55:38Z","lastAt":"2021-07-06T20:36:08Z","messageCount":18,"participants":["Junio C Hamano","Felipe Contreras","Han-Wen Nienhuys","Ævar Arnfjörð Bjarmason","Jeff Hostetler","Jeff King","Elijah Newren","Phillip Wood"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"428858","messageId":"xmqq4kdft122.fsf@gitster.g","threadId":"56034","inReplyTo":null,"subject":"What's cooking in git.git (Jun 2021, #07; Wed, 30)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2021-06-30T22:55:33Z","receivedAt":"2021-06-30T22:55:38Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Here are the topics that have been cooking in my tree.  Commits\nprefixed with '-' are only in 'seen' while commits prefixed with '+'\nare in 'next'.  The ones marked with '.' do not appear in any of the\nintegration branches, but I am still holding onto them.  Generally,\nbeing in 'next' is a sign that a topic is stable enough to be used\nand are candidate to be in a future release, while being in 'seen'\nmeans nothing more than that the maintainer has found it interesting\nfor some reason (like \"it may have hard-to-resolve conflicts with\nanother topic already in flight\" or \"this may turn out to be\nuseful\")---do not read too much into a topic being in (or not in)\n'seen'.\n\nSome new topics have been merged to 'seen' and are listed in this\nreport but there may well be topics that I haven't got around to\npick them up.  Also, among those that are in 'seen', I haven't\nsummarized many of them in this rerpot (let alone decided what to do\nwith them).  Help and patience is greatly appreciated.\n\nCopies of the source code to Git live in many repositories, and the\nfollowing is a list of the ones I push into or their mirrors.  Some\nrepositories have only a subset of branches.\n\nWith maint, master, next, seen, todo:\n\n\tgit://git.kernel.org/pub/scm/git/git.git/\n\tgit://repo.or.cz/alt-git.git/\n\thttps://kernel.googlesource.com/pub/scm/git/git/\n\thttps://github.com/git/git/\n\thttps://gitlab.com/git-vcs/git/\n\nWith all the integration branches and topics broken out:\n\n\thttps://github.com/gitster/git/\n\nEven though the preformatted documentation in HTML and man format\nare not sources, they are published in these repositories for\nconvenience (replace \"htmldocs\" with \"manpages\" for the manual\npages):\n\n\tgit://git.kernel.org/pub/scm/git/git-htmldocs.git/\n\thttps://github.com/gitster/git-htmldocs.git/\n\nRelease tarballs are available at:\n\n\thttps://www.kernel.org/pub/software/scm/git/\n\n--------------------------------------------------\n[New Topics]\n\n* fw/complete-cmd-idx-fix (2021-06-19) 1 commit\n - completion: bash: fix late declaration of __git_cmd_idx\n\n Recent update to completion script (in contrib/) broke those who\n use the __git_complete helper to define completion to their custom\n command.\n\n Will merge to 'next'.\n\n\n* jk/test-avoid-globmatch-with-skip-patterns (2021-06-17) 1 commit\n - test-lib: avoid accidental globbing in match_pattern_list()\n\n We broke \"GIT_SKIP_TESTS=t?000\" to skip certain tests in recent\n update, which got fixed.\n\n Will merge to 'next'.\n\n\n* jk/test-without-readlink-1 (2021-06-19) 1 commit\n - t: use portable wrapper for readlink(1)\n\n Some test scripts assumed that readlink(1) was universally\n installed and available, which is not the case.\n\n Will merge to 'next'.\n\n\n* jx/sideband-cleanup (2021-06-17) 4 commits\n - test: refactor to use \"get_abbrev_oid\" to get abbrev oid\n - test: refactor to use \"test_commit\" to create commits\n - test: compare raw output, not mangle tabs and spaces\n - sideband: don't lose clear-to-eol at packet boundary\n\n The side-band demultiplexer that is used to display progress output\n from the remote end did not clear the line properly when the end of\n line hits at a packet boundary, which has been corrected.  Also\n comes with test clean-ups.\n\n Will merge to 'next'.\n\n\n* ab/bundle-doc (2021-06-30) 3 commits\n - bundle doc: elaborate on rev<->ref restriction\n - bundle doc: split out open v.s. closed discussion from <rev-arg>\n - bundle doc: rewrite the \"DESCRIPTION\" section\n\n Doc update.\n\n\n* ab/fix-columns-to-80-during-tests (2021-06-29) 1 commit\n - test-lib.sh: set COLUMNS=80 for --verbose repeatability\n\n Output from some of our tests were affected by the width of the\n terminal that they were run in, which has been corrected by\n exporting a fixed value in the COLUMNS environment.\n\n Will merge to 'next'.\n\n\n* ab/pack-stdin-packs-fix (2021-06-28) 2 commits\n - pack-objects: fix segfault in --stdin-packs option\n - pack-objects tests: cover blindspots in stdin handling\n\n Input validation of \"git pack-objects --stdin-packs\" has been\n corrected.\n\n cf. <YND3h2l10PlnSNGJ@nand.local>\n\n\n* ar/more-typofix (2021-06-28) 3 commits\n - git-worktree.txt: fix typo in example path\n - t: fix typos in test messages\n - blame: correct name of config option in docs\n\n Typofixes.\n\n Will merge to 'next'.\n\n\n* bk/doc-commit-typofix (2021-06-28) 1 commit\n - Documentation: fix typo in the --patch option of the commit command\n\n\n* dc/p4-binary-submit-fix (2021-06-28) 1 commit\n - git-p4: fix failed submit by skip non-text data files\n\n\n* ds/commit-and-checkout-with-sparse-index (2021-06-28) 5 commits\n - checkout: stop expanding sparse indexes\n - sparse-index: recompute cache-tree\n - commit: integrate with sparse-index\n - p2000: compress repo names\n - p2000: add 'git checkout -' test and decrease depth\n (this branch uses ds/status-with-sparse-index.)\n\n\n* en/merge-dir-rename-corner-case-fix (2021-06-30) 3 commits\n - merge-recursive: handle rename-to-self case\n - merge-ort: ensure we consult df_conflict and path_conflicts\n - t6423: test directory renames causing rename-to-self\n\n\n* ew/many-alternate-optim (2021-06-29) 5 commits\n - oidtree: a crit-bit tree for odb_loose_cache\n - oidcpy_with_padding: constify `src' arg\n - make object_directory.loose_objects_subdir_seen a bitmap\n - avoid strlen via strbuf_addstr in link_alt_odb_entry\n - speed up alt_odb_usable() with many alternates\n\n\n* jk/log-decorate-optim (2021-06-28) 6 commits\n - add_ref_decoration(): rename s/type/deco_type/\n - load_ref_decorations(): avoid parsing non-tag objects\n - object.h: add lookup_object_by_type() function\n - object.h: expand docstring for lookup_unknown_object()\n - log: avoid loading decorations for userformats that don't need it\n - pretty.h: update and expand docstring for userformat_find_requirements()\n\n\n* js/ci-windows-update (2021-06-28) 6 commits\n - ci: accelerate the checkout\n - ci(vs-build): build with NO_GETTEXT\n - ci(windows): transfer also the Git-tracked files to the test jobs\n - ci: upgrade to using actions/{up,down}load-artifacts v2\n - ci (vs-build): use `cmd` to copy the DLLs, not `powershell`\n - ci: use the new GitHub Action to download git-sdk-64-minimal\n\n\n* js/config-mak-windows-pcre-fix (2021-06-28) 1 commit\n - config.mak.uname: PCRE1 cleanup\n\n\n* js/gfw-system-config-loc-fix (2021-06-28) 3 commits\n - config: normalize the path of the system gitconfig\n - cmake(windows): set correct path to the system Git config\n - mingw: move Git for Windows' system config where users expect it\n\n\n* jt/push-negotiation-fixes (2021-06-28) 3 commits\n - fetch: die on invalid --negotiation-tip hash\n - send-pack: fix push nego. when remote has refs\n - send-pack: fix push.negotiate with remote helper\n\n\n* ks/submodule-cleanup (2021-06-28) 1 commit\n - submodule: remove unnecessary `prefix` based option logic\n\n\n* rs/grep-parser-fix (2021-06-30) 1 commit\n - grep: report missing left operand of --and\n\n\n* tb/midx-use-checksum (2021-06-28) 4 commits\n - midx: report checksum mismatches during 'verify'\n - midx: don't reuse corrupt MIDXs when writing\n - commit-graph: rewrite to use checksum_valid()\n - csum-file: introduce checksum_valid()\n\n\n* ab/bundle-updates (2021-06-30) 3 commits\n - bundle: remove \"ref_list\" in favor of string-list.c API\n - bundle.c: use a temporary variable for OIDs and names\n - bundle cmd: stop leaking memory from parse_options_cmd_bundle()\n\n\n* ab/fetch-negotiate-segv-fix (2021-06-30) 3 commits\n - fetch: fix segfault in --negotiate-only without --negotiation-tip=*\n - fetch: document the --negotiate-only option\n - send-pack.c: move \"no refs in common\" abort earlier\n\n\n* ab/make-delete-on-error (2021-06-29) 1 commit\n - Makefile: add and use the \".DELETE_ON_ERROR\" flag\n\n\n* ab/make-tags-cleanup (2021-06-29) 5 commits\n - Makefile: normalize clobbering & xargs for tags targets\n - Makefile: don't use \"FORCE\" for tags targets\n - Makefile: fix \"cscope\" target to refer to cscope.out\n - Makefile: add QUIET_GEN to \"cscope\" target\n - Makefile: move \".PHONY: cscope\" near its target\n\n\n* ew/mmap-failures (2021-06-29) 1 commit\n - xmmap: inform Linux users of tuning knobs on ENOMEM\n\n--------------------------------------------------\n[Stalled]\n\n* hn/reftable (2021-05-20) 28 commits\n . t1404: annotate test cases with REFFILES\n . t1401,t2011: parameterize HEAD.lock for REFTABLE\n . t1301: document what needs to be done for REFTABLE\n . Add \"test-tool dump-reftable\" command.\n . git-prompt: prepare for reftable refs backend\n . Reftable support for git-core\n . reftable: add dump utility\n . reftable: implement stack, a mutable database of reftable files.\n . reftable: implement refname validation\n . reftable: add merged table view\n . reftable: add a heap-based priority queue for reftable records\n . reftable: reftable file level tests\n . reftable: read reftable files\n . reftable: generic interface to tables\n . reftable: write reftable files\n . reftable: a generic binary tree implementation\n . reftable: reading/writing blocks\n . Provide zlib's uncompress2 from compat/zlib-compat.c\n . reftable: (de)serialization for the polymorphic record type.\n . reftable: add blocksource, an abstraction for random access reads\n . reftable: utility functions\n . reftable: add error related functionality\n . reftable: add LICENSE\n . init-db: set the_repository->hash_algo early on\n . hash.h: provide constants for the hash IDs\n . refs/debug: trace into reflog expiry too\n . refs: document reflog_expire_fn's flag argument\n . refs: make explicit that ref_iterator_peel returns boolean\n\n The \"reftable\" backend for the refs API.\n\n Waiting for reviews.\n Seems to break tests when merged to 'seen'.\n\n\n* ao/p4-avoid-decoding (2021-04-12) 2 commits\n - git-p4: do not decode data from perforce by default\n - git-p4: avoid decoding more data from perforce\n\n \"git p4\" in Python-2 days used to accept a lot more kinds of data\n from Perforce server as uninterrupted byte sequence, but after\n switching to Python-3, too many things are expected to be in UTF-8,\n which broke traditional use cases.\n\n Waiting for reviews.\n\n\n* tv/p4-fallback-encoding (2021-04-30) 1 commit\n - git-p4: git-p4.fallbackEncoding to specify non UTF-8 charset\n\n \"git p4\" learns the fallbackEncoding configuration variable to\n safely accept changeset descriptions that aren't written in UTF-8.\n\n Waiting for reviews.\n\n--------------------------------------------------\n[Cooking]\n\n* tb/multi-pack-bitmaps (2021-06-28) 24 commits\n - p5326: perf tests for MIDX bitmaps\n - p5310: extract full and partial bitmap tests\n - midx: respect 'GIT_TEST_MULTI_PACK_INDEX_WRITE_BITMAP'\n - t7700: update to work with MIDX bitmap test knob\n - t5319: don't write MIDX bitmaps in t5319\n - t5310: disable GIT_TEST_MULTI_PACK_INDEX_WRITE_BITMAP\n - t0410: disable GIT_TEST_MULTI_PACK_INDEX_WRITE_BITMAP\n - t5326: test multi-pack bitmap behavior\n - t/helper/test-read-midx.c: add --checksum mode\n - t5310: move some tests to lib-bitmap.sh\n - pack-bitmap: write multi-pack bitmaps\n - pack-bitmap: read multi-pack bitmaps\n - pack-bitmap.c: introduce 'bitmap_is_preferred_refname()'\n - pack-bitmap.c: introduce 'nth_bitmap_object_oid()'\n - pack-bitmap.c: introduce 'bitmap_num_objects()'\n - midx: infer preferred pack when not given one\n - midx: respect 'core.multiPackIndex' when writing\n - midx: clear auxiliary .rev after replacing the MIDX\n - midx: make a number of functions non-static\n - Documentation: describe MIDX-based bitmaps\n - Documentation: build 'technical/bitmap-format' by default\n - pack-bitmap-write.c: free existing bitmaps\n - pack-bitmap-write.c: gracefully fail to write non-closed bitmaps\n - pack-bitmap.c: harden 'test_bitmap_walk()' to check type bitmaps\n\n The reachability bitmap file used to be generated only for a single\n pack, but now we've learned to generate bitmaps for history that\n span across multiple packfiles.\n\n\n* ds/status-with-sparse-index (2021-06-28) 16 commits\n - fsmonitor: integrate with sparse index\n - wt-status: expand added sparse directory entries\n - status: use sparse-index throughout\n - status: skip sparse-checkout percentage with sparse-index\n - diff-lib: handle index diffs with sparse dirs\n - dir.c: accept a directory as part of cone-mode patterns\n - unpack-trees: handle dir/file conflict of sparse entries\n - unpack-trees: unpack sparse directory entries\n - unpack-trees: rename unpack_nondirectories()\n - unpack-trees: compare sparse directories correctly\n - unpack-trees: preserve cache_bottom\n - t1092: add tests for status/add and sparse files\n - t1092: expand repository data shape\n - t1092: replace incorrect 'echo' with 'cat'\n - sparse-index: include EXTENDED flag when expanding\n - sparse-index: skip indexes with unmerged entries\n (this branch is used by ds/commit-and-checkout-with-sparse-index.)\n\n \"git status\" codepath learned to work with sparsely populated index\n without hydrating it fully.\n\n\n* ab/describe-tests-fix (2021-05-11) 5 commits\n - describe tests: support -C in \"check_describe\"\n - describe tests: fix nested \"test_expect_success\" call\n - describe tests: don't rely on err.actual from \"check_describe\"\n - describe tests: refactor away from glob matching\n - describe tests: improve test for --work-tree & --dirty\n\n Various updates to tests around \"git describe\"\n\n Will merge to 'next'.\n\n\n* ab/pickaxe-pcre2 (2021-05-11) 22 commits\n - xdiff-interface: replace discard_hunk_line() with a flag\n - xdiff users: use designated initializers for out_line\n - pickaxe -G: don't special-case create/delete\n - pickaxe -G: terminate early on matching lines\n - xdiff-interface: allow early return from xdiff_emit_line_fn\n - xdiff-interface: prepare for allowing early return\n - pickaxe -S: slightly optimize contains()\n - pickaxe: rename variables in has_changes() for brevity\n - pickaxe -S: support content with NULs under --pickaxe-regex\n - pickaxe: assert that we must have a needle under -G or -S\n - pickaxe: refactor function selection in diffcore-pickaxe()\n - perf: add performance test for pickaxe\n - pickaxe/style: consolidate declarations and assignments\n - diff.h: move pickaxe fields together again\n - pickaxe: die when --find-object and --pickaxe-all are combined\n - pickaxe: die when -G and --pickaxe-regex are combined\n - pickaxe tests: add missing test for --no-pickaxe-regex being an error\n - pickaxe tests: test for -G, -S and --find-object incompatibility\n - pickaxe tests: add test for \"log -S\" not being a regex\n - pickaxe tests: add test for diffgrep_consume() internals\n - pickaxe tests: refactor to use test_commit --append --printf\n - grep/pcre2 tests: reword comments referring to kwset\n\n Rewrite the backend for \"diff -G/-S\" to use pcre2 engine when\n available.\n\n Will merge to 'next'.\n\n\n* ab/config-based-hooks-base (2021-06-29) 33 commits\n - hooks: fix a TOCTOU in \"did we run a hook?\" heuristic\n - receive-pack: convert receive hooks to hook.h\n - post-update: use hook.h library\n - receive-pack: convert 'update' hook to hook.h\n - hooks: allow callers to capture output\n - run-command: allow capturing of collated output\n - reference-transaction: use hook.h to run hooks\n - transport: convert pre-push hook to use config\n - hook: convert 'post-rewrite' hook in sequencer.c to hook.h\n - hook: provide stdin by string_list or callback\n - run-command: add stdin callback for parallelization\n - am: convert 'post-rewrite' hook to hook.h\n - hook: support passing stdin to hooks\n - run-command: allow stdin for run_processes_parallel\n - run-command: remove old run_hook_{le,ve}() hook API\n - receive-pack: convert push-to-checkout hook to hook.h\n - read-cache: convert post-index-change hook to use config\n - commit: use hook.h to execute hooks\n - git-p4: use 'git hook' to run hooks\n - send-email: use 'git hook run' for 'sendemail-validate'\n - git hook run: add an --ignore-missing flag\n - merge: use config-based hooks for post-merge hook\n - hooks: convert 'post-checkout' hook to hook library\n - am: convert applypatch hooks to use config\n - rebase: teach pre-rebase to use hook.h\n - gc: use hook library for pre-auto-gc hook\n - hook: add 'run' subcommand\n - hook-list.h: add a generated list of hooks, like config-list.h\n - hook.c: add a hook_exists() wrapper and use it in bugreport.c\n - hook.[ch]: move find_hook() to this new library\n - Makefile: remove an out-of-date comment\n - Makefile: stop hardcoding {command,config}-list.h\n - Makefile: mark \"check\" target as .PHONY\n\n Restructuring of (a subset of) Emily's config-based-hooks series,\n to demonstrate that a series can be presented as a more logical and\n focused progression.\n\n Waiting for reviews.\n\n\n* ab/config-hooks-path-testfix (2021-06-16) 1 commit\n - pre-commit hook tests: don't leave \"actual\" nonexisting on failure\n\n Test fix.\n\n Will merge to 'next'.\n\n\n* ab/doc-retire-alice-bob (2021-06-16) 6 commits\n - pack-protocol doc: use \"www-data\" in place of \"alice\"\n - doc: replace \"alice\" and \"bob\" with \"jdoe\" and \"msmith\"\n - fast-import doc: change \"bob\" in an example to \"file.txt\"\n - daemon doc + code comments: reword \"alice\" example\n - gitcvs-migration doc: replace \"alice\" and \"bob\" with \"you\" and \"www-data\"\n - gittutorial doc: replace \"alice\" and \"bob\" with \"you\" and \"www-data\"\n\n\n* ab/pre-auto-gc-hook-test (2021-06-16) 1 commit\n - gc tests: add a test for the \"pre-auto-gc\" hook\n\n Test fix.\n\n Will merge to 'next'.\n\n\n* jv/userdiff-csharp-update (2021-06-16) 1 commit\n - userdiff: add support for C# record types\n\n The userdiff pattern for C# learned the token \"record\".\n\n Will merge to 'next'.\n\n\n* en/ort-perf-batch-13 (2021-06-28) 5 commits\n - merge-ort: add prefetching for content merges\n - diffcore-rename: use a different prefetch for basename comparisons\n - diffcore-rename: allow different missing_object_cb functions\n - t6421: add tests checking for excessive object downloads during merge\n - promisor-remote: output trace2 statistics for number of objects fetched\n\n Performance tweaks of \"git merge -sort\" around lazy fetching of objects.\n\n\n* ab/serve-cleanup (2021-06-28) 8 commits\n - upload-pack.c: convert to new serve.c \"startup\" config cb\n - serve: add support for a \"startup\" git_config() callback\n - serve.c: add trace2 regions for advertise & command\n - serve.c: add call_{advertise,command}() indirection\n - serve: use designated initializers\n - transport: use designated initializers\n - transport: rename \"fetch\" in transport_vtable to \"fetch_refs\"\n - serve: mark has_capability() as static\n\n Code clean-up around \"git serve\".\n\n\n* js/stop-exporting-bogus-columns (2021-06-28) 1 commit\n - pager: avoid setting COLUMNS when we're guessing its value\n\n When we cannot figure out how wide the terminal is, we use a\n fallback value of 80 ourselves (which cannot be avoided), but when\n we run the pager, we export it in COLUMNS, which forces the pager\n to use the hardcoded value, even when the pager is perfectly\n capable to figure it out itself.  Stop exporting COLUMNS when we\n fall back on the hardcoded default value for our own use.\n\n Will merge to 'next'.\n\n\n* ah/uninitialized-reads-fix (2021-06-15) 3 commits\n - builtin/checkout--worker: zero-initialise struct to avoid MSAN complaints\n - split-index: use oideq instead of memcmp to compare object_id's\n - bulk-checkin: make buffer reuse more obvious and safer\n\n Make the codebase MSAN clean.\n\n Will merge to 'next'.\n\n\n* ar/typofix (2021-06-14) 1 commit\n - *: fix typos which duplicate a word\n\n Typofixes.\n\n Will merge to 'next'.\n\n\n* jk/revision-squelch-gcc-warning (2021-06-11) 1 commit\n - add_pending_object_with_path(): work around \"gcc -O3\" complaint\n\n Warning fix.\n\n Will merge to 'next'.\n\n\n* jk/union-merge-binary (2021-06-11) 3 commits\n - ll_union_merge(): rename path_unused parameter\n - ll_union_merge(): pass name labels to ll_xdl_merge()\n - ll_binary_merge(): handle XDL_MERGE_FAVOR_UNION\n\n The \"union\" conflict resultion variant misbehaved when used with\n binary merge driver.\n\n Will merge to 'next'.\n\n\n* js/no-more-multimail (2021-06-11) 1 commit\n - multimail: stop shipping a copy\n\n Remove multimail from contrib/\n\n Will merge to 'next'.\n\n\n* js/subtree-on-windows-fix (2021-06-15) 2 commits\n - subtree: fix assumption about the directory separator\n - subtree: fix the GIT_EXEC_PATH sanity check to work on Windows\n\n Update \"git subtree\" to work better on Windows.\n\n Will merge to 'next'.\n\n\n* jt/partial-clone-submodule-1 (2021-06-28) 5 commits\n - promisor-remote: teach lazy-fetch in any repo\n - run-command: refactor subprocess env preparation\n - submodule: refrain from filtering GIT_CONFIG_COUNT\n - promisor-remote: support per-repository config\n - repository: move global r_f_p_c to repo struct\n\n Prepare the internals for lazily fetching objects in submodules\n from their promisor remotes.\n\n Waiting for reviews.\n\n\n* ab/mktag-tests (2021-06-28) 6 commits\n - mktag tests: test fast-export\n - mktag tests: test for-each-ref\n - mktag tests: test update-ref and reachable fsck\n - mktag tests: test hash-object --literally and unreachable fsck\n - mktag tests: invert --no-strict test\n - mktag tests: parse out options in helper\n\n Fill test gaps.\n\n\n* ab/show-branch-tests (2021-06-28) 4 commits\n - show-branch tests: add missing tests\n - show-branch: don't <COLOR></RESET> for space characters\n - show-branch tests: modernize test code\n - show-branch tests: rename the one \"show-branch\" test file\n\n Fill test gaps.\n\n\n* ah/graph-typofix (2021-06-15) 1 commit\n - graph: improve grammar of \"invalid color\" error message\n\n Typofix in an error message.\n\n Will merge to 'next'.\n\n\n* es/superproject-aware-submodules (2021-06-16) 5 commits\n - SQUASH???\n - submodule: cache superproject gitdir during 'update'\n - submodule: cache superproject gitdir during absorbgitdirs\n - introduce submodule.superprojectGitDir cache\n - t7400-submodule-basic: modernize inspect() helper\n\n A configuration variable in a submodule points at the location of\n the superproject it is bound to (RFC).\n\n Waiting for reviews.\n\n\n* fc/pull-cleanups (2021-06-19) 3 commits\n - pull: trivial whitespace style fix\n - pull: trivial cleanup\n - pull: cleanup autostash check\n\n Code cleanup.\n\n Will merge to 'next'.\n\n\n* jk/bitmap-tree-optim (2021-06-15) 1 commit\n - bitmaps: don't recurse into trees already in the bitmap\n\n Avoid duplicated work while building reachability bitmaps.\n\n Will merge to 'next'.\n\n\n* jx/t6020-with-older-bash (2021-06-17) 1 commit\n - t6020: fix incompatible parameter expansion\n\n Work around inefficient glob substitution in older versions of bash\n by rewriting parts of a test.\n\n Will merge to 'next'.\n\n\n* en/zdiff3 (2021-06-15) 2 commits\n - update documentation for new zdiff3 conflictStyle\n - xdiff: implement a zealous diff3, or \"zdiff3\"\n\n \"Zealous diff3\" style of merge conflict presentation has been added.\n\n Expecting a reroll.\n cf. <CABPp-BE7-E03+x38EK-=AE5mwwdST+d50hiiud2eY2Nsf3rM5g@mail.gmail.com>\n\n\n* pw/diff-color-moved-fix (2021-06-15) 10 commits\n - diff --color-moved: intern strings\n - diff --color-moved-ws=allow-indentation-change: improve hash lookups\n - diff --color-moved: stop clearing potential moved blocks\n - diff --color-moved: shrink potential moved blocks as we go\n - diff --color-moved: unify moved block growth functions\n - diff --color-moved: call comparison function directly\n - diff --color-moved-ws=allow-indentation-change: simplify and optimize\n - diff: simplify allow-indentation-change delta calculation\n - diff --color-moved: avoid false short line matches and bad zebra coloring\n - diff --color-moved=zebra: fix alternate coloring\n\n Long-overdue correctness and performance update to \"diff\n --color-moved\" feature.\n\n Waiting for reviews.\n\n\n* hn/refs-errno-cleanup (2021-06-11) 8 commits\n - refs: explicitly propagate errno from refs_read_raw_ref\n - refs: clear errno return in refs_resolve_ref_unsafe()\n - refs: add failure_errno to refs_read_raw_ref() signature\n - refs: use refs_resolve_ref_unsafe_with_errno() where needed\n - refs: make errno output explicit for refs_resolve_ref_unsafe\n - refs: make errno output explicit for read_raw_ref_fn\n - refs/files-backend: stop setting errno from lock_ref_oid_basic\n - refs: remove EINVAL errno output from specification of read_raw_ref_fn\n\n Futz with the way 'errno' is relied on in the refs API to carry the\n failure modes up the callchain.\n\n Will merge to 'next'?\n\n\n* ab/cmd-foo-should-return (2021-06-09) 1 commit\n - builtins + test helpers: use return instead of exit() in cmd_*\n\n Code clean-up.\n\n Will merge to 'next'.\n\n\n* ab/progress-cleanup (2021-06-08) 1 commit\n - read-cache.c: don't guard calls to progress.c API\n\n Code clean-up.\n\n Will merge to 'next'.\n\n\n* ab/test-tool-cache-cleanup (2021-06-08) 4 commits\n - read-cache perf: add a perf test for refresh_index()\n - test-tool: migrate read-cache-again to parse_options()\n - test-tool: migrate read-cache-perf to parse_options()\n - test-tool: split up test-tool read-cache\n\n Test code shuffling.\n\n Waiting for reviews.\n\n\n* ab/xdiff-bug-cleanup (2021-06-08) 1 commit\n - xdiff: use BUG(...), not xdl_bug(...)\n\n Code clean-up.\n\n Will merge to 'next'.\n\n\n* ar/test-code-cleanup (2021-06-08) 1 commit\n - t: fix whitespace around &&\n\n Test code clean-up.\n\n Will merge to 'next'.\n\n\n* ba/object-info (2021-06-08) 1 commit\n - protocol-caps.h: add newline at end of file\n\n Code clean-up.\n\n Will merge to 'next'.\n\n\n* dd/document-log-decorate-default (2021-06-08) 1 commit\n - doc/log: correct default for --decorate\n\n Doc clean-up.\n\n Will merge to 'next'.\n\n\n* fc/doc-default-to-upstream-config (2021-06-08) 1 commit\n - doc: merge: mention default of defaulttoupstream\n\n Doc clean-up.\n\n Will merge to 'next'.\n\n\n* ms/mergetools-kdiff3-on-windows (2021-06-08) 1 commit\n - mergetools/kdiff3: make kdiff3 work on Windows too\n\n On Windows, mergetool has been taught to find kdiff3.exe just like\n it finds winmerge.exe.\n\n Will merge to 'next'.\n\n\n* ab/pack-objects-stdin (2021-06-28) 4 commits\n - pack-objects.c: make use of REV_INFO_STDIN_LINE_PROCESS\n - pack-objects.c: do stdin parsing via revision.c's API\n - revision.h: refactor \"disable_stdin\" and \"read_from_stdin\"\n - upload-pack: run is_repository_shallow() before setup_revisions()\n\n Code clean-up.\n\n\n* ar/doc-libera-chat-in-my-first-contrib (2021-06-09) 1 commit\n - MyFirstContribution: link #git-devel to Libera Chat\n\n Update MyFirst document.\n\n Will merge to 'next'.\n\n\n* ar/mailinfo-memcmp-to-skip-prefix (2021-06-09) 1 commit\n - mailinfo: use starts_with() when checking scissors\n\n Code clean-up.\n\n Will merge to 'next'.\n\n\n* ar/submodule-add (2021-06-16) 3 commits\n - submodule--helper: introduce add-config subcommand\n - submodule--helper: introduce add-clone subcommand\n - submodule--helper: refactor module_clone()\n\n Rewrite of \"git submodule\" in C continues.\n\n Waiting for reviews.\n\n\n* ds/gender-neutral-doc (2021-06-16) 5 commits\n - SQUASH??? replace neutering tips with that of Æver\n - CodingGuidelines: recommend singular they\n - *: fix typos\n - comments: avoid using the gender of our users\n - doc: avoid using the gender of other people\n\n Attempt to update the documentation not to assume users are of\n certain gender and adds to guidelines to do so.\n\n Will wait for conclusion of the on-list discussion.\n It appears that the updates to the text are favourably accepted by\n reviewers, but not necessarily the update to the guidelines.\n\n\n* gh/gitweb-branch-sort (2021-06-10) 1 commit\n - gitweb: use HEAD as secondary sort key in git_get_heads_list()\n\n Tie-break branches that point at the same object in the list of\n branches on GitWeb to show the one pointed at by HEAD early.\n\n Waiting for reviews.\n\n\n* jk/doc-max-pack-size (2021-06-09) 1 commit\n - doc: warn people against --max-pack-size\n\n Doc update.\n\n Will merge to 'next'.\n\n\n* lh/systemd-timers (2021-06-14) 3 commits\n - maintenance: add support for systemd timers on Linux\n - maintenance: `git maintenance run` learned `--scheduler=<scheduler>`\n - cache.h: Introduce a generic \"xdg_config_home_for(…)\" function\n\n \"git maintenance\" scheduler learned to use systemd timers as a\n possible backend.\n\n Waiting for reviews.\n\n\n* dd/svn-test-wo-locale-a (2021-06-08) 1 commit\n - t: use user-specified utf-8 locale for testing svn\n\n \"git-svn\" tests assumed that \"locale -a\", which is used to pick an\n available UTF-8 locale, is available everywhere.  A knob has been\n introduced to allow testers to specify a suitable locale to use.\n\n Will merge to 'next'.\n\n\n* fc/completion-updates (2021-06-07) 4 commits\n - completion: bash: add correct suffix in variables\n - completion: bash: fix for multiple dash commands\n - completion: bash: fix for suboptions with value\n - completion: bash: fix prefix detection in branch.*\n\n Command line completion updates.\n\n Expecting a reroll.\n cf. <60be6f7fa4435_db80d208f2@natae.notmuch>\n\n\n* mr/cmake (2021-06-11) 3 commits\n - cmake: add warning for ignored MSGFMT_EXE\n - cmake: create compile_commands.json by default\n - cmake: add knob to disable vcpkg\n\n CMake update.\n\n Will merge to 'next'.\n\n\n* ab/update-submitting-patches (2021-06-08) 3 commits\n - SubmittingPatches: remove pine-specific hints from MUA hints\n - SubmittingPatches: replace discussion of Travis with GitHub Actions\n - SubmittingPatches: move discussion of Signed-off-by above \"send\"\n\n Reorganize and update the SubmitingPatches document.\n\n Expecting a reroll.\n cf. <20210607172542.GA6312@szeder.dev>\n cf. <nycvar.QRO.7.76.6.2106072346560.55@tvgsbejvaqbjf.bet>\n\n\n* hn/prep-tests-for-reftable (2021-06-02) 22 commits\n - t1415: set REFFILES for test specific to storage format\n - t4202: mark bogus head hash test with REFFILES\n - t7003: check reflog existence only for REFFILES\n - t7900: stop checking for loose refs\n - t1404: mark tests that muck with .git directly as REFFILES.\n - t2017: mark --orphan/logAllRefUpdates=false test as REFFILES\n - t1414: mark corruption test with REFFILES\n - t1407: require REFFILES for for_each_reflog test\n - test-lib: provide test prereq REFFILES\n - t5304: use \"reflog expire --all\" to clear the reflog\n - t5304: restyle: trim empty lines, drop ':' before >\n - t7003: use rev-parse rather than FS inspection\n - t5000: inspect HEAD using git-rev-parse\n - t5000: reformat indentation to the latest fashion\n - t1301: fix typo in error message\n - t1413: use tar to save and restore entire .git directory\n - t1401-symbolic-ref: avoid direct filesystem access\n - t1401: use tar to snapshot and restore repo state\n - t5601: read HEAD using rev-parse\n - t9300: check ref existence using test-helper rather than a file system check\n - t/helper/ref-store: initialize oid in resolve-ref\n - t4202: split testcase for invalid HEAD symref and HEAD hash\n\n Preliminary clean-up of tests before the main reftable changes\n hits the codebase.\n\n Will merge to 'next'.\n\n\n* js/trace2-discard-event-docfix (2021-06-04) 1 commit\n - docs: fix api-trace2 doc for \"too_many_files\" event\n\n Docfix.\n\n Will merge to 'next'.\n\n\n* tk/partial-clone-repack-doc (2021-06-04) 1 commit\n - Remove warning that repack only works on non-promisor packfiles\n\n Docfix.\n\n Will merge to 'next'.\n\n\n* zh/cat-file-batch-fix (2021-06-04) 2 commits\n - cat-file: merge two block into one\n - cat-file: handle trivial --batch format with --batch-all-objects\n\n \"git cat-file --batch-all-objects\"\" misbehaved when \"--batch\" is in\n use and did not ask for certain object traits.\n\n Will merge to 'next'.\n\n\n* fc/push-simple-updates (2021-06-02) 7 commits\n - doc: push: explain default=simple correctly\n - push: remove unused code in setup_push_upstream()\n - push: simplify setup_push_simple()\n - push: reorganize setup_push_simple()\n - push: copy code to setup_push_simple()\n - push: hedge code of default=simple\n - push: rename !triangular to same_remote\n (this branch is used by fc/push-simple-updates-cleanup.)\n\n Some code and doc clarification around \"git push\".\n\n Will merge to 'next'.\n\n\n* fc/push-simple-updates-cleanup (2021-06-02) 13 commits\n - push: don't get a full remote object\n - push: only check same_remote when needed\n - push: remove trivial function\n - push: remove redundant check\n - push: factor out the typical case\n - push: get rid of all the setup_push_* functions\n - push: trivial simplifications\n - push: make setup_push_* return the dst\n - push: only get the branch when needed\n - push: factor out null branch check\n - push: split switch cases\n - push: return immediately in trivial switch case\n - push: create new get_upstream_ref() helper\n (this branch uses fc/push-simple-updates.)\n\n Some more code and doc clarification around \"git push\".\n\n Will merge to 'next'.\n\n\n* tb/complete-diff-anchored (2021-05-31) 1 commit\n - completion: add --anchored to diff's options\n\n The command line completion (in contrib/) learned that \"git diff\"\n takes the \"--anchored\" option.\n\n Will merge to 'next'.\n\n\n* en/ort-perf-batch-12 (2021-06-09) 4 commits\n - merge-ort: miscellaneous touch-ups\n - Fix various issues found in comments\n - diffcore-rename: avoid unnecessary strdup'ing in break_idx\n - merge-ort: replace string_list_df_name_compare with faster alternative\n\n More fix-ups and optimization to \"merge -sort\".\n\n Will merge to 'next'.\n\n\n* zh/ref-filter-raw-data (2021-06-28) 15 commits\n . ref-filter: remove grab_oid() function\n . cat-file: re-implement --textconv, --filters options\n . cat-file: reuse err buf in batch_object_write()\n . cat-file: reuse ref-filter logic\n . cat-file: change batch_objects parameter name\n . cat-file: add has_object_file() check\n . ref-filter: modify the error message and value in get_object\n . ref-filter: add cat_file_mode in struct ref_format\n . ref-filter: introduce free_ref_array_item_value() function\n . ref-filter: pass get_object() return value to their callers\n . ref-filter: add %(rest) atom\n . ref-filter: use non-const ref_format in *_atom_parser()\n . ref-filter: --format=%(raw) re-support --perl\n . ref-filter: add %(raw) atom\n . ref-filter: add obj-type check in grab contents\n\n Prepare the \"ref-filter\" machinery that drives the \"--format\"\n option of \"git for-each-ref\" and its friends to be used in \"git\n cat-file --batch\".\n\n This conflicts with the other topic on \"cat-file --batch\" by the\n same author.\n\n\n* jh/builtin-fsmonitor (2021-05-24) 30 commits\n - t/perf: avoid copying builtin fsmonitor files into test repo\n - t7527: test status with untracked-cache and fsmonitor--daemon\n - p7519: add fsmonitor--daemon\n - t7527: create test for fsmonitor--daemon\n - fsmonitor: force update index after large responses\n - fsmonitor: enhance existing comments\n - fsmonitor--daemon: use a cookie file to sync with file system\n - fsmonitor--daemon: periodically truncate list of modified files\n - fsmonitor--daemon: implement handle_client callback\n - fsmonitor-fs-listen-macos: implement FSEvent listener on MacOS\n - fsmonitor-fs-listen-macos: add macos header files for FSEvent\n - fsmonitor-fs-listen-win32: implement FSMonitor backend on Windows\n - fsmonitor--daemon: create token-based changed path cache\n - fsmonitor--daemon: define token-ids\n - fsmonitor--daemon: add pathname classification\n - fsmonitor--daemon: implement daemon command options\n - fsmonitor-fs-listen-macos: stub in backend for MacOS\n - fsmonitor-fs-listen-win32: stub in backend for Windows\n - t/helper/fsmonitor-client: create IPC client to talk to FSMonitor Daemon\n - fsmonitor--daemon: implement client command options\n - fsmonitor--daemon: add a built-in fsmonitor daemon\n - fsmonitor: introduce `core.useBuiltinFSMonitor` to call the daemon via IPC\n - config: FSMonitor is repository-specific\n - help: include fsmonitor--daemon feature flag in version info\n - fsmonitor-ipc: create client routines for git-fsmonitor--daemon\n - fsmonitor--daemon: update fsmonitor documentation\n - fsmonitor--daemon: man page\n - simple-ipc: preparations for supporting binary messages.\n - Merge branch 'jk/perf-in-worktrees' into HEAD\n - Merge branch 'jh/simple-ipc' into jh/rfc-builtin-fsmonitor\n\n An attempt to write and ship with a watchman equivalent tailored\n for our use.\n\n What's the status of this one?\n\n\n* es/trace2-log-parent-process-name (2021-06-09) 1 commit\n - tr2: log parent process name\n\n trace2 logs learned to show parent process name to see in what\n context Git was invoked.\n\n Will merge to 'next'?\n\n\n* ab/send-email-optim (2021-05-28) 13 commits\n - perl: nano-optimize by replacing Cwd::cwd() with Cwd::getcwd()\n - send-email: move trivial config handling to Perl\n - perl: lazily load some common Git.pm setup code\n - send-email: lazily load modules for a big speedup\n - send-email: get rid of indirect object syntax\n - send-email: use function syntax instead of barewords\n - send-email: lazily shell out to \"git var\"\n - send-email: lazily load config for a big speedup\n - send-email: copy \"config_regxp\" into git-send-email.perl\n - send-email: refactor sendemail.smtpencryption config parsing\n - send-email: remove non-working support for \"sendemail.smtpssl\"\n - send-email tests: test for boolean variables without a value\n - send-email tests: support GIT_TEST_PERL_FATAL_WARNINGS=true\n\n \"git send-email\" optimization.\n\n Will merge to 'next'.\n\n\n* ab/fsck-unexpected-type (2021-06-28) 21 commits\n - fsck: report invalid object type-path combinations\n - fsck: report invalid types recorded in objects\n - object-store.h: move read_loose_object() below 'struct object_info'\n - fsck: don't hard die on invalid object types\n - object-file.c: return -2 on \"header too long\" in unpack_loose_header()\n - object-file.c: return -1, not \"status\" from unpack_loose_header()\n - object-file.c: guard against future bugs in loose_object_info()\n - object-file.c: stop dying in parse_loose_header()\n - object-file.c: split up ternary in parse_loose_header()\n - object-file.c: simplify unpack_loose_short_header()\n - object-file.c: add missing braces to loose_object_info()\n - object-file.c: make parse_loose_header_extended() public\n - object-file.c: don't set \"typep\" when returning non-zero\n - cache.h: move object functions to object-store.h\n - cat-file tests: test for current --allow-unknown-type behavior\n - cat-file tests: add corrupt loose object test\n - rev-list tests: test for behavior with invalid object types\n - cat-file tests: test that --allow-unknown-type isn't on by default\n - cat-file tests: test for missing object with -t and -s\n - fsck tests: add test for fsck-ing an unknown type\n - fsck tests: refactor one test to use a sub-repo\n\n \"git fsck\" has been taught to report mismatch between expected and\n actual types of an object better.\n\n--------------------------------------------------\n[Will Discard]\n\n* ag/merge-strategies-in-c (2021-03-17) 15 commits\n . sequencer: use the \"octopus\" merge 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 . t6407: modernise tests\n\n The resolve and octopus merge strategy backends have been rewritten\n in C.\n\n Will discard.\n We had to wait for a reroll for too long.\n cf. <nycvar.QRO.7.76.6.2103241142220.50@tvgsbejvaqbjf.bet>\n\n\n* mr/bisect-in-c-4 (2021-04-11) 4 commits\n . bisect--helper: retire `--bisect-next-check` subcommand\n . bisect--helper: reimplement `bisect_run` shell function in C\n . bisect--helper: reimplement `bisect_visualize()`shell function in C\n . run-command: make `exists_in_PATH()` non-static\n\n The codepaths involved in running \"git bisect visualize\" and \"git\n bisect run\" has been rewritten in C.\n\n Will discard.\n We had to wait for a reroll for too long.\n cf. <xmqq35vwh8qk.fsf@gitster.g>, <xmqqy2doftrj.fsf@gitster.g>,\n <CAP8UFD3X24F3qgefHpi00PM-KUk+vcqxwy2Dbngbyj7ciavCVQ@mail.gmail.com>\n May want to boost the test coverage.\n\n\n* ma/t0091-bugreport-fix (2021-04-12) 1 commit\n . t0091-bugreport.sh: actually verify some content of report\n\n Test fix.\n\n Will discard.\n We had to wait for a reroll for too long.\n Expecting a reroll.\n cf. <YHYZTLl90rkWWVOr@google.com>, <87a6q22dei.fsf@evledraar.gmail.com>\n\n\n* ls/fast-export-signed (2021-05-03) 5 commits\n . fast-export, fast-import: add support for signed-commits\n . fast-export: do not modify memory from get_commit_buffer\n . git-fast-export.txt: clarify why 'verbatim' may not be a good idea\n . fast-export: rename --signed-tags='warn' to 'warn-verbatim'\n . git-fast-import.txt: add missing LF in the BNF\n\n \"git fast-export\" offers a way to control how signed tags are\n handled; the mechanism has been extended to allow specifying how\n signed commits are handled as well.\n\n Will discard.\n We had to wait for a reroll for too long.\n cf. <xmqqa6pca0pv.fsf@gitster.g>, <xmqq1rao9zev.fsf@gitster.g>\n\n\n* fc/neuter-doc (2021-06-14) 2 commits\n . comments: avoid using the gender of our users\n . doc: avoid using the gender of other people\n\n An attempt to avoid gendered pronouns by rewriting parts of docs.\n\n This seems to give more readable result than Stolee's \"use singular\n they\" topic.\n\n Now is a part of the ds/gender-neutral-doc topic.\n\n\n* es/config-based-hooks (2021-05-27) 37 commits\n . docs: link githooks and git-hook manpages\n . doc: clarify fsmonitor-watchman specification\n . run-command: stop thinking about hooks\n . git-send-email: use 'git hook run' for 'sendemail-validate'\n . bugreport: use hook_exists instead of find_hook\n . receive-pack: convert receive hooks to hook.h\n . post-update: use hook.h library\n . proc-receive: acquire hook list from hook.h\n . receive-pack: convert 'update' hook to hook.h\n . reference-transaction: look for hooks in config\n . transport: convert pre-push hook to use config\n . hook: convert 'post-rewrite' hook to config\n . hooks: convert 'post-checkout' hook to hook library\n . git-p4: use 'git hook' to run hooks\n . receive-pack: convert push-to-checkout hook to hook.h\n . read-cache: convert post-index-change hook to use config\n . rebase: teach pre-rebase to use hook.h\n . gc: use hook library for pre-auto-gc hook\n . merge: use config-based hooks for post-merge hook\n . am: convert applypatch hooks to use config\n . commit: use config-based hooks\n . hooks: allow callers to capture output\n . run-command: allow capturing of collated output\n . hook: provide stdin by string_list or callback\n . run-command: add stdin callback for parallelization\n . hook: allow specifying working directory for hooks\n . hook: allow parallel hook execution\n . run-command: allow stdin for run_processes_parallel\n . hook: support passing stdin to hooks\n . hook: introduce hook_exists()\n . hook: add 'run' subcommand\n . parse-options: parse into strvec\n . hook: implement hookcmd.<name>.skip\n . hook: teach hook.runHookDir\n . hook: include hookdir hook in list\n . hook: introduce git-hook subcommand\n . doc: propose hooks managed by the config\n\n Propose hook scripts defined in the configuration.\n\n Tentatively kicked out to give ab/config-based-hooks-base a chance\n to interact with other topics in flight.\n\n\n* bc/doc-asciidoctor-to-man-wo-xmlto (2021-05-14) 2 commits\n . doc: remove GNU_ROFF option\n . doc: add an option to have Asciidoctor build man pages directly\n\n An option to render the manual pages via AsciiDoctor bypassing\n xmlto has been introduced.\n"},{"id":"428862","messageId":"60dd062a779d7_1634c20840@natae.notmuch","threadId":"56034","inReplyTo":"xmqq4kdft122.fsf@gitster.g","subject":"RE: What's cooking in git.git (Jun 2021, #07; Wed, 30)","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2021-07-01T00:02:50Z","receivedAt":"2021-07-01T00:02:56Z","isPatch":false,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"Junio C Hamano wrote:\n> Here are the topics that have been cooking in my tree.\n\n> --------------------------------------------------\n> [New Topics]\n\nThis is a topic:\n\nhttps://lore.kernel.org/git/20210623004815.1807-1-felipe.contreras@gmail.com/\n\nSo is this:\n\nhttps://lore.kernel.org/git/20210615183157.104999-1-felipe.contreras@gmail.com/\n\nAnd this:\n\nhttps://lore.kernel.org/git/20210628031642.699156-1-felipe.contreras@gmail.com/\n\nAnd this:\n\nhttps://lore.kernel.org/git/20210607185043.893205-1-felipe.contreras@gmail.com/\n\n-- \nFelipe Contreras\n"},{"id":"428876","messageId":"CAFQ2z_Opw4dZmH7Xc4tRzMvD-op-Reh3C7sM31VoNM5eAHaiLA@mail.gmail.com","threadId":"56034","inReplyTo":"CAFQ2z_PYJMzOBcN+KoQSfdD-eYtZVzrArL0Dh-YW-q3R8hgm2w@mail.gmail.com","subject":"Re: What's cooking in git.git (Jun 2021, #07; Wed, 30)","fromName":"Han-Wen Nienhuys","fromEmail":"hanwen@google.com","sentAt":"2021-07-01T07:19:32Z","receivedAt":"2021-07-01T07:19:46Z","isPatch":false,"sender":{"key":"hanwen@google.com","avatar":"https://avatars.githubusercontent.com/u/31547?v=4"},"body":"(now in plain text)\n\nOn Thu, Jul 1, 2021 at 9:17 AM Han-Wen Nienhuys <hanwen@google.com> wrote:\n>>  Futz with the way 'errno' is relied on in the refs API to carry the\n>>  failure modes up the callchain.\n>>\n>>  Will merge to 'next'?\n\nIs there anything I can do to address the '?' here?\n\nI am happy to do a review for a similar sized chunk of other code for\nsomeone else in exchange for review of this.\n\n-- \nHan-Wen Nienhuys - Google Munich\nI work 80%. Don't expect answers from me on Fridays.\n--\nGoogle Germany GmbH, Erika-Mann-Strasse 33, 80636 Munich\nRegistergericht und -nummer: Hamburg, HRB 86891\nSitz der Gesellschaft: Hamburg\nGeschäftsführer: Paul Manicle, Halimah DeLaine Prado\n"},{"id":"428895","messageId":"87zgv6cgxs.fsf@evledraar.gmail.com","threadId":"56034","inReplyTo":"CAFQ2z_Opw4dZmH7Xc4tRzMvD-op-Reh3C7sM31VoNM5eAHaiLA@mail.gmail.com","subject":"Re: What's cooking in git.git (Jun 2021, #07; Wed, 30)","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2021-07-01T13:06:37Z","receivedAt":"2021-07-01T13:16:52Z","isPatch":false,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"\nOn Thu, Jul 01 2021, Han-Wen Nienhuys wrote:\n\n> (now in plain text)\n>\n> On Thu, Jul 1, 2021 at 9:17 AM Han-Wen Nienhuys <hanwen@google.com> wrote:\n>>>  Futz with the way 'errno' is relied on in the refs API to carry the\n>>>  failure modes up the callchain.\n>>>\n>>>  Will merge to 'next'?\n>\n> Is there anything I can do to address the '?' here?\n>\n> I am happy to do a review for a similar sized chunk of other code for\n> someone else in exchange for review of this.\n\nI took you up on that and gave that series a detailed look now[1].\n\nI'm afraid what I had to say there doesn't bode well for getting it to\n'next' (some style/questions etc. there, but also what I believe is a\nserious bug in 8/8 was among the findings[2]).\n\nI'd like you to take a look & review any of my outstanding \"ab/*\"\nserieses (or combinatino thereof) of similar scope/complexity as yours\nin return. I'l leave it to your interests & sense of fairness what to\npick which one(s) :)\n\n1. https://lore.kernel.org/git/pull.1012.v2.git.git.1623329868.gitgitgadget@gmail.com/\n2. https://lore.kernel.org/git/8735sydvzs.fsf@evledraar.gmail.com/\n"},{"id":"428898","messageId":"87r1gicfh1.fsf@evledraar.gmail.com","threadId":"56034","inReplyTo":"xmqq4kdft122.fsf@gitster.g","subject":"Re: What's cooking in git.git (Jun 2021, #07; Wed, 30)","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2021-07-01T13:42:23Z","receivedAt":"2021-07-01T13:48:37Z","isPatch":false,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"\nOn Wed, Jun 30 2021, Junio C Hamano wrote:\n\n> * jh/builtin-fsmonitor (2021-05-24) 30 commits\n>  - t/perf: avoid copying builtin fsmonitor files into test repo\n>  - t7527: test status with untracked-cache and fsmonitor--daemon\n>  - p7519: add fsmonitor--daemon\n>  - t7527: create test for fsmonitor--daemon\n>  - fsmonitor: force update index after large responses\n>  - fsmonitor: enhance existing comments\n>  - fsmonitor--daemon: use a cookie file to sync with file system\n>  - fsmonitor--daemon: periodically truncate list of modified files\n>  - fsmonitor--daemon: implement handle_client callback\n>  - fsmonitor-fs-listen-macos: implement FSEvent listener on MacOS\n>  - fsmonitor-fs-listen-macos: add macos header files for FSEvent\n>  - fsmonitor-fs-listen-win32: implement FSMonitor backend on Windows\n>  - fsmonitor--daemon: create token-based changed path cache\n>  - fsmonitor--daemon: define token-ids\n>  - fsmonitor--daemon: add pathname classification\n>  - fsmonitor--daemon: implement daemon command options\n>  - fsmonitor-fs-listen-macos: stub in backend for MacOS\n>  - fsmonitor-fs-listen-win32: stub in backend for Windows\n>  - t/helper/fsmonitor-client: create IPC client to talk to FSMonitor Daemon\n>  - fsmonitor--daemon: implement client command options\n>  - fsmonitor--daemon: add a built-in fsmonitor daemon\n>  - fsmonitor: introduce `core.useBuiltinFSMonitor` to call the daemon via IPC\n>  - config: FSMonitor is repository-specific\n>  - help: include fsmonitor--daemon feature flag in version info\n>  - fsmonitor-ipc: create client routines for git-fsmonitor--daemon\n>  - fsmonitor--daemon: update fsmonitor documentation\n>  - fsmonitor--daemon: man page\n>  - simple-ipc: preparations for supporting binary messages.\n>  - Merge branch 'jk/perf-in-worktrees' into HEAD\n>  - Merge branch 'jh/simple-ipc' into jh/rfc-builtin-fsmonitor\n>\n>  An attempt to write and ship with a watchman equivalent tailored\n>  for our use.\n>\n>  What's the status of this one?\n\nI think Johannes's reply to the last WC applies[1]:\n\n    I am not Jeff, but I know that he is busy getting back to it, and\n    plans on submitting a third iteration.\n\nFWIW I'm still curious about some details on the performance concerns\nthat seem to have prompted this built-in fsmonitor endeavor, as I asked\nabout (but didn't get a reply to) in [2].\n\nNot as a \"we shouldn't have this, let's keep using the hook\", but just\ncuriosity about why we've seemingly gotten such different performance\nnumbers on the watchman hook v.s. a built-in approach.\n\nI suspect (but don't know) that the reason is that the built-in is\nperhaps integrating differently with git somehow, in a way that we could\nretrofit the hook approach to also do (if anyone still cares about the\nhook approach).\n\nIn any case I'm interested in *why* the new approach is faster, given\nthat I've done some testing (again, noted in [2]) that suggest that\nbottleneck in the previous pipeline wasn't at all what Jeff H. thought\nit was.\n\n1. https://lore.kernel.org/git/nycvar.QRO.7.76.6.2106171135530.57@tvgsbejvaqbjf.bet/#t\n2. https://lore.kernel.org/git/87h7lgfchm.fsf@evledraar.gmail.com/\n"},{"id":"428900","messageId":"4d72a3d6-397e-e55b-cc11-5591070d5e27@jeffhostetler.com","threadId":"56034","inReplyTo":"87r1gicfh1.fsf@evledraar.gmail.com","subject":"Re: What's cooking in git.git (Jun 2021, #07; Wed, 30)","fromName":"Jeff Hostetler","fromEmail":"git@jeffhostetler.com","sentAt":"2021-07-01T14:20:29Z","receivedAt":"2021-07-01T14:20:32Z","isPatch":false,"sender":{"key":"git@jeffhostetler.com","avatar":null},"body":"\n\nOn 7/1/21 9:42 AM, Ævar Arnfjörð Bjarmason wrote:\n> \n> On Wed, Jun 30 2021, Junio C Hamano wrote:\n> \n>> * jh/builtin-fsmonitor (2021-05-24) 30 commits\n>>   - t/perf: avoid copying builtin fsmonitor files into test repo\n>>   - t7527: test status with untracked-cache and fsmonitor--daemon\n>>   - p7519: add fsmonitor--daemon\n>>   - t7527: create test for fsmonitor--daemon\n>>   - fsmonitor: force update index after large responses\n>>   - fsmonitor: enhance existing comments\n>>   - fsmonitor--daemon: use a cookie file to sync with file system\n>>   - fsmonitor--daemon: periodically truncate list of modified files\n>>   - fsmonitor--daemon: implement handle_client callback\n>>   - fsmonitor-fs-listen-macos: implement FSEvent listener on MacOS\n>>   - fsmonitor-fs-listen-macos: add macos header files for FSEvent\n>>   - fsmonitor-fs-listen-win32: implement FSMonitor backend on Windows\n>>   - fsmonitor--daemon: create token-based changed path cache\n>>   - fsmonitor--daemon: define token-ids\n>>   - fsmonitor--daemon: add pathname classification\n>>   - fsmonitor--daemon: implement daemon command options\n>>   - fsmonitor-fs-listen-macos: stub in backend for MacOS\n>>   - fsmonitor-fs-listen-win32: stub in backend for Windows\n>>   - t/helper/fsmonitor-client: create IPC client to talk to FSMonitor Daemon\n>>   - fsmonitor--daemon: implement client command options\n>>   - fsmonitor--daemon: add a built-in fsmonitor daemon\n>>   - fsmonitor: introduce `core.useBuiltinFSMonitor` to call the daemon via IPC\n>>   - config: FSMonitor is repository-specific\n>>   - help: include fsmonitor--daemon feature flag in version info\n>>   - fsmonitor-ipc: create client routines for git-fsmonitor--daemon\n>>   - fsmonitor--daemon: update fsmonitor documentation\n>>   - fsmonitor--daemon: man page\n>>   - simple-ipc: preparations for supporting binary messages.\n>>   - Merge branch 'jk/perf-in-worktrees' into HEAD\n>>   - Merge branch 'jh/simple-ipc' into jh/rfc-builtin-fsmonitor\n>>\n>>   An attempt to write and ship with a watchman equivalent tailored\n>>   for our use.\n>>\n>>   What's the status of this one?\n> \n> I think Johannes's reply to the last WC applies[1]:\n> \n>      I am not Jeff, but I know that he is busy getting back to it, and\n>      plans on submitting a third iteration.\n> \n> FWIW I'm still curious about some details on the performance concerns\n> that seem to have prompted this built-in fsmonitor endeavor, as I asked\n> about (but didn't get a reply to) in [2].\n> \n> Not as a \"we shouldn't have this, let's keep using the hook\", but just\n> curiosity about why we've seemingly gotten such different performance\n> numbers on the watchman hook v.s. a built-in approach.\n> \n> I suspect (but don't know) that the reason is that the built-in is\n> perhaps integrating differently with git somehow, in a way that we could\n> retrofit the hook approach to also do (if anyone still cares about the\n> hook approach).\n> \n> In any case I'm interested in *why* the new approach is faster, given\n> that I've done some testing (again, noted in [2]) that suggest that\n> bottleneck in the previous pipeline wasn't at all what Jeff H. thought\n> it was.\n> \n> 1. https://lore.kernel.org/git/nycvar.QRO.7.76.6.2106171135530.57@tvgsbejvaqbjf.bet/#t\n> 2. https://lore.kernel.org/git/87h7lgfchm.fsf@evledraar.gmail.com/\n> \n\nA quick reply here of Junio's question. Yes, I'm working on a V3\nto submit (any day now -- $DAYJOB notwithstanding (read: meetings)).\n\nI'll push this up and then try to answer the perf questions.\n\nThanks for your patience.\nJeff\n"},{"id":"428902","messageId":"87o8bmcd9a.fsf@evledraar.gmail.com","threadId":"56034","inReplyTo":"xmqq4kdft122.fsf@gitster.g","subject":"Re: What's cooking in git.git (Jun 2021, #07; Wed, 30)","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2021-07-01T14:00:01Z","receivedAt":"2021-07-01T14:36:22Z","isPatch":false,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"\nComments on my outstanding:\n\nOn Wed, Jun 30 2021, Junio C Hamano wrote:\n\n> * ab/bundle-doc (2021-06-30) 3 commits\n>  - bundle doc: elaborate on rev<->ref restriction\n>  - bundle doc: split out open v.s. closed discussion from <rev-arg>\n>  - bundle doc: rewrite the \"DESCRIPTION\" section\n>\n>  Doc update.\n\nWill update per your https://lore.kernel.org/git/xmqqk0mbt5rj.fsf@gitster.g/\n\n> * ab/pack-stdin-packs-fix (2021-06-28) 2 commits\n>  - pack-objects: fix segfault in --stdin-packs option\n>  - pack-objects tests: cover blindspots in stdin handling\n>\n>  Input validation of \"git pack-objects --stdin-packs\" has been\n>  corrected.\n>\n>  cf. <YND3h2l10PlnSNGJ@nand.local>\n\nWill address & re-roll.\n\n> * ab/bundle-updates (2021-06-30) 3 commits\n>  - bundle: remove \"ref_list\" in favor of string-list.c API\n>  - bundle.c: use a temporary variable for OIDs and names\n>  - bundle cmd: stop leaking memory from parse_options_cmd_bundle()\n>\n>\n> * ab/fetch-negotiate-segv-fix (2021-06-30) 3 commits\n>  - fetch: fix segfault in --negotiate-only without --negotiation-tip=*\n>  - fetch: document the --negotiate-only option\n>  - send-pack.c: move \"no refs in common\" abort earlier\n\nHas outstanding comments, will re-roll.\n\n> * ab/make-delete-on-error (2021-06-29) 1 commit\n>  - Makefile: add and use the \".DELETE_ON_ERROR\" flag\n\nI think this is stable at this point & has gotten a lot of\nattention/review:\nhttps://lore.kernel.org/git/patch-1.1-2557117855-20210629T084356Z-avarab@gmail.com/\n\n> * ab/make-tags-cleanup (2021-06-29) 5 commits\n>  - Makefile: normalize clobbering & xargs for tags targets\n>  - Makefile: don't use \"FORCE\" for tags targets\n>  - Makefile: fix \"cscope\" target to refer to cscope.out\n>  - Makefile: add QUIET_GEN to \"cscope\" target\n>  - Makefile: move \".PHONY: cscope\" near its target\n\nWill re-roll per https://lore.kernel.org/git/67c45b13-df8f-8065-377a-fbd2f80992ee@ramsayjones.plus.com/\n\n> * ab/describe-tests-fix (2021-05-11) 5 commits\n>  - describe tests: support -C in \"check_describe\"\n>  - describe tests: fix nested \"test_expect_success\" call\n>  - describe tests: don't rely on err.actual from \"check_describe\"\n>  - describe tests: refactor away from glob matching\n>  - describe tests: improve test for --work-tree & --dirty\n>\n>  Various updates to tests around \"git describe\"\n>\n>  Will merge to 'next'.\n\nThanks!\n\n> * ab/pickaxe-pcre2 (2021-05-11) 22 commits\n>  - xdiff-interface: replace discard_hunk_line() with a flag\n>  - xdiff users: use designated initializers for out_line\n>  - pickaxe -G: don't special-case create/delete\n>  - pickaxe -G: terminate early on matching lines\n>  - xdiff-interface: allow early return from xdiff_emit_line_fn\n>  - xdiff-interface: prepare for allowing early return\n>  - pickaxe -S: slightly optimize contains()\n>  - pickaxe: rename variables in has_changes() for brevity\n>  - pickaxe -S: support content with NULs under --pickaxe-regex\n>  - pickaxe: assert that we must have a needle under -G or -S\n>  - pickaxe: refactor function selection in diffcore-pickaxe()\n>  - perf: add performance test for pickaxe\n>  - pickaxe/style: consolidate declarations and assignments\n>  - diff.h: move pickaxe fields together again\n>  - pickaxe: die when --find-object and --pickaxe-all are combined\n>  - pickaxe: die when -G and --pickaxe-regex are combined\n>  - pickaxe tests: add missing test for --no-pickaxe-regex being an error\n>  - pickaxe tests: test for -G, -S and --find-object incompatibility\n>  - pickaxe tests: add test for \"log -S\" not being a regex\n>  - pickaxe tests: add test for diffgrep_consume() internals\n>  - pickaxe tests: refactor to use test_commit --append --printf\n>  - grep/pcre2 tests: reword comments referring to kwset\n>\n>  Rewrite the backend for \"diff -G/-S\" to use pcre2 engine when\n>  available.\n>\n>  Will merge to 'next'.\n\nThanks!\n\n> * ab/config-based-hooks-base (2021-06-29) 33 commits\n>  - hooks: fix a TOCTOU in \"did we run a hook?\" heuristic\n>  - receive-pack: convert receive hooks to hook.h\n>  - post-update: use hook.h library\n>  - receive-pack: convert 'update' hook to hook.h\n>  - hooks: allow callers to capture output\n>  - run-command: allow capturing of collated output\n>  - reference-transaction: use hook.h to run hooks\n>  - transport: convert pre-push hook to use config\n>  - hook: convert 'post-rewrite' hook in sequencer.c to hook.h\n>  - hook: provide stdin by string_list or callback\n>  - run-command: add stdin callback for parallelization\n>  - am: convert 'post-rewrite' hook to hook.h\n>  - hook: support passing stdin to hooks\n>  - run-command: allow stdin for run_processes_parallel\n>  - run-command: remove old run_hook_{le,ve}() hook API\n>  - receive-pack: convert push-to-checkout hook to hook.h\n>  - read-cache: convert post-index-change hook to use config\n>  - commit: use hook.h to execute hooks\n>  - git-p4: use 'git hook' to run hooks\n>  - send-email: use 'git hook run' for 'sendemail-validate'\n>  - git hook run: add an --ignore-missing flag\n>  - merge: use config-based hooks for post-merge hook\n>  - hooks: convert 'post-checkout' hook to hook library\n>  - am: convert applypatch hooks to use config\n>  - rebase: teach pre-rebase to use hook.h\n>  - gc: use hook library for pre-auto-gc hook\n\nFor this one I'd submitted these independently as\n\n>  - hook: add 'run' subcommand\n>  - hook-list.h: add a generated list of hooks, like config-list.h\n>  - hook.c: add a hook_exists() wrapper and use it in bugreport.c\n>  - hook.[ch]: move find_hook() to this new library\n\nhttps://lore.kernel.org/git/cover-0.3-0000000000-20210629T183325Z-avarab@gmail.com/\n\n(René had a comment about his SOB on a patch there, but I see you\naddressed it while queuing it (changed to Helped-by))\n\nAnd:\n\n>  - Makefile: remove an out-of-date comment\n>  - Makefile: stop hardcoding {command,config}-list.h\n>  - Makefile: mark \"check\" target as .PHONY\n\nhttps://lore.kernel.org/git/cover-0.3-0000000000-20210629T190137Z-avarab@gmail.com/\n\nBut you re-combined them. If it makes things easier for you I'm all for\nit, but it seems better to me keep them split if that mean that some\nparts can advance faster, and thus make the rest easier to review.\n\n>  Restructuring of (a subset of) Emily's config-based-hooks series,\n>  to demonstrate that a series can be presented as a more logical and\n>  focused progression.\n>\n>  Waiting for reviews.\n\nI think it's mainly waiting on Emily giving it a detailed look, per\nhttps://lore.kernel.org/git/YNEoMZ1QhZwmS0v2@google.com/\n\n>\n> * ab/config-hooks-path-testfix (2021-06-16) 1 commit\n>  - pre-commit hook tests: don't leave \"actual\" nonexisting on failure\n>\n>  Test fix.\n>\n>  Will merge to 'next'.\n>\n>\n> * ab/doc-retire-alice-bob (2021-06-16) 6 commits\n>  - pack-protocol doc: use \"www-data\" in place of \"alice\"\n>  - doc: replace \"alice\" and \"bob\" with \"jdoe\" and \"msmith\"\n>  - fast-import doc: change \"bob\" in an example to \"file.txt\"\n>  - daemon doc + code comments: reword \"alice\" example\n>  - gitcvs-migration doc: replace \"alice\" and \"bob\" with \"you\" and \"www-data\"\n>  - gittutorial doc: replace \"alice\" and \"bob\" with \"you\" and \"www-data\"\n\nHaving re-read the discussion now I don't know if there's anything\noutstanding to change about this series. It's gotten a lot of attention\nso far, so it's more of a matter of if you're willing to take this sort\nof documentation change or not.\n\n> * ab/serve-cleanup (2021-06-28) 8 commits\n>  - upload-pack.c: convert to new serve.c \"startup\" config cb\n>  - serve: add support for a \"startup\" git_config() callback\n>  - serve.c: add trace2 regions for advertise & command\n>  - serve.c: add call_{advertise,command}() indirection\n>  - serve: use designated initializers\n>  - transport: use designated initializers\n>  - transport: rename \"fetch\" in transport_vtable to \"fetch_refs\"\n>  - serve: mark has_capability() as static\n>\n>  Code clean-up around \"git serve\".\n\nJeff, what do you think about this version & the upload-pack.c changes?\n\n> * ab/mktag-tests (2021-06-28) 6 commits\n>  - mktag tests: test fast-export\n>  - mktag tests: test for-each-ref\n>  - mktag tests: test update-ref and reachable fsck\n>  - mktag tests: test hash-object --literally and unreachable fsck\n>  - mktag tests: invert --no-strict test\n>  - mktag tests: parse out options in helper\n>\n>  Fill test gaps.\n>\n>\n> * ab/show-branch-tests (2021-06-28) 4 commits\n>  - show-branch tests: add missing tests\n>  - show-branch: don't <COLOR></RESET> for space characters\n>  - show-branch tests: modernize test code\n>  - show-branch tests: rename the one \"show-branch\" test file\n>\n>  Fill test gaps.\n\nI think both of these are ready for next.\n\nThe mktag one had trivial improvement suggestions I've addressed in a\nre-roll already. Felipe gave v2 of ab/show-branch-tests a review here:\nhttps://lore.kernel.org/git/60cbbe9d89cbd_1299208fa@natae.notmuch/\n\nI'm partial to just keeping it as-is, I think in 2/4 it's OK to fix\nvarious test modernization issues at once & they don't need to be split\nup into more commits, it's easy enough to follow along with it. What do\nyou think?\n\n> * ab/cmd-foo-should-return (2021-06-09) 1 commit\n>  - builtins + test helpers: use return instead of exit() in cmd_*\n>\n>  Code clean-up.\n>\n>  Will merge to 'next'.\n>\n>\n> * ab/progress-cleanup (2021-06-08) 1 commit\n>  - read-cache.c: don't guard calls to progress.c API\n>\n>  Code clean-up.\n>\n>  Will merge to 'next'.\n\nThanks!\n\n> * ab/test-tool-cache-cleanup (2021-06-08) 4 commits\n>  - read-cache perf: add a perf test for refresh_index()\n>  - test-tool: migrate read-cache-again to parse_options()\n>  - test-tool: migrate read-cache-perf to parse_options()\n>  - test-tool: split up test-tool read-cache\n>\n>  Test code shuffling.\n>\n>  Waiting for reviews.\n\nEmily gave this a review, waiting on her reply to my reply at\nhttps://lore.kernel.org/git/87pmw3dlkl.fsf@evledraar.gmail.com/\n\nI.e. whether the outstanding is to merely clarify more in commit\nmessages that the goal is to split up a doing-to-many-things function,\nnot to optimize an existing perf test.\n\n> * ab/xdiff-bug-cleanup (2021-06-08) 1 commit\n>  - xdiff: use BUG(...), not xdl_bug(...)\n>\n>  Code clean-up.\n>\n>  Will merge to 'next'.\n\nThanks!\n\n> * ab/pack-objects-stdin (2021-06-28) 4 commits\n>  - pack-objects.c: make use of REV_INFO_STDIN_LINE_PROCESS\n>  - pack-objects.c: do stdin parsing via revision.c's API\n>  - revision.h: refactor \"disable_stdin\" and \"read_from_stdin\"\n>  - upload-pack: run is_repository_shallow() before setup_revisions()\n>\n>  Code clean-up.\n\nEmily & Jonathan Tan gave the version preceding this one a detailed\nonce-over, my re-roll addressed their concerns, but it would be good to\nhear if they agree with that assessment.\n\n> * ab/update-submitting-patches (2021-06-08) 3 commits\n>  - SubmittingPatches: remove pine-specific hints from MUA hints\n>  - SubmittingPatches: replace discussion of Travis with GitHub Actions\n>  - SubmittingPatches: move discussion of Signed-off-by above \"send\"\n>\n>  Reorganize and update the SubmitingPatches document.\n>\n>  Expecting a reroll.\n>  cf. <20210607172542.GA6312@szeder.dev>\n>  cf. <nycvar.QRO.7.76.6.2106072346560.55@tvgsbejvaqbjf.bet>\n\nWill try to re-roll sooner than later...\n\n> * ab/send-email-optim (2021-05-28) 13 commits\n>  - perl: nano-optimize by replacing Cwd::cwd() with Cwd::getcwd()\n>  - send-email: move trivial config handling to Perl\n>  - perl: lazily load some common Git.pm setup code\n>  - send-email: lazily load modules for a big speedup\n>  - send-email: get rid of indirect object syntax\n>  - send-email: use function syntax instead of barewords\n>  - send-email: lazily shell out to \"git var\"\n>  - send-email: lazily load config for a big speedup\n>  - send-email: copy \"config_regxp\" into git-send-email.perl\n>  - send-email: refactor sendemail.smtpencryption config parsing\n>  - send-email: remove non-working support for \"sendemail.smtpssl\"\n>  - send-email tests: test for boolean variables without a value\n>  - send-email tests: support GIT_TEST_PERL_FATAL_WARNINGS=true\n>\n>  \"git send-email\" optimization.\n>\n>  Will merge to 'next'.\n\nThanks!\n\n> * ab/fsck-unexpected-type (2021-06-28) 21 commits\n>  - fsck: report invalid object type-path combinations\n>  - fsck: report invalid types recorded in objects\n>  - object-store.h: move read_loose_object() below 'struct object_info'\n>  - fsck: don't hard die on invalid object types\n>  - object-file.c: return -2 on \"header too long\" in unpack_loose_header()\n>  - object-file.c: return -1, not \"status\" from unpack_loose_header()\n>  - object-file.c: guard against future bugs in loose_object_info()\n>  - object-file.c: stop dying in parse_loose_header()\n>  - object-file.c: split up ternary in parse_loose_header()\n>  - object-file.c: simplify unpack_loose_short_header()\n>  - object-file.c: add missing braces to loose_object_info()\n>  - object-file.c: make parse_loose_header_extended() public\n>  - object-file.c: don't set \"typep\" when returning non-zero\n>  - cache.h: move object functions to object-store.h\n>  - cat-file tests: test for current --allow-unknown-type behavior\n>  - cat-file tests: add corrupt loose object test\n>  - rev-list tests: test for behavior with invalid object types\n>  - cat-file tests: test that --allow-unknown-type isn't on by default\n>  - cat-file tests: test for missing object with -t and -s\n>  - fsck tests: add test for fsck-ing an unknown type\n>  - fsck tests: refactor one test to use a sub-repo\n>\n>  \"git fsck\" has been taught to report mismatch between expected and\n>  actual types of an object better.\n\nThere's one trivial comment on the v4, will re-roll as a v5:\nhttps://lore.kernel.org/git/87mtrezya9.fsf@evledraar.gmail.com/\n\nI think this should be ready to move to 'next', but it would be good to\nget other eyes on the mid-series changes I made in v4:\nhttps://lore.kernel.org/git/cover-00.21-00000000000-20210624T191754Z-avarab@gmail.com/\n"},{"id":"428941","messageId":"xmqqczs2rrew.fsf@gitster.g","threadId":"56034","inReplyTo":"87o8bmcd9a.fsf@evledraar.gmail.com","subject":"Re: What's cooking in git.git (Jun 2021, #07; Wed, 30)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2021-07-01T15:21:27Z","receivedAt":"2021-07-01T15:21:30Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes:\n\n> And:\n>\n>>  - Makefile: remove an out-of-date comment\n>>  - Makefile: stop hardcoding {command,config}-list.h\n>>  - Makefile: mark \"check\" target as .PHONY\n>\n> https://lore.kernel.org/git/cover-0.3-0000000000-20210629T190137Z-avarab@gmail.com/\n>\n> But you re-combined them. If it makes things easier for you I'm all for\n> it, but it seems better to me keep them split if that mean that some\n> parts can advance faster, and thus make the rest easier to review.\n\nI am all for that reasoning, but in this case (and other topics that\nI \"combined\", as well), one had textual dependencies on the other,\nand didn't make sense to split them into two, as neither part can\nadvance without affecting the other, if I recall correctly.\n\n>> * ab/doc-retire-alice-bob (2021-06-16) 6 commits\n>>  - pack-protocol doc: use \"www-data\" in place of \"alice\"\n>>  - doc: replace \"alice\" and \"bob\" with \"jdoe\" and \"msmith\"\n>>  - fast-import doc: change \"bob\" in an example to \"file.txt\"\n>>  - daemon doc + code comments: reword \"alice\" example\n>>  - gitcvs-migration doc: replace \"alice\" and \"bob\" with \"you\" and \"www-data\"\n>>  - gittutorial doc: replace \"alice\" and \"bob\" with \"you\" and \"www-data\"\n>\n> Having re-read the discussion now I don't know if there's anything\n> outstanding to change about this series. It's gotten a lot of attention\n> so far, so it's more of a matter of if you're willing to take this sort\n> of documentation change or not.\n\nI actually do agree with the objection that www-data is a contrived\nthing to use in the examples, not because the user www-data is\ndistro specific, but because it involves one physical human user\nacting on two accounts, instead of two human users interacting with\neach other.\n"},{"id":"428967","messageId":"YN3rOhJDVnyOxctX@coredump.intra.peff.net","threadId":"56034","inReplyTo":"xmqq4kdft122.fsf@gitster.g","subject":"Re: What's cooking in git.git (Jun 2021, #07; Wed, 30)","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2021-07-01T16:20:10Z","receivedAt":"2021-07-01T16:20:12Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Wed, Jun 30, 2021 at 03:55:33PM -0700, Junio C Hamano wrote:\n\n> * jk/test-avoid-globmatch-with-skip-patterns (2021-06-17) 1 commit\n>  - test-lib: avoid accidental globbing in match_pattern_list()\n> \n>  We broke \"GIT_SKIP_TESTS=t?000\" to skip certain tests in recent\n>  update, which got fixed.\n> \n>  Will merge to 'next'.\n\nNot a big deal, but the commit message needs s/t?0000/t?000/ (the\nnumbers are four-digits, and the \"?\" is one of them). If it's not too\nmuch trouble before it hits 'next'. :)\n\n-Peff\n"},{"id":"428972","messageId":"YN3ygh98zikMmd1S@coredump.intra.peff.net","threadId":"56034","inReplyTo":"87o8bmcd9a.fsf@evledraar.gmail.com","subject":"Re: What's cooking in git.git (Jun 2021, #07; Wed, 30)","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2021-07-01T16:51:14Z","receivedAt":"2021-07-01T16:51:17Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Thu, Jul 01, 2021 at 04:00:01PM +0200, Ævar Arnfjörð Bjarmason wrote:\n\n> > * ab/serve-cleanup (2021-06-28) 8 commits\n> >  - upload-pack.c: convert to new serve.c \"startup\" config cb\n> >  - serve: add support for a \"startup\" git_config() callback\n> >  - serve.c: add trace2 regions for advertise & command\n> >  - serve.c: add call_{advertise,command}() indirection\n> >  - serve: use designated initializers\n> >  - transport: use designated initializers\n> >  - transport: rename \"fetch\" in transport_vtable to \"fetch_refs\"\n> >  - serve: mark has_capability() as static\n> >\n> >  Code clean-up around \"git serve\".\n> \n> Jeff, what do you think about this version & the upload-pack.c changes?\n\nI left some comments on the config-callback patches. I think it's an OK\ndirection.\n\nI didn't read the final patch carefully. And I do think it needs to be\ngone over very carefully to make sure there are not any subtleties,\ngiven the problems we've had with config-reading there in the past.  I'm\nnot enthused about doing so for what I consider to be a pretty\nunexciting outcome. I.e., it feels a bit like high-risk churn for low\nreward.\n\n-Peff\n"},{"id":"429037","messageId":"87tulcand1.fsf@evledraar.gmail.com","threadId":"56034","inReplyTo":"xmqqczs2rrew.fsf@gitster.g","subject":"Re: What's cooking in git.git (Jun 2021, #07; Wed, 30)","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2021-07-02T12:18:40Z","receivedAt":"2021-07-02T12:53:19Z","isPatch":false,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"\nOn Thu, Jul 01 2021, Junio C Hamano wrote:\n\n> Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes:\n>\n>> And:\n>>\n>>>  - Makefile: remove an out-of-date comment\n>>>  - Makefile: stop hardcoding {command,config}-list.h\n>>>  - Makefile: mark \"check\" target as .PHONY\n>>\n>> https://lore.kernel.org/git/cover-0.3-0000000000-20210629T190137Z-avarab@gmail.com/\n>>\n>> But you re-combined them. If it makes things easier for you I'm all for\n>> it, but it seems better to me keep them split if that mean that some\n>> parts can advance faster, and thus make the rest easier to review.\n>\n> I am all for that reasoning, but in this case (and other topics that\n> I \"combined\", as well), one had textual dependencies on the other,\n> and didn't make sense to split them into two, as neither part can\n> advance without affecting the other, if I recall correctly.\n\nYes, they can't be advanced separately, one depends on the other in\nturn. I'm suggesting that the parts that go earlier in the sequence\ncould advance before the later parts.\n\nI.e. the whole hook topic is a big chunk that needs careful review, but\ne.g. the relatively trivial Makefile topic could hopefully advance\nfairly fast to \"master\", followed by the easily understood hook-list.h\ntopic.\n\nThe result would be peeling the first 6 commits off\nab/config-based-hooks-base, leaving a smaller set of commits for\nreviewers of the \"meaty\" part of ab/config-based-hooks-base to focus on.\n\nI started doing it this way because of your comments that I tended to\nproduce fairl long-winded topics. Often e.g. I'll find a bug, only to\ndiscover that we don't test anything in that area. I thought that by\nputting trivial cleanups, tests etc. first some of the \"prep\" work could\ngraduate early, leaving the more complex for later.\n\nBut perhaps that's not something you'd like to see. I'm still trying to\nget a feel for if/how you'd like to split certain things.\n\nOne alternative in general (although not so much with the\nab/config-based-hooks-base topic) would be to simply withhold some of\nthe later parts locally. I've got several topics outstanding that are in\nthat state, the real \"meaty\" part hasn't landed on-list yet.\n\nThat has the advantage of having smaller and more incremental parts\non-list for review, but at the drawback that those earlier topics often\nseem like a \"bridge to nohere\". E.g. my \"handle_stdin_line\" topic[1] is\none such topic. I found an existing in-tree user for it, but the real\nthing I wanted it for still hasn't materialized.\n\n>>> * ab/doc-retire-alice-bob (2021-06-16) 6 commits\n>>>  - pack-protocol doc: use \"www-data\" in place of \"alice\"\n>>>  - doc: replace \"alice\" and \"bob\" with \"jdoe\" and \"msmith\"\n>>>  - fast-import doc: change \"bob\" in an example to \"file.txt\"\n>>>  - daemon doc + code comments: reword \"alice\" example\n>>>  - gitcvs-migration doc: replace \"alice\" and \"bob\" with \"you\" and \"www-data\"\n>>>  - gittutorial doc: replace \"alice\" and \"bob\" with \"you\" and \"www-data\"\n>>\n>> Having re-read the discussion now I don't know if there's anything\n>> outstanding to change about this series. It's gotten a lot of attention\n>> so far, so it's more of a matter of if you're willing to take this sort\n>> of documentation change or not.\n>\n> I actually do agree with the objection that www-data is a contrived\n> thing to use in the examples, not because the user www-data is\n> distro specific, but because it involves one physical human user\n> acting on two accounts, instead of two human users interacting with\n> each other.\n\nThe www-thing is arbitrary, and I can change that bit, but what do you\nthink of what I mentioned at the end of [2], i.e.:\n\n    \n    [...] the purposes of the example in the guide replacing Alice & Bob\n    with You & another version of you removes a lot of potential\n    confusion, we don't need to cover permissions, the other user doing\n    unexpected things like non-ff updates, pruning branches you may have\n    relied on through the --local clone etc.\n    \n    It's implicit that both \"users\" are you, so we only have to discuss the\n    point of the actual example, how to pull and push between two different\n    repos, the \"different users\" in this case was always a distraction.\n\nI do think it makes things simpler in that sense, i.e. aside from \"alice\nand bob\" if we start talking about two human users on the same system\nthe reader we'll need to be more guarded about describing repository\ninteractions, because we're acting on remote data that the reader might\ncorrectly assume is changing concurrently outside of their control.\n\nWhereas the real point of that example is to demonstrate how one user\nmight push and pull between two repositories under their control, so\nusing an example that assumes one user implicitly side-steps any such\ndiscussion of permissions, data races etc.\n\n1. https://lore.kernel.org/git/cover-0.4-00000000000-20210621T150651Z-avarab@gmail.com/\n2. https://lore.kernel.org/git/875yyc5i6x.fsf@evledraar.gmail.com/\n"},{"id":"429061","messageId":"xmqqim1spupt.fsf@gitster.g","threadId":"56034","inReplyTo":"87tulcand1.fsf@evledraar.gmail.com","subject":"Re: What's cooking in git.git (Jun 2021, #07; Wed, 30)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2021-07-02T16:05:18Z","receivedAt":"2021-07-02T16:05:24Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes:\n\n> Yes, they can't be advanced separately, one depends on the other in\n> turn. I'm suggesting that the parts that go earlier in the sequence\n> could advance before the later parts.\n\nYes, as long as the bottom half is more stable and ready than the\ntop half.  When I queued these topics, however, I didn't get that\nimpression.  After all, if we have to replace the bottom half as\noften as the top half, having to maintain two topic branches is a\nneedless chore.\n\n>> I actually do agree with the objection that www-data is a contrived\n>> thing to use in the examples, not because the user www-data is\n>> distro specific, but because it involves one physical human user\n>> acting on two accounts, instead of two human users interacting with\n>> each other.\n>\n> The www-thing is arbitrary, and I can change that bit, but what do you\n> think of what I mentioned at the end of [2], i.e.:\n>\n>     \n>     [...] the purposes of the example in the guide replacing Alice & Bob\n>     with You & another version of you removes a lot of potential\n>     confusion, we don't need to cover permissions, the other user doing\n>     unexpected things like non-ff updates, pruning branches you may have\n>     relied on through the --local clone etc.\n\nwww-data requiring more privilege than you do require permissions.\n\nI still think two distinct human users coorporating towards the same\ngoal would make easier-to-follow sample sessions.  IOW, ...\n\n>     It's implicit that both \"users\" are you, so we only have to discuss the\n>     point of the actual example, how to pull and push between two different\n>     repos, the \"different users\" in this case was always a distraction.\n\n... my view of the presence of two distinct human users in the\nexample is completely opposite from yours---to explain Git as a\nplatform for inter-developer coordination, it is a crucial part, not\na distraction, to have them as two separate human beings.  It is OK\nto pretend, while practicing, that you who are typing to this window\nares playing the role of Tom and you in the other window are playing\nthe role of Mary, and that would be what learners would do, but the\nscenario the learners learn from should resemble more what they\nwould do in the real world when they become fluent for the learning\nexperience to be more useful and lasting in their minds., I would\nexpect.\n\nThanks.\n\n"},{"id":"429097","messageId":"CABPp-BGF4K4mAXf6X1rTpTmPCfQgwdhv_VUVg2icGk5Dw7Q1yw@mail.gmail.com","threadId":"56034","inReplyTo":"xmqq4kdft122.fsf@gitster.g","subject":"Re: What's cooking in git.git (Jun 2021, #07; Wed, 30)","fromName":"Elijah Newren","fromEmail":"newren@gmail.com","sentAt":"2021-07-03T05:04:42Z","receivedAt":"2021-07-03T05:04:58Z","isPatch":false,"sender":{"key":"newren@gmail.com","avatar":"https://avatars.githubusercontent.com/u/5455730?v=4"},"body":"On Wed, Jun 30, 2021 at 3:58 PM Junio C Hamano <gitster@pobox.com> wrote:\n\nI've started reviewing v7 of ds/status-with-sparse-index and will also\nlook at ds/commit-and-checkout-with-sparse-index afterwards.\n\n> * en/ort-perf-batch-13 (2021-06-28) 5 commits\n>  - merge-ort: add prefetching for content merges\n>  - diffcore-rename: use a different prefetch for basename comparisons\n>  - diffcore-rename: allow different missing_object_cb functions\n>  - t6421: add tests checking for excessive object downloads during merge\n>  - promisor-remote: output trace2 statistics for number of objects fetched\n>\n>  Performance tweaks of \"git merge -sort\" around lazy fetching of objects.\n\nMost of the discussion on this series was actually about related\nimprovements and ideas, with both Stolee and I agreeing those belong\noutside this particular series.\n\nI also tried to ask Jonathan Tan to review since it was based upon his\nsimilar work to do the original prefetching in diffcore-rename, but I\nthink he's been busy.\n\nI'm not aware of further tweaks needed right now, but it's not at all\nclear to me whether I addressed your and Stolee's feedback to each of\nyour satisfaction.  So this one could potentially be marked \"Waiting\nfor reviews (to finalize)\"\n\n> * en/zdiff3 (2021-06-15) 2 commits\n>  - update documentation for new zdiff3 conflictStyle\n>  - xdiff: implement a zealous diff3, or \"zdiff3\"\n>\n>  \"Zealous diff3\" style of merge conflict presentation has been added.\n>\n>  Expecting a reroll.\n>  cf. <CABPp-BE7-E03+x38EK-=AE5mwwdST+d50hiiud2eY2Nsf3rM5g@mail.gmail.com>\n\nI've got some updates for this RFC series (in part thanks to Phillip's\nawesome digging!), including some tests.  However, I've also got a\nclear testcase failure, and another\nI-know-I-saw-a-real-problem-in-real-testing-but-lost-the-details\nissue.  So I may wait a while to resend.\n\n> * pw/diff-color-moved-fix (2021-06-15) 10 commits\n>  - diff --color-moved: intern strings\n>  - diff --color-moved-ws=allow-indentation-change: improve hash lookups\n>  - diff --color-moved: stop clearing potential moved blocks\n>  - diff --color-moved: shrink potential moved blocks as we go\n>  - diff --color-moved: unify moved block growth functions\n>  - diff --color-moved: call comparison function directly\n>  - diff --color-moved-ws=allow-indentation-change: simplify and optimize\n>  - diff: simplify allow-indentation-change delta calculation\n>  - diff --color-moved: avoid false short line matches and bad zebra coloring\n>  - diff --color-moved=zebra: fix alternate coloring\n>\n>  Long-overdue correctness and performance update to \"diff\n>  --color-moved\" feature.\n>\n>  Waiting for reviews.\n\nI read through the first half the patches or so and didn't spot\nanything wrong, though I'm very unfamiliar with this code area and\ndon't yet understand much of what's going on; it was more of a spot\ncheck.  I'll try to get back to it again, but my review probably won't\nbe that deep.\n\n> * en/ort-perf-batch-12 (2021-06-09) 4 commits\n>  - merge-ort: miscellaneous touch-ups\n>  - Fix various issues found in comments\n>  - diffcore-rename: avoid unnecessary strdup'ing in break_idx\n>  - merge-ort: replace string_list_df_name_compare with faster alternative\n>\n>  More fix-ups and optimization to \"merge -sort\".\n>\n>  Will merge to 'next'.\n\nSounds good, thanks.  As far as I can tell, I addressed everyone's\nfeedback a month or more ago (including getting Stolee's Reviewed-by);\nno changes since then.\n\n> * ab/send-email-optim (2021-05-28) 13 commits\n>  - perl: nano-optimize by replacing Cwd::cwd() with Cwd::getcwd()\n>  - send-email: move trivial config handling to Perl\n>  - perl: lazily load some common Git.pm setup code\n>  - send-email: lazily load modules for a big speedup\n>  - send-email: get rid of indirect object syntax\n>  - send-email: use function syntax instead of barewords\n>  - send-email: lazily shell out to \"git var\"\n>  - send-email: lazily load config for a big speedup\n>  - send-email: copy \"config_regxp\" into git-send-email.perl\n>  - send-email: refactor sendemail.smtpencryption config parsing\n>  - send-email: remove non-working support for \"sendemail.smtpssl\"\n>  - send-email tests: test for boolean variables without a value\n>  - send-email tests: support GIT_TEST_PERL_FATAL_WARNINGS=true\n>\n>  \"git send-email\" optimization.\n>\n>  Will merge to 'next'.\n\nWahoo!\n"},{"id":"429191","messageId":"87czrx6v9w.fsf@evledraar.gmail.com","threadId":"56034","inReplyTo":"CABPp-BGF4K4mAXf6X1rTpTmPCfQgwdhv_VUVg2icGk5Dw7Q1yw@mail.gmail.com","subject":"Re: What's cooking in git.git (Jun 2021, #07; Wed, 30)","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2021-07-05T08:03:58Z","receivedAt":"2021-07-05T08:05:04Z","isPatch":false,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"\nOn Fri, Jul 02 2021, Elijah Newren wrote:\n\n> On Wed, Jun 30, 2021 at 3:58 PM Junio C Hamano <gitster@pobox.com> wrote:\n> [...]\n>> * ab/send-email-optim (2021-05-28) 13 commits\n>>  - perl: nano-optimize by replacing Cwd::cwd() with Cwd::getcwd()\n>>  - send-email: move trivial config handling to Perl\n>>  - perl: lazily load some common Git.pm setup code\n>>  - send-email: lazily load modules for a big speedup\n>>  - send-email: get rid of indirect object syntax\n>>  - send-email: use function syntax instead of barewords\n>>  - send-email: lazily shell out to \"git var\"\n>>  - send-email: lazily load config for a big speedup\n>>  - send-email: copy \"config_regxp\" into git-send-email.perl\n>>  - send-email: refactor sendemail.smtpencryption config parsing\n>>  - send-email: remove non-working support for \"sendemail.smtpssl\"\n>>  - send-email tests: test for boolean variables without a value\n>>  - send-email tests: support GIT_TEST_PERL_FATAL_WARNINGS=true\n>>\n>>  \"git send-email\" optimization.\n>>\n>>  Will merge to 'next'.\n>\n> Wahoo!\n\nDo you find the faster t9001*.sh helps your overall test-run time?\n"},{"id":"429196","messageId":"YOL0voZhJolF8Rg4@coredump.intra.peff.net","threadId":"56034","inReplyTo":"87czrx6v9w.fsf@evledraar.gmail.com","subject":"Re: What's cooking in git.git (Jun 2021, #07; Wed, 30)","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2021-07-05T12:02:06Z","receivedAt":"2021-07-05T12:02:09Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Mon, Jul 05, 2021 at 10:03:58AM +0200, Ævar Arnfjörð Bjarmason wrote:\n\n> >> * ab/send-email-optim (2021-05-28) 13 commits\n> >>  - perl: nano-optimize by replacing Cwd::cwd() with Cwd::getcwd()\n> >>  - send-email: move trivial config handling to Perl\n> >>  - perl: lazily load some common Git.pm setup code\n> >>  - send-email: lazily load modules for a big speedup\n> >>  - send-email: get rid of indirect object syntax\n> >>  - send-email: use function syntax instead of barewords\n> >>  - send-email: lazily shell out to \"git var\"\n> >>  - send-email: lazily load config for a big speedup\n> >>  - send-email: copy \"config_regxp\" into git-send-email.perl\n> >>  - send-email: refactor sendemail.smtpencryption config parsing\n> >>  - send-email: remove non-working support for \"sendemail.smtpssl\"\n> >>  - send-email tests: test for boolean variables without a value\n> >>  - send-email tests: support GIT_TEST_PERL_FATAL_WARNINGS=true\n> >>\n> >>  \"git send-email\" optimization.\n> >>\n> >>  Will merge to 'next'.\n> >\n> > Wahoo!\n> \n> Do you find the faster t9001*.sh helps your overall test-run time?\n\nIt does for me. Running \"make test\" on 'next' (with -O2, and prove\n--state=slow,save so it realizes t9001 is slow) takes 66 wallclock\nseconds total. The top culprits are:\n\n  $ perl -MYAML -e ' \n      $_ = do { local $/; <> };\n      # prove puts this non-YAML cruft at the end\n      s/\\.\\.\\.$//s;\n      my $t = YAML::Load($_)->{tests};\n      print \"$_->[1] $_->[0]\\n\" for\n        sort { $b->[1] <=> $a->[1] } \n        map { [$_, $t->{$_}->{elapsed}] } \n        keys(%$t);\n    ' t/.prove  | head\n  65.4168658256531 t9001-send-email.sh\n  40.6761560440063 t0027-auto-crlf.sh\n  26.0225791931152 t7610-mergetool.sh\n  24.0760719776154 t7112-reset-submodule.sh\n  23.5710120201111 t5572-pull-submodule.sh\n  23.2557609081268 t9500-gitweb-standalone-no-errors.sh\n  20.9747018814087 t1701-racy-split-index.sh\n  20.7937531471252 t7400-submodule-basic.sh\n  20.4510779380798 t5510-fetch.sh\n  20.0246639251709 t3305-notes-fanout.sh\n\nMerging in ab/send-email-optim takes 60s, and yields:\n\n  39.4541878700256 t0027-auto-crlf.sh\n  27.2213349342346 t7610-mergetool.sh\n  26.2227098941803 t9001-send-email.sh\n  25.0272631645203 t5572-pull-submodule.sh\n  24.3502569198608 t7112-reset-submodule.sh\n  24.0568630695343 t9500-gitweb-standalone-no-errors.sh\n  22.3555839061737 t7400-submodule-basic.sh\n  21.2626190185547 t3305-notes-fanout.sh\n  21.1138088703156 t5510-fetch.sh\n  21.0040950775146 t1701-racy-split-index.sh\n\nSo we shaved 39s of CPU (albeit clocked-down-due-to-parallelism\nseconds), and gained 6 wallclock seconds. Subsequent runs seem to shave\noff a few more seconds even (presumably because the test-slowness\nordering is updated).\n\n-Peff\n"},{"id":"429292","messageId":"d04c71fe-acfb-abe0-b00f-cfa01651dd57@gmail.com","threadId":"56034","inReplyTo":"CABPp-BGF4K4mAXf6X1rTpTmPCfQgwdhv_VUVg2icGk5Dw7Q1yw@mail.gmail.com","subject":"Re: What's cooking in git.git (Jun 2021, #07; Wed, 30)","fromName":"Phillip Wood","fromEmail":"phillip.wood123@gmail.com","sentAt":"2021-07-06T13:46:23Z","receivedAt":"2021-07-06T15:12:58Z","isPatch":false,"sender":{"key":"phillip.wood@dunelm.org.uk","avatar":null},"body":"On 03/07/2021 06:04, Elijah Newren wrote:\n> On Wed, Jun 30, 2021 at 3:58 PM Junio C Hamano <gitster@pobox.com> wrote:\n>> * pw/diff-color-moved-fix (2021-06-15) 10 commits\n>>   - diff --color-moved: intern strings\n>>   - diff --color-moved-ws=allow-indentation-change: improve hash lookups\n>>   - diff --color-moved: stop clearing potential moved blocks\n>>   - diff --color-moved: shrink potential moved blocks as we go\n>>   - diff --color-moved: unify moved block growth functions\n>>   - diff --color-moved: call comparison function directly\n>>   - diff --color-moved-ws=allow-indentation-change: simplify and optimize\n>>   - diff: simplify allow-indentation-change delta calculation\n>>   - diff --color-moved: avoid false short line matches and bad zebra coloring\n>>   - diff --color-moved=zebra: fix alternate coloring\n>>\n>>   Long-overdue correctness and performance update to \"diff\n>>   --color-moved\" feature.\n>>\n>>   Waiting for reviews.\n> \n> I read through the first half the patches or so and didn't spot\n> anything wrong, though I'm very unfamiliar with this code area and\n> don't yet understand much of what's going on; it was more of a spot\n> check.  I'll try to get back to it again, but my review probably won't\n> be that deep.\n\nThanks for taking a look, do let me know if I can do anything to make \nreviewing this series easier\n\nBest Wishes\n\nPhillip\n"},{"id":"429326","messageId":"87k0m35jfi.fsf@evledraar.gmail.com","threadId":"56034","inReplyTo":"d04c71fe-acfb-abe0-b00f-cfa01651dd57@gmail.com","subject":"Re: What's cooking in git.git (Jun 2021, #07; Wed, 30)","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2021-07-06T19:29:38Z","receivedAt":"2021-07-06T19:30:47Z","isPatch":false,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"\nOn Tue, Jul 06 2021, Phillip Wood wrote:\n\n> On 03/07/2021 06:04, Elijah Newren wrote:\n>> On Wed, Jun 30, 2021 at 3:58 PM Junio C Hamano <gitster@pobox.com> wrote:\n>>> * pw/diff-color-moved-fix (2021-06-15) 10 commits\n>>>   - diff --color-moved: intern strings\n>>>   - diff --color-moved-ws=allow-indentation-change: improve hash lookups\n>>>   - diff --color-moved: stop clearing potential moved blocks\n>>>   - diff --color-moved: shrink potential moved blocks as we go\n>>>   - diff --color-moved: unify moved block growth functions\n>>>   - diff --color-moved: call comparison function directly\n>>>   - diff --color-moved-ws=allow-indentation-change: simplify and optimize\n>>>   - diff: simplify allow-indentation-change delta calculation\n>>>   - diff --color-moved: avoid false short line matches and bad zebra coloring\n>>>   - diff --color-moved=zebra: fix alternate coloring\n>>>\n>>>   Long-overdue correctness and performance update to \"diff\n>>>   --color-moved\" feature.\n>>>\n>>>   Waiting for reviews.\n>> I read through the first half the patches or so and didn't spot\n>> anything wrong, though I'm very unfamiliar with this code area and\n>> don't yet understand much of what's going on; it was more of a spot\n>> check.  I'll try to get back to it again, but my review probably won't\n>> be that deep.\n>\n> Thanks for taking a look, do let me know if I can do anything to make\n> reviewing this series easier\n\nFWIW I put it on my own mental list of \"waiting for re-roll\" per [1].\n\n(Not that you need to take my suggestion there, just that you seemed to\nagree, so I thought I'd take a look when that happened in a v2).\n\n1. https://lore.kernel.org/git/094f5e5f-d447-8867-a9a7-be5c8827bba6@gmail.com/\n"},{"id":"429342","messageId":"xmqqbl7fi3il.fsf@gitster.g","threadId":"56034","inReplyTo":"87k0m35jfi.fsf@evledraar.gmail.com","subject":"Re: What's cooking in git.git (Jun 2021, #07; Wed, 30)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2021-07-06T20:36:02Z","receivedAt":"2021-07-06T20:36:08Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Ævar Arnfjörð Bjarmason <avarab@gmail.com> writes:\n\n> FWIW I put it on my own mental list of \"waiting for re-roll\" per [1].\n>\n> (Not that you need to take my suggestion there, just that you seemed to\n> agree, so I thought I'd take a look when that happened in a v2).\n>\n> 1. https://lore.kernel.org/git/094f5e5f-d447-8867-a9a7-be5c8827bba6@gmail.com/\n\nThanks for a reminder.  I should have noticed the message myself.\n"}]}