{"thread":{"id":"57242","subject":"What's cooking in git.git (Jan 2022, #03; Thu, 13)","startedAt":"2022-01-14T00:49:04Z","lastAt":"2022-01-18T16:14:10Z","messageCount":19,"participants":["Junio C Hamano","Elijah Newren","Ævar Arnfjörð Bjarmason","Taylor Blau","David Aguilar","Patrick Steinhardt","Derrick Stolee"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"446193","messageId":"xmqq35lrf8g4.fsf@gitster.g","threadId":"57242","inReplyTo":null,"subject":"What's cooking in git.git (Jan 2022, #03; Thu, 13)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2022-01-14T00:48:59Z","receivedAt":"2022-01-14T00:49:04Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"There are a few \"oops, what we merged recently is broken\" topics\nthat still are not in 'master', but otherwise what we have should\nbe pretty much what we'll have in the final one.\n\n - I am reasonably happy with ab/refs-errno-cleanup (just one patch)\n that fixes the incorrect state of the code left by the earlier\n parts of the topic that have already been merged during this\n cycle.\n\n - I am also OK with ab/reftable-build-fixes (two patches), one for\n general type correctness fix, the other for helping older sub-C99\n compilers.\n\nIf there are fixes for regressions that we introduced during this\ncycle other than these two topics, I certainly am missing them, so\nplease holler loudly and quickly, hopefully in time for me to tag\nthe -rc1 tomorrow.\n\nHere are the topics that have been cooking in my tree. Commits\nprefixed with '+' are in 'next' (being in 'next' is a sign that a\ntopic is stable enough to be used and are candidate to be in a\nfuture release). Commits prefixed with '-' are only in 'seen',\nwhich means nothing more than that I have found them of interest for\nsome 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'. The ones marked with '.' do not appear in any of the\nintegration branches, but I am still holding onto them.\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[Graduated to 'master']\n\n* ab/do-not-limit-stash-help-to-push (2021-12-16) 1 commit\n (merged to 'next' on 2022-01-03 at adfc0c1f17)\n + stash: don't show \"git stash push\" usage on bad \"git stash\" usage\n\n \"git stash\" by default triggers its \"push\" action, but its\n implementation also made \"git stash -h\" to show short help only for\n \"git stash push\", which has been corrected.\n source: \n\n\n* ab/makefile-hook-list-dependency-fix (2021-12-25) 1 commit\n (merged to 'next' on 2022-01-03 at 8c8ca0cbce)\n + Makefile: correct the dependency graph of hook-list.h\n\n Fix dependency rules to generate hook-list.h header file.\n source: \n\n\n* ab/makefile-msgfmt-wo-stats (2021-12-25) 1 commit\n (merged to 'next' on 2022-01-03 at 2e2097c310)\n + Makefile: don't invoke msgfmt with --statistics\n\n Make the recipe that runs msgfmt less noisy.\n source: \n\n\n* ab/makefile-pager-env-is-used-only-by-pager.c (2021-12-25) 1 commit\n (merged to 'next' on 2022-01-03 at 7ab1e1c5c4)\n + Makefile: move -DPAGER_ENV from BASIC_CFLAGS to EXTRA_CPPFLAGS\n\n source: \n\n\n* ab/reflog-prep (2021-12-22) 9 commits\n (merged to 'next' on 2022-01-03 at 07d2d78d86)\n + reflog + refs-backend: move \"verbose\" out of the backend\n + refs files-backend: assume cb->newlog if !EXPIRE_REFLOGS_DRY_RUN\n + reflog: reduce scope of \"struct rev_info\"\n + reflog expire: don't use lookup_commit_reference_gently()\n + reflog expire: refactor & use \"tip_commit\" only for UE_NORMAL\n + reflog expire: use \"switch\" over enum values\n + reflog: change one->many worktree->refnames to use a string_list\n + reflog expire: narrow scope of \"cb\" in cmd_reflog_expire()\n + reflog delete: narrow scope of \"cmd\" passed to count_reflog_ent()\n (this branch is used by jc/reflog-parse-options.)\n\n Code refactoring in the reflog part of refs API.\n source: \n\n\n* ab/usage-die-message (2021-12-07) 6 commits\n (merged to 'next' on 2022-01-03 at 0ef3860099)\n + config API: use get_error_routine(), not vreportf()\n + usage.c + gc: add and use a die_message_errno()\n + gc: return from cmd_gc(), don't call exit()\n + usage.c API users: use die_message() for error() + exit 128\n + usage.c API users: use die_message() for \"fatal :\" + exit 128\n + usage.c: add a die_message() routine\n\n Code clean-up to hide vreportf() from public API.\n source: \n\n\n* ak/protect-any-current-branch (2022-01-12) 1 commit\n (merged to 'next' on 2022-01-12 at 5ad87959d2)\n + branch: missing space fix at line 313\n\n source: <20220111123627.58625-1-bagasdotme@gmail.com>\n\n\n* ds/fetch-pull-with-sparse-index (2021-12-22) 5 commits\n (merged to 'next' on 2022-01-03 at f58da4a878)\n + test-read-cache: remove --table, --expand options\n + t1091/t3705: remove 'test-tool read-cache --table'\n + t1092: replace 'read-cache --table' with 'ls-files --sparse'\n + ls-files: add --sparse option\n + fetch/pull: use the sparse index\n\n \"git fetch\" and \"git pull\" are now declared sparse-index clean.\n Also \"git ls-files\" learns the \"--sparse\" option to help debugging.\n source: \n\n\n* ds/sparse-checkout-malformed-pattern-fix (2021-12-30) 3 commits\n (merged to 'next' on 2021-12-30 at c8b2ade48c)\n + sparse-checkout: refuse to add to bad patterns\n + sparse-checkout: fix OOM error with mixed patterns\n + sparse-checkout: fix segfault on malformed patterns\n\n Certain sparse-checkout patterns that are valid in non-cone mode\n led to segfault in cone mode, which has been corrected.\n source: \n\n\n* en/merge-ort-renorm-with-rename-delete-conflict-fix (2021-12-30) 1 commit\n (merged to 'next' on 2022-01-05 at bb81dd432d)\n + merge-ort: fix bug with renormalization and rename/delete conflicts\n\n A corner case bug in the ort merge strategy has been corrected.\n source: \n\n\n* en/stash-df-fix (2022-01-04) 1 commit\n (merged to 'next' on 2022-01-06 at 22d0c6ccf7)\n + stash: do not return before restoring untracked files\n\n \"git stash apply\" forgot to attempt restoring untracked files when\n it failed to restore changes to tracked ones.\n source: \n\n\n* fs/gpg-unknown-key-test-fix (2022-01-12) 1 commit\n (merged to 'next' on 2022-01-12 at 80d3416168)\n + t/gpg: simplify test for unknown key\n\n Test simplification.\n source: <20220112120757.874714-1-fs@gigacodes.de>\n\n\n* fs/ssh-signing-key-lifetime (2022-01-10) 1 commit\n (merged to 'next' on 2022-01-12 at 8541f3399c)\n + fmt-merge-msg: prevent use-after-free with signed tags\n\n \"git merge $signed_tag\" started to drop the tag message from the\n default merge message it uses by accident, which has been corrected.\n source: <6e08b73d602853b3de71257117e85e32b96b5c19.1641849502.git.me@ttaylorr.com>\n\n\n* hn/ref-api-tests-update (2021-12-22) 4 commits\n (merged to 'next' on 2022-01-03 at 6815d4d8c1)\n + t7004: use \"test-tool ref-store\" for reflog inspection\n + t7004: create separate tags for different tests\n + t5550: require REFFILES\n + t5540: require REFFILES\n\n Test updates.\n source: \n\n\n* hn/refs-debug-update (2021-12-22) 3 commits\n (merged to 'next' on 2022-01-03 at 1cd451c66c)\n + refs: centralize initialization of the base ref_store.\n + refs: print error message in debug output\n + refs: pass gitdir to packed_ref_store_create\n\n Debugging support for refs API.\n source: \n\n\n* hn/reftable-fixes (2021-12-23) 3 commits\n (merged to 'next' on 2022-01-03 at 532f9bbb27)\n + reftable: support preset file mode for writing\n + reftable: signal overflow\n + reftable: fix typo in header\n\n Assorted fixlets in reftable code.\n source: \n\n\n* hn/test-ref-store-show-hash-algo (2021-12-21) 1 commit\n (merged to 'next' on 2022-01-03 at 3cdbfeeaf1)\n + test-ref-store: print hash algorithm\n\n Debugging support for refs API.\n source: \n\n\n* ja/i18n-similar-messages (2022-01-05) 11 commits\n (merged to 'next' on 2022-01-06 at 6440d8fdf2)\n + i18n: turn even more messages into \"cannot be used together\" ones\n + i18n: ref-filter: factorize \"%(foo) atom used without %(bar) atom\"\n + i18n: factorize \"--foo outside a repository\"\n + i18n: refactor \"unrecognized %(foo) argument\" strings\n + i18n: factorize \"no directory given for --foo\"\n + i18n: factorize \"--foo requires --bar\" and the like\n + i18n: tag.c factorize i18n strings\n + i18n: standardize \"cannot open\" and \"cannot read\"\n + i18n: turn \"options are incompatible\" into \"cannot be used together\"\n + i18n: refactor \"%s, %s and %s are mutually exclusive\"\n + i18n: refactor \"foo and bar are mutually exclusive\"\n\n Similar message templates have been consolidated so that\n translators need to work on fewer number of messages.\n source: \n\n\n* ja/perf-use-specified-shell (2021-12-25) 1 commit\n (merged to 'next' on 2022-01-03 at 8ca35ee3d0)\n + t/perf: do not run tests in user's $SHELL\n\n Perf tests were run with end-user's shell, but it has been\n corrected to use the shell specified by $TEST_SHELL_PATH.\n source: <20211225081656.1311583-1-aclopte@gmail.com>\n\n\n* jc/doc-submitting-patches-choice-of-base (2021-12-30) 1 commit\n (merged to 'next' on 2022-01-05 at eb97c01c24)\n + SubmittingPatchs: clarify choice of base and testing\n\n Extend the guidance to choose the base commit to build your work\n on, and hint/nudge contributors to read others' changes.\n source: \n\n\n* jh/p4-human-unit-numbers (2021-12-20) 2 commits\n (merged to 'next' on 2022-01-03 at c385766ff4)\n + git-p4: show progress as an integer\n + git-p4: print size values in appropriate units\n\n The way \"git p4\" shows file sizes in its output has been updated to\n use human-readable units.\n source: <20211219154028.3288756-1-jholdsworth@nvidia.com>\n\n\n* jh/p4-remove-unused (2021-12-22) 2 commits\n (merged to 'next' on 2022-01-03 at 43f61b2fef)\n + git-p4: remove \"rollback\" verb\n + git-p4: remove \"debug\" verb\n\n Remove a few commands from \"git p4\" that aren't very useful.\n source: <20211222145552.93786-1-jholdsworth@nvidia.com>\n\n\n* jl/subtree-check-parents-argument-passing-fix (2022-01-04) 1 commit\n (merged to 'next' on 2022-01-05 at 7dc995f026)\n + subtree: fix argument handling in check_parents\n\n Fix performance-releated bug in \"git subtree\" (in contrib/).\n source: \n\n\n* js/branch-track-inherit (2021-12-20) 3 commits\n (merged to 'next' on 2022-01-03 at ab5fb34db3)\n + config: require lowercase for branch.*.autosetupmerge\n + branch: add flags and config to inherit tracking\n + branch: accept multiple upstream branches for tracking\n (this branch is used by gc/branch-recurse-submodules.)\n\n \"git -c branch.autosetupmerge=inherit branch new old\" makes \"new\"\n to have the same upstream as the \"old\" branch, instead of marking\n \"old\" itself as its upstream.\n source: \n\n\n* js/l10n-mention-ngettext-early-in-readme (2021-12-21) 1 commit\n (merged to 'next' on 2022-01-03 at a644f898ce)\n + l10n: README: call more attention to plural strings\n\n Localization doc update.\n source: <25107068cbbf8c9ce6886e66e25dff19e072583f.1639425295.git.steadmon@google.com>\n\n\n* jt/pack-header-lshift-overflow (2022-01-12) 1 commit\n (merged to 'next' on 2022-01-12 at fa27107ab3)\n + packfile: fix off-by-one error in decoding logic\n\n source: \n\n\n* jz/apply-3-corner-cases (2021-12-20) 1 commit\n (merged to 'next' on 2022-01-03 at cb8e19c304)\n + git-apply: skip threeway in add / rename cases\n\n \"git apply --3way\" bypasses the attempt to do a three-way\n application in more cases to address the regression caused by the\n recent change to use direct application as a fallback.\n source: <20211217224328.7646-1-jerry@skydio.com>\n\n\n* lh/use-gnu-color-in-grep (2022-01-05) 1 commit\n (merged to 'next' on 2022-01-05 at a0e2068cb7)\n + grep: align default colors with GNU grep ones\n\n The color palette used by \"git grep\" has been updated to match that\n of GNU grep.\n source: <20220105081835.4949-1-lenaic@lhuard.fr>\n\n\n* ma/header-dup-cleanup (2022-01-10) 1 commit\n (merged to 'next' on 2022-01-12 at de7891f5e8)\n + cache.h: drop duplicate `ensure_full_index()` declaration\n\n Code clean-up.\n source: <20220110184134.18675-1-martin.agren@gmail.com>\n\n\n* ma/windows-dynload-fix (2022-01-09) 1 commit\n (merged to 'next' on 2022-01-12 at af0a2e6c90)\n + lazyload: use correct calling conventions\n\n Fix calling dynamically loaded functions on Windows.\n source: \n\n\n* ms/t-readme-typofix (2022-01-04) 1 commit\n (merged to 'next' on 2022-01-06 at 59288ebb22)\n + t/README: fix typo\n\n Typofix.\n source: \n\n\n* ps/lockfile-cleanup-fix (2022-01-07) 1 commit\n (merged to 'next' on 2022-01-10 at ff0ae63f07)\n + fetch: fix deadlock when cleaning up lockfiles in async signals\n\n Some lockfile code called free() in signal-death code path, which\n has been corrected.\n source: <555ec6717ecab0fe6ef5660bcf0d61d59f84ef8b.1641552500.git.ps@pks.im>\n\n\n* rb/nonstop-lacks-uncompress2 (2022-01-12) 1 commit\n (merged to 'next' on 2022-01-12 at 2387a47655)\n + build: NonStop ships with an older zlib\n\n source: \n\n\n* ws/fast-export-with-revision-options (2021-12-21) 1 commit\n (merged to 'next' on 2022-01-03 at f94f7f70b6)\n + fast-export: fix surprising behavior with --first-parent\n\n Use of certain \"git rev-list\" options with \"git fast-export\"\n created nonsense results (the worst two of which being \"--reverse\"\n and \"--invert-grep --grep=\"). The use of \"--first-parent\" is\n made to behave a bit more sensible than before.\n source: \n\n--------------------------------------------------\n[New Topics]\n\n* jh/p4-fix-use-of-process-error-exception (2022-01-06) 1 commit\n (merged to 'next' on 2022-01-10 at 49d529bfd7)\n + git-p4: fix instantiation of CalledProcessError\n\n Will cook in 'next'.\n source: <20220106214156.90967-1-jholdsworth@nvidia.com>\n\n\n* jh/p4-spawning-external-commands-cleanup (2022-01-06) 3 commits\n (merged to 'next' on 2022-01-10 at 54b36b4e66)\n + git-p4: don't print shell commands as python lists\n + git-p4: pass command arguments as lists instead of using shell\n + git-p4: don't select shell mode using the type of the command argument\n\n Will cook in 'next'.\n source: <20220106214035.90725-1-jholdsworth@nvidia.com>\n\n\n* pb/pull-rebase-autostash-fix (2022-01-06) 1 commit\n - pull --rebase: honor rebase.autostash even when fast-forwarding\n\n \"git pull --rebase\" ignored the rebase.autostash configuration\n variable when the remote history is a descendant of our history,\n which has been corrected.\n\n Expecting an ack or two.\n source: <4fe0e850-1c30-da70-5535-e2a4105ba734@gmail.com>\n\n\n* rs/grep-expr-cleanup (2022-01-06) 4 commits\n (merged to 'next' on 2022-01-10 at b70a3bb0fa)\n + grep: use grep_and_expr() in compile_pattern_and()\n + grep: extract grep_binexp() from grep_or_expr()\n + grep: use grep_not_expr() in compile_pattern_not()\n + grep: use grep_or_expr() in compile_pattern_or()\n\n Code clean-up.\n\n Will cook in 'next'.\n source: \n\n\n* fs/ssh-signing-crlf (2022-01-07) 1 commit\n - gpg-interface: trim CR from ssh-keygen\n\n The code path that verifies signatures made with ssh were made to\n work better on a system with CRLF line endings.\n\n Will merge to 'next'?\n source: <20220103095337.600536-1-fs@gigacodes.de>\n\n\n* jc/qsort-s-alignment-fix (2022-01-07) 2 commits\n (merged to 'next' on 2022-01-10 at 329fd6e09a)\n + stable-qsort: avoid using potentially unaligned access\n + compat/qsort_s.c: avoid using potentially unaligned access\n\n Fix a hand-rolled alloca() imitation that may have violated\n alignment requirement of data being sorted in compatibility\n implementation of qsort_s() and stable qsort().\n\n Will cook in 'next'.\n source: \n source: \n\n\n* ps/avoid-unnecessary-hook-invocation-with-packed-refs (2022-01-12) 7 commits\n - fixup! refs: open-code deletion of packed refs\n - refs: skip hooks when deleting uncovered packed refs\n - refs: do not execute reference-transaction hook on packing refs\n - refs: demonstrate excessive execution of the reference-transaction hook\n - refs: allow skipping the reference-transaction hook\n - refs: allow passing flags when beginning transactions\n - refs: open-code deletion of packed refs\n\n Because a deletion of ref would need to remove it from both the\n loose ref store and the packed ref store, a delete-ref operation\n that logically removes one ref may end up invoking ref-transaction\n hook twice, which has been corrected.\n\n Under review.\n source: \n\n\n* rs/apply-symlinks-use-strset (2022-01-07) 1 commit\n (merged to 'next' on 2022-01-10 at 32497a67d5)\n + apply: use strsets to track symlinks\n\n \"git apply\" (ab)used the util pointer of the string-list to keep\n track of how each symbolic link needs to be handled, which has been\n simplified by using strset.\n\n Will cook in 'next'.\n source: <8739caad-aa3d-1f0f-b5dd-6174a8e059f6@web.de>\n\n\n* ld/sparse-index-bash-completion (2022-01-10) 3 commits\n - sparse-checkout: limit tab completion to a single level\n - sparse-checkout: custom tab completion\n - sparse-checkout: custom tab completion tests\n\n The command line completion (in contrib/) learns to complete\n arguments give to \"git sparse-checkout\" command.\n source: \n\n\n* bc/clarify-eol-attr (2022-01-12) 2 commits\n - docs: correct documentation about eol attribute\n - t0027: add tests for eol without text in .gitattributes\n\n Doc and test update around the eol attribute.\n source: <20220111021507.531736-1-sandals@crustytoothpaste.net>\n\n\n* jz/rev-list-exclude-first-parent-only (2022-01-12) 1 commit\n - git-rev-list: add --exclude-first-parent-only flag\n\n \"git log\" and friends learned an option --exclude-first-parent-only\n to propagate UNINTERESTING bit down only along the first-parent\n chain, just like --first-parent option shows commits that lack the\n UNINTERESTING bit only along the first-parent chain.\n source: <20220111213941.30129-1-jerry@skydio.com>\n\n\n* ab/refs-errno-cleanup (2022-01-13) 1 commit\n - refs API: use \"failure_errno\", not \"errno\"\n\n A brown-paper-bag fix on top of a topic that was merged during this\n cycle.\n\n Will merge to 'next' and then to 'master'.\n source: \n\n\n* ab/reftable-build-fixes (2022-01-13) 2 commits\n - reftable tests: avoid \"int\" overflow, use \"uint64_t\"\n - reftable: avoid initializing structs from structs\n\n A few portability tweaks.\n\n Will merge to 'next' and then to 'master'?\n source: \n\n\n* en/present-despite-skipped (2022-01-13) 6 commits\n . Accelerate clear_skip_worktree_from_present_files() by caching\n . Update documentation related to sparsity and the skip-worktree bit\n . repo_read_index: clear SKIP_WORKTREE bit from files present in worktree\n . unpack-trees: fix accidental loss of user changes\n . t1011: add testcase demonstrating accidental loss of user modifications\n . Merge branch 'vd/sparse-clean-etc' into en/present-despite-skipped\n (this branch uses vd/sparse-clean-etc.)\n\n With this topic merged, 'seen' seems to fail t1092.\n source: \n\n\n* js/t1450-making-it-writable-does-not-need-full-posixperm (2022-01-13) 1 commit\n - t1450-fsck: exec-bit is not needed to make loose object writable\n\n Test fix.\n\n Will merge to 'next'.\n source: <529d99f3-13c8-ffc1-dd4c-3fc70330262a@kdbg.org>\n\n--------------------------------------------------\n[Stalled]\n\n* ds/sparse-checkout-requires-per-worktree-config (2021-12-21) 5 commits\n . sparse-checkout: use repo_config_set_worktree_gently()\n . config: add repo_config_set_worktree_gently()\n . worktree: add upgrade_to_worktree_config()\n . config: make some helpers repo-aware\n . setup: use a repository when upgrading format\n\n \"git sparse-checkout\" wants to work with per-worktree configration,\n but did not work well in a worktree attached to a bare repository.\n source: \n\n\n* pw/add-p-hunk-split-fix (2022-01-12) 2 commits\n - builtin add -p: fix hunk splitting\n - t3701: clean up hunk splitting tests\n\n \"git add -p\" rewritten in C regressed hunk splitting in some cases,\n which has been corrected.\n\n Will merge to 'next'?\n source: \n\n\n* gc/fetch-negotiate-only-early-return (2022-01-12) 3 commits\n - fetch --negotiate-only: do not update submodules\n - fetch: skip tasks related to fetching objects\n - fetch: use goto cleanup in cmd_fetch()\n\n \"git fetch --nogotiate-only\" is an internal command used by \"git\n push\" to figure out which part of our history is missing from the\n other side. It should never recurse into submodules even when\n fetch.recursesubmodules configuration variable is set, nor it\n should trigger \"gc\". The code has been tightened up to ensure it\n only does common ancestry discovery and nothing else.\n\n Almost there.\n source: <20220113004501.78822-1-chooglen@google.com>\n\n\n* je/http-better-error-output (2021-12-03) 1 commit\n . http-backend: give a hint that web browser access is not supported\n\n When the http-backend program, which is the server-side component\n for the smart HTTP transport, sends a \"404 Not found\" error, we\n deliberately did not say anything to the requesting client. We now\n send a message back to the browser to tell the user that they do\n not want to visit the URL via their browser, instead of a totally\n blank page.\n\n Expecting a reroll.\n Breaks its self tests.\n cf. <7r23s082-o3q0-479o-srqn-r45q778s5nq7@vanv.qr>\n source: <20211202102855.23907-1-jengelh@inai.de>\n\n\n* cb/save-term-across-editor-invocation (2021-12-01) 3 commits\n - fixup! editor: allow for saving/restoring terminal state\n - editor: allow for saving/restoring terminal state\n - terminal: teach save_term to fail when not foreground\n\n Some editors are reported to leave the terminal in funny state\n after they exit on Windows. Work it around by saving and restoring\n the terminal state when needed.\n\n Expecting a reroll.\n cf. \n source: <20211202035446.1154-1-carenas@gmail.com>\n\n\n* ar/submodule-update (2021-10-13) 9 commits\n . submodule--helper: rename helper functions\n . submodule--helper: remove unused helpers\n . submodule: move core cmd_update() logic to C\n . submodule--helper: run update using child process struct\n . submodule--helper: allow setting superprefix for init_submodule()\n . submodule--helper: refactor get_submodule_displaypath()\n . submodule--helper: rename helpers for update-clone\n . submodule--helper: get remote names from any repository\n . submodule--helper: split up ensure_core_worktree()\n\n Rewrite of \"git submodule update\" in C.\n\n Expecting a reroll?\n cf. \n source: <20211013051805.45662-10-raykar.ath@gmail.com>\n\n--------------------------------------------------\n[Cooking]\n\n* bc/csprng-mktemps (2022-01-04) 2 commits\n - wrapper: use a CSPRNG to generate random file names\n - wrapper: add a helper to generate numbers from a CSPRNG\n\n Pick a better random number generator and use it when we prepare\n temporary filenames.\n\n Expecting a reroll.\n cf. \n source: <20220104015555.3387101-1-sandals@crustytoothpaste.net>\n\n\n* jc/reflog-parse-options (2022-01-10) 2 commits\n (merged to 'next' on 2022-01-12 at 1659e49c4b)\n + builtin/reflog.c: use parse-options api for expire, delete subcommands\n + Merge branch 'ab/reflog-prep' into jc/reflog-parse-options\n\n Use the parse-options API in \"git reflog\" command.\n\n Will cook in 'next'.\n source: \n\n\n* vd/sparse-clean-etc (2022-01-13) 9 commits\n - update-index: reduce scope of index expansion in do_reupdate\n - update-index: integrate with sparse index\n - update-index: add tests for sparse-checkout compatibility\n - checkout-index: integrate with sparse index\n - checkout-index: add --ignore-skip-worktree-bits option\n - checkout-index: expand sparse checkout compatibility tests\n - clean: integrate with sparse index\n - reset: reorder wildcard pathspec conditions\n - reset: fix validation in sparse index test\n (this branch is used by en/present-despite-skipped.)\n\n \"git update-index\", \"git checkout-index\", and \"git clean\" are\n taught to work better with the sparse checkout feature.\n source: \n\n\n* ms/update-index-racy (2022-01-07) 4 commits\n - update-index: refresh should rewrite index in case of racy timestamps\n - t7508: add tests capturing racy timestamp handling\n - t7508: fix bogus mtime verification\n - test-lib: introduce API for verifying file mtime\n\n \"git update-index --refresh\" has been taught to deal better with\n racy timestamps (just like \"git status\" already does).\n\n Will merge to 'next'.\n source: \n\n\n* jc/find-header (2022-01-06) 1 commit\n (merged to 'next' on 2022-01-10 at 8a13b4f0b3)\n + receive-pack.c: consolidate find header logic\n\n Code clean-up.\n\n Will cook in 'next'.\n source: \n\n\n* jc/name-rev-stdin (2022-01-10) 2 commits\n - name-rev.c: use strbuf_getline instead of limited size buffer\n - name-rev: deprecate --stdin in favor of --annotate-stdin\n\n \"git name-rev --stdin\" does not behave like usual \"--stdin\" at\n all. Start the process of renaming it to \"--annotate-stdin\".\n\n Will merge to 'next'?\n source: \n\n\n* en/remerge-diff (2021-12-30) 10 commits\n - merge-ort: mark conflict/warning messages from inner merges as omittable\n - show, log: include conflict/warning messages in --remerge-diff headers\n - diff: add ability to insert additional headers for paths\n - merge-ort: format messages slightly different for use in headers\n - merge-ort: mark a few more conflict messages as omittable\n - merge-ort: capture and print ll-merge warnings in our preferred fashion\n - ll-merge: make callers responsible for showing warnings\n - log: clean unneeded objects during `log --remerge-diff`\n - show, log: provide a --remerge-diff capability\n - Merge branch 'ns/tmp-objdir' into en/remerge-diff\n\n \"git log --remerge-diff\" shows the difference from mechanical merge\n result and the merge result that is actually recorded.\n\n Will merge to 'next'?\n source: \n\n\n* bs/forbid-i18n-of-protocol-token-in-fetch-pack (2021-12-22) 2 commits\n - fixup! fetch-pack: parameterize message containing 'ready' keyword\n - fetch-pack: parameterize message containing 'ready' keyword\n\n L10n support for a few error messages.\n\n Expecting an ack for fixup.\n source: <20211222075805.19027-1-bagasdotme@gmail.com>\n\n\n* gc/branch-recurse-submodules (2022-01-10) 6 commits\n - branch: add --recurse-submodules option for branch creation\n - builtin/branch: clean up action-picking logic in cmd_branch()\n - branch: add a dry_run parameter to create_branch()\n - branch: make create_branch() always create a branch\n - branch: move --set-upstream-to behavior to dwim_and_setup_tracking()\n - Merge branch 'js/branch-track-inherit' into gc/branch-recurse-submodules\n\n \"git branch\" learned the \"--recurse-submodules\" option.\n\n Will merge to 'next'?\n source: <20211220233459.45739-1-chooglen@google.com>\n\n\n* hn/reftable-coverity-fixes (2021-12-22) 18 commits\n - reftable: be more paranoid about 0-length memcpy calls\n - reftable: add print functions to the record types\n - reftable: make reftable_record a tagged union\n - reftable: remove outdated file reftable.c\n - reftable: implement record equality generically\n - reftable: make reftable-record.h function signatures const correct\n - reftable: handle null refnames in reftable_ref_record_equal\n - reftable: drop stray printf in readwrite_test\n - reftable: order unittests by complexity\n - reftable: all xxx_free() functions accept NULL arguments\n - reftable: fix resource warning\n - reftable: ignore remove() return value in stack_test.c\n - reftable: check reftable_stack_auto_compact() return value\n - reftable: fix resource leak blocksource.c\n - reftable: fix resource leak in block.c error path\n - reftable: fix OOB stack write in print functions\n - Merge branch 'hn/create-reflog-simplify' into hn/reftable-coverity-fixes\n - Merge branch 'hn/reftable' into hn/reftable-coverity-fixes\n\n Problems identified by Coverity in the reftable code have been\n corrected.\n\n Expecting a reroll.\n cf. <211228.86lf04j0lg.gmgdl@evledraar.gmail.com>\n source: \n\n\n* tb/midx-bitmap-corruption-fix (2022-01-04) 9 commits\n - pack-bitmap.c: gracefully fallback after opening pack/MIDX\n - midx: read `RIDX` chunk when present\n - t/lib-bitmap.sh: parameterize tests over reverse index source\n - t5326: move tests to t/lib-bitmap.sh\n - t5326: extract `test_rev_exists`\n - t5326: drop unnecessary setup\n - pack-revindex.c: instrument loading on-disk reverse index\n - midx.c: make changing the preferred pack safe\n - t5326: demonstrate bitmap corruption after permutation\n\n A bug that made multi-pack bitmap and the object order out-of-sync\n (hence the .midx data gets corrupted) has been fixed.\n\n Waiting for a hopefully final review.\n cf. \n source: \n\n\n* pw/fix-some-issues-in-reset-head (2021-12-08) 14 commits\n - rebase -m: don't fork git checkout\n - rebase --apply: set ORIG_HEAD correctly\n - rebase --apply: fix reflog\n - reset_head(): take struct rebase_head_opts\n - rebase: cleanup reset_head() calls\n - reset_head(): make default_reflog_action optional\n - reset_head(): factor out ref updates\n - create_autostash(): remove unneeded parameter\n - reset_head(): remove action parameter\n - rebase --apply: don't run post-checkout hook if there is an error\n - rebase: do not remove untracked files on checkout\n - rebase: pass correct arguments to post-checkout hook\n - t5403: refactor rebase post-checkout hook tests\n - rebase: factor out checkout for up to date branch\n\n Fix \"some issues\" in a helper function reset_head().\n\n Expecting a reroll.\n cf. \n cf. \n cf. \n cf. \n cf. \n source: \n\n\n* ab/cat-file (2022-01-12) 12 commits\n (merged to 'next' on 2022-01-12 at ee4d43041d)\n + cat-file: s/_/-/ in typo'd usage_msg_optf() message\n + cat-file: don't whitespace-pad \"(...)\" in SYNOPSIS and usage output\n (merged to 'next' on 2022-01-05 at e145efa605)\n + cat-file: use GET_OID_ONLY_TO_DIE in --(textconv|filters)\n + object-name.c: don't have GET_OID_ONLY_TO_DIE imply *_QUIETLY\n + cat-file: correct and improve usage information\n + cat-file: fix remaining usage bugs\n + cat-file: make --batch-all-objects a CMDMODE\n + cat-file: move \"usage\" variable to cmd_cat_file()\n + cat-file docs: fix SYNOPSIS and \"-h\" output\n + parse-options API: add a usage_msg_optf()\n + cat-file tests: test messaging on bad objects/paths\n + cat-file tests: test bad usage\n\n Assorted updates to \"git cat-file\", especially \"-h\".\n\n Will cook in 'next'.\n source: \n source: \n\n\n* ab/grep-patterntype (2021-12-27) 7 commits\n - grep API: call grep_config() after grep_init()\n - grep.c: don't pass along NULL callback value\n - built-ins: trust the \"prefix\" from run_builtin()\n - fixup! grep tests: add missing \"grep.patternType\" config tests\n - grep tests: add missing \"grep.patternType\" config tests\n - log tests: check if grep_config() is called by \"log\"-like cmds\n - grep.h: remove unused \"regex_t regexp\" from grep_opt\n\n Some code clean-up in the \"git grep\" machinery.\n\n Reroll exists.\n source: \n\n\n* js/use-builtin-add-i (2021-12-01) 2 commits\n - add -i: default to the built-in implementation\n - t2016: require the PERL prereq only when necessary\n\n \"git add -i\" was rewritten in C some time ago and has been in\n testing; the reimplementation is now exposed to general public by\n default.\n\n On hold.\n There are known breakages on macOS.\n cf. \n source: \n\n\n* jt/conditional-config-on-remote-url (2021-12-15) 2 commits\n - config: include file if remote URL matches a glob\n - config: make git_config_include() static\n\n The conditional inclusion mechanism of configuration files using\n \"[includeIf ]\" learns to base its decision on the\n URL of the remote repository the repository interacts with.\n\n How does this one look these days?\n source: \n\n\n* ab/ambiguous-object-name (2022-01-13) 6 commits\n - object-name: re-use \"struct strbuf\" in show_ambiguous_object()\n - object-name: iterate ambiguous objects before showing header\n - object-name: show date for ambiguous tag objects\n - object-name: make ambiguous object output translatable\n - object-name: explicitly handle OBJ_BAD in show_ambiguous_object()\n - object-name tests: add tests for ambiguous object blind spots\n\n Error output given in response to an ambiguous object name has been\n improved.\n source: \n\n\n* tl/ls-tree-oid-only (2022-01-06) 9 commits\n - cocci: allow padding with `strbuf_addf()`\n - ls-tree.c: introduce \"--format\" option\n - ls-tree.c: introduce struct \"show_tree_data\"\n - ls-tree.c: support --object-only option for \"git-ls-tree\"\n - ls-tree: optimize naming and handling of \"return\" in show_tree()\n - ls-tree: use \"size_t\", not \"int\" for \"struct strbuf\"'s \"len\"\n - ls-tree: use \"enum object_type\", not {blob,tree,commit}_type\n - ls-tree: add missing braces to \"else\" arms\n - ls-tree: remove commented-out code\n\n \"git ls-tree\" learns \"--oid-only\" option, similar to \"--name-only\",\n and more generalized \"--format\" option.\n source: \n\n\n* ab/config-based-hooks-2 (2022-01-07) 17 commits\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 to use hook.h\n - commit: convert {pre-commit,prepare-commit-msg} hook to hook.h\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 - hooks: convert worktree 'post-checkout' hook to hook library\n - hooks: convert non-worktree 'post-checkout' hook to hook library\n - merge: convert post-merge to use hook.h\n - am: convert applypatch-msg to use hook.h\n - rebase: convert pre-rebase to use hook.h\n - hook API: add a run_hooks_l() wrapper\n - am: convert {pre,post}-applypatch to use hook.h\n - gc: use hook library for pre-auto-gc hook\n - hook API: add a run_hooks() wrapper\n - hook: add 'run' subcommand\n\n More \"config-based hooks\".\n\n Will merge to 'next', with minor nits?\n source: \n\n\n* jh/builtin-fsmonitor-part2 (2021-12-25) 31 commits\n - fixup! t7527: create test for fsmonitor--daemon\n - fixup! t/perf/p7519: speed up test on Windows\n - t7527: test status with untracked-cache and fsmonitor--daemon\n - fsmonitor: force update index after large responses\n - fsmonitor--daemon: use a cookie file to sync with file system\n - fsmonitor--daemon: periodically truncate list of modified files\n - t/perf/p7519: add fsmonitor--daemon test cases\n - t/perf/p7519: speed up test on Windows\n - t/helper/test-chmtime: skip directories on Windows\n - t/perf: avoid copying builtin fsmonitor files into test repo\n - t7527: create test for fsmonitor--daemon\n - t/helper/fsmonitor-client: create IPC client to talk to FSMonitor Daemon\n - help: include fsmonitor--daemon feature flag in version info\n - fsmonitor--daemon: implement handle_client callback\n - compat/fsmonitor/fsm-listen-darwin: implement FSEvent listener on MacOS\n - compat/fsmonitor/fsm-listen-darwin: add macos header files for FSEvent\n - compat/fsmonitor/fsm-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 'start' command\n - fsmonitor--daemon: implement 'run' command\n - compat/fsmonitor/fsm-listen-darwin: stub in backend for Darwin\n - compat/fsmonitor/fsm-listen-win32: stub in backend for Windows\n - fsmonitor--daemon: implement 'stop' and 'status' commands\n - fsmonitor--daemon: add a built-in fsmonitor daemon\n - fsmonitor: document builtin fsmonitor\n - fsmonitor: use IPC to query the builtin FSMonitor daemon\n - fsmonitor: config settings are repository-specific\n - fsmonitor-ipc: create client routines for git-fsmonitor--daemon\n - fsmonitor: enhance existing comments\n\n Built-in fsmonitor (part 2).\n\n Expecting a reroll.\n Seems that the discussion stalled.\n cf. \n source: \n\n\n* es/superproject-aware-submodules (2021-11-18) 5 commits\n - submodule: use config to find superproject worktree\n - submodule: record superproject gitdir during 'update'\n - submodule: record superproject gitdir during absorbgitdirs\n - introduce submodule.superprojectGitDir record\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 Expecting a reroll.\n cf. <20211117234300.2598132-1-jonathantanmy@google.com>\n source: <20211117005701.371808-1-emilyshaffer@google.com>\n\n\n* ab/only-single-progress-at-once (2022-01-07) 7 commits\n - *.c: use isatty(0|2), not isatty(STDIN_FILENO|STDERR_FILENO)\n - pack-bitmap-write.c: don't return without stop_progress()\n - progress.c: add temporary variable from progress struct\n - progress.c tests: test some invalid usage\n - progress.c tests: make start/stop commands on stdin\n - progress.c test helper: add missing braces\n - leak tests: fix a memory leak in \"test-progress\" helper\n\n Further tweaks on progress API.\n\n Getting there.\n source: \n\n--------------------------------------------------\n[Discarded]\n\n* ms/customizable-ident-expansion (2021-12-25) 2 commits\n . fixup! keyword expansion: make \"$Id$\" string configurable\n . keyword expansion: make \"$Id$\" string configurable\n\n Instead of \"$Id$\", user-specified string (like $FreeBSD$) can be\n used as an in-blob placeholder for keyword expansion.\n\n Will discard.\n Stalled for too long.\n cf. \n cf. <211101.86fssf3bn3.gmgdl@evledraar.gmail.com>\n source: \n\n\n* tb/cruft-packs (2021-11-29) 17 commits\n . sha1-file.c: don't freshen cruft packs\n . builtin/gc.c: conditionally avoid pruning objects via loose\n . builtin/repack.c: add cruft packs to MIDX during geometric repack\n . builtin/repack.c: use named flags for existing_packs\n . builtin/repack.c: allow configuring cruft pack generation\n . builtin/repack.c: support generating a cruft pack\n . builtin/pack-objects.c: --cruft with expiration\n . reachable: report precise timestamps from objects in cruft packs\n . reachable: add options to add_unseen_recent_objects_to_traversal\n . builtin/pack-objects.c: --cruft without expiration\n . builtin/pack-objects.c: return from create_object_entry()\n . t/helper: add 'pack-mtimes' test-tool\n . pack-mtimes: support writing pack .mtimes files\n . chunk-format.h: extract oid_version()\n . pack-write: pass 'struct packing_data' to 'stage_tmp_packfiles'\n . pack-mtimes: support reading .mtimes files\n . Documentation/technical: add cruft-packs.txt\n\n Instead of leaving unreachable objects in loose form when packing,\n or ejecting them into loose form when repacking, gather them in a\n packfile with an auxiliary file that records the last-use time of\n these objects.\n\n Dropped per author request.\n cf. \n cf. <865b99dd-0b18-9a07-49c1-3959a777c685@gmail.com>\n cf. \n cf. \n source: \n"},{"id":"446199","messageId":"xmqqk0f3dk5o.fsf@gitster.g","threadId":"57242","inReplyTo":"xmqq35lrf8g4.fsf@gitster.g","subject":"Re: What's cooking in git.git (Jan 2022, #03; Thu, 13)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2022-01-14T04:18:59Z","receivedAt":"2022-01-14T04:19:04Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Junio C Hamano writes:\n\n> There are a few \"oops, what we merged recently is broken\" topics\n> that still are not in 'master', but otherwise what we have should\n> be pretty much what we'll have in the final one.\n>\n> - I am reasonably happy with ab/refs-errno-cleanup (just one patch)\n> that fixes the incorrect state of the code left by the earlier\n> parts of the topic that have already been merged during this\n> cycle.\n>\n> - I am also OK with ab/reftable-build-fixes (two patches), one for\n> general type correctness fix, the other for helping older sub-C99\n> compilers.\n>\n> If there are fixes for regressions that we introduced during this\n> cycle other than these two topics, I certainly am missing them, so\n> please holler loudly and quickly, hopefully in time for me to tag\n> the -rc1 tomorrow.\n\nOh, by the way, the tip of 'seen' has consistently failing the\nleak-check test. I didn't have chance, time or energy to see if\nthey are failing merely because an existing test script that used to\nbe leak-clean gained a use of command that has been known to be\nleak-unclean without introducing any new leaks, or our recent change\ndid introduce new leaks to commands that have been leak-clean.\nSomebody with too much time on their hand should go in and check to\nhelp, before CI testing on 'seen' becomes useful again.\n\nThanks.\n"},{"id":"446220","messageId":"CABPp-BFj8NyXG6cQPJeDE46eicbfnP=TgEirRZVcnMM+YNG7OQ@mail.gmail.com","threadId":"57242","inReplyTo":"xmqq35lrf8g4.fsf@gitster.g","subject":"Mistakes in the stalled category? (Was: Re: What's cooking in git.git (Jan 2022, #03; Thu, 13))","fromName":"Elijah Newren","fromEmail":"newren@gmail.com","sentAt":"2022-01-14T15:44:36Z","receivedAt":"2022-01-14T15:44:51Z","isPatch":false,"sender":{"key":"newren@gmail.com","avatar":"https://avatars.githubusercontent.com/u/5455730?v=4"},"body":"Hi,\n\nAre there some errors with the stalled category this time around? In\nparticular...\n\nOn Fri, Jan 14, 2022 at 7:16 AM Junio C Hamano wrote:\n>\n> --------------------------------------------------\n> [Stalled]\n>\n> * ds/sparse-checkout-requires-per-worktree-config (2021-12-21) 5 commits\n> . sparse-checkout: use repo_config_set_worktree_gently()\n> . config: add repo_config_set_worktree_gently()\n> . worktree: add upgrade_to_worktree_config()\n> . config: make some helpers repo-aware\n> . setup: use a repository when upgrading format\n>\n> \"git sparse-checkout\" wants to work with per-worktree configration,\n> but did not work well in a worktree attached to a bare repository.\n> source: \n\nIt has been two weeks since the last submission and emails about this\ntopic, so maybe you put this one in \"stalled\" intentionally. (If so,\nand Stolee if this really is stalled, would you like me to try\nupdating? I know it has expanded quite a bit from the early simple\nfix you were trying to provide, but you've got most the code I think\nyou need and some important fixes I wouldn't want to see dropped.)\nBut I'm wondering if marking this topic as stalled was intentional,\nbecause:\n\n> * pw/add-p-hunk-split-fix (2022-01-12) 2 commits\n> - builtin add -p: fix hunk splitting\n> - t3701: clean up hunk splitting tests\n>\n> \"git add -p\" rewritten in C regressed hunk splitting in some cases,\n> which has been corrected.\n>\n> Will merge to 'next'?\n> source: \n>\n>\n> * gc/fetch-negotiate-only-early-return (2022-01-12) 3 commits\n> - fetch --negotiate-only: do not update submodules\n> - fetch: skip tasks related to fetching objects\n> - fetch: use goto cleanup in cmd_fetch()\n>\n> \"git fetch --nogotiate-only\" is an internal command used by \"git\n> push\" to figure out which part of our history is missing from the\n> other side. It should never recurse into submodules even when\n> fetch.recursesubmodules configuration variable is set, nor it\n> should trigger \"gc\". The code has been tightened up to ensure it\n> only does common ancestry discovery and nothing else.\n>\n> Almost there.\n> source: <20220113004501.78822-1-chooglen@google.com>\n\nThese last two series were submitted in the two days; they don't seem\nstalled to me.\n\n(The other series in \"stalled\" besides these three all seemed to\nbelong there to me.)\n"},{"id":"446221","messageId":"CABPp-BFc=_Gns6oDCDvJ9TjtKYAVhYVq_c74hyZJtL=96GqH=w@mail.gmail.com","threadId":"57242","inReplyTo":"xmqq35lrf8g4.fsf@gitster.g","subject":"en/present-despite-skipped & en/remerge-diff (Was: Re: What's cooking in git.git (Jan 2022, #03; Thu, 13))","fromName":"Elijah Newren","fromEmail":"newren@gmail.com","sentAt":"2022-01-14T15:54:36Z","receivedAt":"2022-01-14T15:54:50Z","isPatch":false,"sender":{"key":"newren@gmail.com","avatar":"https://avatars.githubusercontent.com/u/5455730?v=4"},"body":"On Fri, Jan 14, 2022 at 7:16 AM Junio C Hamano wrote:\n>\n> * en/present-despite-skipped (2022-01-13) 6 commits\n> . Accelerate clear_skip_worktree_from_present_files() by caching\n> . Update documentation related to sparsity and the skip-worktree bit\n> . repo_read_index: clear SKIP_WORKTREE bit from files present in worktree\n> . unpack-trees: fix accidental loss of user changes\n> . t1011: add testcase demonstrating accidental loss of user modifications\n> . Merge branch 'vd/sparse-clean-etc' into en/present-despite-skipped\n> (this branch uses vd/sparse-clean-etc.)\n>\n> With this topic merged, 'seen' seems to fail t1092.\n> source: \n\nDoh! Sorry about that. I tried to merge with seen to check for\nsemantic conflicts, because this series was more likely to have them\nthan others. That's how I noticed the issue with vd/sparse-clean-etc.\nBut I apparently missed ds/fetch-pull-with-sparse-index. Since the\nlatter has now merged to master and your version of\nvd/sparse-clean-etc builds on top of a version of master that has\nds/fetch-pull-with-sparse-index in it, I can simply submit an updated\nseries and you won't have to merge anything extra into that topic.\nI'll submit it soon.\n\n> * en/remerge-diff (2021-12-30) 10 commits\n> - merge-ort: mark conflict/warning messages from inner merges as omittable\n> - show, log: include conflict/warning messages in --remerge-diff headers\n> - diff: add ability to insert additional headers for paths\n> - merge-ort: format messages slightly different for use in headers\n> - merge-ort: mark a few more conflict messages as omittable\n> - merge-ort: capture and print ll-merge warnings in our preferred fashion\n> - ll-merge: make callers responsible for showing warnings\n> - log: clean unneeded objects during `log --remerge-diff`\n> - show, log: provide a --remerge-diff capability\n> - Merge branch 'ns/tmp-objdir' into en/remerge-diff\n>\n> \"git log --remerge-diff\" shows the difference from mechanical merge\n> result and the merge result that is actually recorded.\n>\n> Will merge to 'next'?\n> source: \n\nBack in September, when you last asked this about an earlier version\nof this series, I suggested waiting off[1]. Crazy how many things\nhave happened since then. Anyway, this time we've gotten good\nfeedback from multiple reviewers and I've addressed all of it, so this\ntime I think it's ready to merge down.\n\n[1] https://lore.kernel.org/git/CABPp-BH2C5HFw_=NtK71oNGXkntzcJd2un0TgMb=UnJ5yJxAAA@mail.gmail.com/\n"},{"id":"446229","messageId":"CABPp-BFGxKBzi5RYDuiJv6Vz7yyGYTOdJC9cL_EkPGNJ5BksYQ@mail.gmail.com","threadId":"57242","inReplyTo":"xmqqk0f3dk5o.fsf@gitster.g","subject":"Re: What's cooking in git.git (Jan 2022, #03; Thu, 13)","fromName":"Elijah Newren","fromEmail":"newren@gmail.com","sentAt":"2022-01-14T16:27:54Z","receivedAt":"2022-01-14T16:28:08Z","isPatch":false,"sender":{"key":"newren@gmail.com","avatar":"https://avatars.githubusercontent.com/u/5455730?v=4"},"body":"On Fri, Jan 14, 2022 at 8:19 AM Junio C Hamano wrote:\n>\n> Junio C Hamano writes:\n>\n> > There are a few \"oops, what we merged recently is broken\" topics\n> > that still are not in 'master', but otherwise what we have should\n> > be pretty much what we'll have in the final one.\n> >\n> > - I am reasonably happy with ab/refs-errno-cleanup (just one patch)\n> > that fixes the incorrect state of the code left by the earlier\n> > parts of the topic that have already been merged during this\n> > cycle.\n> >\n> > - I am also OK with ab/reftable-build-fixes (two patches), one for\n> > general type correctness fix, the other for helping older sub-C99\n> > compilers.\n> >\n> > If there are fixes for regressions that we introduced during this\n> > cycle other than these two topics, I certainly am missing them, so\n> > please holler loudly and quickly, hopefully in time for me to tag\n> > the -rc1 tomorrow.\n>\n> Oh, by the way, the tip of 'seen' has consistently failing the\n> leak-check test. I didn't have chance, time or energy to see if\n> they are failing merely because an existing test script that used to\n> be leak-clean gained a use of command that has been known to be\n> leak-unclean without introducing any new leaks, or our recent change\n> did introduce new leaks to commands that have been leak-clean.\n\nIt's apparently the latter, because there have been no test script\nchanges in the relevant tests.\n\n> Somebody with too much time on their hand should go in and check to\n> help, before CI testing on 'seen' becomes useful again.\n\nThis \"fixes\" seen:\nhttps://lore.kernel.org/git/pull.1192.git.git.1642176433017.gitgitgadget@gmail.com/\n\nI briefly looked at a couple leak traces and thought they looked ref\nrelated, but I don't have time to go hunt down memory leaks right now.\nI figure this thread has reported them, so let's just get \"seen\" back\nto green.\n"},{"id":"446231","messageId":"220114.86fspqrxbw.gmgdl@evledraar.gmail.com","threadId":"57242","inReplyTo":"xmqqk0f3dk5o.fsf@gitster.g","subject":"Re: What's cooking in git.git (Jan 2022, #03; Thu, 13)","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2022-01-14T18:12:15Z","receivedAt":"2022-01-14T18:23:04Z","isPatch":false,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"\nOn Thu, Jan 13 2022, Junio C Hamano wrote:\n\n> Junio C Hamano writes:\n>\n>> There are a few \"oops, what we merged recently is broken\" topics\n>> that still are not in 'master', but otherwise what we have should\n>> be pretty much what we'll have in the final one.\n>>\n>> - I am reasonably happy with ab/refs-errno-cleanup (just one patch)\n>> that fixes the incorrect state of the code left by the earlier\n>> parts of the topic that have already been merged during this\n>> cycle.\n>>\n>> - I am also OK with ab/reftable-build-fixes (two patches), one for\n>> general type correctness fix, the other for helping older sub-C99\n>> compilers.\n>>\n>> If there are fixes for regressions that we introduced during this\n>> cycle other than these two topics, I certainly am missing them, so\n>> please holler loudly and quickly, hopefully in time for me to tag\n>> the -rc1 tomorrow.\n>\n> Oh, by the way, the tip of 'seen' has consistently failing the\n> leak-check test. I didn't have chance, time or energy to see if\n> they are failing merely because an existing test script that used to\n> be leak-clean gained a use of command that has been known to be\n> leak-unclean without introducing any new leaks, or our recent change\n> did introduce new leaks to commands that have been leak-clean.\n> Somebody with too much time on their hand should go in and check to\n> help, before CI testing on 'seen' becomes useful again.\n\nIt's a regression in\nps/avoid-unnecessary-hook-invocation-with-packed-refs, Patrick could you\nlook into it? On your current \"seen\" doing a:\n\n git revert -m 1 48b388cbf31\n\nWill make those 3x failing tests pass:\nhttps://github.com/git/git/runs/4811683950?check_suite_focus=true\n\n(That commit being: 48b388cbf31 (Merge branch\n'ps/avoid-unnecessary-hook-invocation-with-packed-refs' into seen,\n2022-01-13))\n\nI didn't have much time to look now, but this mostly untested fix-up\nfixes up the topic under SANITIZE=leak (but may break something else). I\nran the broken tests with SANITIZE=leak, and the normal tests without\nSANITIZE=leak, but didn't have time for further testing:\n\ndiff --git a/refs/packed-backend.c b/refs/packed-backend.c\nindex ff96ee482a0..b8012f97009 100644\n--- a/refs/packed-backend.c\n+++ b/refs/packed-backend.c\n@@ -1577,6 +1577,7 @@ int packed_refs_delete_refs(struct ref_store *ref_store,\n error(_(\"could not delete references: %s\"), err.buf);\n }\n \n+ ref_transaction_free(transaction);\n strbuf_release(&err);\n return ret;\n }\n\nI.e. the moving around of the ref_transaction_free() is at fault\nsomehow, probably...\n\nJunio: Is there something you're doing to hide\nhttps://github.com/gitster/git/actions ? My version of that is visible\nwithout being logged-in, but yours is a 404 (maybe it needs to be set\nup?)\n\nIf CI is setup to run on your topic push-outs it'll become easier both\nfor you & others to see what topics in \"seen\" may be broken. Usually it\nwould be a matter of browsing the page of pushed-out topics for failures\n(except in cases where the merger causes the faulure...)\n"},{"id":"446240","messageId":"YeHRYkl2RaQbrtmx@nand.local","threadId":"57242","inReplyTo":"xmqq35lrf8g4.fsf@gitster.g","subject":"tb/midx-bitmap-corruption-fix (was: Re: What's cooking in git.git (Jan 2022, #03; Thu, 13))","fromName":"Taylor Blau","fromEmail":"me@ttaylorr.com","sentAt":"2022-01-14T19:39:14Z","receivedAt":"2022-01-14T19:39:17Z","isPatch":false,"sender":{"key":"me@ttaylorr.com","avatar":"https://avatars.githubusercontent.com/u/301000140?v=4"},"body":"On Thu, Jan 13, 2022 at 04:48:59PM -0800, Junio C Hamano wrote:\n> * tb/midx-bitmap-corruption-fix (2022-01-04) 9 commits\n> - pack-bitmap.c: gracefully fallback after opening pack/MIDX\n> - midx: read `RIDX` chunk when present\n> - t/lib-bitmap.sh: parameterize tests over reverse index source\n> - t5326: move tests to t/lib-bitmap.sh\n> - t5326: extract `test_rev_exists`\n> - t5326: drop unnecessary setup\n> - pack-revindex.c: instrument loading on-disk reverse index\n> - midx.c: make changing the preferred pack safe\n> - t5326: demonstrate bitmap corruption after permutation\n>\n> A bug that made multi-pack bitmap and the object order out-of-sync\n> (hence the .midx data gets corrupted) has been fixed.\n>\n> Waiting for a hopefully final review.\n> cf. \n> source: \n\nI would really like to get this into 2.35 since it's fixing an important\nsource of repository corruption, but I think it should have a careful\nround of review before merging. And it is pretty late into the cycle\nanyway, so we may be too late to merge the whole thing.\n\nBut the first two patches:\n\n - midx.c: make changing the preferred pack safe\n - t5326: demonstrate bitmap corruption after permutation\n\ncould be applied as-is and the rest of the series left for later, which\nshould be a safer approach (and would be sufficient to resolve the bug\nat the expense of some redundant bytes on disk[1]).\n\nI think Stolee is probably the most familiar with this topic, but he is\noff currently and I'm not sure whether or not he'll be back with enough\ntime to get this merged before 2.35.\n\nOn the other hand, the bug is pretty difficult to trigger, is affecting\na very new feature, shouldn't ever cause permanent damage, and can be\nrecovered from fairly easily (by dropping existing bitmaps). So perhaps\nit's OK to let it sit out for another release like this...\n\nThanks,\nTaylor\n\n[1]: More or less storing the contents of the multi-pack-index-$HASH.rev\n file twice: once in the .rev file itself, and again as an optional\n write-only chunk in the MIDX.\n"},{"id":"446243","messageId":"xmqqmtjyaylt.fsf@gitster.g","threadId":"57242","inReplyTo":"CABPp-BFGxKBzi5RYDuiJv6Vz7yyGYTOdJC9cL_EkPGNJ5BksYQ@mail.gmail.com","subject":"Re: What's cooking in git.git (Jan 2022, #03; Thu, 13)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2022-01-14T19:47:26Z","receivedAt":"2022-01-14T19:47:31Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Elijah Newren writes:\n\n> It's apparently the latter, because there have been no test script\n> changes in the relevant tests.\n>\n>> Somebody with too much time on their hand should go in and check to\n>> help, before CI testing on 'seen' becomes useful again.\n>\n> This \"fixes\" seen:\n> https://lore.kernel.org/git/pull.1192.git.git.1642176433017.gitgitgadget@gmail.com/\n>\n> I briefly looked at a couple leak traces and thought they looked ref\n> related, but I don't have time to go hunt down memory leaks right now.\n> I figure this thread has reported them, so let's just get \"seen\" back\n> to green.\n\nIf it were \"we added a use of known-to-leak command in an otherwise\nclean test, without adding a new leak\", I would wholeheartedly\nsupport such a change, but if it is the other way around, it may\nmake sense to leave it broken as an incentive for people who care\nabout leaks to go in and fix them up.\n\nIf we toggle it off any time leak-checker CI job starts complaining\non a test script, the leak-checker CI job serves no useful purpose,\nno?\n\nAn obvious alternative, based on the same attitude, is to rip out\nthe whole fragile leak-checker thing from the CI. I've mentioned\nan ideal alternative (disregarding feasibility) already elsewhere\nso I won't repeat it.\n\nThanks.\n\n"},{"id":"446255","messageId":"CABPp-BGOqK0YJXna3PqnFmTcW_KxzAGbqjpUvRjgAxAwYzG4bw@mail.gmail.com","threadId":"57242","inReplyTo":"xmqqmtjyaylt.fsf@gitster.g","subject":"Re: What's cooking in git.git (Jan 2022, #03; Thu, 13)","fromName":"Elijah Newren","fromEmail":"newren@gmail.com","sentAt":"2022-01-14T21:49:11Z","receivedAt":"2022-01-14T21:49:27Z","isPatch":false,"sender":{"key":"newren@gmail.com","avatar":"https://avatars.githubusercontent.com/u/5455730?v=4"},"body":"On Fri, Jan 14, 2022 at 11:47 AM Junio C Hamano wrote:\n>\n> Elijah Newren writes:\n>\n> > It's apparently the latter, because there have been no test script\n> > changes in the relevant tests.\n> >\n> >> Somebody with too much time on their hand should go in and check to\n> >> help, before CI testing on 'seen' becomes useful again.\n> >\n> > This \"fixes\" seen:\n> > https://lore.kernel.org/git/pull.1192.git.git.1642176433017.gitgitgadget@gmail.com/\n> >\n> > I briefly looked at a couple leak traces and thought they looked ref\n> > related, but I don't have time to go hunt down memory leaks right now.\n> > I figure this thread has reported them, so let's just get \"seen\" back\n> > to green.\n>\n> If it were \"we added a use of known-to-leak command in an otherwise\n> clean test, without adding a new leak\", I would wholeheartedly\n> support such a change, but if it is the other way around, it may\n> make sense to leave it broken as an incentive for people who care\n> about leaks to go in and fix them up.\n\nPerhaps. Waiting can make sense up to a point.\n\n> If we toggle it off any time leak-checker CI job starts complaining\n> on a test script, the leak-checker CI job serves no useful purpose,\n> no?\n\nFolks who use PRs for the purpose of getting the cross-platform CI\ntesting before submitting to the list can still get early notification\nof potential leaks in their own series, due to the remaining tests\nbeing marked as leak-free. They can then fix up their series before\nsubmitting them to the list. That seems like a useful purpose to me.\n\nFurther, these CI jobs did notify us of an issue in someone else's\npatches (we don't yet know whose), and we were able to report it much\nlike any other bug report. That gives people a heads up and allows\nthem to take action on it. (And if they do so, they can remark the\ntest as leak-free.) That also seems like a useful purpose to me.\n\nIn contrast, if we leave the leak-checker failing and the failing job\nspreads to next and master, then we'll just end up training everyone\nto ignore it -- both for their own PRs and in general. To me, that's\nwhat making the leak-checker serve no useful purpose would look like.\n"},{"id":"446258","messageId":"xmqqfspq9dqn.fsf@gitster.g","threadId":"57242","inReplyTo":"CABPp-BGOqK0YJXna3PqnFmTcW_KxzAGbqjpUvRjgAxAwYzG4bw@mail.gmail.com","subject":"Re: What's cooking in git.git (Jan 2022, #03; Thu, 13)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2022-01-14T22:03:28Z","receivedAt":"2022-01-14T22:03:34Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Elijah Newren writes:\n\n> In contrast, if we leave the leak-checker failing and the failing job\n> spreads to next and master, then we'll just end up training everyone\n> to ignore it -- both for their own PRs and in general. To me, that's\n> what making the leak-checker serve no useful purpose would look like.\n\nWhat you proposed is no better than that.\n\nMarking a test as \"OK to fail\", because somebody added a new leak,\nis a small step of removing the leak-checking job from the CI.\nAmong 226 such tests, you killed one of them and 225 more to go.\n\nAnd after you are done, nobody's PR will be blocked because they do\nnot see a leak-checker breakage.\n"},{"id":"446260","messageId":"xmqq7db29d23.fsf@gitster.g","threadId":"57242","inReplyTo":"xmqqfspq9dqn.fsf@gitster.g","subject":"Re: What's cooking in git.git (Jan 2022, #03; Thu, 13)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2022-01-14T22:18:12Z","receivedAt":"2022-01-14T22:18:17Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Junio C Hamano writes:\n\n> Elijah Newren writes:\n>\n>> In contrast, if we leave the leak-checker failing and the failing job\n>> spreads to next and master,...\n\nIn any case, I think Ævar identified an offending topic, so I kicked\nit out from 'seen'. If it makes the leak-checker job pass, that\nwould be a much better outcome than anything else we have been\ndiscussing on this thread so far, and we'd hopefully set a good\nprecedent to follow. (1) if your change adds a new leak, you'll not\nhit 'next', (2) as an exception, if new test added by you use a tool\nthat wasn't used in it, and if the test is known to be leaky, it is\nOK to mark the test leak-checker-unclean.\n\nLet's see how well the rest of the topics do.\n"},{"id":"446264","messageId":"xmqqr19999ly.fsf@gitster.g","threadId":"57242","inReplyTo":"CABPp-BFj8NyXG6cQPJeDE46eicbfnP=TgEirRZVcnMM+YNG7OQ@mail.gmail.com","subject":"Re: Mistakes in the stalled category?","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2022-01-14T23:32:41Z","receivedAt":"2022-01-14T23:32:48Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Elijah Newren writes:\n\n> Are there some errors with the stalled category this time around? In\n> particular...\n\nYeah, I agree that the more recent ones would not belong there.\n"},{"id":"446266","messageId":"xmqqlezh98uj.fsf@gitster.g","threadId":"57242","inReplyTo":"CABPp-BFj8NyXG6cQPJeDE46eicbfnP=TgEirRZVcnMM+YNG7OQ@mail.gmail.com","subject":"Re: Mistakes in the stalled category?","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2022-01-14T23:49:08Z","receivedAt":"2022-01-14T23:49:17Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Elijah Newren writes:\n\n>> [Stalled]\n>>\n>> * ds/sparse-checkout-requires-per-worktree-config (2021-12-21) 5 commits\n>> . sparse-checkout: use repo_config_set_worktree_gently()\n>> . config: add repo_config_set_worktree_gently()\n>> . worktree: add upgrade_to_worktree_config()\n>> . config: make some helpers repo-aware\n>> . setup: use a repository when upgrading format\n>>\n>> \"git sparse-checkout\" wants to work with per-worktree configration,\n>> but did not work well in a worktree attached to a bare repository.\n>> source: \n>\n> It has been two weeks since the last submission and emails about this\n> topic, so maybe you put this one in \"stalled\" intentionally. (If so,\n> and Stolee if this really is stalled, would you like me to try\n> updating? I know it has expanded quite a bit from the early simple\n> fix you were trying to provide, but you've got most the code I think\n> you need and some important fixes I wouldn't want to see dropped.)\n\n[Stalled] being early in the report is primarily a way to remind us\nthat an update is overdue.\n\n<1db0f601-4769-15c0-cd58-ecddfa1fc9d5@gmail.com> is what prompted me\nto consider that I can put it on backburner and spend my time on\nother topics.\n\n>> * pw/add-p-hunk-split-fix (2022-01-12) 2 commits\n\nI think what happened was that the previous iteration of this was on\n2021-12-20 and marked as \"Expecting a reroll\" for a few issues of\nthe \"What's cooking\" report. And then I've moved it to [Stalled]\nsection, but before I had a chance to issue another \"What's cooking\"\nreport, an update came, and when I updated the list of commits, I\nforgot to move the topic out of the [Stalled] category.\n\n>> * gc/fetch-negotiate-only-early-return (2022-01-12) 3 commits\n\nLikewise, the previous round was on 2021-12-21.\n\n"},{"id":"446295","messageId":"CAJDDKr7bYP7JNmY7XML0rkxv8uCKAbO-LdAAn0KkyTq3UX4hkg@mail.gmail.com","threadId":"57242","inReplyTo":"xmqq35lrf8g4.fsf@gitster.g","subject":"Re: What's cooking in git.git (Jan 2022, #03; Thu, 13)","fromName":"David Aguilar","fromEmail":"davvid@gmail.com","sentAt":"2022-01-15T16:45:02Z","receivedAt":"2022-01-15T16:45:41Z","isPatch":false,"sender":{"key":"davvid@gmail.com","avatar":"https://avatars.githubusercontent.com/u/13196?v=4"},"body":"On Fri, Jan 14, 2022 at 7:16 AM Junio C Hamano wrote:\n>\n> There are a few \"oops, what we merged recently is broken\" topics\n> that still are not in 'master', but otherwise what we have should\n> be pretty much what we'll have in the final one.\n>\n> - I am reasonably happy with ab/refs-errno-cleanup (just one patch)\n> that fixes the incorrect state of the code left by the earlier\n> parts of the topic that have already been merged during this\n> cycle.\n>\n> - I am also OK with ab/reftable-build-fixes (two patches), one for\n> general type correctness fix, the other for helping older sub-C99\n> compilers.\n\nI ran into the c99 test balloon. I get the (expected?) c99 build\nerrors and an uncompress2 build error with RHEL7.9's stock gcc 4.8.5.\n\nAre we dropping support for RHEL7.9?\n\nUsing NO_UNCOMPRESS2=YesPlease and CFLAGS += -std=c99 in config.mak\nworks but I'm not sure if we actually expect users to deal with those\ndetails themselves.\n\nSorry for being out of the loop on the plan for older toolchains, but\nI wasn't sure if we should be expecting a patch that configures the\ndefault flags so that things build out of the box again or if this is\nconsidered the new status quo and we're okay regressing?\n\n$ rpm -q zlib-devel\nzlib-devel-1.2.7-19.el7_9.x86_64\n\n$ gcc -v\n...\ngcc version 4.8.5 20150623 (Red Hat 4.8.5-44) (GCC)\n\nWas part of the intention of the test balloon to get reports of this\nnature? I checked the commit logs and related discussion but that\ndidn't really clarify things.\n\nIt seems like that was the intention based on the note of, \"any\nwarnings will eventually surface.\" in the commit message for\n5f46385309 (config.mak.dev: specify -std=gnu99 for gcc/clang, 2021-12-08).\n\nI'll keep an eye out for any patches that I should test in this area.\n\ncheers,\n--\nDavid\n"},{"id":"446297","messageId":"xmqqo84c4xqh.fsf@gitster.g","threadId":"57242","inReplyTo":"CAJDDKr7bYP7JNmY7XML0rkxv8uCKAbO-LdAAn0KkyTq3UX4hkg@mail.gmail.com","subject":"Re: What's cooking in git.git (Jan 2022, #03; Thu, 13)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2022-01-15T19:15:02Z","receivedAt":"2022-01-15T19:15:07Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"David Aguilar writes:\n\n> I ran into the c99 test balloon. I get the (expected?) c99 build\n> errors and an uncompress2 build error with RHEL7.9's stock gcc 4.8.5.\n>\n> Are we dropping support for RHEL7.9?\n\nNo. Our expectation is that those who care about various distros\nand platforms supply patches like ...\n\n> Using NO_UNCOMPRESS2=YesPlease and CFLAGS += -std=c99 in config.mak\n\n... this, but not locallly but to config.mak.uname, to help the\nusers and builders on their favorite systems. It is better done\nsooner rather than later.\n\n> works but I'm not sure if we actually expect users to deal with those\n> details themselves.\n\nUsers would not have to, as long as somebody else on the list uses\nthe smea system as theirs cares enough to supply such patches to\nconfig.mak.uname.\n\n> Sorry for being out of the loop on the plan for older toolchains, but\n> I wasn't sure if we should be expecting a patch that configures the\n> default flags so that things build out of the box again or if this is\n> considered the new status quo and we're okay regressing?\n\nObviously the former. This is a team effort and everybody who finds\nnew issues is very appreciated to chip in.\n\n"},{"id":"446298","messageId":"xmqqiluk4wo4.fsf@gitster.g","threadId":"57242","inReplyTo":"xmqqlezh98uj.fsf@gitster.g","subject":"Re: Mistakes in the stalled category?","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2022-01-15T19:38:03Z","receivedAt":"2022-01-15T19:38:08Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Junio C Hamano writes:\n\n> Elijah Newren writes:\n>\n>>> [Stalled]\n>>>\n>>> * ds/sparse-checkout-requires-per-worktree-config (2021-12-21) 5 commits\n>\n> [Stalled] being early in the report is primarily a way to remind us\n> that an update is overdue.\n>\n> <1db0f601-4769-15c0-cd58-ecddfa1fc9d5@gmail.com> is what prompted me\n> to consider that I can put it on backburner and spend my time on\n> other topics.\n\nThe pushout of 'seen' I did after -rc1 last night included this\ntopic again; https://github.com/git/git/actions/runs/1700157728\n\nThe CI job triggered Leak Sanitizer in \"git init\" (the first thing\ntests do is to prepare a test repository with working tree, so there\nmay be other new breakages, but we wouldn't know) and broke CI.\n\nThis morning, just to test, I ejected this topic from 'seen' and\npushed it out; https://github.com/git/git/actions/runs/1702373554\n\nAs the only difference between these two runs is this topic, it may\nbe worth looking into locating and fixing new leaks introduced by\nit.\n\nThanks.\n"},{"id":"446301","messageId":"CAJDDKr61fZhSOmEYzwzMgmsR7e84SwpJJ9==NV-4aepAKp2u8w@mail.gmail.com","threadId":"57242","inReplyTo":"xmqqo84c4xqh.fsf@gitster.g","subject":"Re: What's cooking in git.git (Jan 2022, #03; Thu, 13)","fromName":"David Aguilar","fromEmail":"davvid@gmail.com","sentAt":"2022-01-16T02:15:10Z","receivedAt":"2022-01-16T02:15:50Z","isPatch":false,"sender":{"key":"davvid@gmail.com","avatar":"https://avatars.githubusercontent.com/u/13196?v=4"},"body":"On Sat, Jan 15, 2022 at 11:15 AM Junio C Hamano wrote:\n>\n> David Aguilar writes:\n>\n> > I ran into the c99 test balloon. I get the (expected?) c99 build\n> > errors and an uncompress2 build error with RHEL7.9's stock gcc 4.8.5.\n> >\n> > Are we dropping support for RHEL7.9?\n>\n> No. Our expectation is that those who care about various distros\n> and platforms supply patches like ...\n>\n> > Using NO_UNCOMPRESS2=YesPlease and CFLAGS += -std=c99 in config.mak\n>\n> ... this, but not locallly but to config.mak.uname, to help the\n> users and builders on their favorite systems. It is better done\n> sooner rather than later.\n>\n> > works but I'm not sure if we actually expect users to deal with those\n> > details themselves.\n>\n> Users would not have to, as long as somebody else on the list uses\n> the smea system as theirs cares enough to supply such patches to\n> config.mak.uname.\n>\n> > Sorry for being out of the loop on the plan for older toolchains, but\n> > I wasn't sure if we should be expecting a patch that configures the\n> > default flags so that things build out of the box again or if this is\n> > considered the new status quo and we're okay regressing?\n>\n> Obviously the former. This is a team effort and everybody who finds\n> new issues is very appreciated to chip in.\n>\n\n\nThanks for verifying; patch sent. A simple $(uname_R) check worked for me\nso hopefully that's not too much cruft to carry around for a few years.\n\ncheers,\n-- \nDavid\n"},{"id":"446340","messageId":"YeUYR8TwJf+31wcl@ncase","threadId":"57242","inReplyTo":"220114.86fspqrxbw.gmgdl@evledraar.gmail.com","subject":"Re: What's cooking in git.git (Jan 2022, #03; Thu, 13)","fromName":"Patrick Steinhardt","fromEmail":"ps@pks.im","sentAt":"2022-01-17T07:18:31Z","receivedAt":"2022-01-17T07:18:35Z","isPatch":false,"sender":{"key":"ps@pks.im","avatar":"https://avatars.githubusercontent.com/u/4056630?v=4"},"body":"On Fri, Jan 14, 2022 at 07:12:15PM +0100, Ævar Arnfjörð Bjarmason wrote:\n> \n> On Thu, Jan 13 2022, Junio C Hamano wrote:\n> \n> > Junio C Hamano writes:\n> >\n> >> There are a few \"oops, what we merged recently is broken\" topics\n> >> that still are not in 'master', but otherwise what we have should\n> >> be pretty much what we'll have in the final one.\n> >>\n> >> - I am reasonably happy with ab/refs-errno-cleanup (just one patch)\n> >> that fixes the incorrect state of the code left by the earlier\n> >> parts of the topic that have already been merged during this\n> >> cycle.\n> >>\n> >> - I am also OK with ab/reftable-build-fixes (two patches), one for\n> >> general type correctness fix, the other for helping older sub-C99\n> >> compilers.\n> >>\n> >> If there are fixes for regressions that we introduced during this\n> >> cycle other than these two topics, I certainly am missing them, so\n> >> please holler loudly and quickly, hopefully in time for me to tag\n> >> the -rc1 tomorrow.\n> >\n> > Oh, by the way, the tip of 'seen' has consistently failing the\n> > leak-check test. I didn't have chance, time or energy to see if\n> > they are failing merely because an existing test script that used to\n> > be leak-clean gained a use of command that has been known to be\n> > leak-unclean without introducing any new leaks, or our recent change\n> > did introduce new leaks to commands that have been leak-clean.\n> > Somebody with too much time on their hand should go in and check to\n> > help, before CI testing on 'seen' becomes useful again.\n> \n> It's a regression in\n> ps/avoid-unnecessary-hook-invocation-with-packed-refs, Patrick could you\n> look into it? On your current \"seen\" doing a:\n> \n> git revert -m 1 48b388cbf31\n> \n> Will make those 3x failing tests pass:\n> https://github.com/git/git/runs/4811683950?check_suite_focus=true\n> \n> (That commit being: 48b388cbf31 (Merge branch\n> 'ps/avoid-unnecessary-hook-invocation-with-packed-refs' into seen,\n> 2022-01-13))\n> \n> I didn't have much time to look now, but this mostly untested fix-up\n> fixes up the topic under SANITIZE=leak (but may break something else). I\n> ran the broken tests with SANITIZE=leak, and the normal tests without\n> SANITIZE=leak, but didn't have time for further testing:\n> \n> diff --git a/refs/packed-backend.c b/refs/packed-backend.c\n> index ff96ee482a0..b8012f97009 100644\n> --- a/refs/packed-backend.c\n> +++ b/refs/packed-backend.c\n> @@ -1577,6 +1577,7 @@ int packed_refs_delete_refs(struct ref_store *ref_store,\n> error(_(\"could not delete references: %s\"), err.buf);\n> }\n> \n> + ref_transaction_free(transaction);\n> strbuf_release(&err);\n> return ret;\n> }\n> \n> I.e. the moving around of the ref_transaction_free() is at fault\n> somehow, probably...\n\nThanks for digging! The bug is actually in the files backend, where\n`files_delete_refs()` has two different exit paths, but I added the free\nof the packed-refs backend only to one of both. So the following patch\nfixes it:\n\ndiff --git a/refs/files-backend.c b/refs/files-backend.c\nindex 9a20cb8fa8..844918cbd8 100644\n--- a/refs/files-backend.c\n+++ b/refs/files-backend.c\n@@ -1280,6 +1280,7 @@ static int files_delete_refs(struct ref_store *ref_store, const char *msg,\n \t\t\tresult |= error(_(\"could not remove reference %s\"), refname);\n \t}\n \n+\tref_transaction_free(transaction);\n \tstrbuf_release(&err);\n \treturn result;\n \n\nI'll send a reroll of my series.\n\nPatrick\n"},{"id":"446426","messageId":"0d6269b5-95ea-c2cd-6e5f-f10e874525d5@gmail.com","threadId":"57242","inReplyTo":"xmqqiluk4wo4.fsf@gitster.g","subject":"Re: Mistakes in the stalled category?","fromName":"Derrick Stolee","fromEmail":"stolee@gmail.com","sentAt":"2022-01-18T16:11:28Z","receivedAt":"2022-01-18T16:14:10Z","isPatch":false,"sender":{"key":"stolee@gmail.com","avatar":"https://avatars.githubusercontent.com/u/570044?v=4"},"body":"On 1/15/2022 2:38 PM, Junio C Hamano wrote:\n> Junio C Hamano writes:\n> \n>> Elijah Newren writes:\n>>\n>>>> [Stalled]\n>>>>\n>>>> * ds/sparse-checkout-requires-per-worktree-config (2021-12-21) 5 commits\n>>\n>> [Stalled] being early in the report is primarily a way to remind us\n>> that an update is overdue.\n>>\n>> <1db0f601-4769-15c0-cd58-ecddfa1fc9d5@gmail.com> is what prompted me\n>> to consider that I can put it on backburner and spend my time on\n>> other topics.\n> \n> The pushout of 'seen' I did after -rc1 last night included this\n> topic again; https://github.com/git/git/actions/runs/1700157728\n> \n> The CI job triggered Leak Sanitizer in \"git init\" (the first thing\n> tests do is to prepare a test repository with working tree, so there\n> may be other new breakages, but we wouldn't know) and broke CI.\n> \n> This morning, just to test, I ejected this topic from 'seen' and\n> pushed it out; https://github.com/git/git/actions/runs/1702373554\n> \n> As the only difference between these two runs is this topic, it may\n> be worth looking into locating and fixing new leaks introduced by\n> it.\n\nThanks for pointing this out. I will work to fix this leak in the\nnext version. Feel free to eject it until the new version is\navailable.\n\nThanks,\n-Stolee\n"}]}