{"thread":{"id":"58098","subject":"What's cooking in git.git (Jul 2022, #01; Fri, 1)","startedAt":"2022-07-01T23:08:20Z","lastAt":"2022-07-06T22:16:04Z","messageCount":10,"participants":["Junio C Hamano","Ævar Arnfjörð Bjarmason","Derrick Stolee","Elijah Newren","Glen Choo"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"458426","messageId":"xmqqo7y85t44.fsf@gitster.g","threadId":"58098","inReplyTo":null,"subject":"What's cooking in git.git (Jul 2022, #01; Fri, 1)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2022-07-01T23:08:11Z","receivedAt":"2022-07-01T23:08:20Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Here are the topics that have been cooking in my tree. Commits\nprefixed with '+' are in 'next' (being in 'next' is a sign that a\ntopic is stable enough to be used and are candidate to be in a\nfuture release). Commits prefixed with '-' are only in 'seen',\nand aren't considered \"accepted\" at all.\n\nGit 2.37 final was released, many in the northan hemisphere are\nabout to be in summer vacation week. We probably will have a small\nupdate 2.37.1 near mid July to deal with a small \"regression\" in\n2.37 but otherwise things are expected to be slow, and slow is good\n;-)\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* ds/git-rebase-doc-markup (2022-06-30) 1 commit\n - git-rebase.txt: use back-ticks consistently\n\n Correct \"git rebase\" documentation mark-up.\n\n Will merge to 'next'.\n source: \n\n\n* ds/rebase-update-ref (2022-06-28) 8 commits\n - rebase: add rebase.updateRefs config option\n - rebase: update refs from 'update-ref' commands\n - rebase: add --update-refs option\n - sequencer: add update-ref command\n - sequencer: define array with enum values\n - rebase-interactive: update 'merge' description\n - branch: consider refs under 'update-refs'\n - t2407: test branches currently using apply backend\n (this branch uses ds/branch-checked-out.)\n\n \"git rebase -i\" learns to update branches whose tip appear in the\n rebased range.\n\n Will merge to 'next'?\n source: \n\n\n* ds/vscode-settings (2022-06-27) 1 commit\n - vscode: improve tab size and wrapping\n\n source: \n\n\n* js/add-i-delete (2022-06-28) 1 commit\n (merged to 'next' on 2022-06-28 at 8ac04bfd24)\n + add --interactive: allow `update` to stage deleted files\n\n Rewrite of \"git add -i\" in C that appeared in Git 2.25 didn't\n correctly record a removed file to the index, which was fixed.\n\n Will merge to 'master'.\n source: \n\n\n* tb/pack-objects-remove-pahole-comment (2022-06-28) 1 commit\n - pack-objects.h: remove outdated pahole results\n\n Comment fix.\n\n Will merge to 'next'.\n source: <1379af2e9d271b501ef3942398e7f159a9c77973.1656440978.git.me@ttaylorr.com>\n\n\n* ab/leakfix (2022-07-01) 11 commits\n - pull: fix a \"struct oid_array\" memory leak\n - cat-file: fix a common \"struct object_context\" memory leak\n - gc: fix a memory leak\n - checkout: avoid \"struct unpack_trees_options\" leak\n - merge-file: fix memory leaks on error path\n - merge-file: refactor for subsequent memory leak fix\n - cat-file: fix a memory leak in --batch-command mode\n - revert: free \"struct replay_opts\" members\n - submodule.c: free() memory from xgetcwd()\n - clone: fix memory leak in wanted_peer_refs()\n - check-ref-format: fix trivial memory leak\n\n Plug various memory leaks.\n\n source: \n\n\n* ab/test-tool-leakfix (2022-07-01) 9 commits\n - test-tool delta: fix a memory leak\n - test-tool ref-store: fix a memory leak\n - test-tool bloom: fix memory leaks\n - test-tool json-writer: fix memory leaks\n - test-tool regex: call regfree(), fix memory leaks\n - test-tool urlmatch-normalization: fix a memory leak\n - test-tool {dump,scrap}-cache-tree: fix memory leaks\n - test-tool path-utils: fix a memory leak\n - test-tool test-hash: fix a memory leak\n\n Plug various memory leaks in test-tool commands.\n\n source: \n\n\n* en/t6429-test-must-be-empty-fix (2022-06-30) 1 commit\n - t6429: fix use of non-existent function\n\n A test fix.\n\n Will merge to 'next'.\n source: \n\n\n* gc/submodule-use-super-prefix (2022-06-30) 8 commits\n - submodule--helper: remove display path helper\n - submodule--helper update: use --super-prefix\n - submodule--helper: remove unused SUPPORT_SUPER_PREFIX flags\n - submodule--helper: use correct display path helper\n - submodule--helper: don't recreate recursive prefix\n - submodule--helper update: use display path helper\n - submodule--helper tests: add missing \"display path\" coverage\n - Merge branch 'ab/submodule-cleanup' into gc/submodule-use-super-prefix\n (this branch uses ab/submodule-cleanup.)\n\n Another step to rewrite more parts of \"git submodule\" in C.\n\n Will merge to 'next'?\n source: <20220701021157.88858-1-chooglen@google.com>\n\n\n* hx/lookup-commit-in-graph-fix (2022-06-30) 1 commit\n - commit-graph.c: no lazy fetch in lookup_commit_in_graph()\n\n A corner case bug where lazily fetching objects from a promisor\n remote resulted in infinite recursion has been corrected.\n\n Will merge to 'next'.\n source: <96d4bb71505d87ed501c058bbd89bfc13d08b24a.1656593279.git.hanxin.hx@bytedance.com>\n\n\n* ll/ls-files-tests-update (2022-07-01) 1 commit\n - ls-files: update test style\n\n Test update.\n\n Will merge to 'next'.\n source: \n\n\n* pw/xdiff-alloc (2022-06-30) 3 commits\n - xdiff: introduce XDL_ALLOC_GROW()\n - xdiff: introduce XDL_CALLOC_ARRAY()\n - xdiff: introduce XDL_ALLOC_ARRAY()\n\n Add a level of redirection to array allocation API in xdiff part,\n to make it easier to share with the libgit2 project.\n\n Waiting for review responses.\n source: \n\n\n* sy/mv-out-of-cone (2022-07-01) 8 commits\n - mv: add check_dir_in_index() and solve general dir check issue\n - mv: use flags mode for update_mode\n - mv: check if exists in index to handle overwriting\n - mv: check if out-of-cone file exists in index with SKIP_WORKTREE bit\n - mv: decouple if/else-if checks using goto\n - mv: update sparsity after moving from out-of-cone to in-cone\n - t1092: mv directory from out-of-cone to in-cone\n - t7002: add tests for moving out-of-cone file/directory\n\n \"git mv A B\" in a sparsely populated working tree can be asked to\n move a path between directories that are \"in cone\" (i.e. expected\n to be materialized in the working tree) and \"out of cone\"\n (i.e. expected to be hidden). The handling of such cases has been\n improved.\n\n Will merge to 'next'.\n source: <20220630023737.473690-1-shaoxuan.yuan02@gmail.com>\n\n--------------------------------------------------\n[Stalled]\n\n* bc/stash-export (2022-04-08) 4 commits\n - builtin/stash: provide a way to import stashes from a ref\n - builtin/stash: provide a way to export stashes to a ref\n - builtin/stash: factor out revision parsing into a function\n - object-name: make get_oid quietly return an error\n\n A mechanism to export and import stash entries to and from a normal\n commit to transfer it across repositories has been introduced.\n\n Expecting a reroll.\n cf. \n source: <20220407215352.3491567-1-sandals@crustytoothpaste.net>\n\n\n* cw/remote-object-info (2022-05-06) 11 commits\n - SQUASH??? coccicheck\n - SQUASH??? ensure that coccicheck is happy\n - SQUASH??? compilation fix\n - cat-file: add --batch-command remote-object-info command\n - cat-file: move parse_cmd and DEFAULT_FORMAT up\n - transport: add object-info fallback to fetch\n - transport: add client side capability to request object-info\n - object-info: send attribute packet regardless of object ids\n - object-store: add function to free object_info contents\n - fetch-pack: move fetch default settings\n - fetch-pack: refactor packet writing\n\n A client component to talk with the object-info endpoint.\n\n Expecting a reroll.\n source: <20220502170904.2770649-1-calvinwan@google.com>\n\n--------------------------------------------------\n[Cooking]\n\n* ab/squelch-empty-fsync-traces (2022-06-30) 1 commit\n . trace2: don't include \"fsync\" events in all trace2 logs\n\n Omit fsync-related trace2 entries when their values are all zero.\n\n Breaks tests in hx/unpack-streaming with an interesting interaction.\n source: \n\n\n* cl/grep-max-count (2022-06-22) 1 commit\n - grep: add --max-count command line option\n\n \"git grep -m\" is a way to limit the hits shown per file.\n\n Will merge to 'next'.\n source: \n\n\n* jk/revisions-doc-markup-fix (2022-06-22) 1 commit\n - revisions.txt: escape \"...\" to avoid asciidoc horizontal ellipsis\n\n Documentation mark-up fix.\n\n Will merge to 'next'.\n source: \n\n\n* tk/rev-parse-doc-clarify-at-u (2022-06-23) 1 commit\n - rev-parse: documentation adjustment - mention remote tracking with @{u}\n\n Doc update.\n\n Will merge to 'next'.\n source: \n\n\n* en/merge-tree (2022-06-22) 17 commits\n - git-merge-tree.txt: add a section on potentional usage mistakes\n - merge-tree: add a --allow-unrelated-histories flag\n - merge-tree: allow `ls-files -u` style info to be NUL terminated\n - merge-ort: optionally produce machine-readable output\n - merge-ort: store more specific conflict information\n - merge-ort: make `path_messages` a strmap to a string_list\n - merge-ort: store messages in a list, not in a single strbuf\n - merge-tree: provide easy access to `ls-files -u` style info\n - merge-tree: provide a list of which files have conflicts\n - merge-ort: remove command-line-centric submodule message from merge-ort\n - merge-ort: provide a merge_get_conflicted_files() helper function\n - merge-tree: support including merge messages in output\n - merge-ort: split out a separate display_update_messages() function\n - merge-tree: implement real merges\n - merge-tree: add option parsing and initial shell for real merge function\n - merge-tree: move logic for existing merge into new function\n - merge-tree: rename merge_trees() to trivial_merge_trees()\n\n A new command is introduced that takes two commits and computes a\n tree that would be contained in the resulting merge commit, if the\n histories leading to these two commits were to be merged, and is\n added as a new mode of \"git merge-tree\" subcommand.\n\n Will merge to 'next'.\n source: \n\n\n* dr/i18n-die-warn-error-usage (2022-06-21) 1 commit\n - i18n: mark message helpers prefix for translation\n\n Give _() markings to fatal/warning/usage: that is shown in front of\n these messages.\n\n Will merge to 'next'.\n source: \n\n\n* ds/t5510-brokequote (2022-06-21) 1 commit\n - t5510: replace 'origin' with URL more carefully\n\n Test fix.\n source: <484a330e-0902-6e1b-8189-63c72dcea494@github.com>\n\n\n* en/merge-restore-to-pristine (2022-06-21) 6 commits\n - merge: do not exit restore_state() prematurely\n - merge: ensure we can actually restore pre-merge state\n - merge: make restore_state() restore staged state too\n - merge: fix save_state() to work when there are racy-dirty files\n - merge: remove unused variable\n - t6424: make sure a failed merge preserves local changes\n\n When \"git merge\" finds that it cannot perform a merge, it should\n restore the working tree to the state before the command was\n initiated, but in some corner cases it didn't.\n\n Needs review.\n source: \n\n\n* rs/combine-diff-with-incompatible-options (2022-06-21) 2 commits\n - combine-diff: abort if --output is given\n - combine-diff: abort if --ignore-matching-lines is given\n\n Certain diff options are currently ignored when combined-diff is\n shown; mark them as incompatible with the feature.\n\n Will merge to 'next'.\n source: <220524.86v8tuvfl1.gmgdl@evledraar.gmail.com>\n\n\n* tk/apply-case-insensitive (2022-06-21) 3 commits\n - apply: support case-only renames in case-insensitive filesystems\n - reset: new failing test for reset of case-insensitive duplicate in index\n - t4141: test \"git apply\" with core.ignorecase\n\n \"git apply\" barfed on a patch that makes a case-only rename on a\n case-insensitive filesystem.\n\n Needs review.\n source: \n\n\n* zh/ls-files-format (2022-06-27) 1 commit\n - ls-files: introduce \"--format\" option\n\n \"git ls-files\" learns the \"--format\" option to tweak its output.\n\n Expecting a reroll.\n cf. \n source: \n\n\n* ab/test-quoting-fix (2022-06-30) 3 commits\n - config tests: fix harmless but broken \"rm -r\" cleanup\n - test-lib.sh: fix prepend_var() quoting issue\n - tests: add missing double quotes to included library paths\n\n Fixes for tests when the source directory has unusual characters in\n its path, e.g. whitespaces, double-quotes, etc.\n\n Will merge to 'next'.\n source: \n\n\n* en/merge-dual-dir-renames-fix (2022-06-30) 5 commits\n - merge-ort: fix issue with dual rename and add/add conflict\n - merge-ort: shuffle the computation and cleanup of potential collisions\n - merge-ort: make a separate function for freeing struct collisions\n - merge-ort: small cleanups of check_for_directory_rename\n - t6423: add tests of dual directory rename plus add/add conflict\n\n Fixes a long-standing corner case bug around directory renames in\n the merge-ort strategy.\n\n Will merge to 'next'?\n source: \n\n\n* cr/setup-bug-typo (2022-06-17) 1 commit\n (merged to 'next' on 2022-06-17 at 8834ffe0ab)\n + setup: fix function name in a BUG() message\n\n Typofix in a BUG() message.\n\n Will cook in 'next'.\n source: \n\n\n* zk/push-use-bitmaps (2022-06-17) 1 commit\n - send-pack.c: add config push.useBitmaps\n\n \"git push\" sometimes perform poorly when reachability bitmaps are\n used, even in a repository where other operations are helped by\n bitmaps. The push.useBitmaps configuration variable is introduced\n to allow disabling use of reachability bitmaps only for \"git push\".\n\n Will merge to 'next'.\n source: \n\n\n* jk/remote-show-with-negative-refspecs (2022-06-17) 1 commit\n - remote: handle negative refspecs in git remote show\n (this branch is used by jk/t5505-restructure.)\n\n \"git remote show [-n] frotz\" now pays attention to negative\n pathspecs.\n\n Will merge to 'next'.\n source: <20220617002036.1577-2-jacob.keller@gmail.com>\n\n\n* js/commit-graph-parsing-without-repo-settings (2022-06-15) 1 commit\n - commit-graph: refactor to avoid prepare_repo_settings\n\n Expecting a reroll.\n source: <9b56496b0809cc8a25af877ea97042e2cb7f2af6.1655246092.git.steadmon@google.com>\n\n\n* jk/optim-promisor-object-enumeration (2022-06-16) 1 commit\n (merged to 'next' on 2022-06-16 at ce0712a74c)\n + is_promisor_object(): walk promisor packs in pack-order\n\n Collection of what is referenced by objects in promisor packs have\n been optimized to inspect these objects in the in-pack order.\n\n Will cook in 'next'.\n source: \n\n\n* ro/mktree-allow-missing-fix (2022-06-21) 1 commit\n - mktree: do not check type of remote objects\n\n \"git mktree --missing\" lazily fetched objects that are missing from\n the local object store, which was totally unnecessary.\n\n Will merge to 'next'.\n source: <748f39a9-65aa-2110-cf92-7ddf81b5f507@roku.com>\n\n\n* ll/curl-accept-language (2022-06-13) 2 commits\n - PREP??? give initializer to rpc_state\n - remote-curl: send Accept-Language header to server\n\n source: \n\n\n* pb/diff-doc-raw-format (2022-06-13) 3 commits\n - diff-index.txt: update raw output format in examples\n - diff-format.txt: correct misleading wording\n - diff-format.txt: dst can be 0* SHA-1 when path is deleted, too\n\n source: \n\n\n* rs/archive-with-internal-gzip (2022-06-15) 6 commits\n (merged to 'next' on 2022-06-17 at ab5af6acd1)\n + archive-tar: use internal gzip by default\n + archive-tar: use OS_CODE 3 (Unix) for internal gzip\n + archive-tar: add internal gzip implementation\n + archive-tar: factor out write_block()\n + archive: rename archiver data field to filter_command\n + archive: update format documentation\n\n Teach \"git archive\" to (optionally and then by default) avoid\n spawning an external \"gzip\" process when creating \".tar.gz\" (and\n \".tgz\") archives.\n\n Will cook in 'next'.\n source: <9df761c3-355a-ede9-7971-b32687fe9abb@web.de>\n\n\n* ds/branch-checked-out (2022-06-21) 7 commits\n (merged to 'next' on 2022-06-21 at e42bc4566f)\n + branch: drop unused worktrees variable\n + fetch: stop passing around unused worktrees variable\n (merged to 'next' on 2022-06-17 at c881874257)\n + branch: fix branch_checked_out() leaks\n + branch: use branch_checked_out() when deleting refs\n + fetch: use new branch_checked_out() and add tests\n + branch: check for bisects and rebases\n + branch: add branch_checked_out() helper\n (this branch is used by ds/rebase-update-ref.)\n\n Introduce a helper to see if a branch is already being worked on\n (hence should not be newly checked out in a working tree), which\n performs much better than the existing find_shared_symref() to\n replace many uses of the latter.\n\n Will cook in 'next'.\n source: \n\n\n* jt/connected-show-missing-from-which-side (2022-06-10) 1 commit\n - fetch,fetch-pack: clarify connectivity check error\n\n We may find an object missing after a \"git fetch\" stores the\n objects it obtained from the other side, but it is not necessarily\n because the remote failed to send necessary objects. Reword the\n messages in an attempt to help users explore other possibilities\n when they hit this error.\n\n Expecting a reroll.\n source: <20220610195247.1177549-1-jonathantanmy@google.com>\n\n\n* ab/submodule-cleanup (2022-06-28) 12 commits\n - git-sh-setup.sh: remove \"say\" function, change last users\n - git-submodule.sh: use \"$quiet\", not \"$GIT_QUIET\"\n - submodule--helper: eliminate internal \"--update\" option\n - submodule--helper: understand --checkout, --merge and --rebase synonyms\n - submodule--helper: report \"submodule\" as our name in some \"-h\" output\n - submodule--helper: rename \"absorb-git-dirs\" to \"absorbgitdirs\"\n - submodule update: remove \"-v\" option\n - submodule--helper: have --require-init imply --init\n - git-submodule.sh: remove unused top-level \"--branch\" argument\n - git-submodule.sh: make the \"$cached\" variable a boolean\n - git-submodule.sh: remove unused $prefix variable\n - git-submodule.sh: remove unused sanitize_submodule_env()\n (this branch is used by gc/submodule-use-super-prefix.)\n\n Further preparation to turn git-submodule.sh into a builtin.\n\n Will merge to 'next'.\n source: \n\n\n* jc/resolve-undo (2022-06-09) 1 commit\n (merged to 'next' on 2022-06-15 at c195e5a2d9)\n + revision: mark blobs needed for resolve-undo as reachable\n\n The resolve-undo information in the index was not protected against\n GC, which has been corrected.\n\n Will cook in 'next'.\n source: \n\n\n* ab/build-gitweb (2022-06-28) 8 commits\n - gitweb/Makefile: add a \"NO_GITWEB\" parameter\n - Makefile: build 'gitweb' in the default target\n - gitweb/Makefile: include in top-level Makefile\n - gitweb: remove \"test\" and \"test-installed\" targets\n - gitweb/Makefile: prepare to merge into top-level Makefile\n - gitweb/Makefile: clear up and de-duplicate the gitweb.{css,js} vars\n - gitweb/Makefile: add a $(GITWEB_ALL) variable\n - gitweb/Makefile: define all .PHONY prerequisites inline\n\n Teach \"make all\" to build gitweb as well.\n source: \n\n\n* ab/test-without-templates (2022-06-06) 7 commits\n - tests: don't assume a .git/info for .git/info/sparse-checkout\n - tests: don't assume a .git/info for .git/info/exclude\n - tests: don't assume a .git/info for .git/info/refs\n - tests: don't assume a .git/info for .git/info/attributes\n - tests: don't assume a .git/info for .git/info/grafts\n - tests: don't depend on template-created .git/branches\n - t0008: don't rely on default \".git/info/exclude\"\n\n Tweak tests so that they still work when the \"git init\" template\n did not create .git/info directory.\n\n Will merge to 'next'?\n source: \n\n\n* ac/bitmap-format-doc (2022-06-16) 3 commits\n (merged to 'next' on 2022-06-16 at 5591d11601)\n + bitmap-format.txt: add information for trailing checksum\n + bitmap-format.txt: fix some formatting issues\n + bitmap-format.txt: feed the file to asciidoc to generate html\n\n Adjust technical/bitmap-format to be formatted by AsciiDoc, and\n add some missing information to the documentation.\n\n Will cook in 'next'.\n source: \n\n\n* hx/unpack-streaming (2022-06-13) 6 commits\n - unpack-objects: use stream_loose_object() to unpack large objects\n - core doc: modernize core.bigFileThreshold documentation\n - object-file.c: add \"stream_loose_object()\" to handle large object\n - object-file.c: factor out deflate part of write_loose_object()\n - object-file.c: refactor write_loose_object() to several steps\n - unpack-objects: low memory footprint for get_data() in dry_run mode\n\n Allow large objects read from a packstream to be streamed into a\n loose object file straight, without having to keep it in-core as a\n whole.\n\n Will merge to 'next'.\n source: \n\n\n* tb/show-ref-count (2022-06-06) 2 commits\n - builtin/show-ref.c: limit output with `--count`\n - builtin/show-ref.c: rename `found_match` to `matches_nr`\n\n \"git show-ref\" learned to stop after emitting N refs with the new\n \"--count=N\" option.\n\n Expecting a reroll.\n cf. \n source: \n\n\n* ds/bundle-uri-more (2022-06-06) 6 commits\n - fetch: add 'refs/bundle/' to log.excludeDecoration\n - bundle-uri: add support for http(s):// and file://\n - fetch: add --bundle-uri option\n - bundle-uri: create basic file-copy logic\n - remote-curl: add 'get' capability\n - docs: document bundle URI standard\n\n The \"bundle URI\" topic.\n\n Needs review.\n source: \n\n\n* js/bisect-in-c (2022-06-27) 16 commits\n - bisect: no longer try to clean up left-over `.git/head-name` files\n - bisect: remove Cogito-related code\n - Turn `git bisect` into a full built-in\n - bisect: move even the command-line parsing to `bisect--helper`\n - bisect: teach the `bisect--helper` command to show the correct usage strings\n - bisect--helper: return only correct exit codes in `cmd_*()`\n - bisect--helper: move the `BISECT_STATE` case to the end\n - bisect--helper: make `--bisect-state` optional\n - bisect--helper: align the sub-command order with git-bisect.sh\n - bisect--helper: using `--bisect-state` without an argument is a bug\n - bisect--helper: really retire `--bisect-autostart`\n - bisect--helper: really retire --bisect-next-check\n - bisect--helper: retire the --no-log option\n - bisect: avoid double-quoting when printing the failed command\n - bisect run: fix the error message\n - bisect: verify that a bogus option won't try to start a bisection\n\n Final bits of \"git bisect.sh\" have been rewritten in C.\n source: \n\n\n* gc/bare-repo-discovery (2022-06-30) 5 commits\n - setup.c: create `discovery.bare`\n - safe.directory: use git_protected_config()\n - config: learn `git_protected_config()`\n - Documentation: define protected configuration\n - Documentation/git-config.txt: add SCOPES section\n\n Introduce a discovery.barerepository configuration variable that\n allows users to forbid discovery of bare repositories.\n\n Will merge to 'next'?\n source: \n\n\n* gg/worktree-from-the-above (2022-06-21) 2 commits\n - dir: minor refactoring / clean-up\n - dir: traverse into repository\n\n With a non-bare repository, with core.worktree pointing at a\n directory that has the repository as its subdirectory, regressed in\n Git 2.27 days.\n\n Will merge to 'next'.\n source: <20220616234433.225-1-gg.oss@outlook.com>\n source: <20220616231956.154-1-gg.oss@outlook.com>\n\n\n* ar/send-email-confirm-by-default (2022-04-22) 1 commit\n - send-email: always confirm by default\n\n \"git send-email\" is changed so that by default it asks for\n confirmation before sending each message out.\n\n Will discard.\n\n I wanted to like this, and had it in the version of Git I use\n myself for daily work, but the prompting turned out to be somewhat\n distracting.\n\n Thoughts?\n source: <20220422083629.1404989-1-hi@alyssa.is>\n"},{"id":"458435","messageId":"220704.8635fhb84h.gmgdl@evledraar.gmail.com","threadId":"58098","inReplyTo":"xmqqo7y85t44.fsf@gitster.g","subject":"gc/submodule-use-super-prefix & ab/submodule-cleanup (was: What's cooking in git.git (Jul 2022, #01; Fri, 1))","fromName":"Ævar Arnfjörð Bjarmason","fromEmail":"avarab@gmail.com","sentAt":"2022-07-04T08:12:19Z","receivedAt":"2022-07-04T08:21:58Z","isPatch":false,"sender":{"key":"avarab@gmail.com","avatar":"https://avatars.githubusercontent.com/u/45301?v=4"},"body":"\nOn Fri, Jul 01 2022, Junio C Hamano wrote:\n\n> * gc/submodule-use-super-prefix (2022-06-30) 8 commits\n> - submodule--helper: remove display path helper\n> - submodule--helper update: use --super-prefix\n> - submodule--helper: remove unused SUPPORT_SUPER_PREFIX flags\n> - submodule--helper: use correct display path helper\n> - submodule--helper: don't recreate recursive prefix\n> - submodule--helper update: use display path helper\n> - submodule--helper tests: add missing \"display path\" coverage\n> - Merge branch 'ab/submodule-cleanup' into gc/submodule-use-super-prefix\n> (this branch uses ab/submodule-cleanup.)\n>\n> Another step to rewrite more parts of \"git submodule\" in C.\n>\n> Will merge to 'next'?\n> source: <20220701021157.88858-1-chooglen@google.com>\n\n[...]\n\n> * ab/submodule-cleanup (2022-06-28) 12 commits\n> - git-sh-setup.sh: remove \"say\" function, change last users\n> - git-submodule.sh: use \"$quiet\", not \"$GIT_QUIET\"\n> - submodule--helper: eliminate internal \"--update\" option\n> - submodule--helper: understand --checkout, --merge and --rebase synonyms\n> - submodule--helper: report \"submodule\" as our name in some \"-h\" output\n> - submodule--helper: rename \"absorb-git-dirs\" to \"absorbgitdirs\"\n> - submodule update: remove \"-v\" option\n> - submodule--helper: have --require-init imply --init\n> - git-submodule.sh: remove unused top-level \"--branch\" argument\n> - git-submodule.sh: make the \"$cached\" variable a boolean\n> - git-submodule.sh: remove unused $prefix variable\n> - git-submodule.sh: remove unused sanitize_submodule_env()\n> (this branch is used by gc/submodule-use-super-prefix.)\n>\n> Further preparation to turn git-submodule.sh into a builtin.\n>\n> Will merge to 'next'.\n> source: \n\nThanks, with regards to gc/submodule-use-super-prefix that series gets\nmy enthusiastic (if it helps):\n\n\tReviewed-by: Ævar Arnfjörð Bjarmason \n\nLikewise the ab/submodule-cleanup parent topic has Glen's:\n\n\tReviewed-by: Glen Choo \n\nPer [1] and [2]. I.e. the latest is the v4, but the Reviewed-By was\ngiven to v3 in [1], and [4] notes on v4 that the one remaining concern\nwas addressed.\n\nThanks both, it's great to see us this close to getting rid of another\n*.sh built-in. I plan to submit the migration to the builtin/submodule.c\nafter these two land, possibly after some memory leak fixes to\nbuiltin/submodule--helper.c.\n\n1. https://lore.kernel.org/git/kl6lh74aivc6.fsf@chooglen-macbookpro.roam.corp.google.com/\n2. https://lore.kernel.org/git/kl6l1qv8ivcs.fsf@chooglen-macbookpro.roam.corp.google.com/\n"},{"id":"458447","messageId":"15631ea2-6722-fd24-c8a6-0cee638b0602@github.com","threadId":"58098","inReplyTo":"xmqqo7y85t44.fsf@gitster.g","subject":"ds/rebase-update-re (was Re: What's cooking in git.git (Jul 2022, #01; Fri, 1))","fromName":"Derrick Stolee","fromEmail":"derrickstolee@github.com","sentAt":"2022-07-04T13:15:02Z","receivedAt":"2022-07-04T13:15:28Z","isPatch":false,"sender":{"key":"stolee@gmail.com","avatar":"https://avatars.githubusercontent.com/u/570044?v=4"},"body":"On 7/1/22 5:08 PM, Junio C Hamano wrote:\n\n> * ds/rebase-update-ref (2022-06-28) 8 commits\n> - rebase: add rebase.updateRefs config option\n> - rebase: update refs from 'update-ref' commands\n> - rebase: add --update-refs option\n> - sequencer: add update-ref command\n> - sequencer: define array with enum values\n> - rebase-interactive: update 'merge' description\n> - branch: consider refs under 'update-refs'\n> - t2407: test branches currently using apply backend\n> (this branch uses ds/branch-checked-out.)\n> \n> \"git rebase -i\" learns to update branches whose tip appear in the\n> rebased range.\n> \n> Will merge to 'next'?\n> source: \n\nPlease expect at least one more re-roll. Here are the things I\nintend to fix in the next re-roll, which should arrive sometime\nthis week.\n\n* Some commit message tweaks.\n* Actually interrupt 'git bisect' and 'git rebase' in the tests.\n* Be careful to update the update-refs file when the user edits\n the todo file.\n* Consider storing the \"before\" values in the update-refs file\n and removing refs whose update-ref steps were removed by the\n user.\n* Elijah had a question about focusing on \"pick\" actions, not\n \"fixup\" or \"squash\" steps. This might be worth marking the\n option as experimental so we have flexibility in changing the\n behavior as new workflows are tested against this option.\n* Elijah also had some optimization ideas, but I'm not sure if\n we should prioritize them here or have them as future\n improvements after the feature is stable.\n\nThanks,\n-Stolee\n\n"},{"id":"458448","messageId":"CAPc5daXCq-JB0Uy_mGk-itLtJBug7sZJ0iE3JX+qX3Y_fpWRDQ@mail.gmail.com","threadId":"58098","inReplyTo":"15631ea2-6722-fd24-c8a6-0cee638b0602@github.com","subject":"Re: ds/rebase-update-re (was Re: What's cooking in git.git (Jul 2022, #01; Fri, 1))","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2022-07-04T15:59:40Z","receivedAt":"2022-07-04T15:59:56Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Thanks for a heads-up. I thought that all of you were supposed to be\noff this week, by the way ;-)?\n"},{"id":"458451","messageId":"CABPp-BEHj6gXe+GNR8cw+UUG27E0Mn-wiBb=Dp=ZgX_2gT915w@mail.gmail.com","threadId":"58098","inReplyTo":"15631ea2-6722-fd24-c8a6-0cee638b0602@github.com","subject":"Re: ds/rebase-update-re (was Re: What's cooking in git.git (Jul 2022, #01; Fri, 1))","fromName":"Elijah Newren","fromEmail":"newren@gmail.com","sentAt":"2022-07-04T16:58:48Z","receivedAt":"2022-07-04T16:59:03Z","isPatch":false,"sender":{"key":"newren@gmail.com","avatar":"https://avatars.githubusercontent.com/u/5455730?v=4"},"body":"On Mon, Jul 4, 2022 at 6:26 AM Derrick Stolee wrote:\n>\n> On 7/1/22 5:08 PM, Junio C Hamano wrote:\n>\n> > * ds/rebase-update-ref (2022-06-28) 8 commits\n> > - rebase: add rebase.updateRefs config option\n> > - rebase: update refs from 'update-ref' commands\n> > - rebase: add --update-refs option\n> > - sequencer: add update-ref command\n> > - sequencer: define array with enum values\n> > - rebase-interactive: update 'merge' description\n> > - branch: consider refs under 'update-refs'\n> > - t2407: test branches currently using apply backend\n> > (this branch uses ds/branch-checked-out.)\n> >\n> > \"git rebase -i\" learns to update branches whose tip appear in the\n> > rebased range.\n> >\n> > Will merge to 'next'?\n> > source: \n>\n> Please expect at least one more re-roll. Here are the things I\n> intend to fix in the next re-roll, which should arrive sometime\n> this week.\n>\n> * Some commit message tweaks.\n> * Actually interrupt 'git bisect' and 'git rebase' in the tests.\n> * Be careful to update the update-refs file when the user edits\n> the todo file.\n> * Consider storing the \"before\" values in the update-refs file\n> and removing refs whose update-ref steps were removed by the\n> user.\n\nFirst half sounds good to me. Not sure I understand the second half\nabout \"removing refs\"; do you mean actually deleting the ref, or just\nnot updating it, or something else?\n\n> * Elijah had a question about focusing on \"pick\" actions, not\n> \"fixup\" or \"squash\" steps. This might be worth marking the\n> option as experimental so we have flexibility in changing the\n> behavior as new workflows are tested against this option.\n\nThis sounds good to me. I do sometimes want to fixup or squash into a\ncommit that a branch I depend upon directly points to, so I'm worried\nyour feature may update refs incorrectly without this support.\n\n> * Elijah also had some optimization ideas, but I'm not sure if\n> we should prioritize them here or have them as future\n> improvements after the feature is stable.\n\nYou know, thinking about it more, with every single commit, sequencer\nalready updates the index, the working tree, HEAD, and *several*\ncontrol files. So these optimizations would probably just be lost in\nthe noise. Unless we restructure sequencer.c, it may not be worth\nimplementing these ideas at all. Either way, I'd say this certainly\nshouldn't hold up this series.\n"},{"id":"458457","messageId":"9662ac37-1458-8cf2-c0dd-5cd2b9107e4b@github.com","threadId":"58098","inReplyTo":"CABPp-BEHj6gXe+GNR8cw+UUG27E0Mn-wiBb=Dp=ZgX_2gT915w@mail.gmail.com","subject":"Re: ds/rebase-update-re (was Re: What's cooking in git.git (Jul 2022, #01; Fri, 1))","fromName":"Derrick Stolee","fromEmail":"derrickstolee@github.com","sentAt":"2022-07-04T20:17:55Z","receivedAt":"2022-07-04T20:18:16Z","isPatch":false,"sender":{"key":"stolee@gmail.com","avatar":"https://avatars.githubusercontent.com/u/570044?v=4"},"body":"On 7/4/22 10:58 AM, Elijah Newren wrote:\n> On Mon, Jul 4, 2022 at 6:26 AM Derrick Stolee wrote:\n>>\n>> On 7/1/22 5:08 PM, Junio C Hamano wrote:\n>>\n>>> * ds/rebase-update-ref (2022-06-28) 8 commits\n>>> - rebase: add rebase.updateRefs config option\n>>> - rebase: update refs from 'update-ref' commands\n>>> - rebase: add --update-refs option\n>>> - sequencer: add update-ref command\n>>> - sequencer: define array with enum values\n>>> - rebase-interactive: update 'merge' description\n>>> - branch: consider refs under 'update-refs'\n>>> - t2407: test branches currently using apply backend\n>>> (this branch uses ds/branch-checked-out.)\n>>>\n>>> \"git rebase -i\" learns to update branches whose tip appear in the\n>>> rebased range.\n>>>\n>>> Will merge to 'next'?\n>>> source: \n>>\n>> Please expect at least one more re-roll. Here are the things I\n>> intend to fix in the next re-roll, which should arrive sometime\n>> this week.\n>>\n>> * Some commit message tweaks.\n>> * Actually interrupt 'git bisect' and 'git rebase' in the tests.\n>> * Be careful to update the update-refs file when the user edits\n>> the todo file.\n>> * Consider storing the \"before\" values in the update-refs file\n>> and removing refs whose update-ref steps were removed by the\n>> user.\n> \n> First half sounds good to me. Not sure I understand the second half\n> about \"removing refs\"; do you mean actually deleting the ref, or just\n> not updating it, or something else?\n\nI mean removing it from the $GIT_DIR/rebase-merge/update-refs file\nso it is no longer blocking others from using it (and the value in\nthe file can be used for a --force-with-lease type update).\n\n>> * Elijah had a question about focusing on \"pick\" actions, not\n>> \"fixup\" or \"squash\" steps. This might be worth marking the\n>> option as experimental so we have flexibility in changing the\n>> behavior as new workflows are tested against this option.\n> \n> This sounds good to me. I do sometimes want to fixup or squash into a\n> commit that a branch I depend upon directly points to, so I'm worried\n> your feature may update refs incorrectly without this support.\n\nI've been using this quite a bit in my own work, and it properly\nmoves fixups and squashes into the right branches, but I've only\ndone that with fixups and squashes on the tip branch, not inside\nthe earlier branches. It's also been really good when I rebase\nand add \"break\" or \"edit\" steps and do really complicated commit\nsquashing or splitting operations.\n\nI'll try to collect some example runs, but it would also help a\ngreat deal if you added these patches to your local version of\nGit and try it own in your workflow. Use the rebase.updateRefs=true\nconfig setting so you don't need to think about it. Victoria has\nbeen testing this for me in her own workflows, so maybe she has\nideas here.\n\nThanks,\n-Stolee\n"},{"id":"458459","messageId":"CABPp-BGTYCsa8LNPUNKpfbRqZKba-fx=R22ryNn3HB8XDoarXQ@mail.gmail.com","threadId":"58098","inReplyTo":"xmqqo7y85t44.fsf@gitster.g","subject":"en/merge-dual-dir-renames-fix (Was: Re: What's cooking in git.git (Jul 2022, #01; Fri, 1))","fromName":"Elijah Newren","fromEmail":"newren@gmail.com","sentAt":"2022-07-05T01:31:42Z","receivedAt":"2022-07-05T01:31:57Z","isPatch":false,"sender":{"key":"newren@gmail.com","avatar":"https://avatars.githubusercontent.com/u/5455730?v=4"},"body":"On Fri, Jul 1, 2022 at 4:23 PM Junio C Hamano wrote:\n\n> * en/merge-dual-dir-renames-fix (2022-06-30) 5 commits\n> - merge-ort: fix issue with dual rename and add/add conflict\n> - merge-ort: shuffle the computation and cleanup of potential collisions\n> - merge-ort: make a separate function for freeing struct collisions\n> - merge-ort: small cleanups of check_for_directory_rename\n> - t6423: add tests of dual directory rename plus add/add conflict\n>\n> Fixes a long-standing corner case bug around directory renames in\n> the merge-ort strategy.\n>\n> Will merge to 'next'?\n> source: \n\nÆvar pointed out a for-initializer that shouldn't yet be used since\nwe've only used that in a \"weather balloon\" so far. Let me send in a\nre-roll to fix that one issue before you merge down.\n"},{"id":"458529","messageId":"kl6lo7y2rxez.fsf@chooglen-macbookpro.roam.corp.google.com","threadId":"58098","inReplyTo":"220704.8635fhb84h.gmgdl@evledraar.gmail.com","subject":"Re: gc/submodule-use-super-prefix & ab/submodule-cleanup (was: What's cooking in git.git (Jul 2022, #01; Fri, 1))","fromName":"Glen Choo","fromEmail":"chooglen@google.com","sentAt":"2022-07-06T16:57:56Z","receivedAt":"2022-07-06T16:59:02Z","isPatch":false,"sender":{"key":"glencbz@gmail.com","avatar":"https://avatars.githubusercontent.com/u/58092771?v=4"},"body":"Ævar Arnfjörð Bjarmason writes:\n\n> On Fri, Jul 01 2022, Junio C Hamano wrote:\n>\n>> * gc/submodule-use-super-prefix (2022-06-30) 8 commits\n>> - submodule--helper: remove display path helper\n>> - submodule--helper update: use --super-prefix\n>> - submodule--helper: remove unused SUPPORT_SUPER_PREFIX flags\n>> - submodule--helper: use correct display path helper\n>> - submodule--helper: don't recreate recursive prefix\n>> - submodule--helper update: use display path helper\n>> - submodule--helper tests: add missing \"display path\" coverage\n>> - Merge branch 'ab/submodule-cleanup' into gc/submodule-use-super-prefix\n>> (this branch uses ab/submodule-cleanup.)\n>>\n>> Another step to rewrite more parts of \"git submodule\" in C.\n>>\n>> Will merge to 'next'?\n>> source: <20220701021157.88858-1-chooglen@google.com>\n>\n> [...]\n>\n>> * ab/submodule-cleanup (2022-06-28) 12 commits\n>> - git-sh-setup.sh: remove \"say\" function, change last users\n>> - git-submodule.sh: use \"$quiet\", not \"$GIT_QUIET\"\n>> - submodule--helper: eliminate internal \"--update\" option\n>> - submodule--helper: understand --checkout, --merge and --rebase synonyms\n>> - submodule--helper: report \"submodule\" as our name in some \"-h\" output\n>> - submodule--helper: rename \"absorb-git-dirs\" to \"absorbgitdirs\"\n>> - submodule update: remove \"-v\" option\n>> - submodule--helper: have --require-init imply --init\n>> - git-submodule.sh: remove unused top-level \"--branch\" argument\n>> - git-submodule.sh: make the \"$cached\" variable a boolean\n>> - git-submodule.sh: remove unused $prefix variable\n>> - git-submodule.sh: remove unused sanitize_submodule_env()\n>> (this branch is used by gc/submodule-use-super-prefix.)\n>>\n>> Further preparation to turn git-submodule.sh into a builtin.\n>>\n>> Will merge to 'next'.\n>> source: \n>\n> Thanks, with regards to gc/submodule-use-super-prefix that series gets\n> my enthusiastic (if it helps):\n>\n> \tReviewed-by: Ævar Arnfjörð Bjarmason \n>\n> Likewise the ab/submodule-cleanup parent topic has Glen's:\n>\n> \tReviewed-by: Glen Choo \n>\n> Per [1] and [2]. I.e. the latest is the v4, but the Reviewed-By was\n> given to v3 in [1], and [4] notes on v4 that the one remaining concern\n> was addressed.\n>\n> Thanks both, it's great to see us this close to getting rid of another\n> *.sh built-in. I plan to submit the migration to the builtin/submodule.c\n> after these two land, possibly after some memory leak fixes to\n> builtin/submodule--helper.c.\n>\n> 1. https://lore.kernel.org/git/kl6lh74aivc6.fsf@chooglen-macbookpro.roam.corp.google.com/\n> 2. https://lore.kernel.org/git/kl6l1qv8ivcs.fsf@chooglen-macbookpro.roam.corp.google.com/\n\nYup, I think this is an accurate summary of both topics.\n\nLikewise, I'm really excited to get rid of another *.sh built-in.\nThanks!\n"},{"id":"458534","messageId":"kl6lh73urup7.fsf@chooglen-macbookpro.roam.corp.google.com","threadId":"58098","inReplyTo":"xmqqo7y85t44.fsf@gitster.g","subject":"gc/bare-repo-discovery (was Re: What's cooking in git.git (Jul 2022, #01; Fri, 1))","fromName":"Glen Choo","fromEmail":"chooglen@google.com","sentAt":"2022-07-06T17:56:36Z","receivedAt":"2022-07-06T17:56:43Z","isPatch":false,"sender":{"key":"glencbz@gmail.com","avatar":"https://avatars.githubusercontent.com/u/58092771?v=4"},"body":"Junio C Hamano writes:\n\n> * gc/bare-repo-discovery (2022-06-30) 5 commits\n> - setup.c: create `discovery.bare`\n> - safe.directory: use git_protected_config()\n> - config: learn `git_protected_config()`\n> - Documentation: define protected configuration\n> - Documentation/git-config.txt: add SCOPES section\n>\n> Introduce a discovery.barerepository configuration variable that\n> allows users to forbid discovery of bare repositories.\n>\n> Will merge to 'next'?\n> source: \n\nI'll send a reroll soon. Taylor Blau left a bunch of great\ncomments/suggestions that I intend to address (though I don't think any\nof them were meant to be blocking?)\n"},{"id":"458544","messageId":"xmqq1quxhopw.fsf@gitster.g","threadId":"58098","inReplyTo":"kl6lh73urup7.fsf@chooglen-macbookpro.roam.corp.google.com","subject":"Re: gc/bare-repo-discovery (was Re: What's cooking in git.git (Jul 2022, #01; Fri, 1))","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2022-07-06T22:15:55Z","receivedAt":"2022-07-06T22:16:04Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Glen Choo writes:\n\n> Junio C Hamano writes:\n>\n>> * gc/bare-repo-discovery (2022-06-30) 5 commits\n>> - setup.c: create `discovery.bare`\n>> - safe.directory: use git_protected_config()\n>> - config: learn `git_protected_config()`\n>> - Documentation: define protected configuration\n>> - Documentation/git-config.txt: add SCOPES section\n>>\n>> Introduce a discovery.barerepository configuration variable that\n>> allows users to forbid discovery of bare repositories.\n>>\n>> Will merge to 'next'?\n>> source: \n>\n> I'll send a reroll soon. Taylor Blau left a bunch of great\n> comments/suggestions that I intend to address (though I don't think any\n> of them were meant to be blocking?)\n\nOK. I too found Taylor's comments very helpful.\n\nThanks for a status update.\n"}]}