{"thread":{"id":"51080","subject":"What's cooking in git.git (May 2019, #02; Tue, 14)","startedAt":"2019-05-13T16:11:35Z","lastAt":"2019-05-21T22:28:25Z","messageCount":9,"participants":["Junio C Hamano","Elijah Newren","Johannes Schindelin"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"375393","messageId":"xmqqa7fqbahj.fsf@gitster-ct.c.googlers.com","threadId":"51080","inReplyTo":null,"subject":"What's cooking in git.git (May 2019, #02; Tue, 14)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2019-05-13T16:11:20Z","receivedAt":"2019-05-13T16:11:35Z","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. Commits prefixed with\n'-' are only in 'pu' (proposed updates) while commits prefixed with\n'+' are in 'next'. The ones marked with '.' do not appear in any of\nthe integration branches, but I am still holding onto them.\n\nAn early preview 2.22-rc0 has been tagged. Among the topics merged\nto 'master' there might be some that are a bit undercooked, but\nhopefully we can stabilize them quickly enough.\n\nYou can find the changes described here in the integration branches\nof the repositories listed at\n\n http://git-blame.blogspot.com/p/git-public-repositories.html\n\n--------------------------------------------------\n[Graduated to \"master\"]\n\n* cc/access-on-aix-workaround (2019-04-25) 1 commit\n (merged to 'next' on 2019-05-09 at 79b25b1954)\n + git-compat-util: work around for access(X_OK) under root\n\n Workaround for standard-compliant but less-than-useful behaviour of\n access(2) for the root user.\n\n\n* dl/no-extern-in-func-decl (2019-05-05) 3 commits\n (merged to 'next' on 2019-05-09 at d165ac4cee)\n + *.[ch]: manually align parameter lists\n + *.[ch]: remove extern from function declarations using sed\n + *.[ch]: remove extern from function declarations using spatch\n\n Mechanically and systematically drop \"extern\" from function\n declarlation.\n\n\n* dl/rev-tilde-doc-clarify (2019-05-08) 4 commits\n (merged to 'next' on 2019-05-09 at 6efd564b11)\n + revisions.txt: remove ambibuity between : and :\n + revisions.txt: mention ~ form\n + revisions.txt: mark optional rev arguments with []\n + revisions.txt: change \"rev\" to \"\"\n\n Docfix.\n\n\n* ew/repack-with-bitmaps-by-default (2019-03-18) 3 commits\n (merged to 'next' on 2019-05-09 at 4f8e8b01c8)\n + pack-objects: default to writing bitmap hash-cache\n + t5310: correctly remove bitmaps for jgit test\n + repack: enable bitmaps by default on bare repos\n\n The connectivity bitmaps are created by default in bare\n repositories now; also the pathname hash-cache is created by\n default to avoid making crappy deltas when repacking.\n cf. <87a7g2iuem.fsf@evledraar.gmail.com>\n\n\n* jc/make-dedup-ls-files-output (2019-04-22) 1 commit\n (merged to 'next' on 2019-05-09 at e3d5825003)\n + Makefile: dedup list of files obtained from ls-files\n\n A \"ls-files\" that emulates \"find\" to enumerate files in the working\n tree resulted in duplicated Makefile rules that caused the build to\n issue an unnecessary warning during a trial build after merge\n conflicts are resolved in working tree *.h files but before the\n resolved results are added to the index. This has been corrected.\n A not-so-low hanging fruit to teach ls-files to dedup either optionally\n or always has also been discussed, which probably is a good idea,\n to prevent mistakes similar to the bug this topic fixes in the future.\n\n\n* jh/trace2 (2019-05-10) 2 commits\n (merged to 'next' on 2019-05-10 at 8b4b5564af)\n + trace2: fix up a missing \"leave\" entry point\n (merged to 'next' on 2019-05-09 at e1bba8aeac)\n + trace2: fix incorrect function pointer check\n\n A few embarrassing bugfixes.\n\n\n* jh/trace2-sid-fix (2019-05-07) 11 commits\n (merged to 'next' on 2019-05-10 at 71c39675ec)\n + trace2: fixup access problem on /etc/gitconfig in read_very_early_config\n (merged to 'next' on 2019-04-25 at a5c08f1226)\n + trace2: update docs to describe system/global config settings\n + trace2: make SIDs more unique\n + trace2: clarify UTC datetime formatting\n + trace2: report peak memory usage of the process\n + trace2: use system/global config for default trace2 settings\n + config: add read_very_early_config()\n + trace2: find exec-dir before trace2 initialization\n + trace2: add absolute elapsed time to start event\n + trace2: refactor setting process starting time\n + config: initialize opts structure in repo_read_config()\n\n Polishing of the new trace2 facility continues. The system-level\n configuration can specify site-wide trace2 settings, which can be\n overridden with per-user configuration and environment variables.\n cf. (v4)\n\n\n* jk/ls-files-doc-markup-fix (2019-04-23) 1 commit\n (merged to 'next' on 2019-05-09 at a68fe0ae72)\n + doc/ls-files: put nested list for \"-t\" option into block\n\n Docfix.\n\n\n* jk/p5302-avoid-collision-check-cost (2019-04-23) 1 commit\n (merged to 'next' on 2019-05-09 at 8dc92cad96)\n + p5302: create the repo in each index-pack test\n\n Fix index-pack perf test so that the repeated invocations always\n run in an empty repository, which emulates the initial clone\n situation better.\n\n\n* jk/perf-aggregate-wo-libjson (2019-04-24) 1 commit\n (merged to 'next' on 2019-05-09 at e697c1993b)\n + t/perf: depend on perl JSON only when using --codespeed\n\n The script to aggregate perf result unconditionally depended on\n libjson-perl even though it did not have to, which has been\n corrected.\n\n\n* js/partial-clone-connectivity-check (2019-05-05) 2 commits\n (merged to 'next' on 2019-05-10 at e1d3a6ebde)\n + t/perf: add perf script for partial clones\n (merged to 'next' on 2019-04-25 at ebd8b4bffd)\n + clone: do faster object check for partial clones\n\n During an initial \"git clone --depth=...\" partial clone, it is\n pointless to spend cycles for a large portion of the connectivity\n check that enumerates and skips promisor objects (which by\n definition is all objects fetched from the other side). This has\n been optimized out.\n\n\n* pw/clean-sequencer-state-upon-final-commit (2019-04-17) 2 commits\n (merged to 'next' on 2019-05-09 at cf6cce8ca7)\n + fix cherry-pick/revert status after commit\n + commit/reset: try to clean up sequencer state\n\n \"git chery-pick\" (and \"revert\" that shares the same runtime engine)\n that deals with multiple commits got confused when the final step\n gets stopped with a conflict and the user concluded the sequence\n with \"git commit\". Attempt to fix it by cleaning up the state\n files used by these commands in such a situation.\n\n\n* pw/rebase-i-internal (2019-04-19) 13 commits\n (merged to 'next' on 2019-05-09 at 1206aa6865)\n + rebase -i: run without forking rebase--interactive\n + rebase: use a common action enum\n + rebase -i: use struct rebase_options in do_interactive_rebase()\n + rebase -i: use struct rebase_options to parse args\n + rebase -i: use struct object_id for squash_onto\n + rebase -i: use struct commit when parsing options\n + rebase -i: remove duplication\n + rebase -i: combine rebase--interactive.c with rebase.c\n + rebase: use OPT_RERERE_AUTOUPDATE()\n + rebase: rename write_basic_state()\n + rebase: don't translate trace strings\n + sequencer: always discard index after checkout\n + Merge branch 'ag/sequencer-reduce-rewriting-todo' into pw/rebase-i-internal\n (this branch is used by pw/rebase-abort-clean-rewritten.)\n\n The internal implementation of \"git rebase -i\" has been updated to\n avoid forking a separate \"rebase--interactive\" process.\n\n--------------------------------------------------\n[New Topics]\n\n* cw/diff-highlight (2019-05-09) 1 commit\n (merged to 'next' on 2019-05-13 at 050d707166)\n + diff-highlight: use correct /dev/null for UNIX and Windows\n\n Portability fix for a diff-highlight tool (in contrib/).\n\n Will merge to 'master'.\n\n\n* dl/warn-tagging-a-tag (2019-05-09) 1 commit\n (merged to 'next' on 2019-05-13 at a2f6365923)\n + tag: fix typo in nested tagging hint\n\n Typofix.\n\n Will merge to 'master'.\n\n\n* js/difftool-no-index (2019-05-09) 1 commit\n - difftool --no-index: error out on --dir-diff (and don't crash)\n\n The \"--dir-diff\" mode of \"git difftool\" is not useful in \"--no-index\"\n mode; they are now explicitly marked as mutually incompatible.\n\n Will merge to 'next'.\n\n\n* nd/submodule-helper-incomplete-line-fix (2019-05-08) 1 commit\n (merged to 'next' on 2019-05-13 at cd52f49825)\n + submodule--helper: add a missing \\n\n\n Typofix.\n\n Will merge to 'master'.\n\n\n* sb/format-patch-base-patch-id-fix (2019-05-08) 2 commits\n - format-patch: make --base patch-id output stable\n - format-patch: inform user that patch-id generation is unstable\n\n The recently added \"--base\" option of \"format-patch\" computed the\n patch-ids for prerequisite patches in an unstable way, which has\n been updated to compute in a way that is compatible with \"git\n patch-id --stable\".\n\n Will merge to 'next'.\n\n\n* ab/send-email-transferencoding-fix (2019-05-13) 3 commits\n (merged to 'next' on 2019-05-13 at 38c6a1e7e0)\n + send-email: do defaults -> config -> getopt in that order\n + send-email: rename the @bcclist variable for consistency\n + send-email: move the read_config() function above getopts\n\n Since \"git send-email\" learned to take 'auto' as the value for the\n transfer-encoding, it by mistake stopped honoring the values given\n to the configuration variables sendemail.transferencoding and/or\n sendemail..transferencoding. This has been corrected to\n (finally) redoing the order of setting the default, reading the\n configuration and command line options.\n\n Will merge to 'master'.\n\n\n* ab/trace2-typofix (2019-05-13) 1 commit\n (merged to 'next' on 2019-05-13 at f1229d4d9d)\n + trace2: fix up a missing \"leave\" entry point\n\n Typofix.\n\n Will merge to 'master'.\n\n\n* dl/format-patch-notes-config (2019-05-13) 2 commits\n - format-patch: teach format.notes config option\n - git-format-patch.txt: document --no-notes option\n\n \"git format-patch\" learns a configuration to set the default for\n its --notes= option.\n\n The interaction between config and option may be a little iffy.\n cf. \n\n\n* ds/midx-too-many-packs (2019-05-07) 2 commits\n - midx: add packs to packed_git linked list\n - midx: pass a repository pointer\n\n The code to generate the multi-pack idx file was not prepared to\n see too many packfiles and ran out of open file descriptor, which\n has been corrected.\n\n Will merge to 'next'.\n\n\n* ds/trace2-document-env-vars (2019-05-13) 1 commit\n - trace2: add variable description to git.txt\n\n Doc update.\n\n Will merge to 'next'.\n\n\n* es/check-non-portable-pre-5.10 (2019-05-13) 1 commit\n - check-non-portable-shell: support Perl versions older than 5.10\n\n Developer support update.\n\n Will merge to 'next'.\n\n\n* id/windows-dep-aslr (2019-05-13) 2 commits\n (merged to 'next' on 2019-05-13 at 33d57069f8)\n + mingw: enable DEP and ASLR\n + mingw: do not let ld strip relocations\n\n Allow DEP and ASLR for Windows build to for security hardening.\n\n Will merge to 'master'.\n\n\n* jk/unused-params-final-batch (2019-05-13) 14 commits\n - verify-commit: simplify parameters to run_gpg_verify()\n - show-branch: drop unused parameter from show_independent()\n - rev-list: drop unused void pointer from finish_commit()\n - remove_all_fetch_refspecs(): drop unused \"remote\" parameter\n - receive-pack: drop unused \"commands\" from prepare_shallow_update()\n - pack-objects: drop unused rev_info parameters\n - name-rev: drop unused parameters from is_better_name()\n - mktree: drop unused length parameter\n - wt-status: drop unused status parameter\n - read-cache: drop unused parameter from threaded load\n - clone: drop dest parameter from copy_alternates()\n - submodule: drop unused prefix parameter from some functions\n - builtin: consistently pass cmd_* prefix to parse_options\n - cmd_{read,write}_tree: rename \"unused\" variable that is used\n\n Remove many unused parameters throughout the codebase, with the\n ultimate aim to allow us compile with -Wunused-parameter cleanly.\n\n Will merge to 'next'.\n\n\n* js/rebase-i-label-shown-in-status-fix (2019-05-13) 1 commit\n - status: fix display of rebase -ir's `label` command\n\n \"git status\" did not know that the \"label\" instruction in the\n todo-list \"rebase -i -r\" uses should not be shown as a hex object\n name.\n\n Will merge to 'next'.\n\n\n* nd/init-relative-template-fix (2019-05-13) 1 commit\n - init: make --template path relative to $CWD\n\n A relative pathname given to \"git init --template= \"\n ought to be relative to the directory \"git init\" gets invoked in,\n but it instead was made relative to the repository, which has been\n corrected.\n\n Will merge to 'next'.\n\n--------------------------------------------------\n[Stalled]\n\n* jn/unknown-index-extensions (2018-11-21) 2 commits\n - index: offer advice for unknown index extensions\n - index: do not warn about unrecognized extensions\n\n A bit too alarming warning given when unknown index extensions\n exist is getting revamped.\n\n Expecting a reroll.\n\n\n* jc/format-patch-delay-message-id (2019-04-05) 1 commit\n - format-patch: move message-id and related headers to the end\n\n The location \"git format-patch --thread\" adds the Message-Id:\n header in the series of header fields has been moved down, which\n may help working around a suspected bug in GMail MSA, reported at\n \n\n Waiting for feedback to see if it truly helps.\n Needs tests.\n\n\n* jt/fetch-cdn-offload (2019-03-12) 9 commits\n - SQUASH???\n - upload-pack: send part of packfile response as uri\n - fetch-pack: support more than one pack lockfile\n - upload-pack: refactor reading of pack-objects out\n - Documentation: add Packfile URIs design doc\n - Documentation: order protocol v2 sections\n - http-fetch: support fetching packfiles by URL\n - http: improve documentation of http_pack_request\n - http: use --stdin when getting dumb HTTP pack\n\n WIP for allowing a response to \"git fetch\" to instruct the bulk of\n the pack contents to be instead taken from elsewhere (aka CDN).\n\n Still being discussed.\n\n\n* js/add-i-coalesce-after-editing-hunk (2018-08-28) 1 commit\n - add -p: coalesce hunks before testing applicability\n\n Applicability check after a patch is edited in a \"git add -i/p\"\n session has been improved.\n\n Will hold.\n cf. \n\n\n* js/protocol-advertise-multi (2018-12-28) 1 commit\n - protocol: advertise multiple supported versions\n\n The transport layer has been updated so that the protocol version\n used can be negotiated between the parties, by the initiator\n listing the protocol versions it is willing to talk, and the other\n side choosing from one of them.\n\n Expecting a reroll.\n cf. \n\n\n* mk/use-size-t-in-zlib (2018-10-15) 1 commit\n - zlib.c: use size_t for size\n\n The wrapper to call into zlib followed our long tradition to use\n \"unsigned long\" for sizes of regions in memory, which have been\n updated to use \"size_t\".\n\n\n* dl/remote-save-to-push (2018-12-11) 1 commit\n - remote: add --save-to-push option to git remote set-url\n\n \"git remote set-url\" learned a new option that moves existing value\n of the URL field to pushURL field of the remote before replacing\n the URL field with a new value.\n\n Anybody who wants to champion this topic?\n I am personally not yet quite convinced if this is worth pursuing.\n\n--------------------------------------------------\n[Cooking]\n\n* ab/perf-installed-fix (2019-05-08) 6 commits\n (merged to 'next' on 2019-05-13 at ccd997a142)\n + perf-lib.sh: forbid the use of GIT_TEST_INSTALLED\n + perf tests: add \"bindir\" prefix to git tree test results\n + perf-lib.sh: remove GIT_TEST_INSTALLED from perf-lib.sh\n + perf-lib.sh: make \"./run \" use the correct gits\n + perf aggregate: remove GIT_TEST_INSTALLED from --codespeed\n + perf README: correct docs for 3c8f12c96c regression\n\n Performance test framework has been broken and measured the version\n of Git that happens to be on $PATH, not the specified one to\n measure, for a while, which has been corrected.\n\n Will merge to 'master'.\n cf. <20190507105434.9600-1-avarab@gmail.com>\n\n\n* an/ignore-doc-update (2019-05-08) 1 commit\n - gitignore.txt: make slash-rules more readable\n\n The description about slashes in gitignore patterns (used to\n indicate things like \"anchored to this level only\" and \"only\n matches directories\") has been revamped.\n\n Almost there.\n cf. <20190507104507.18735-1-admin@in-ici.net>\n\n\n* bl/t4253-exit-code-from-format-patch (2019-05-07) 1 commit\n - t4253-am-keep-cr-dos: avoid using pipes\n\n Avoid patterns to pipe output from a git command to feed another\n command in tests.\n\n Will merge to 'next'.\n\n\n* cm/notes-comment-fix (2019-05-07) 1 commit\n - notes: correct documentation of format_display_notes()\n\n A stale in-code comment has been updated.\n\n Will merge to 'next'.\n\n\n* dl/branch-from-3dot-merge-base (2019-05-07) 2 commits\n (merged to 'next' on 2019-05-13 at 2eb387070e)\n + branch: make create_branch accept a merge base rev\n + t2018: cleanup in current test\n\n \"git branch new A...B\" and \"git checkout -b new A...B\" have been\n taught that in their contexts, the notation A...B means \"the merge\n base between these two commits\", just like \"git checkout A...B\"\n detaches HEAD at that commit.\n\n Will merge to 'master'.\n cf. \n\n\n* ds/cvsexportcommit-force-text (2019-05-07) 1 commit\n - cvsexportcommit: force crlf translation\n\n \"git cvsexportcommit\" running on msys did not expect cvsnt showed\n \"cvs status\" output with CRLF line endings.\n\n Will merge to 'next'.\n\n\n* en/fast-export-encoding (2019-05-13) 5 commits\n - fast-export: do automatic reencoding of commit messages only if requested\n - fast-export: differentiate between explicitly utf-8 and implicitly utf-8\n - fast-export: avoid stripping encoding header if we cannot reencode\n - fast-import: support 'encoding' commit header\n - t9350: fix encoding test to actually test reencoding\n\n The \"git fast-export/import\" pair has been taught to handle commits\n with log messages in encoding other than UTF-8 better.\n\n Will merge to 'next'.\n cf. <20190510205335.19968-1-newren@gmail.com>\n\n\n* jk/apache-lsan (2019-05-08) 1 commit\n (merged to 'next' on 2019-05-13 at 1a055a6df0)\n + t/lib-httpd: pass LSAN_OPTIONS through apache\n\n Allow tests that involve httpd to be run under leak sanitizer, just\n like we can already do so under address sanitizer.\n\n Will merge to 'master'.\n\n\n* jk/cocci-batch (2019-05-08) 2 commits\n (merged to 'next' on 2019-05-13 at 8bbbfd3438)\n + coccicheck: make batch size of 0 mean \"unlimited\"\n + coccicheck: optionally batch spatch invocations\n\n Optionally \"make coccicheck\" can feed multiple source files to\n spatch, gaining performance while spending more memory.\n\n Will merge to 'master'.\n\n\n* js/commit-graph-parse-leakfix (2019-05-07) 1 commit\n (merged to 'next' on 2019-05-13 at 18df8ac778)\n + commit-graph: fix memory leak\n\n Leakfix.\n\n Will merge to 'master'.\n\n\n* js/fsmonitor-refresh-after-discarding-index (2019-05-08) 2 commits\n - fsmonitor: force a refresh after the index was discarded\n - fsmonitor: demonstrate that it is not refreshed after discard_index()\n\n The fsmonitor interface got out of sync after the in-core index\n file gets discarded, which has been corrected.\n\n Will merge to 'next'.\n\n\n* js/t5580-unc-alternate-test (2019-05-07) 1 commit\n - t5580: verify that alternates can be UNC paths\n\n An additional test for MinGW\n\n Will merge to 'next'.\n\n\n* js/t6500-use-windows-pid-on-mingw (2019-05-08) 1 commit\n (merged to 'next' on 2019-05-13 at 24e5e27eb7)\n + t6500(mingw): use the Windows PID of the shell\n\n Future-proof a test against an update to MSYS2 runtime v3.x series.\n\n Will merge to 'master'.\n cf. \n It might be helpful in the longer term to encapsulate the code that\n uses /proc/$$/winpid into a helper function and use it anywhere $$\n is referenced, but let's defer it until we see such a callsite that\n would be helped by such a move.\n\n\n* mh/http-fread-api-fix (2019-05-08) 1 commit\n (merged to 'next' on 2019-05-13 at f6af2d38d7)\n + Make fread/fwrite-like functions in http.c more like fread/fwrite.\n\n A pair of private functions in http.c that had names similar to\n fread/fwrite did not return the number of elements, which was found\n to be confusing.\n\n Will merge to 'master'.\n\n\n* nd/merge-quit (2019-05-13) 2 commits\n - merge: add --quit\n - merge: remove drop_save() in favor of remove_merge_branch_state()\n\n \"git merge\" learned \"--quit\" option that cleans up the in-progress\n merge while leaving the working tree and the index still in a mess.\n\n On hold.\n cf. \n Windows seems to have problems with the code, and may need a\n follow-up fixes.\n\n\n* nd/parse-options-aliases (2019-05-07) 1 commit\n (merged to 'next' on 2019-05-13 at c77cc06452)\n + parse-options: don't emit \"ambiguous option\" for aliases\n\n Attempt to use an abbreviated option in \"git clone --recurs\" is\n responded by a request to disambiguate between --recursive and\n --recurse-submodules, which is bad because these two are synonyms.\n The parse-options API has been extended to define such synonyms\n more easily and not produce an unnecessary failure.\n\n Will merge to 'master'.\n\n\n* pw/rebase-abort-clean-rewritten (2019-05-08) 1 commit\n - rebase --abort: cleanup refs/rewritten\n\n \"git rebase --abort\" used to leave refs/rewritten/ when concluding\n \"git rebase -r\", which has been corrected.\n\n On hold.\n cf. <6726f99a-ec27-b72f-39d7-03eecb555197@gmail.com>\n\n\n* sg/ci-libsvn-perl (2019-05-07) 1 commit\n - ci: install 'libsvn-perl' instead of 'git-svn'\n\n To run tests for Git SVN, our scripts for CI used to install the\n git-svn package (in the hope that it would bring in the right\n dependencies). This has been updated to install the more direct\n dependency, namely, libsvn-perl.\n\n Will merge to 'next'.\n\n\n* tt/no-ipv6-fallback-for-winxp (2019-05-07) 1 commit\n - mingw: remove obsolete IPv6-related code\n\n Code cleanup.\n\n Will merge to 'next'.\n\n\n* nb/branch-show-other-worktrees-head (2019-05-07) 3 commits\n - branch: add worktree info on verbose output\n - branch: update output to include worktree info\n - ref-filter: add worktreepath atom\n\n \"git branch --list\" learned to show branches that are checked out\n in other worktrees connected to the same repository prefixed with\n '+', similar to the way the currently checked out branch is shown\n with '*' in front.\n\n\n* en/unicode-in-refnames (2019-04-26) 1 commit\n - Honor core.precomposeUnicode in more places\n\n On a filesystem like HFS+, the names of the refs stored as filesystem\n entities may become different from what the end-user expects, just\n like files in the working tree get \"renamed\". Work around the\n mismatch by paying attention to the core.precomposeUnicode\n configuration.\n\n Will merge to 'next'.\n\n\n* dl/difftool-mergetool (2019-05-13) 7 commits\n - difftool: fallback on merge.guitool\n - difftool: make --gui, --tool and --extcmd mutually exclusive\n - mergetool: fallback to tool when guitool unavailable\n - mergetool--lib: create gui_mode function\n - mergetool: use get_merge_tool function\n - t7610: add mergetool --gui tests\n - t7610: unsuppress output\n\n Update \"git difftool\" and \"git mergetool\" so that the combinations\n of {diff,merge}.{tool,guitool} configuration variables serve as\n fallback settings of each other in a sensible order.\n\n Will merge to 'next'.\n cf. (v5)\n\n\n* es/first-contrib-tutorial (2019-05-08) 2 commits\n - documentation: add anchors to MyFirstContribution\n - documentation: add tutorial for first contribution\n\n A new tutorial targetting specifically aspiring git-core\n developers.\n\n I think we are almost there.\n cf. <20190507213040.151799-1-emilyshaffer@google.com> (v5)\n\n\n* cc/multi-promisor (2019-04-15) 17 commits\n - Move core_partial_clone_filter_default to promisor-remote.c\n - Move repository_format_partial_clone to promisor-remote.c\n - Remove fetch-object.{c,h} in favor of promisor-remote.{c,h}\n - remote: add promisor and partial clone config to the doc\n - partial-clone: add multiple remotes in the doc\n - t0410: test fetching from many promisor remotes\n - builtin/fetch: remove unique promisor remote limitation\n - promisor-remote: parse remote.*.partialclonefilter\n - diff: use promisor-remote.h instead of fetch-object.h\n - Use promisor_remote_get_direct() and has_promisor_remote()\n - promisor-remote: use repository_format_partial_clone\n - promisor-remote: add promisor_remote_reinit()\n - promisor-remote: implement promisor_remote_get_direct()\n - Add initial support for many promisor remotes\n - fetch-object: make functions return an error code\n - t0410: remove pipes after git commands\n - Merge branch 'jt/batch-fetch-blobs-in-diff' into cc/multi-promisor\n\n Teach the lazy clone machinery that there can be more than one\n promisor remote and consult them in order when downloading missing\n objects on demand.\n\n Needs review.\n\n\n* nd/switch-and-restore (2019-05-07) 43 commits\n - Declare both git-switch and git-restore experimental\n - help: move git-diff and git-reset to different groups\n - doc: promote \"git restore\"\n - user-manual.txt: prefer 'merge --abort' over 'reset --hard'\n - completion: support restore\n - t: add tests for restore\n - restore: support --patch\n - restore: replace --force with --ignore-unmerged\n - restore: default to --source=HEAD when only --staged is specified\n - restore: reject invalid combinations with --staged\n - restore: add --worktree and --staged\n - checkout: factor out worktree checkout code\n - restore: disable overlay mode by default\n - restore: make pathspec mandatory\n - restore: take tree-ish from --source option instead\n - checkout: split part of it to new command 'restore'\n - doc: promote \"git switch\"\n - completion: support switch\n - t: add tests for switch\n - switch: make --orphan switch to an empty tree\n - switch: reject if some operation is in progress\n - switch: no worktree status unless real branch switch happens\n - switch: implicit dwim, use --no-guess to disable it\n - switch: add short option for --detach\n - switch: only allow explicit detached HEAD\n - switch: reject \"do nothing\" case\n - switch: stop accepting pathspec\n - switch: remove -l\n - switch: add --discard-changes\n - switch: better names for -b and -B\n - checkout: split part of it to new command 'switch'\n - checkout: split options[] array in three pieces\n - checkout: move 'confict_style' and 'dwim_..' to checkout_opts\n - checkout: make \"opts\" in cmd_checkout() a pointer\n - checkout: factor out some code in parse_branchname_arg()\n - checkout: keep most #include sorted\n - checkout: inform the user when removing branch state\n - checkout: advice how to get out of detached HEAD mode\n - t: rename t2014-switch.sh to t2014-checkout-switch.sh\n - git-checkout.txt: fix monospace typeset\n - doc: document --overwrite-ignore\n - git-checkout.txt: fix one syntax line\n - git-checkout.txt: spell out --no-option\n\n Two new commands \"git switch\" and \"git restore\" are introduced to\n split \"checking out a branch to work on advancing its history\" and\n \"checking out paths out of the index and/or a tree-ish to work on\n advancing the current history\" out of the single \"git checkout\"\n command.\n\n The \"switch\" part seems more or less ready for testing. Perhaps\n we should split this back into two topics and merge it to 'next'.\n cf. <20190329103919.15642-1-pclouds@gmail.com> (switch v6)\n cf. <20190425094600.15673-1-pclouds@gmail.com> (restore v3)\n\n\n* jc/format-patch-noclobber (2019-02-22) 1 commit\n - format-patch: --no-clobber refrains from overwriting output files\n\n \"git format-patch\" used to overwrite an existing patch/cover-letter\n file. A new \"--no-clobber\" option stops it.\n\n Undecided but inclined to discard.\n\n\n* am/p4-branches-excludes (2019-04-02) 8 commits\n - git-p4: respect excluded paths when detecting branches\n - git-p4: add failing test for \"git-p4: respect excluded paths when detecting branches\"\n - git-p4: don't exclude other files with same prefix\n - git-p4: add failing test for \"don't exclude other files with same prefix\"\n - git-p4: don't groom exclude path list on every commit\n - git-p4: match branches case insensitively if configured\n - git-p4: add failing test for \"git-p4: match branches case insensitively if configured\"\n - git-p4: detect/prevent infinite loop in gitCommitByP4Change()\n\n \"git p4\" update.\n\n Is this ready for 'next'?\n\n\n* dl/rebase-i-keep-base (2019-04-25) 6 commits\n - rebase: teach rebase --keep-base\n - rebase: fast-forward --fork-point in more cases\n - rebase: fast-forward --onto in more cases\n - rebase: refactor can_fast_forward into goto tower\n - t3432: test rebase fast-forward behavior\n - t3431: add rebase --fork-point tests\n\n \"git rebase --keep-base \" tries to find the original base\n of the topic being rebased and rebase on top of that same base,\n which is useful when running the \"git rebase -i\" (and its limited\n variant \"git rebase -x\").\n\n The command also has learned to fast-forward in more cases where it\n can instead of replaying to recreate identical commits.\n\n On hold.\n cf. <20190508001252.15752-1-avarab@gmail.com>\n cf. \n\n\n* nd/worktree-name-sanitization (2019-03-20) 2 commits\n - SQUASH???\n - worktree add: sanitize worktree names\n\n In recent versions of Git, per-worktree refs are exposed in\n refs/worktrees// hierarchy, which means that worktree names\n must be a valid refname component. The code now sanitizes the names\n given to worktrees, to make sure these refs are well-formed.\n\n I am inclined to squash the fix at the tip in and merge the result\n to 'next'. Opinions?\n\n\n* ds/commit-graph-write-refactor (2019-05-13) 11 commits\n - commit-graph: extract write_commit_graph_file()\n - commit-graph: extract copy_oids_to_commits()\n - commit-graph: extract count_distinct_commits()\n - commit-graph: extract fill_oids_from_all_packs()\n - commit-graph: extract fill_oids_from_commit_hex()\n - commit-graph: extract fill_oids_from_packs()\n - commit-graph: create write_commit_graph_context\n - commit-graph: remove Future Work section\n - commit-graph: collapse parameters into flags\n - commit-graph: return with errors during write\n - commit-graph: fix the_repository reference\n\n Renamed from commit-graph-format-v2 and changed scope.\n\n Expecting a reroll.\n I think it is almost there, modulo a few internal API details..\n cf. (v4)\n cf. <17829620-1084-74e5-54ad-aa95990f4dbd@gmail.com>\n\n\n* br/blame-ignore (2019-04-14) 6 commits\n - blame: use a fingerprint heuristic to match ignored lines\n - blame: optionally track line fingerprints during fill_blame_origin()\n - blame: add config options to handle output for ignored lines\n - blame: add the ability to ignore commits and their changes\n - blame: use a helper function in blame_chunk()\n - Move init_skiplist() outside of fsck\n\n \"git blame\" learned to \"ignore\" commits in the history, whose\n effects (as well as their presence) get ignored.\n\n Expecting a reroll.\n cf. <20190410162409.117264-1-brho@google.com> (v6)\n cf. \n cf. <3db6bad3-e7a5-af1d-3fe2-321bd17db2c6@google.com>\n\n--------------------------------------------------\n[Discarded]\n\n* nd/precious (2019-04-09) 1 commit\n . Introduce \"precious\" file concept\n\n \"git clean\" learned to pay attention to the 'precious' attributes\n and keep untracked paths with the attribute instead of removing\n when the \"--keep-precious\" is given.\n\n Retracted.\n cf. \n\n I am not sure what aspect of this longer-term \"precious\" vision,\n which gets taught to various commands and use cases individually\n and incrementally, Ævar finds problematic, which I understand is\n the reason of redtraction.\n\n\n* nd/config-move-to (2019-01-14) 7 commits\n . config.h: fix hdr-check warnings\n . config: add --move-to\n . config: factor out set_config_source_file()\n . config: use OPT_FILENAME()\n . config.c: add repo_config_set_worktree_gently()\n . worktree.c: add get_worktree_config()\n . config.c: avoid git_path() in do_git_config_sequence()\n\n Dropped.\n cf. \n\n\n* dm/some-stdio-functions-are-macro-on-freebsd (2019-02-01) 1 commit\n . http: cast result to FILE *\n\n Variants of BSD define fileno(fh) as a macro, breaking a program\n that passes a \"void *\" to it.\n\n Can be safely discarded.\n cf. \n\n\n* hs/send-email-transferencoding-fix (2019-04-10) 1 commit\n . send-email: honor transferencoding config option again\n\n cf. \n Replaced by jc/send-email-transferencoding-fix\n\n\n* jc/send-email-transferencoding-fix (2019-05-09) 2 commits\n . send-email: honor transferencoding config option again\n . send-email: update the mechanism to set default configuration values\n\n Replaced by Ævar's \"do the default in the right order\" approach.\n cf. <20190508105607.178244-1-gitster@pobox.com> (v2)\n\n\n* ds/midx-expire-repack (2019-04-25) 11 commits\n . t5319-multi-pack-index.sh: test batch size zero\n . midx: add test that 'expire' respects .keep files\n . multi-pack-index: test expire while adding packs\n . midx: implement midx_repack()\n . multi-pack-index: prepare 'repack' subcommand\n . multi-pack-index: implement 'expire' subcommand\n . midx: refactor permutation logic and pack sorting\n . midx: simplify computation of pack name lengths\n . multi-pack-index: prepare for 'expire' subcommand\n . Docs: rearrange subcommands for multi-pack-index\n . repack: refactor pack deletion for future use\n\n \"git multi-pack-index expire/repack\" are new subcommands that\n consult midx file and are used to drop unused pack files and\n coalesce small pack files that are still in use.\n\n Tentatively dropped\n cf. <20190424151428.170316-1-dstolee@microsoft.com> (v5)\n cf. <1733843e-f0e4-0e25-468b-64727f6855d5@gmail.com>\n"},{"id":"375422","messageId":"CABPp-BGPrDfHjQoASFFbyt2v3Aa6vie5H5U2KOWUnu=_NEiPyA@mail.gmail.com","threadId":"51080","inReplyTo":"xmqqa7fqbahj.fsf@gitster-ct.c.googlers.com","subject":"Re: What's cooking in git.git (May 2019, #02; Tue, 14)","fromName":"Elijah Newren","fromEmail":"newren@gmail.com","sentAt":"2019-05-13T22:20:51Z","receivedAt":"2019-05-13T22:21:05Z","isPatch":false,"sender":{"key":"newren@gmail.com","avatar":"https://avatars.githubusercontent.com/u/5455730?v=4"},"body":"On Mon, May 13, 2019 at 12:27 PM Junio C Hamano wrote:\n> * en/fast-export-encoding (2019-05-13) 5 commits\n> - fast-export: do automatic reencoding of commit messages only if requested\n> - fast-export: differentiate between explicitly utf-8 and implicitly utf-8\n> - fast-export: avoid stripping encoding header if we cannot reencode\n> - fast-import: support 'encoding' commit header\n> - t9350: fix encoding test to actually test reencoding\n>\n> The \"git fast-export/import\" pair has been taught to handle commits\n> with log messages in encoding other than UTF-8 better.\n>\n> Will merge to 'next'.\n> cf. <20190510205335.19968-1-newren@gmail.com>\n\nI sent out a v4 this morning with Torsten's suggested YES|NO wording,\nyour suggestion of more aliases for \"yes\"/\"no\", and based on Dscho's\nclarification I was able to update the tests to be more precise. I\nagain tested with Dscho's gitgitgadget Azure pipelines integration to\nverify it works on the major platforms. Could you merge it (cf.\n<20190513164722.31534-1-newren@gmail.com>) instead of the v3 you\ncurrently have in en/fast-export-encoding?\n\nElijah\n"},{"id":"375494","messageId":"nycvar.QRO.7.76.6.1905141153150.44@tvgsbejvaqbjf.bet","threadId":"51080","inReplyTo":"xmqqa7fqbahj.fsf@gitster-ct.c.googlers.com","subject":"js/difftool-no-index, was Re: What's cooking in git.git (May 2019, #02; Tue, 14)","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2019-05-14T09:54:26Z","receivedAt":"2019-05-14T09:54:49Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi Junio,\n\nOn Tue, 14 May 2019, Junio C Hamano wrote:\n\n> * js/difftool-no-index (2019-05-09) 1 commit\n> - difftool --no-index: error out on --dir-diff (and don't crash)\n>\n> The \"--dir-diff\" mode of \"git difftool\" is not useful in \"--no-index\"\n> mode; they are now explicitly marked as mutually incompatible.\n>\n> Will merge to 'next'.\n\nSeeing as the following is a part of v2.22.0-rc0's release notes:\n\n\t * \"git difftool\" can now run outside a repository.\n\nmaybe we could get this to `master` instead of `next`?\n\nThanks,\nDscho\n"},{"id":"375582","messageId":"xmqqef508q0y.fsf@gitster-ct.c.googlers.com","threadId":"51080","inReplyTo":"nycvar.QRO.7.76.6.1905141153150.44@tvgsbejvaqbjf.bet","subject":"Re: js/difftool-no-index, was Re: What's cooking in git.git (May 2019, #02; Tue, 14)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2019-05-15T01:28:29Z","receivedAt":"2019-05-15T01:28:34Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Johannes Schindelin writes:\n\n> Hi Junio,\n>\n> On Tue, 14 May 2019, Junio C Hamano wrote:\n>\n>> * js/difftool-no-index (2019-05-09) 1 commit\n>> - difftool --no-index: error out on --dir-diff (and don't crash)\n>>\n>> The \"--dir-diff\" mode of \"git difftool\" is not useful in \"--no-index\"\n>> mode; they are now explicitly marked as mutually incompatible.\n>>\n>> Will merge to 'next'.\n>\n> Seeing as the following is a part of v2.22.0-rc0's release notes:\n>\n> \t * \"git difftool\" can now run outside a repository.\n>\n> maybe we could get this to `master` instead of `next`?\n\nThere is no 'instead of'. Topics advance one step at a time.\n\nI do not recall seeing you respond to my review comments on the\n\"error out\" patch , but\nthinking about it further, assuming that my rephrasing of your\njustification in my review was correct, I wonder if it is even more\ncorrect if we do not error out but simply behave as if \"--dir-diff\"\noption were not given.\n\nThat is, if there were some way to distinguish tracked and untracked\npaths in --no-index mode, --dir-diff would copy only the tracked parts\nto temporary directories and run tree comparison. Since there is no\n\"untracked paths\" by definition in --no-index mode, we do not even\nhave to make copies but just compare the two directories given using\nthe same tree comparison tool.\n\nWouldn't that be more useful behaviour than telling the end users to\nrun the backend tree comparison tool themselves?\n"},{"id":"375608","messageId":"nycvar.QRO.7.76.6.1905151015540.44@tvgsbejvaqbjf.bet","threadId":"51080","inReplyTo":"xmqqef508q0y.fsf@gitster-ct.c.googlers.com","subject":"Re: js/difftool-no-index, was Re: What's cooking in git.git (May 2019, #02; Tue, 14)","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2019-05-15T08:24:40Z","receivedAt":"2019-05-15T08:24:40Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi Junio,\n\nOn Wed, 15 May 2019, Junio C Hamano wrote:\n\n> Johannes Schindelin writes:\n>\n> > On Tue, 14 May 2019, Junio C Hamano wrote:\n> >\n> >> * js/difftool-no-index (2019-05-09) 1 commit\n> >> - difftool --no-index: error out on --dir-diff (and don't crash)\n> >>\n> >> The \"--dir-diff\" mode of \"git difftool\" is not useful in \"--no-index\"\n> >> mode; they are now explicitly marked as mutually incompatible.\n> >>\n> >> Will merge to 'next'.\n> >\n> > Seeing as the following is a part of v2.22.0-rc0's release notes:\n> >\n> > \t * \"git difftool\" can now run outside a repository.\n> >\n> > maybe we could get this to `master` instead of `next`?\n>\n> There is no 'instead of'. Topics advance one step at a time.\n\nWell, some topics seem to enter `next` via `master`...\n\n> I do not recall seeing you respond to my review comments on the\n> \"error out\" patch , but\n> thinking about it further, assuming that my rephrasing of your\n> justification in my review was correct, I wonder if it is even more\n> correct if we do not error out but simply behave as if \"--dir-diff\"\n> option were not given.\n\nI misunderstood that to be a rhetoric question, like in the \"thinking out\nloud\" you favor. Sorry.\n\n> That is, if there were some way to distinguish tracked and untracked\n> paths in --no-index mode, --dir-diff would copy only the tracked parts\n> to temporary directories and run tree comparison. Since there is no\n> \"untracked paths\" by definition in --no-index mode, we do not even\n> have to make copies but just compare the two directories given using\n> the same tree comparison tool.\n\nAs far as `--no-index` is concerned, all files are untracked anyway. There\nis no index, so there are no staged/committed/tracked files.\n\n> Wouldn't that be more useful behaviour than telling the end users to\n> run the backend tree comparison tool themselves?\n\nI am lost. What behavior are you talking about? I *think* you mean to say\nthat the `--dir-diff` mode under `--no-index` should essentially do what\n`--no-dir-diff` does? I think that's not the way I want to go. The\n`--dir-diff` mode only makes sense when there is a mix of\ntracked/untracked files, which simply is not the case for `--no-index`.\nAnd we should not make it \"fall back\" to its opposite. The user asked for\n`--dir-diff` and it just does not make sense.\n\nIn any case, the patch series I'd like to see advanced to `master` is not\nabout supporting or not supporting `--dir-diff` with `--no-index`, but\nabout preventing a segmentation fault when a user calls the (non-sense)\ncombination of `--no-index` and `--dir-diff`.\n\nCiao,\nDscho\n"},{"id":"375613","messageId":"xmqqsgtg5cff.fsf@gitster-ct.c.googlers.com","threadId":"51080","inReplyTo":"nycvar.QRO.7.76.6.1905151015540.44@tvgsbejvaqbjf.bet","subject":"Re: js/difftool-no-index, was Re: What's cooking in git.git (May 2019, #02; Tue, 14)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2019-05-15T08:50:28Z","receivedAt":"2019-05-15T08:50:40Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Johannes Schindelin writes:\n\n> As far as `--no-index` is concerned, all files are untracked anyway. There\n> is no index, so there are no staged/committed/tracked files.\n> ...\n\nAh, I see I completely misspoke; sorry, there should be no \"pretend\nas if there were no --dir-diff\".\n\nIn the normal usage, \"difftool --dir-diff\" materializes the two\ntree-shaped things (I am using an invented term loosely here---it\nmay be a tree-ish, or what is in the index, or the working tree\nfiles) in two temporary directories and runs a tool to compare two\ndirectories, and what is copied out of the working tree is limited\nto the tracked ones (for the obvious purpose to avoid showing them\nthe the directory comparison tool).\n\nExtending that semantics literally to \"--no-index\" use would result\nin nonsense. As you say, there is no \"tracked\" thing under the\nmode, so doing what \"--dir-diff\" literally does (i.e. only copy out\nwhat is tracked for the purpose of comparison) would be to compare\nnothing against nothing. That is not useful.\n\nWhat I was wondering when I wrote the message you are responding to\nwas complete opposite. I was imagining what would happen if we\ntreat _everything_ in the two directories being compared by\n\"difftool --dir-diff --no-index\" as if it is tracked. We do not\neven have to make copies (as we are not doing a selective \"only the\ntracked ones\" copy), but if it is easier to implement we could make\nredundant copies. In either case, the two directories given to the\n\"difftool --no-index\" would be compared with the same directory\ncomparison tool the end user is used to (as that is the one the\n\"--dir-diff\" mode uses to compare without \"--no-index\"). When the\nend user does this sequence, for example:\n\n $ rm -fr /var/tmp/1 /var/tmp/2 && mkdir /var/tmp/1 /var/tmp/2\n $ (cd /var/tmp/1 && tar xvf -) Johannes Schindelin writes:\n>\n> I was imagining what would happen if we treat _everything_ in the two\n> directories being compared by \"difftool --dir-diff --no-index\" as if it\n> is tracked.\n\nIsn't this exactly what `git difftool --no-index` *without* `--dir-diff`\ndoes already (although without copying or hardlinking or symlinking any\nfiles)?\n\nCiao,\nDscho\n"},{"id":"375868","messageId":"xmqqsgtb1am1.fsf@gitster-ct.c.googlers.com","threadId":"51080","inReplyTo":"nycvar.QRO.7.76.6.1905172025450.46@tvgsbejvaqbjf.bet","subject":"Re: js/difftool-no-index, was Re: What's cooking in git.git (May 2019, #02; Tue, 14)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2019-05-19T01:44:38Z","receivedAt":"2019-05-19T01:44:46Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Johannes Schindelin writes:\n\n> Hi Junio,\n>\n> On Wed, 15 May 2019, Junio C Hamano wrote:\n>\n>> Johannes Schindelin writes:\n>>\n>> I was imagining what would happen if we treat _everything_ in the two\n>> directories being compared by \"difftool --dir-diff --no-index\" as if it\n>> is tracked.\n>\n> Isn't this exactly what `git difftool --no-index` *without* `--dir-diff`\n> does already (although without copying or hardlinking or symlinking any\n> files)?\n\nIf that is the case, then I would imagine that running that command\nfor the user, instead of refusing to work, would give a more\npleasant end-user experience. No?\n\nUnless we anticipate that we might dwim incorrectly and mistake a\nrequest to compare two things, to which the distinction between\ntracked and untracked matters, as a request to compare two\ndirectories that are not under Git control, that is. If such an\nincorrect dwim were a possibility, then it is helpful to refuse with\n\"when comparing two non-git-controlled directories, you cannot use\nthe '--dir-diff' mode\", as that would not silently give an incorrect\noutput to the users.\n\nIn any case, all of the above can be left for future improvements.\nGetting close to the final, I think it is preferrable to have a\n\"refuse to stop early\" (i.e. the patch that is already in 'next')\ninstead of \"do what the user meant\" whose implementation may become\nmore involved (and error prone).\n\nThanks.\n\n"},{"id":"376021","messageId":"nycvar.QRO.7.76.6.1905220025440.46@tvgsbejvaqbjf.bet","threadId":"51080","inReplyTo":"xmqqsgtb1am1.fsf@gitster-ct.c.googlers.com","subject":"Re: js/difftool-no-index, was Re: What's cooking in git.git (May 2019, #02; Tue, 14)","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2019-05-21T22:28:00Z","receivedAt":"2019-05-21T22:28:25Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi Junio,\n\nOn Sun, 19 May 2019, Junio C Hamano wrote:\n\n> Johannes Schindelin writes:\n>\n> > On Wed, 15 May 2019, Junio C Hamano wrote:\n> >\n> >> Johannes Schindelin writes:\n> >>\n> >> I was imagining what would happen if we treat _everything_ in the two\n> >> directories being compared by \"difftool --dir-diff --no-index\" as if it\n> >> is tracked.\n> >\n> > Isn't this exactly what `git difftool --no-index` *without* `--dir-diff`\n> > does already (although without copying or hardlinking or symlinking any\n> > files)?\n>\n> If that is the case, then I would imagine that running that command\n> for the user, instead of refusing to work, would give a more\n> pleasant end-user experience. No?\n\nThe elephant in the room is that a user might think that --dir-diff makes\ncopies (as promised) and lets their difftool clobber them files happily,\nonly to find out that they clobbered the original files. And since they\nasked for `--no-index`, chances are that those files really are not\ntracked, so now they do not even have a way to revert that clobbering.\n\nSeriously, I'd rather not DWIM `--no-index --dir-diff` to essentially do\n`--no-index --no-dir-diff`.\n\nCiao,\nDscho\n\n> Unless we anticipate that we might dwim incorrectly and mistake a\n> request to compare two things, to which the distinction between\n> tracked and untracked matters, as a request to compare two\n> directories that are not under Git control, that is. If such an\n> incorrect dwim were a possibility, then it is helpful to refuse with\n> \"when comparing two non-git-controlled directories, you cannot use\n> the '--dir-diff' mode\", as that would not silently give an incorrect\n> output to the users.\n>\n> In any case, all of the above can be left for future improvements.\n> Getting close to the final, I think it is preferrable to have a\n> \"refuse to stop early\" (i.e. the patch that is already in 'next')\n> instead of \"do what the user meant\" whose implementation may become\n> more involved (and error prone).\n>\n> Thanks.\n>\n>\n"}]}