{"thread":{"id":"57835","subject":"What's cooking in git.git (May 2022, #01; Mon, 2)","startedAt":"2022-05-02T22:27:58Z","lastAt":"2022-05-21T21:49:22Z","messageCount":12,"participants":["Junio C Hamano","Derrick Stolee","brian m. carlson","Elijah Newren","Johannes Schindelin"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"454753","messageId":"xmqqilqnvacd.fsf@gitster.g","threadId":"57835","inReplyTo":null,"subject":"What's cooking in git.git (May 2022, #01; Mon, 2)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2022-05-02T22:27:46Z","receivedAt":"2022-05-02T22:27:58Z","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\nIt is unfortunate that the tree is not really open to new\ndevelopment yet, even after 10 working days since the last release,\nwhich unfortunately turned out to be a release with unusually high\nnumber of regressions. We have dealt with a few of them on the\n'master' front already, there are a few that look simple and obvious\nthat should graduate before 2.36.1, but there are a few more whose\nfixes have not been adequately reviewed or agreed upon.\n\n - hooks no longer is connected directly to the standard output.\n cf. \n\n - \"sudo make install\" does not work out-of-the-box.\n cf. <4ef9287b-6260-9538-7c89-cffb611520ee@maurel.de>\n\nUntil we clear them, we cannot really start merging non-fix commits\nthat are otherwise good ones down to 'next'.\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* ep/equals-null-cocci (2022-05-02) 2 commits\n - tree-wide: apply equals-null.cocci\n - Merge branch 'ep/maint-equals-null-cocci' into ep/equals-null-cocci\n (this branch uses ep/maint-equals-null-cocci.)\n\n Merges up ep/maint-equals-null-cocci to the current codebase.\n\n Will merge to 'next'?\n\n\n* ep/maint-equals-null-cocci (2022-05-02) 4 commits\n - tree-wide: apply equals-null.cocci\n - Merge branch 'ep/maint-equals-null-cocci' for maint-2.35\n - tree-wide: apply equals-null.cocci\n - contrib/coccinnelle: add equals-null.cocci\n (this branch is used by ep/equals-null-cocci.)\n\n Introduce and apply coccinelle rule to discourage an explicit\n comparison between a pointer and NULL, and applies the clean-up to\n the maintenance track.\n\n Will merge to 'next'?\n\n\n* jc/clone-remote-name-leak-fix (2022-04-30) 1 commit\n - clone: plug a miniscule leak\n\n \"git clone --origin X\" leaked piece of memory that held value read\n from the clone.defaultRemoteName configuration variable, which has\n been plugged.\n\n Will merge to 'next'.\n source: \n\n\n* jc/cocci-xstrdup-or-null-fix (2022-04-30) 1 commit\n - cocci: drop bogus xstrdup_or_null() rule\n\n Get rid of a bogus and over-eager coccinelle rule.\n\n Will merge to 'next'.\n source: \n\n\n* js/wait-or-whine-can-fail (2022-04-28) 1 commit\n - run-command: don't spam trace2_child_exit()\n\n We used to log an error return from wait_or_whine() as process\n termination of the waited child, which was incorrect.\n\n Needs clarifying \"in rare cases\".\n source: <4616d09ffa632bd2c9e308a713c4bdf2a1328c3c.1651179450.git.steadmon@google.com>\n\n\n* jc/show-pathspec-fix (2022-04-29) 1 commit\n - 2.36 show regression fix\n\n \"git show ... -- \" lost the pathspec\n when showing the second and subsequent commits, which has been\n corrected.\n\n Will merge to 'next' and then to 'master'.\n source: \n\n\n* rs/format-patch-pathspec-fix (2022-04-30) 1 commit\n - 2.36 format-patch regression fix\n\n \"git format-patch -- \" lost the pathspec when\n showing the second and subsequent commits, which has been\n corrected.\n\n Will merge to 'next' and then to 'master'.\n source: \n\n\n* rs/fast-export-pathspec-fix (2022-04-30) 1 commit\n - 2.36 fast-export regression fix\n\n \"git fast-export -- \" lost the pathspec when showing the\n second and subsequent commits, which has been corrected.\n\n Will merge to 'next' and then to 'master'.\n source: <2c988c7b-0efe-4222-4a43-8124fe1a9da6@web.de>\n\n\n--------------------------------------------------\n[Stalled]\n\n* ab/commit-plug-leaks (2022-02-16) 2 commits\n - commit: use strbuf_release() instead of UNLEAK()\n - commit: fix \"author_ident\" leak\n\n Leakfixes in the top-level called-once function.\n\n Expecting a reroll.\n I think UNLEAK->strbuf_release() is a regression.\n source: \n\n\n* dl/prompt-pick-fix (2022-03-25) 1 commit\n . git-prompt: fix sequencer/todo detection\n\n Fix shell prompt script (in contrib/) for those who set\n rebase.abbreviateCommands; we failed to recognize that we were in a\n multi-step cherry-pick session.\n\n Is this even needed? How?\n cf. \n source: <20220325145301.3370-1-danny0838@gmail.com>\n\n\n* es/superproject-aware-submodules (2022-03-09) 3 commits\n . rev-parse: short-circuit superproject worktree when config unset\n . introduce submodule.hasSuperproject record\n . t7400-submodule-basic: modernize inspect() helper\n\n A configuration variable in a repository tells if it is (or is not)\n a submodule of a superproject.\n\n Expecting a reroll.\n cf. \n source: <20220310004423.2627181-1-emilyshaffer@google.com>\n\n\n* cw/remote-object-info (2022-03-30) 5 commits\n . fixup! transfer.advertiseObjectInfo: add object-info config\n . fixup! object-info: add option for retrieving object info\n . object-info: add option for retrieving object info\n . transfer.advertiseObjectInfo: add object-info config\n . fetch-pack: refactor packet writing and fetch options\n\n Attempt to add a client component to talk with object-info\n endpoint.\n\n Expecting a reroll.\n source: <20220328191112.3092139-1-calvinwan@google.com>\n\n--------------------------------------------------\n[Cooking]\n\n* ds/sparse-colon-path (2022-04-26) 5 commits\n - rev-parse: integrate with sparse index\n - object-name: diagnose trees in index properly\n - object-name: reject trees found in the index\n - show: integrate with the sparse index\n - t1092: add compatibility tests for 'git show'\n\n \"git show :\" learned to work better with the sparse-index\n feature.\n\n Will merge to 'next'?\n source: \n\n\n* cb/path-owner-check-with-sudo (2022-04-28) 3 commits\n - t: add tests for safe.directory when running with sudo\n - Documentation: explain how safe.directory works when running under sudo\n - git-compat-util: avoid failing dir ownership checks if running privileged\n\n With a recent update to refuse access to repositories of other\n people by default, \"sudo make install\" and \"sudo git describe\"\n stopped working. This series intends to loosen it while keeping\n the safety.\n source: <20220428105852.94449-1-carenas@gmail.com>\n\n\n* pb/ggg-in-mfc-doc (2022-04-27) 3 commits\n - MyFirstContribution: drop PR description for GGG single-patch contributions\n - MyFirstContribution: also explain cover letter in GitGitGadget section\n - MyFirstContribution: move cover letter description to a separate file\n\n Documentation update.\n\n Expecting a reroll.\n cf. <706e34fb-fb0c-e86d-5d0f-0bd052566b3a@gmail.com>\n source: \n\n\n* sg/safe-directory-tests-and-docs (2022-04-27) 3 commits\n - safe.directory: document and check that it's ignored in the environment\n - t0033-safe-directory: check when 'safe.directory' is ignored\n - t0033-safe-directory: check the error message without matching the trash dir\n\n New tests for the safe.directory mechanism.\n\n Will merge to 'next'.\n source: <20220427170649.4949-1-szeder.dev@gmail.com>\n\n\n* ah/rebase-keep-base-fix (2022-04-21) 1 commit\n - rebase: use correct base for --keep-base when a branch is given\n\n \"git rebase --keep-base \" computed the\n commit to rebase onto incorrectly, which has been corrected.\n\n Will merge to 'next'.\n source: <20220421044233.894255-1-alexhenrie24@gmail.com>\n\n\n* cg/tools-for-git-doc (2022-04-21) 1 commit\n - Documentation/ToolsForGit.txt: Tools for developing Git\n\n A new doc that lists tips for tools to work with Git's codebase.\n\n Thoughts?\n source: <20220421084515.21236-2-cogoni.guillaume@gmail.com>\n\n\n* ea/rebase-code-simplify (2022-04-20) 1 commit\n - rebase: simplify an assignment of options.type in cmd_rebase\n\n Code clean-up.\n\n Will merge to 'next'.\n source: <20220418172721.215898-1-eantoranz@gmail.com>\n\n\n* kt/commit-graph-plug-fp-leak-on-error (2022-04-20) 1 commit\n - commit-graph: close file before returning NULL\n\n Fix a leak of FILE * in an error codepath.\n\n Will merge to 'next'.\n source: \n\n\n* ab/valgrind-fixes (2022-04-21) 4 commits\n - object-file: fix a unpack_loose_header() regression in 3b6a8db3b03\n - commit-graph.c: don't assume that stat() succeeds\n - log test: skip a failing mkstemp() test under valgrind\n - tests: make RUNTIME_PREFIX compatible with --valgrind\n\n A bit of test framework fixes with a few fixes to issues found by\n valgrind.\n\n Expecting a reroll.\n source: \n\n\n* cb/ci-make-p4-optional (2022-04-25) 6 commits\n - fixup! ci: make failure to find perforce more user friendly\n - fixup! CI: use https, not http to download binaries from perforce.com\n - CI: use https, not http to download binaries from perforce.com\n - ci: reintroduce prevention from perforce being quarantined in macOS\n - ci: avoid brew for installing perforce\n - ci: make failure to find perforce more user friendly\n\n macOS CI jobs have been occasionally flaky due to tentative version\n skew between perforce and the homebrew packager. Instead of\n failing the whole CI job, just let it skip the p4 tests when this\n happens.\n\n Expecting a reroll.\n source: <20220423142559.32507-1-carenas@gmail.com>\n source: <0220423142559.32507-5-carenas@gmail.com>\n\n\n* ds/midx-normalize-pathname-before-comparison (2022-04-25) 3 commits\n - cache: use const char * for get_object_directory()\n - multi-pack-index: use --object-dir real path\n - midx: use real paths in lookup_multi_pack_index()\n\n The path taken by \"git multi-pack-index\" command from the end user\n was compared with path internally prepared by the tool withut first\n normalizing, which lead to duplicated paths not being noticed,\n which has been corrected.\n\n Will merge to 'next'.\n source: \n\n\n* jc/show-branch-g-current (2022-04-21) 1 commit\n - show-branch: -g and --current are incompatible\n\n The \"--current\" option of \"git show-branch\" should have been made\n incompatible with the \"--reflog\" mode, but this was not enforced,\n which has been corrected.\n source: \n\n\n* rs/external-diff-tempfile (2022-04-20) 2 commits\n - diff: use mks_tempfile_dt()\n - tempfile: add mks_tempfile_dt()\n\n The temporary files fed to external diff command are now generated\n inside a new temporary directory under the same basename.\n\n Will merge to 'next'.\n source: <2ad1dd80-e79f-7304-219c-db24bb269c4d@web.de>\n\n\n* ab/cc-package-fixes (2022-04-22) 1 commit\n - CI: select CC based on CC_PACKAGE (again)\n\n Correct choices of C compilers used in various CI jobs.\n\n Will merge to 'next'.\n source: \n\n\n* ab/hooks-regression-fix (2022-04-21) 6 commits\n - hook API: fix v2.36.0 regression: hooks should be connected to a TTY\n - hook API: don't redundantly re-set \"no_stdin\" and \"stdout_to_stderr\"\n - hook tests: fix redirection logic error in 96e7225b310\n - run-command: add an \"ungroup\" option to run_process_parallel()\n - run-command tests: test stdout of run_command_parallel()\n - run-command API: replace run_processes_parallel_tr2() with opts struct\n\n In Git 2.36 we revamped the way how hooks are invoked. One change\n that is end-user visible is that the output of a hook is no longer\n directly connected to the standard output of \"git\" that spawns the\n hook, which was noticed post release. This is getting corrected.\n\n Expecting a reroll.\n source: \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 Thoughts?\n source: <20220422083629.1404989-1-hi@alyssa.is>\n\n\n* mv/log-since-as-filter (2022-04-23) 1 commit\n - log: \"--since-as-filter\" option is a non-terminating \"--since\" variant\n\n \"git log --since=X\" will stop traversal upon seeing a commit that\n is older than X, but there may be commits behind it that is younger\n than X when the commit was created with a faulty clock. A new\n option is added to keep digging without stopping, and instead\n filter out commits with timestamp older than X.\n\n Will merge to 'next'.\n source: \n\n\n* ds/do-not-call-bug-on-bad-refs (2022-04-25) 1 commit\n - clone: die() instead of BUG() on bad refs\n\n Code clean-up.\n\n Will merge to 'next'.\n source: \n\n\n* vd/sparse-stash (2022-04-27) 7 commits\n - unpack-trees: preserve index sparsity\n - stash: merge applied stash with merge-ort\n - merge-recursive: add merge function arg to 'merge_recursive_generic'\n - read-cache: set sparsity when index is new\n - sparse-index: expose 'is_sparse_index_allowed()'\n - stash: integrate with sparse index\n - stash: expand sparse-checkout compatibility testing\n\n Teach \"git stash\" to work better with sparse index entries.\n source: \n\n\n* cm/reftable-0-length-memset (2022-04-15) 1 commit\n - reftable: avoid undefined behaviour breaking t0032\n\n Code clean-up.\n\n Will merge to 'next'.\n source: <20220415083058.29495-1-carenas@gmail.com>\n\n\n* sa/t1011-use-helpers (2022-04-12) 1 commit\n - t1011: replace test -f with test_path_is_file\n\n A GSoC practice.\n\n Will merge to 'next'.\n source: <20220412203722.10484-1-siddharthasthana31@gmail.com>\n\n\n* ep/coverage-report-wants-test-to-have-run (2022-04-13) 1 commit\n - Makefile: add a prerequisite to the coverage-report target\n\n source: <20220414022513.31465-1-gitter.spiros@gmail.com>\n\n\n* pw/test-malloc-with-sanitize-address (2022-04-11) 1 commit\n - tests: make SANITIZE=address imply TEST_NO_MALLOC_CHECK\n\n Avoid problems from interaction between malloc_check and address\n sanitizer.\n\n Will merge to 'next'.\n source: \n\n\n* rs/t7812-pcre2-ws-bug-test (2022-04-11) 1 commit\n - t7812: test PCRE2 whitespace bug\n\n A test to ensure workaround for an earlier pcre2 bug does work.\n\n Will merge to 'next'.\n source: <3a49649d-8ff9-e5a7-e3fd-33fee5068ae8@web.de>\n\n\n* ah/convert-warning-message (2022-04-08) 1 commit\n - convert: clarify line ending conversion warning\n\n Update a few end-user facing messages around eol conversion.\n\n Will merge to 'next'.\n source: <20220408044154.9947-1-alexhenrie24@gmail.com>\n\n\n* cg/vscode-with-gdb (2022-04-08) 1 commit\n - contrib/vscode/: debugging with VS Code and gdb\n\n VS code configuration updates.\n\n Will merge to 'next'.\n source: <20220407204001.112287-2-cogoni.guillaume@gmail.com>\n\n\n* gf/unused-includes (2022-04-06) 2 commits\n - apply.c: remove unnecessary include\n - serve.c: remove unnecessary include\n\n Remove unused includes.\n\n Will merge to 'next'.\n source: <20220331194436.58005-1-garrit@slashdev.space>\n\n\n* km/t3501-use-test-helpers (2022-04-06) 1 commit\n - t3501: remove test -f and stop ignoring git exit code\n\n Test script updates.\n\n Will merge to 'next'.\n source: <20220405134742.17526-2-khalid.masum.92@gmail.com>\n\n\n* pb/submodule-recurse-mode-enum (2022-04-06) 1 commit\n - submodule.h: use a named enum for RECURSE_SUBMODULES_*\n\n Small code clean-up.\n\n Will merge to 'next'.\n source: \n\n\n* rs/commit-summary-wo-break-rewrite (2022-04-06) 1 commit\n - commit, sequencer: turn off break_opt for commit summary\n\n The commit summary shown after making a commit is matched to what\n is given in \"git status\" not to use the break-rewrite heuristics.\n\n Will merge to 'next'.\n source: \n\n\n* tk/p4-utf8-bom (2022-04-06) 1 commit\n - git-p4: preserve utf8 BOM when importing from p4 to git\n\n \"git p4\" update.\n\n Will merge to 'next'.\n source: \n\n\n* tk/p4-with-explicity-sync (2022-04-06) 1 commit\n - git-p4: support explicit sync of arbitrary existing git-p4 refs\n\n \"git p4\" update.\n\n Will merge to 'next'.\n source: \n\n\n* ab/env-array (2022-04-06) 3 commits\n - run-command API users: use \"env\" not \"env_array\" in comments & names\n - run-command API: rename \"env_array\" to \"env\"\n - cocci: add a rename of \"struct child_process\"'s \"env_array\" to \"env\"\n\n Rename .env_array member to .env in the child_process structure.\n\n On hold.\n source: \n\n\n* ab/misc-cleanup (2022-04-01) 6 commits\n (merged to 'next' on 2022-04-04 at c5fb674865)\n + alloc.[ch]: remove alloc_report() function\n + object-store.h: remove unused has_sha1_file*()\n + pack-bitmap-write: remove unused bitmap_reset() function\n + xdiff/xmacros.h: remove unused XDL_PTRFREE\n + configure.ac: remove USE_PIC comment\n + run-command.h: remove always unused \"clean_on_exit_handler_cbdata\"\n\n Code clean-up.\n\n Will cook in 'next'.\n source: \n\n\n* gf/shorthand-version-and-help (2022-03-31) 1 commit\n - cli: add -v and -h shorthands\n\n \"git -v\" and \"git -h\" are now understood as \"git --version\" and\n \"git --help\".\n\n Will merge to 'next'.\n source: <20220331212709.36036-1-garrit@slashdev.space>\n\n\n* ea/progress-partial-blame (2022-04-06) 1 commit\n (merged to 'next' on 2022-04-07 at 7df8392d71)\n + blame: report correct number of lines in progress when using ranges\n\n The progress meter of \"git blame\" was showing incorrect numbers\n when processing only parts of the file.\n\n Will cook in 'next'.\n source: <20220406181320.16911-1-eantoranz@gmail.com>\n\n\n* ab/plug-leak-in-revisions (2022-04-13) 28 commits\n - revisions API: add a TODO for diff_free(&revs->diffopt)\n - revisions API: have release_revisions() release \"topo_walk_info\"\n - revisions API: have release_revisions() release \"date_mode\"\n - revisions API: call diff_free(&revs->pruning) in revisions_release()\n - revisions API: release \"reflog_info\" in release revisions()\n - revisions API: clear \"boundary_commits\" in release_revisions()\n - revisions API: have release_revisions() release \"prune_data\"\n - revisions API: have release_revisions() release \"grep_filter\"\n - revisions API: have release_revisions() release \"filter\"\n - revisions API: have release_revisions() release \"cmdline\"\n - revisions API: have release_revisions() release \"mailmap\"\n - revisions API: have release_revisions() release \"commits\"\n - revisions API users: use release_revisions() for \"prune_data\" users\n - revisions API users: use release_revisions() with UNLEAK()\n - revisions API users: use release_revisions() in builtin/log.c\n - revisions API users: use release_revisions() in http-push.c\n - revisions API users: add \"goto cleanup\" for release_revisions()\n - stash: always have the owner of \"stash_info\" free it\n - revisions API users: use release_revisions() needing REV_INFO_INIT\n - revision.[ch]: document and move code declared around \"init\"\n - revisions API users: add straightforward release_revisions()\n - revision.[ch]: provide and start using a release_revisions()\n - cocci: add and apply free_commit_list() rules\n - format-patch: don't leak \"extra_headers\" or \"ref_message_ids\"\n - string_list API users: use string_list_init_{no,}dup\n - blame: use \"goto cleanup\" for cleanup_scoreboard()\n - t/helper/test-fast-rebase.c: don't leak \"struct strbuf\"\n - Merge branch 'ds/partial-bundle-more' into ab/plug-leak-in-revisions\n\n Plug the memory leaks from the trickiest API of all, the revision\n walker.\n\n On hold.\n source: \n\n\n* ab/ci-github-workflow-markup (2022-04-21) 11 commits\n . ci: call `finalize_test_case_output` a little later\n . ci: use `--github-workflow-markup` in the GitHub workflow\n . ci: optionally mark up output in the GitHub workflow\n . test(junit): avoid line feeds in XML attributes\n . tests: refactor --write-junit-xml code\n . ci: make it easier to find failed tests' logs in the GitHub workflow\n . CI: stop setting FAILED_TEST_ARTIFACTS N times\n . CI: don't include \"test-results/\" in ci/print-test-failures.sh output\n . CI: add --exit-code to ci/print-test-failures.sh\n . CI: don't \"cd\" in ci/print-test-failures.sh\n . Merge branch 'ab/ci-setup-simplify' into ab/ci-github-workflow-markup\n (this branch uses ab/ci-setup-simplify.)\n\n Build a moral equivalent of js/ci-github-workflow-markup on top of\n ab/ci-setup-simplify.\n source: \n\n\n* ab/ci-setup-simplify (2022-04-21) 29 commits\n . CI: make it easy to use ci/*.sh outside of CI\n . CI: don't use \"set -x\" in \"ci/lib.sh\" output\n . CI: set PYTHON_PATH setting for osx-{clang,gcc} into \"$jobname\" case\n . CI: set SANITIZE=leak in MAKEFLAGS directly\n . CI: set CC in MAKEFLAGS directly, don't add it to the environment\n . CI: add more variables to MAKEFLAGS, except under vs-build\n . CI: narrow down variable definitions in --build and --test\n . CI: only invoke ci/lib.sh as \"steps\" in main.yml\n . CI: pre-select test slice in Windows & VS tests\n . ci/run-test-slice.sh: replace shelling out with \"echo\"\n . CI: move \"env\" definitions into ci/lib.sh\n . CI: combine ci/install{,-docker}-dependencies.sh\n . CI: split up and reduce \"ci/test-documentation.sh\"\n . CI: invoke \"make artifacts-tar\" directly in windows-build\n . CI: check ignored unignored build artifacts in \"win[+VS] build\" too\n . CI: make ci/{lib,install-dependencies}.sh POSIX-compatible\n . CI: remove \"run-build-and-tests.sh\", run \"make [test]\" directly\n . CI: export variables via a wrapper\n . CI: consistently use \"export\" in ci/lib.sh\n . CI: move p4 and git-lfs variables to ci/install-dependencies.sh\n . CI: have \"static-analysis\" run \"check-builtins\", not \"documentation\"\n . CI: have \"static-analysis\" run a \"make ci-static-analysis\" target\n . CI: don't have \"git grep\" invoke a pager in tree content check\n . CI/lib.sh: stop adding leading whitespace to $MAKEFLAGS\n . CI: remove unused Azure ci/* code\n . CI: remove dead \"tree skipping\" code\n . CI: remove more dead Travis CI support\n . CI: make \"$jobname\" explicit, remove fallback\n . CI: run \"set -ex\" early in ci/lib.sh\n (this branch is used by ab/ci-github-workflow-markup.)\n\n Drive more actions done in CI via the Makefile instead of shell\n commands sprinkled in .github/workflows/main.yml\n source: \n\n\n* kf/p4-multiple-remotes (2022-03-21) 1 commit\n - git-p4: fix issue with multiple perforce remotes\n\n \"git p4\" update.\n\n Will merge to 'next'?\n source: \n\n\n* fr/vimdiff-layout (2022-04-03) 4 commits\n (merged to 'next' on 2022-04-04 at 5d1c8197d0)\n + mergetools: add description to all diff/merge tools\n + vimdiff: add tool documentation\n + vimdiff: integrate layout tests in the unit tests framework ('t' folder)\n + vimdiff: new implementation with layout support\n\n Reimplement \"vimdiff[123]\" mergetool drivers with a more generic\n layout mechanism.\n\n Will cook in 'next'.\n source: <20220330191909.294610-1-greenfoo@u92.eu>\n\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 Will merge to 'next'?\n source: <20220407215352.3491567-1-sandals@crustytoothpaste.net>\n\n\n* ns/batch-fsync (2022-04-06) 13 commits\n - core.fsyncmethod: performance tests for batch mode\n - t/perf: add iteration setup mechanism to perf-lib\n - core.fsyncmethod: tests for batch mode\n - test-lib-functions: add parsing helpers for ls-files and ls-tree\n - core.fsync: use batch mode and sync loose objects by default on Windows\n - unpack-objects: use the bulk-checkin infrastructure\n - update-index: use the bulk-checkin infrastructure\n - builtin/add: add ODB transaction around add_files_to_cache\n - cache-tree: use ODB transaction around writing a tree\n - core.fsyncmethod: batched disk flushes for loose-objects\n - bulk-checkin: rebrand plug/unplug APIs as 'odb transactions'\n - bulk-checkin: rename 'state' variable and separate 'plugged' boolean\n - Merge branch 'ns/core-fsyncmethod' into ns/batch-fsync\n\n Introduce a filesystem-dependent mechanism to optimize the way the\n bits for many loose object files are ensured to hit the disk\n platter.\n\n Will merge to 'next'?\n source: \n\n\n* en/sparse-cone-becomes-default (2022-04-21) 9 commits\n - Documentation: some sparsity wording clarifications\n - git-sparse-checkout.txt: mark non-cone mode as deprecated\n - git-sparse-checkout.txt: flesh out pattern set sections a bit\n - git-sparse-checkout.txt: add a new EXAMPLES section\n - git-sparse-checkout.txt: shuffle some sections and mark as internal\n - git-sparse-checkout.txt: update docs for deprecation of 'init'\n - git-sparse-checkout.txt: wording updates for the cone mode default\n - sparse-checkout: make --cone the default\n - tests: stop assuming --no-cone is the default mode for sparse-checkout\n\n Deprecate non-cone mode of the sparse-checkout feature.\n\n Will merge to 'next'?\n source: \n\n\n* tb/cruft-packs (2022-03-02) 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 A mechanism to pack unreachable objects into a \"cruft pack\",\n instead of ejecting them into loose form to be reclaimed later, has\n been introduced.\n\n Waiting for discussion to settle.\n cf. \n source: \n\n\n* js/ci-github-workflow-markup (2022-03-01) 9 commits\n - ci: call `finalize_test_case_output` a little later\n - ci: use `--github-workflow-markup` in the GitHub workflow\n - ci: optionally mark up output in the GitHub workflow\n - test(junit): avoid line feeds in XML attributes\n - tests: refactor --write-junit-xml code\n - ci/run-build-and-tests: add some structure to the GitHub workflow output\n - ci: make it easier to find failed tests' logs in the GitHub workflow\n - ci/run-build-and-tests: take a more high-level view\n - ci: fix code style\n\n Update the GitHub workflow support to make it quicker to get to the\n failing test.\n\n Expecting a reroll.\n cf. <52382f7c-5b6a-63b6-2eb2-26c12f31f529@gmail.com>\n source: \n\n\n* et/xdiff-indirection (2022-02-17) 1 commit\n - xdiff: provide indirection to git functions\n\n Insert a layer of preprocessor macros for common functions in xdiff\n codebase.\n\n Expecting a reroll.\n cf. \n source: <20220217225408.GB7@edef91d97c94>\n\n\n* tk/simple-autosetupmerge (2022-04-29) 3 commits\n - push: new config option \"push.autoSetupRemote\" supports \"simple\" push\n - push: default to single remote even when not named origin\n - branch: new autosetupmerge option 'simple' for matching branches\n\n \"git -c branch.autosetupmerge=simple branch $A $B\" will set the $B\n as $A's upstream only when $A and $B shares the same name, and \"git\n -c push.default=simple\" on branch $A would push to update the\n branch $A at the remote $B came from. Also more places use the\n sole remote, if exists, before defaulting to 'origin'.\n\n source: \n\n\n* tk/untracked-cache-with-uall (2022-04-01) 2 commits\n (merged to 'next' on 2022-04-04 at 2e11f1ac0c)\n + untracked-cache: support '--untracked-files=all' if configured\n + untracked-cache: test untracked-cache-bypassing behavior with -uall\n\n The performance of the \"untracked cache\" feature has been improved\n when \"--untracked-files=\" and \"status.showUntrackedFiles\"\n are combined.\n\n Will cook in 'next'.\n source: \n\n\n* jh/builtin-fsmonitor-part3 (2022-04-22) 29 commits\n - fsmonitor--daemon: allow --super-prefix argument\n - t7527: test Unicode NFC/NFD handling on MacOS\n - t/lib-unicode-nfc-nfd: helper prereqs for testing unicode nfc/nfd\n - fsmonitor: on macOS also emit NFC spelling for NFD pathname\n - t7527: test FSMonitor on case insensitive+preserving file system\n - fsmonitor: never set CE_FSMONITOR_VALID on submodules\n - t/perf/p7527: add perf test for builtin FSMonitor\n - t7527: FSMonitor tests for directory moves\n - fsmonitor: optimize processing of directory events\n - fsm-listen-darwin: shutdown daemon if worktree root is moved/renamed\n - fsm-health-win32: force shutdown daemon if worktree root moves\n - fsm-health-win32: add polling framework to monitor daemon health\n - fsmonitor--daemon: stub in health thread\n - fsmonitor--daemon: rename listener thread related variables\n - fsmonitor--daemon: prepare for adding health thread\n - fsmonitor--daemon: cd out of worktree root\n - fsm-listen-darwin: ignore FSEvents caused by xattr changes on macOS\n - unpack-trees: initialize fsmonitor_has_run_once in o->result\n - fsmonitor-settings: NTFS and FAT32 on MacOS are incompatible\n - fsmonitor-settings: remote repos on Windows are incompatible\n - fsmonitor-settings: remote repos on macOS are incompatible\n - fsmonitor-settings: stub in macOS-specific incompatibility checking\n - fsmonitor-settings: VFS for Git virtual repos are incompatible\n - fsmonitor-settings: stub in Win32-specific incompatibility checking\n - fsmonitor-settings: bare repos are incompatible with FSMonitor\n - t/helper/fsmonitor-client: create stress test\n - t7527: test FSMonitor on repos with Unicode root paths\n - fsm-listen-win32: handle shortnames\n - Merge branch 'jh/builtin-fsmonitor-part2' into jh/builtin-fsmonitor-part3\n\n More fsmonitor--daemon.\n source: \n\n\n* js/bisect-in-c (2022-02-23) 14 commits\n - bisect: no longer try to clean up left-over `.git/head-name` files\n - bisect: remove Cogito-related code\n - bisect: turn `git bisect` into a full built-in\n - bisect: move even the option parsing to `bisect--helper`\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\n Final bits of \"git bisect.sh\" have been rewritten in C.\n\n Expecting a reroll.\n cf. <220225.86ilt27uln.gmgdl@evledraar.gmail.com>\n source: \n\n\n* js/scalar-diagnose (2022-02-06) 6 commits\n - scalar: teach `diagnose` to gather loose objects information\n - scalar: teach `diagnose` to gather packfile info\n - scalar diagnose: include disk space information\n - scalar: add `diagnose`\n - scalar: validate the optional enlistment argument\n - archive: optionally add \"virtual\" files\n\n Implementation of \"scalar diagnose\" subcommand.\n\n On hold.\n cf. \n source: \n\n\n* en/merge-tree (2022-02-23) 13 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-tree: provide easy access to `ls-files -u` style info\n - merge-tree: provide a list of which files have conflicts\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 - Merge branch 'en/remerge-diff' into en/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 On hold.\n cf. \n source: \n\n\n* jh/p4-various-fixups (2022-04-01) 22 commits\n (merged to 'next' on 2022-04-04 at 251b14976f)\n + git-p4: sort imports\n + git-p4: seperate multiple statements onto seperate lines\n + git-p4: move inline comments to line above\n + git-p4: only seperate code blocks by a single empty line\n + git-p4: compare to singletons with \"is\" and \"is not\"\n + git-p4: normalize indentation of lines in conditionals\n + git-p4: ensure there is a single space around all operators\n + git-p4: ensure every comment has a single #\n + git-p4: remove spaces between dictionary keys and colons\n + git-p4: remove redundant backslash-continuations inside brackets\n + git-p4: remove extraneous spaces before function arguments\n + git-p4: place a single space after every comma\n + git-p4: removed brackets when assigning multiple return values\n + git-p4: remove spaces around default arguments\n + git-p4: remove padding from lists, tuples and function arguments\n + git-p4: sort and de-duplcate pylint disable list\n + git-p4: remove commented code\n + git-p4: convert descriptive class and function comments into docstrings\n + git-p4: improve consistency of docstring formatting\n + git-p4: indent with 4-spaces\n + git-p4: remove unneeded semicolons from statements\n + git-p4: add blank lines between functions and class definitions\n\n Various cleanups to \"git p4\".\n\n Will cook in 'next'.\n source: <20220401142504.58995-1-jholdsworth@nvidia.com>\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\n What's the status of the \"known breakage\"?\n Are we ready to switch if we wanted to?\n There are known breakages on macOS.\n cf. \n source: \n"},{"id":"454800","messageId":"0facc01f-ee36-333a-eb25-9c98d616700e@github.com","threadId":"57835","inReplyTo":"xmqqilqnvacd.fsf@gitster.g","subject":"Re: What's cooking in git.git (May 2022, #01; Mon, 2)","fromName":"Derrick Stolee","fromEmail":"derrickstolee@github.com","sentAt":"2022-05-03T21:19:57Z","receivedAt":"2022-05-03T21:20:03Z","isPatch":false,"sender":{"key":"stolee@gmail.com","avatar":"https://avatars.githubusercontent.com/u/570044?v=4"},"body":"On 5/2/2022 6:27 PM, Junio C Hamano wrote:\n\n> * ds/sparse-colon-path (2022-04-26) 5 commits\n> - rev-parse: integrate with sparse index\n> - object-name: diagnose trees in index properly\n> - object-name: reject trees found in the index\n> - show: integrate with the sparse index\n> - t1092: add compatibility tests for 'git show'\n> \n> \"git show :\" learned to work better with the sparse-index\n> feature.\n> \n> Will merge to 'next'?\n> source: \n\nI think this is stable and ready to go. Unless anyone else wanted to\ntake a stab at reviewing it. I think we've got good eyes on it already.\n\n> * en/sparse-cone-becomes-default (2022-04-21) 9 commits\n> - Documentation: some sparsity wording clarifications\n> - git-sparse-checkout.txt: mark non-cone mode as deprecated\n> - git-sparse-checkout.txt: flesh out pattern set sections a bit\n> - git-sparse-checkout.txt: add a new EXAMPLES section\n> - git-sparse-checkout.txt: shuffle some sections and mark as internal\n> - git-sparse-checkout.txt: update docs for deprecation of 'init'\n> - git-sparse-checkout.txt: wording updates for the cone mode default\n> - sparse-checkout: make --cone the default\n> - tests: stop assuming --no-cone is the default mode for sparse-checkout\n> \n> Deprecate non-cone mode of the sparse-checkout feature.\n> \n> Will merge to 'next'?\n> source: \n\nThis has been quite stable and I'd like to see it merge early in the\nrelease cycle in case that catches anything during local testing.\n\nThanks,\n-Stolee\n"},{"id":"454829","messageId":"xmqqy1zhmftk.fsf@gitster.g","threadId":"57835","inReplyTo":"0facc01f-ee36-333a-eb25-9c98d616700e@github.com","subject":"Re: What's cooking in git.git (May 2022, #01; Mon, 2)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2022-05-04T16:18:47Z","receivedAt":"2022-05-04T16:18:58Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Derrick Stolee writes:\n\n> On 5/2/2022 6:27 PM, Junio C Hamano wrote:\n>\n>> * ds/sparse-colon-path (2022-04-26) 5 commits\n>> - rev-parse: integrate with sparse index\n>> - object-name: diagnose trees in index properly\n>> - object-name: reject trees found in the index\n>> - show: integrate with the sparse index\n>> - t1092: add compatibility tests for 'git show'\n>> \n>> \"git show :\" learned to work better with the sparse-index\n>> feature.\n>> \n>> Will merge to 'next'?\n>> source: \n>\n> I think this is stable and ready to go. Unless anyone else wanted to\n> take a stab at reviewing it. I think we've got good eyes on it already.\n\nThanks. I read over the conversions and they all looked sensible.\n\n>> * en/sparse-cone-becomes-default (2022-04-21) 9 commits\n>> - Documentation: some sparsity wording clarifications\n>> - git-sparse-checkout.txt: mark non-cone mode as deprecated\n>> - git-sparse-checkout.txt: flesh out pattern set sections a bit\n>> - git-sparse-checkout.txt: add a new EXAMPLES section\n>> - git-sparse-checkout.txt: shuffle some sections and mark as internal\n>> - git-sparse-checkout.txt: update docs for deprecation of 'init'\n>> - git-sparse-checkout.txt: wording updates for the cone mode default\n>> - sparse-checkout: make --cone the default\n>> - tests: stop assuming --no-cone is the default mode for sparse-checkout\n>> \n>> Deprecate non-cone mode of the sparse-checkout feature.\n>> \n>> Will merge to 'next'?\n>> source: \n>\n> This has been quite stable and I'd like to see it merge early in the\n> release cycle in case that catches anything during local testing.\n\nThis one I am not worried about stability all that much; I am more\nworried about breaking those who do not believe in \"cone is the\nfuture\" and managing their expectations.\n\nThanks.\n\n"},{"id":"454840","messageId":"YnL2d4Vr9Vr7W4Hj@camp.crustytoothpaste.net","threadId":"57835","inReplyTo":"xmqqilqnvacd.fsf@gitster.g","subject":"Re: What's cooking in git.git (May 2022, #01; Mon, 2)","fromName":"brian m. carlson","fromEmail":"sandals@crustytoothpaste.net","sentAt":"2022-05-04T21:56:07Z","receivedAt":"2022-05-04T21:56:21Z","isPatch":false,"sender":{"key":"sandals@crustytoothpaste.net","avatar":"https://avatars.githubusercontent.com/u/497054?v=4"},"body":"On 2022-05-02 at 22:27:46, Junio C Hamano wrote:\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> Will merge to 'next'?\n> source: <20220407215352.3491567-1-sandals@crustytoothpaste.net>\n\nI think Ævar had some requests for a v5, so I'll try to get to one\nrelatively soon.\n-- \nbrian m. carlson (he/him or they/them)\nToronto, Ontario, CA\n"},{"id":"454841","messageId":"xmqqa6bwluay.fsf@gitster.g","threadId":"57835","inReplyTo":"YnL2d4Vr9Vr7W4Hj@camp.crustytoothpaste.net","subject":"Re: What's cooking in git.git (May 2022, #01; Mon, 2)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2022-05-05T00:03:33Z","receivedAt":"2022-05-05T00:06:47Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"\"brian m. carlson\" writes:\n\n> On 2022-05-02 at 22:27:46, Junio C Hamano wrote:\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>> Will merge to 'next'?\n>> source: <20220407215352.3491567-1-sandals@crustytoothpaste.net>\n>\n> I think Ævar had some requests for a v5, so I'll try to get to one\n> relatively soon.\n\nThanks.\n"},{"id":"454862","messageId":"CABPp-BHDKRX4sW_Jjqw5j7Voas0X_xkFZgg5Jqk0TkNZOd7k1g@mail.gmail.com","threadId":"57835","inReplyTo":"xmqqy1zhmftk.fsf@gitster.g","subject":"Re: What's cooking in git.git (May 2022, #01; Mon, 2)","fromName":"Elijah Newren","fromEmail":"newren@gmail.com","sentAt":"2022-05-05T15:04:49Z","receivedAt":"2022-05-05T15:05:16Z","isPatch":false,"sender":{"key":"newren@gmail.com","avatar":"https://avatars.githubusercontent.com/u/5455730?v=4"},"body":"On Wed, May 4, 2022 at 9:18 AM Junio C Hamano wrote:\n>\n> >> * en/sparse-cone-becomes-default (2022-04-21) 9 commits\n> >> - Documentation: some sparsity wording clarifications\n> >> - git-sparse-checkout.txt: mark non-cone mode as deprecated\n> >> - git-sparse-checkout.txt: flesh out pattern set sections a bit\n> >> - git-sparse-checkout.txt: add a new EXAMPLES section\n> >> - git-sparse-checkout.txt: shuffle some sections and mark as internal\n> >> - git-sparse-checkout.txt: update docs for deprecation of 'init'\n> >> - git-sparse-checkout.txt: wording updates for the cone mode default\n> >> - sparse-checkout: make --cone the default\n> >> - tests: stop assuming --no-cone is the default mode for sparse-checkout\n> >>\n> >> Deprecate non-cone mode of the sparse-checkout feature.\n> >>\n> >> Will merge to 'next'?\n> >> source: \n> >\n> > This has been quite stable and I'd like to see it merge early in the\n> > release cycle in case that catches anything during local testing.\n>\n> This one I am not worried about stability all that much; I am more\n> worried about breaking those who do not believe in \"cone is the\n> future\" and managing their expectations.\n\nWhat path forward would you like to see?\n\nOn v2 you said, \"There are a few things that I found questionable, but\nthey were mostly documentation issues\". You brought up several points\nin v2 where my wording in the Documentation was admittedly suboptimal,\nthough I think I addressed all of those in v3.\n\nI also brought up an alternative change for consideration in the cover\nletter, though you didn't comment on it and other reviewers, while\nokay with the alternative, seemed to agree with my proposed changes in\nthe series over the alternative.\n\nAlso, it might be worth noting:\n - sparse-checkout, the capability in Git since ~2010, remains\nunmodified by these changes\n - sparse-checkout, the subcommand added a couple years ago, still\nsupports all non-cone mode behaviors it did before\n - the change here is that the sparse-checkout subcommand's default\nbecomes cone mode, along with documentation updates\n"},{"id":"454863","messageId":"720e86b3-367d-ab83-d3ba-a05cdf9bf0c2@github.com","threadId":"57835","inReplyTo":"CABPp-BHDKRX4sW_Jjqw5j7Voas0X_xkFZgg5Jqk0TkNZOd7k1g@mail.gmail.com","subject":"Re: What's cooking in git.git (May 2022, #01; Mon, 2)","fromName":"Derrick Stolee","fromEmail":"derrickstolee@github.com","sentAt":"2022-05-05T15:13:18Z","receivedAt":"2022-05-05T15:13:33Z","isPatch":false,"sender":{"key":"stolee@gmail.com","avatar":"https://avatars.githubusercontent.com/u/570044?v=4"},"body":"On 5/5/2022 11:04 AM, Elijah Newren wrote:\n> On Wed, May 4, 2022 at 9:18 AM Junio C Hamano wrote:\n>>\n>>>> * en/sparse-cone-becomes-default (2022-04-21) 9 commits\n>>>> - Documentation: some sparsity wording clarifications\n>>>> - git-sparse-checkout.txt: mark non-cone mode as deprecated\n>>>> - git-sparse-checkout.txt: flesh out pattern set sections a bit\n>>>> - git-sparse-checkout.txt: add a new EXAMPLES section\n>>>> - git-sparse-checkout.txt: shuffle some sections and mark as internal\n>>>> - git-sparse-checkout.txt: update docs for deprecation of 'init'\n>>>> - git-sparse-checkout.txt: wording updates for the cone mode default\n>>>> - sparse-checkout: make --cone the default\n>>>> - tests: stop assuming --no-cone is the default mode for sparse-checkout\n>>>>\n>>>> Deprecate non-cone mode of the sparse-checkout feature.\n>>>>\n>>>> Will merge to 'next'?\n>>>> source: \n>>>\n>>> This has been quite stable and I'd like to see it merge early in the\n>>> release cycle in case that catches anything during local testing.\n>>\n>> This one I am not worried about stability all that much; I am more\n>> worried about breaking those who do not believe in \"cone is the\n>> future\" and managing their expectations.\n> \n> What path forward would you like to see?\n> \n> On v2 you said, \"There are a few things that I found questionable, but\n> they were mostly documentation issues\". You brought up several points\n> in v2 where my wording in the Documentation was admittedly suboptimal,\n> though I think I addressed all of those in v3.\n> \n> I also brought up an alternative change for consideration in the cover\n> letter, though you didn't comment on it and other reviewers, while\n> okay with the alternative, seemed to agree with my proposed changes in\n> the series over the alternative.\n> \n> Also, it might be worth noting:\n> - sparse-checkout, the capability in Git since ~2010, remains\n> unmodified by these changes\n> - sparse-checkout, the subcommand added a couple years ago, still\n> supports all non-cone mode behaviors it did before\n> - the change here is that the sparse-checkout subcommand's default\n> becomes cone mode, along with documentation updates\n\nI think Junio's concerns are the same ones we have had that delayed\nthis switch from the start: we don't know who might be surprised by\nthis change.\n\nWe've given this enough time to collect opinions from interested\nparties that watch the list, but we need the next level of feedback\nat this point. That's why I'm advocating to merge this now and have\na long time before the release to gather feedback.\n\nThanks,\n-Stolee\n"},{"id":"454865","messageId":"xmqqmtfwezx5.fsf@gitster.g","threadId":"57835","inReplyTo":"CABPp-BHDKRX4sW_Jjqw5j7Voas0X_xkFZgg5Jqk0TkNZOd7k1g@mail.gmail.com","subject":"Re: What's cooking in git.git (May 2022, #01; Mon, 2)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2022-05-05T15:56:22Z","receivedAt":"2022-05-05T15:56:29Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Elijah Newren writes:\n\n> On v2 you said, \"There are a few things that I found questionable, but\n> they were mostly documentation issues\". You brought up several points\n> in v2 where my wording in the Documentation was admittedly suboptimal,\n> though I think I addressed all of those in v3.\n\nCorrect. The above is all assuming that \"cone is the future\" is\nwhat is shared by the population that depends on sparse-checkout. I\ndo not have strong opinions to declare that, but to help the topic\nmove forward, I was giving reviews under the condition, i.e. \"If\nthere is such a concensus, here are the things in your current\niteration that would need to be addressed\".\n\nI vaguely recall there were folks who didn't like the change of\ndefault the last time you brought this topic up. Convincing them is\nnot my job---it is yours. Mine has been to help prepare the code\nready for public consumption when that happens.\n"},{"id":"454867","messageId":"CABPp-BH8VsH+Y3UxAvZM2kua8XGRE1RyenrESeYwofcq-=kjbQ@mail.gmail.com","threadId":"57835","inReplyTo":"xmqqmtfwezx5.fsf@gitster.g","subject":"Re: What's cooking in git.git (May 2022, #01; Mon, 2)","fromName":"Elijah Newren","fromEmail":"newren@gmail.com","sentAt":"2022-05-05T16:15:31Z","receivedAt":"2022-05-05T16:15:58Z","isPatch":false,"sender":{"key":"newren@gmail.com","avatar":"https://avatars.githubusercontent.com/u/5455730?v=4"},"body":"On Thu, May 5, 2022 at 8:56 AM Junio C Hamano wrote:\n>\n> Elijah Newren writes:\n>\n> > On v2 you said, \"There are a few things that I found questionable, but\n> > they were mostly documentation issues\". You brought up several points\n> > in v2 where my wording in the Documentation was admittedly suboptimal,\n> > though I think I addressed all of those in v3.\n>\n> Correct. The above is all assuming that \"cone is the future\" is\n> what is shared by the population that depends on sparse-checkout. I\n> do not have strong opinions to declare that, but to help the topic\n> move forward, I was giving reviews under the condition, i.e. \"If\n> there is such a concensus, here are the things in your current\n> iteration that would need to be addressed\".\n\nThanks for the clarification.\n\n> I vaguely recall there were folks who didn't like the change of\n> default the last time you brought this topic up. Convincing them is\n> not my job---it is yours. Mine has been to help prepare the code\n> ready for public consumption when that happens.\n\nThat's a surprise to me; I don't remember anyone bringing that up,\never. Did I somehow miss it?? I'd be happy to talk to anyone and\nhear their concerns if they do hold such an opinion. Over the last\nyear or so both Stolee and Victoria have suggested such a change or\nsaid they wondered why I didn't include such a change with other\nsparse-checkout changes we were making, and I mentioned a few times I\nthought it'd be a good future plan and finally submitted it. To the\nbest of my memory, no one ever offered a counter opinion.\n\nAre you perhaps mixing this up with the case where people brought up\nconcerns with suggested tab-completion changes? That's the only other\nrelated series recently where folks brought up concerns with suggested\nchanges.\n"},{"id":"454874","messageId":"CABPp-BF9ftVVp7-ZZuhak456x12-H941Nj4qV7gNf71rrugGhQ@mail.gmail.com","threadId":"57835","inReplyTo":"CABPp-BH8VsH+Y3UxAvZM2kua8XGRE1RyenrESeYwofcq-=kjbQ@mail.gmail.com","subject":"Re: What's cooking in git.git (May 2022, #01; Mon, 2)","fromName":"Elijah Newren","fromEmail":"newren@gmail.com","sentAt":"2022-05-05T18:53:47Z","receivedAt":"2022-05-05T18:54:07Z","isPatch":false,"sender":{"key":"newren@gmail.com","avatar":"https://avatars.githubusercontent.com/u/5455730?v=4"},"body":"On Thu, May 5, 2022 at 9:15 AM Elijah Newren wrote:\n>\n> On Thu, May 5, 2022 at 8:56 AM Junio C Hamano wrote:\n> >\n[...]\n> > I vaguely recall there were folks who didn't like the change of\n> > default the last time you brought this topic up. Convincing them is\n> > not my job---it is yours. Mine has been to help prepare the code\n> > ready for public consumption when that happens.\n>\n> That's a surprise to me; I don't remember anyone bringing that up,\n> ever. Did I somehow miss it?? I'd be happy to talk to anyone and\n> hear their concerns if they do hold such an opinion. Over the last\n> year or so both Stolee and Victoria have suggested such a change or\n> said they wondered why I didn't include such a change with other\n> sparse-checkout changes we were making, and I mentioned a few times I\n> thought it'd be a good future plan and finally submitted it. To the\n> best of my memory, no one ever offered a counter opinion.\n>\n> Are you perhaps mixing this up with the case where people brought up\n> concerns with suggested tab-completion changes? That's the only other\n> related series recently where folks brought up concerns with suggested\n> changes.\n\nSo, I did a few searches through the list history for anyone that may\nhave commented on cone mode as default. The only comments I can find\nare Stolee, Victoria, Lessley, and I being in favor. Here's what I\nthink is a pretty good search on lore.kernel.org/git[1]:\n\n \"cone mode\" default -f:stolee -f:gitster -f:newren -f:dye -f:dennington\n\n(Where the reason for the exclusions are because Stolee, Victoria,\nLessley, and I have all spoken up in favor of the change, and you\naren't stating that you're opposed to it just that you thought someone\nelse had been. Plus those exclusions massively reduce the number of\nemails to read through, since I think we're the only active\ncontributors to the sparse-checkout subcommand and related\nfunctionality.)\n\nLooking at all those links plus related discussion, the closest thing\nI could find was\nhttps://lore.kernel.org/git/20220118222234.GB2632@szeder.dev/ (which\nwas later in the thread from [2] which was the direct link from the\nabove search). However, this wasn't an argument against cone mode as\ndefault, that thread was where SZEDER argued that tab completion\nshould not be limited to directories in non-cone mode[3]. Now, I did\nbring up making cone mode the default, but only in _response_ to that\nemail because it'd handle his example _better_ than his hand-picked\nexample of why he wanted completion to include files for non-cone\nmode. SZEDER didn't respond, which doesn't tell us much about his\nopinion on this area, but he certainly didn't push back on the\nsuggestion to make cone mode the default. And that's the closest I\ncan find to anyone stating a disagreement with this proposed change.\n\nDid I miss some other email in my searches that you remember where\nsomeone did actually have concerns with this change?\n\n\n[1] Quick link:\nhttps://lore.kernel.org/git/?q=%C2%A0%22cone+mode%22+default+-f%3Astolee+-f%3Agitster+-f%3Anewren+-f%3Adye+-f%3Adennington\n\n[2] https://lore.kernel.org/git/20220115095725.GA1738@szeder.dev/\n\n[3] Though the example he used was perhaps unfortunate since it wasn't\ntab-completable with files anyway, and any completion that would have\nbeen provided by file completion would have been wrong since it'd\ncomplete on files in the wrong directory (namely, the root directory).\n"},{"id":"454878","messageId":"xmqqh763dbay.fsf@gitster.g","threadId":"57835","inReplyTo":"CABPp-BF9ftVVp7-ZZuhak456x12-H941Nj4qV7gNf71rrugGhQ@mail.gmail.com","subject":"Re: What's cooking in git.git (May 2022, #01; Mon, 2)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2022-05-05T19:33:25Z","receivedAt":"2022-05-05T19:33:43Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Elijah Newren writes:\n\n> On Thu, May 5, 2022 at 9:15 AM Elijah Newren wrote:\n>>\n>> On Thu, May 5, 2022 at 8:56 AM Junio C Hamano wrote:\n>> >\n> [...]\n>> > I vaguely recall there were folks who didn't like the change of\n>> > default the last time you brought this topic up. Convincing them is\n>> > not my job---it is yours. Mine has been to help prepare the code\n>> > ready for public consumption when that happens.\n>>\n>> That's a surprise to me; I don't remember anyone bringing that up,\n>> ever. Did I somehow miss it?? I'd be happy to talk to anyone and\n>> hear their concerns if they do hold such an opinion. Over the last\n>> year or so both Stolee and Victoria have suggested such a change or\n>> said they wondered why I didn't include such a change with other\n>> sparse-checkout changes we were making, and I mentioned a few times I\n>> thought it'd be a good future plan and finally submitted it. To the\n>> best of my memory, no one ever offered a counter opinion.\n>>\n>> Are you perhaps mixing this up with the case where people brought up\n>> concerns with suggested tab-completion changes? That's the only other\n>> related series recently where folks brought up concerns with suggested\n>> changes.\n>\n> So, I did a few searches through the list history for anyone that may\n> have commented on cone mode as default. The only comments I can find\n> are Stolee, Victoria, Lessley, and I being in favor.\n\nOK, so I misremembered. Let's hope that the list subscriber base is\nwide enough and those who will be affected are all in favor, and see\nhow the reception by those who are not on the list goes.\n\nHopefully some organizations may have a canary channel for their\nusers to try out something based on 'next', and we will hear from\nthem, either positive or negative, about their experience.\n\n"},{"id":"455710","messageId":"nycvar.QRO.7.76.6.2205212347060.352@tvgsbejvaqbjf.bet","threadId":"57835","inReplyTo":"CABPp-BF9ftVVp7-ZZuhak456x12-H941Nj4qV7gNf71rrugGhQ@mail.gmail.com","subject":"cone mode as default for sparse-checkout?, was Re: What's cooking in git.git (May 2022, #01; Mon, 2)","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2022-05-21T21:48:57Z","receivedAt":"2022-05-21T21:49:22Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi Elijah,\n\nOn Thu, 5 May 2022, Elijah Newren wrote:\n\n> On Thu, May 5, 2022 at 9:15 AM Elijah Newren wrote:\n> >\n> > On Thu, May 5, 2022 at 8:56 AM Junio C Hamano wrote:\n> > >\n> [...]\n> > > I vaguely recall there were folks who didn't like the change of\n> > > default the last time you brought this topic up. Convincing them is\n> > > not my job---it is yours. Mine has been to help prepare the code\n> > > ready for public consumption when that happens.\n> >\n> > That's a surprise to me; I don't remember anyone bringing that up,\n> > ever. Did I somehow miss it?? I'd be happy to talk to anyone and\n> > hear their concerns if they do hold such an opinion. Over the last\n> > year or so both Stolee and Victoria have suggested such a change or\n> > said they wondered why I didn't include such a change with other\n> > sparse-checkout changes we were making, and I mentioned a few times I\n> > thought it'd be a good future plan and finally submitted it. To the\n> > best of my memory, no one ever offered a counter opinion.\n> >\n> > Are you perhaps mixing this up with the case where people brought up\n> > concerns with suggested tab-completion changes? That's the only other\n> > related series recently where folks brought up concerns with suggested\n> > changes.\n>\n> So, I did a few searches through the list history for anyone that may\n> have commented on cone mode as default.\n\nFWIW even though I am a rather heavy user of non-cone mode (the structure\nof Git for Windows' SDK does not lend itself to code mode, but e.g. for\nthe minimal SDK we use in our CI/PR build we definitely need a subset of\nthe otherwise too-clunky SDK), I am very much in favor of making cone-mode\nthe default.\n\nThanks,\nDscho\n"}]}