{"thread":{"id":"54013","subject":"What's cooking in git.git (Aug 2020, #02; Mon, 10)","startedAt":"2020-08-10T19:44:38Z","lastAt":"2020-08-12T08:15:57Z","messageCount":18,"participants":["Junio C Hamano","Eric Sunshine","Jeff King","Patrick Steinhardt","brian m. carlson","Sergey Organov","Taylor Blau","Matheus Tavares Bernardino"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"403280","messageId":"xmqqsgcub811.fsf@gitster.c.googlers.com","threadId":"54013","inReplyTo":null,"subject":"What's cooking in git.git (Aug 2020, #02; Mon, 10)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2020-08-10T19:44:26Z","receivedAt":"2020-08-10T19:44:38Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Here are the topics that have been cooking.  Commits prefixed with '-' are\nonly in 'seen' (formerly 'pu'---proposed updates) while commits prefixed\nwith '+' are in 'next'.  The ones marked with '.' do not appear in any of\nthe integration branches, but I am still holding onto them.\n\nYou can find the changes described here in the integration branches of the\nrepositories listed at\n\n    http://git-blame.blogspot.com/p/git-public-repositories.html\n\n--------------------------------------------------\n[Graduated to 'master']\n\n* en/eol-attrs-gotchas (2020-08-03) 4 commits\n  (merged to 'next' on 2020-08-05 at 46e73f4c94)\n + checkout: support renormalization with checkout -m <paths>\n + merge: make merge.renormalize work for all uses of merge machinery\n + t6038: remove problematic test\n + t6038: make tests fail for the right reason\n\n All \"mergy\" operations that internally use the merge-recursive\n machinery should honor the merge.renormalize configuration, but\n many of them didn't.\n\n\n* en/merge-recursive-comment-fixes (2020-08-02) 1 commit\n  (merged to 'next' on 2020-08-04 at ec14b8ea94)\n + merge-recursive: fix unclear and outright wrong comments\n\n Comment fix.\n\n\n* es/adjust-subtree-test-for-merge-msg-update (2020-08-03) 1 commit\n  (merged to 'next' on 2020-08-04 at 634b1fcbac)\n + Revert \"contrib: subtree: adjust test to change in fmt-merge-msg\"\n\n Adjust tests in contrib/ to the recent change to fmt-merge-msg.\n\n\n* es/worktree-cleanup (2020-07-31) 4 commits\n  (merged to 'next' on 2020-08-04 at 798f642f66)\n + worktree: retire special-case normalization of main worktree path\n + worktree: drop bogus and unnecessary path munging\n + worktree: drop unused code from get_linked_worktree()\n + worktree: drop pointless strbuf_release()\n\n Code cleanup around \"worktree\" API implementation.\n\n\n* jk/compiler-fixes-and-workarounds (2020-08-04) 3 commits\n  (merged to 'next' on 2020-08-04 at a1caf8edbc)\n + revision: avoid leak when preparing bloom filter for \"/\"\n + revision: avoid out-of-bounds read/write on empty pathspec\n + config: work around gcc-10 -Wstringop-overflow warning\n\n Small fixes and workarounds.\n\n\n* jk/strvec (2020-07-30) 11 commits\n  (merged to 'next' on 2020-08-04 at 61addb841f)\n + strvec: rename struct fields\n + strvec: drop argv_array compatibility layer\n + strvec: update documention to avoid argv_array\n + strvec: fix indentation in renamed calls\n + strvec: convert remaining callers away from argv_array name\n + strvec: convert more callers away from argv_array name\n + strvec: convert builtin/ callers away from argv_array name\n + quote: rename sq_dequote_to_argv_array to mention strvec\n + strvec: rename files from argv-array to strvec\n + argv-array: rename to strvec\n + argv-array: use size_t for count and alloc\n (this branch is used by ds/maintenance and ds/maintenance-part-2.)\n\n The argv_array API is useful for not just managing argv but any\n \"vector\" (NULL-terminated array) of strings, and has seen adoption\n to a certain degree.  It has been renamed to \"strvec\" to reduce the\n barrier to adoption.\n\n\n* jt/pack-objects-prefetch-in-batch (2020-07-21) 2 commits\n  (merged to 'next' on 2020-08-03 at 29424e614d)\n + pack-objects: prefetch objects to be packed\n + pack-objects: refactor to oid_object_info_extended\n\n Originally merged to 'next' on 2020-08-01\n\n While packing many objects in a repository with a promissor remote,\n lazily fetching missing objects from the promissor remote one by\n one may be inefficient---the code now attempts to fetch all the\n missing objects in batch (obviously this won't work for a lazy\n clone that lazily fetches tree objects as you cannot even enumerate\n what blobs are missing until you learn which trees are missing).\n\n\n* jt/pretend-object-never-come-from-elsewhere (2020-07-21) 1 commit\n  (merged to 'next' on 2020-08-03 at 36cd23aae5)\n + sha1-file: make pretend_object_file() not prefetch\n\n Originally merged to 'next' on 2020-08-01\n\n The pretend-object mechanism checks if the given object already\n exists in the object store before deciding to keep the data\n in-core, but the check would have triggered lazy fetching of such\n an object from a promissor remote.\n\n\n* ma/t1450-quotefix (2020-08-01) 1 commit\n  (merged to 'next' on 2020-08-04 at 0e762e3553)\n + t1450: fix quoting of NUL byte when corrupting pack\n\n Test fix.\n\n\n* mp/complete-show-color-moved (2020-07-15) 1 commit\n  (merged to 'next' on 2020-08-03 at c90fea8e5e)\n + completion: add show --color-moved[-ws]\n\n Originally merged to 'next' on 2020-08-01\n\n Command line completion (in contrib/) update.\n A follow-up patch to reduce duplication may be warranted.\n\n\n* ny/notes-doc-sample-update (2020-08-03) 1 commit\n  (merged to 'next' on 2020-08-04 at a63dcb2c55)\n + docs: improve the example that illustrates git-notes path names\n\n Doc updates.\n\n\n* pb/guide-docs (2020-08-04) 4 commits\n  (merged to 'next' on 2020-08-05 at 031cab2bc2)\n + git.txt: add list of guides\n + Documentation: don't hardcode command categories twice\n + help: drop usage of 'common' and 'useful' for guides\n + command-list.txt: add missing 'gitcredentials' and 'gitremote-helpers'\n\n Update \"git help guides\" documentation organization.\n\n\n* rs/bisect-oid-to-hex-fix (2020-08-02) 1 commit\n  (merged to 'next' on 2020-08-04 at dc3c8ea699)\n + bisect: use oid_to_hex_r() instead of memcpy()+oid_to_hex()\n\n Code cleanup.\n\n\n* so/rev-parser-errormessage-fix (2020-08-04) 1 commit\n  (merged to 'next' on 2020-08-05 at dd9653da77)\n + revision: fix die() message for \"--unpacked=\"\n\n Error message fix.\n\n--------------------------------------------------\n[New Topics]\n\n* bc/sha-256-cvs-svn-updates (2020-08-05) 1 commit\n  (merged to 'next' on 2020-08-07 at b1ce7e5ec5)\n + git-cvsexportcommit: support Perl before 5.10.1\n\n Portability fix.\n\n Will merge to 'master'.\n\n\n* ds/maintenance-part-2 (2020-08-06) 9 commits\n - maintenance: add incremental-repack auto condition\n - maintenance: auto-size incremental-repack batch\n - maintenance: add incremental-repack task\n - midx: use start_delayed_progress()\n - midx: enable core.multiPackIndex by default\n - maintenance: create auto condition for loose-objects\n - maintenance: add loose-objects task\n - maintenance: add prefetch task\n - fetch: optionally allow disabling FETCH_HEAD update\n (this branch uses ds/maintenance.)\n\n\n* jt/has_object (2020-08-06) 4 commits\n  (merged to 'next' on 2020-08-07 at ed08abb693)\n + fsck: do not lazy fetch known non-promisor object\n + pack-objects: no fetch when allow-{any,promisor}\n + apply: do not lazy fetch when applying binary\n + sha1-file: introduce no-lazy-fetch has_object()\n\n A new helper function has_object() has been introduced to make it\n easier to mark object existence checks that do and don't want to\n trigger lazy fetches, and a few such checks are converted using it.\n\n Will merge to 'master'.\n\n\n* ma/doc-sha-256-is-experimental (2020-08-06) 1 commit\n - Documentation: mark `--object-format=sha256` as experimental\n\n The recent addition of SHA-256 support is marked as experimental in\n the documentation.\n\n\n* ma/test-quote-cleanup (2020-08-06) 2 commits\n  (merged to 'next' on 2020-08-10 at 63a95c2926)\n + t4104: modernize and simplify quoting\n + t: don't spuriously close and reopen quotes\n\n Test cleanup.\n\n Will merge to 'master'.\n\n\n* rp/apply-cached-with-i-t-a (2020-08-09) 3 commits\n - t4140: test apply with i-t-a paths\n - apply: make i-t-a entries never match worktree\n - apply: allow \"new file\" patches on i-t-a entries\n\n Recent versions of \"git diff-files\" shows a diff between the index\n and the working tree for \"intent-to-add\" paths as a \"new file\"\n patch; \"git apply --cached\" should be able to take \"git diff-files\"\n and should act as an equivalent to \"git add\" for the path, but the\n command failed to do so for such a path.\n\n Will merge to 'next'.\n\n\n* rp/blame-first-parent-doc (2020-08-06) 1 commit\n  (merged to 'next' on 2020-08-10 at 3fdbebe1ea)\n + blame-options.txt: document --first-parent option\n\n The \"git blame --first-parent\" option was not documented, but now\n it is.\n\n Will merge to 'master'.\n\n\n* jc/noop-withstatic-inline (2020-08-06) 1 commit\n - compat-util: type-check parameters of no-op replacement functions\n\n A no-op replacement function implemented as a C preprocessor macro\n does not perform as good a job as one implemented as a \"static\n inline\" function in catching errors in parameters; replace the\n former with the latter in <git-compat-util.h> header.\n\n\n* ps/ref-transaction-hook (2020-08-07) 1 commit\n  (merged to 'next' on 2020-08-10 at d8ad7cc8f6)\n + refs: fix interleaving hook calls with reference-transaction hook\n\n The logic to find the ref transaction hook script attempted to\n cache the path to the found hook without realizing that it needed\n to keep a copied value, as the API it used returned a transitory\n buffer space.  This has been corrected.\n\n Will merge to 'master'.\n to be followed by a removal of the caching feature, which does not\n seem to help even as a negative cache.\n\n\n* ss/submodule-summary-in-c (2020-08-06) 5 commits\n - submodule: port submodule subcommand 'summary' from shell to C\n - t7421: introduce a test script for verifying 'summary' output\n - submodule: rename helper functions to avoid ambiguity\n - submodule: remove extra line feeds between callback struct and macro\n - submodule: expose the '--for-status' option of summary\n\n Yet another subcommand of \"git submodule\" is getting rewritten in C.\n\n\n* am/ci-wsfix (2020-08-10) 1 commit\n - ci: fix inconsistent indentation\n\n\n* es/init-no-separate-git-dir-in-bare (2020-08-10) 1 commit\n - init: disallow --separate-git-dir with bare repository\n\n \"git init --separate-git-dir\" can be used in an existing repository\n with a working tree to move its .git/ directory away from the\n working tree.  Even though this re-init feature makes no sense in\n an existing bare repository, it was not erroring out.  Now it does.\n\n\n* es/test-cmp-typocatcher (2020-08-09) 1 commit\n - test_cmp: diagnose incorrect arguments\n\n Test framework update.\n\n Will merge to 'next'.\n\n\n* jk/sideband-error-l10n (2020-08-07) 1 commit\n - sideband: mark \"remote error:\" prefix for translation\n\n Mark error message for i18n.\n\n Will merge to 'next'.\n\n\n* rp/ita-diff-modefix (2020-08-09) 1 commit\n - diff-lib: use worktree mode in diffs from i-t-a entries\n\n \"git diff [<tree-ish>] $path\" for a $path that is marked with i-t-a\n bit was not showing the mode bits from the working tree.\n\n--------------------------------------------------\n[Stalled]\n\n* pw/rebase-i-more-options (2020-07-16) 5 commits\n - rebase: add --reset-author-date\n - rebase -i: support --ignore-date\n - sequencer: rename amend_author to author_to_free\n - rebase -i: support --committer-date-is-author-date\n - rebase -i: add --ignore-whitespace flag\n\n \"git rebase -i\" learns a bit more options.\n\n Waiting for a (hopefully final) review.\n\n\n* mt/grep-sparse-checkout (2020-06-12) 6 commits\n - config: add setting to ignore sparsity patterns in some cmds\n - grep: honor sparse checkout patterns\n - config: correctly read worktree configs in submodules\n - t/helper/test-config: facilitate addition of new cli options\n - t/helper/test-config: return exit codes consistently\n - doc: grep: unify info on configuration variables\n\n \"git grep\" has been tweaked to be limited to the sparse checkout\n paths.\n\n Review needed on 4/6; otherwise looking sane.\n cf. <CABPp-BGdEyEeajYZj_rdxp=MyEQdszuyjVTax=hhYj3fOtRQUQ@mail.gmail.com>\n\n\n* rs/more-buffered-io (2020-08-02) 3 commits\n - upload-pack: use buffered I/O to talk to rev-list\n - midx: use buffered I/O to talk to pack-objects\n - connected: use buffered I/O to talk to rev-list\n\n Expecting a reroll.\n\n\n* ls/mergetool-meld-auto-merge (2020-07-12) 2 commits\n - SQUASH???\n - Support auto-merge for meld to follow the vim-diff behavior\n\n The 'meld' backend of the \"git mergetool\" learned to give the\n underlying 'meld' the '--auto-merge' option, which would help\n reduce the amount of text that requires manual merging.\n\n Expecting a reroll.\n\n\n* jx/proc-receive-hook (2020-05-18) 11 commits\n . doc: add documentation for the proc-receive hook\n . transport: parse report options for tracking refs\n . t5411: test updates of remote-tracking branches\n . receive-pack: new config receive.procReceiveRefs\n . refs.c: refactor to reuse ref_is_hidden()\n . receive-pack: feed report options to post-receive\n . doc: add document for capability report-status-v2\n . New capability \"report-status-v2\" for git-push\n . receive-pack: add new proc-receive hook\n . t5411: add basic test cases for proc-receive hook\n . transport: not report a non-head push as a branch\n\n \"git receive-pack\" that accepts requests by \"git push\" learned to\n outsource most of the ref updates to the new \"proc-receive\" hook.\n\n Ejected out of 'seen'; somehow its tests seem to break with clang\n cf. https://travis-ci.org/github/git/git/builds/713443572\n\n\n* mf/submodule-summary-with-correct-repository (2020-06-24) 2 commits\n - submodule: use submodule repository when preparing summary\n - revision: use repository from rev_info when parsing commits\n\n \"git diff/show\" on a change that involves a submodule used to read\n the information on commits in the submodule from a wrong repository\n and gave a wrong information when the commit-graph is involved.\n\n Needs tests.\n\n\n* dr/push-remoteref-fix (2020-04-23) 1 commit\n - remote.c: fix handling of %(push:remoteref)\n\n The \"%(push:remoteref)\" placeholder in the \"--format=\" argument of\n \"git format-patch\" (and friends) only showed what got explicitly\n configured, not what ref at the receiving end would be updated when\n \"git push\" was used, as it ignored the default behaviour (e.g. update\n the same ref as the source).\n\n Expecting a reroll.\n cf. <20200416152145.wp2zeibxmuyas6y6@feanor>\n\n\n* mr/bisect-in-c-2 (2020-07-17) 14 commits\n . SQUASH??? do not add new users of git_path_bisect_head()\n . bisect--helper: retire `--bisect-autostart` subcommand\n . bisect--helper: retire `--write-terms` subcommand\n . bisect--helper: retire `--check-expected-revs` subcommand\n . bisect--helper: reimplement `bisect_state` & `bisect_head` shell functions in C\n . bisect--helper: retire `--next-all` subcommand\n . bisect--helper: retire `--bisect-clean-state` subcommand\n . bisect--helper: finish porting `bisect_start()` to C\n . bisect--helper: reimplement `bisect_next` and `bisect_auto_next` shell functions in C\n . bisect: call 'clear_commit_marks_all()' in 'bisect_next_all()'\n . bisect--helper: reimplement `bisect_autostart` shell function in C\n . bisect--helper: introduce new `write_in_file()` function\n . bisect--helper: use '-res' in 'cmd_bisect__helper' return\n . bisect--helper: BUG() in cmd_*() on invalid subcommand\n\n Rewrite of the remainder of \"git bisect\" script in C continues.\n\n Needs more work.\n Ejected out of 'seen'; al/bisect-first-parent topic has a bit of\n textual conflict with this topic.\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[Cooking]\n\n* es/worktree-doc-cleanups (2020-08-03) 5 commits\n  (merged to 'next' on 2020-08-04 at cbb53ca464)\n + git-worktree.txt: link to man pages when citing other Git commands\n + git-worktree.txt: make start of new sentence more obvious\n + git-worktree.txt: fix minor grammatical issues\n + git-worktree.txt: consistently use term \"working tree\"\n + git-worktree.txt: employ fixed-width typeface consistently\n\n Doc cleanup around \"worktree\".\n\n Will merge to 'master'.\n\n\n* jk/log-fp-implies-m (2020-07-29) 7 commits\n  (merged to 'next' on 2020-08-03 at 39fefa6b82)\n + doc/git-log: clarify handling of merge commit diffs\n + doc/git-log: move \"-t\" into diff-options list\n + doc/git-log: drop \"-r\" diff option\n + doc/git-log: move \"Diff Formatting\" from rev-list-options\n + log: enable \"-m\" automatically with \"--first-parent\"\n + revision: add \"--no-diff-merges\" option to counteract \"-m\"\n + log: drop \"--cc implies -m\" logic\n\n Originally merged to 'next' on 2020-08-01\n\n \"git log --first-parent -p\" showed patches only for single-parent\n commits on the first-parent chain; the \"--first-parent\" option has\n been made to imply \"-m\".  Use \"--no-diff-merges\" to restore the\n previous behaviour to omit patches for merge commits.\n\n On hold a bit.\n cf. <20200804200018.GB2014743@coredump.intra.peff.net>\n\n\n* pd/mergetool-nvimdiff (2020-07-29) 2 commits\n - mergetools: add support for nvimdiff (neovim) family\n - mergetool--lib: improve support for vimdiff-style tool variants\n\n The existing backends for \"git mergetool\" based on variants of vim\n have been refactored and then support for \"nvim\" has been added.\n\n Will merge to 'next'.\n\n\n* al/bisect-first-parent (2020-08-07) 5 commits\n - bisect: combine args passed to find_bisection()\n - bisect: introduce first-parent flag\n - cmd_bisect__helper: defer parsing no-checkout flag\n - rev-list: allow bisect and first-parent flags\n - t6030: modernize \"git bisect run\" tests\n\n \"git bisect\" learns the \"--first-parent\" option to find the first\n breakage along the first-parent chain.\n\n Will merge to 'next'.\n\n\n* dd/send-email-config (2020-07-23) 1 commit\n - git-send-email: die if sendmail.* config is set\n\n Stop when \"sendmail.*\" configuration variables are defined, which\n could be a mistaken attempt to define \"sendemail.*\" variables.\n\n Will merge to 'next'.\n\n\n* bc/sha-256-part-3 (2020-07-30) 39 commits\n  (merged to 'next' on 2020-08-04 at 57115e548f)\n + t: remove test_oid_init in tests\n + docs: add documentation for extensions.objectFormat\n + ci: run tests with SHA-256\n + t: make SHA1 prerequisite depend on default hash\n + t: allow testing different hash algorithms via environment\n + t: add test_oid option to select hash algorithm\n + repository: enable SHA-256 support by default\n + setup: add support for reading extensions.objectformat\n + bundle: add new version for use with SHA-256\n + builtin/verify-pack: implement an --object-format option\n + http-fetch: set up git directory before parsing pack hashes\n + t0410: mark test with SHA1 prerequisite\n + t5308: make test work with SHA-256\n + t9700: make hash size independent\n + t9500: ensure that algorithm info is preserved in config\n + t9350: make hash size independent\n + t9301: make hash size independent\n + t9300: use $ZERO_OID instead of hard-coded object ID\n + t9300: abstract away SHA-1-specific constants\n + t8011: make hash size independent\n + t8003: make hash size independent\n + t8002: make hash size independent\n + t7508: use $ZERO_OID instead of hard-coded constant\n + t7506: avoid checking for SHA-1-specific constants\n + t7405: make hash size independent\n + t7400: make hash size independent\n + t7102: abstract away SHA-1-specific constants\n + t7201: abstract away SHA-1-specific constants\n + t7063: make hash size independent\n + t7003: compute appropriate length constant\n + t6501: avoid hard-coded objects\n + t6500: specify test values for SHA-256\n + t6301: make hash size independent\n + t6101: make hash size independent\n + t6100: make hash size independent\n + t3404: prepare 'short SHA-1 collision' tests for SHA-256\n + t3305: make hash agnostic\n + t1001: use $ZERO_OID\n + t: make test-bloom initialize repository\n\n The final leg of SHA-256 transition.\n\n Will merge to 'master'.\n\n\n* hn/reftable-prep-part-2 (2020-07-27) 3 commits\n - Make HEAD a PSEUDOREF rather than PER_WORKTREE.\n - Modify pseudo refs through ref backend storage\n - t1400: use git rev-parse for testing PSEUDOREF existence\n\n Further preliminary change to refs API.\n\n Will merge to 'next'.\n\n\n* ds/maintenance (2020-08-06) 11 commits\n - maintenance: add trace2 regions for task execution\n - maintenance: add auto condition for commit-graph task\n - maintenance: use pointers to check --auto\n - maintenance: create maintenance.<task>.enabled config\n - maintenance: take a lock on the objects directory\n - maintenance: add --task option\n - maintenance: add commit-graph task\n - maintenance: initialize task array\n - maintenance: replace run_auto_gc()\n - maintenance: add --quiet option\n - maintenance: create basic maintenance runner\n (this branch is used by ds/maintenance-part-2.)\n\n A \"git gc\"'s big brother has been introduced to take care of more\n repository maintenance tasks, not limited to the object database\n cleaning.\n\n\n* tb/upload-pack-filters (2020-08-05) 4 commits\n  (merged to 'next' on 2020-08-05 at 918e7092fe)\n + t5616: use test_i18ngrep for upload-pack errors\n  (merged to 'next' on 2020-08-04 at 3ae57cc90c)\n + upload-pack.c: introduce 'uploadpackfilter.tree.maxDepth'\n + upload-pack.c: allow banning certain object filter(s)\n + list_objects_filter_options: introduce 'list_object_filter_config_name'\n\n The component to respond to \"git fetch\" request is made more\n configurable to selectively allow or reject object filtering\n specification used for partial cloning.\n\n Will merge to 'master'.\n cf. <20200804003722.GA2726931@coredump.intra.peff.net>\n\n\n* mt/hash-to-hex-thread-safety (2020-06-26) 2 commits\n - hex: make hash_to_hex_algop() and friends thread-safe\n - compat/win32/pthread: add pthread_once()\n\n hash_to_hex() used a set of rotating static buffers, which was not\n safe to use in a threaded environment.  This has been made safer by\n using thread-local storage.\n\n Expecting a reroll.\n cf. <CAHd-oW6A2aBHg80R9kyifvF7thwzam5EYYoN9d2TaBcHJrcKWw@mail.gmail.com>\n\n\n* ss/cmake-build (2020-06-26) 8 commits\n  (merged to 'next' on 2020-08-03 at a0d70165c1)\n + ci: modification of main.yml to use cmake for vs-build job\n + cmake: support for building git on windows with msvc and clang.\n + cmake: support for building git on windows with mingw\n + cmake: support for testing git when building out of the source tree\n + cmake: support for testing git with ctest\n + cmake: installation support for git\n + cmake: generate the shell/perl/python scripts and templates, translations\n + Introduce CMake support for configuring Git\n\n Originally merged to 'next' on 2020-08-01\n\n CMake support to build with MSVC for Windows bypassing the Makefile.\n\n Will merge to 'master'.\n\n\n* es/config-hooks (2020-07-30) 6 commits\n - hook: add 'run' subcommand\n - parse-options: parse into argv_array\n - hook: add --porcelain to list command\n - hook: add list command\n - hook: scaffolding for git-hook subcommand\n - doc: propose hooks managed by the config\n\n The \"hooks defined in config\" topic.\n\n Expecting a reroll.\n Now jk/strvec is in 'master', we may want to see the topic reworked\n on top of it.  Are there unresolved issues, or does the topic need\n a round of detailed review?\n cf. <xmqqmu3i9kvg.fsf@gitster.c.googlers.com>\n\n--------------------------------------------------\n[Discarded]\n\n* jk/fast-export-anonym (2020-06-22) 4 commits\n  (merged to 'next' on 2020-06-22 at b517b2f707)\n + fast-export: allow dumping the path mapping\n + fast-export: refactor path printing to not rely on stdout\n + fast-export: anonymize \"master\" refname\n + fast-export: allow dumping the refname mapping\n\n The way refnames are anonymized has been updated and a way to help\n debugging using the anonymized output hsa been added.\n\n Superseded by 'jk/fast-export-anonym-alt'.\n\n\n* jk/t6000-timestamp-fix (2020-07-07) 1 commit\n  (merged to 'next' on 2020-07-09 at 633bcd552f)\n + t6000: use test_tick consistently\n\n Test update.\n\n Now it is part of jk/tests-timestamp-fix with a larger scope.\n\n\n* jc/no-update-fetch-head (2020-07-29) 1 commit\n . fetch: optionally allow disabling FETCH_HEAD update\n\n \"git fetch\" learned the \"--[no-]write-fetch-head\" option to\n optionally stop describing what was fetched in FETCH_HEAD.\n\n Now it is part of ds/maintenance topic.\n"},{"id":"403331","messageId":"CAPig+cRtS3iMp3zLRKBOvEchrDPrCmozndbFpWtjiosk5nqGew@mail.gmail.com","threadId":"54013","inReplyTo":"xmqqsgcub811.fsf@gitster.c.googlers.com","subject":"Re: What's cooking in git.git (Aug 2020, #02; Mon, 10)","fromName":"Eric Sunshine","fromEmail":"sunshine@sunshineco.com","sentAt":"2020-08-11T05:01:27Z","receivedAt":"2020-08-11T05:01:41Z","isPatch":false,"sender":{"key":"sunshine@sunshineco.com","avatar":"https://avatars.githubusercontent.com/u/163641?v=4"},"body":"On Mon, Aug 10, 2020 at 3:45 PM Junio C Hamano <gitster@pobox.com> wrote:\n> * es/init-no-separate-git-dir-in-bare (2020-08-10) 1 commit\n> - init: disallow --separate-git-dir with bare repository\n>\n> \"git init --separate-git-dir\" can be used in an existing repository\n> with a working tree to move its .git/ directory away from the\n> working tree. Even though this re-init feature makes no sense in\n> an existing bare repository, it was not erroring out. Now it does.\n\nThis description is a bit misleading since it focuses only on existing\nrepositories, however, --separate-git-dir is intended for use with\nboth new and existing repositories. So, perhaps the description could\nbe rewritten something like this:\n\n    The purpose of \"git init --separate-git-dir\" is to initialize a\n    new project with the repository separate from the working tree,\n    or, in the case of an existing project, to move the repository\n    (the .git/ directory) out of the working tree. It does not make\n    sense to use --separate-git-dir with a bare repository for which\n    there is no working tree, so disallow its use with bare\n    repositories.\n"},{"id":"403337","messageId":"20200811065347.GA1486938@coredump.intra.peff.net","threadId":"54013","inReplyTo":"xmqqsgcub811.fsf@gitster.c.googlers.com","subject":"Re: What's cooking in git.git (Aug 2020, #02; Mon, 10)","fromName":"Jeff King","fromEmail":"peff@peff.net","sentAt":"2020-08-11T06:53:47Z","receivedAt":"2020-08-11T06:53:51Z","isPatch":false,"sender":{"key":"peff@peff.net","avatar":"https://avatars.githubusercontent.com/u/45925?v=4"},"body":"On Mon, Aug 10, 2020 at 12:44:26PM -0700, Junio C Hamano wrote:\n\n> * ps/ref-transaction-hook (2020-08-07) 1 commit\n>   (merged to 'next' on 2020-08-10 at d8ad7cc8f6)\n>  + refs: fix interleaving hook calls with reference-transaction hook\n> \n>  The logic to find the ref transaction hook script attempted to\n>  cache the path to the found hook without realizing that it needed\n>  to keep a copied value, as the API it used returned a transitory\n>  buffer space.  This has been corrected.\n> \n>  Will merge to 'master'.\n>  to be followed by a removal of the caching feature, which does not\n>  seem to help even as a negative cache.\n\nI noticed this had a CI problem. Fix is below.\n\n-- >8 --\nSubject: [PATCH] t1416: avoid hard-coded sha1 ids\n\nThe test added by e5256c82e5 (refs: fix interleaving hook calls with\nreference-transaction hook, 2020-08-07) uses hard-coded sha1 object ids\nin its expected output. This causes it to fail when run with\nGIT_TEST_DEFAULT_HASH=sha256.\n\nLet's make use of the oid variables we define earlier, as the rest of\nthe nearby tests do.\n\nSigned-off-by: Jeff King <peff@peff.net>\n---\n t/t1416-ref-transaction-hooks.sh | 5 +++--\n 1 file changed, 3 insertions(+), 2 deletions(-)\n\ndiff --git a/t/t1416-ref-transaction-hooks.sh b/t/t1416-ref-transaction-hooks.sh\nindex d4d19194bf..f6e741c6c0 100755\n--- a/t/t1416-ref-transaction-hooks.sh\n+++ b/t/t1416-ref-transaction-hooks.sh\n@@ -7,6 +7,7 @@ test_description='reference transaction hooks'\n test_expect_success setup '\n \tmkdir -p .git/hooks &&\n \ttest_commit PRE &&\n+\tPRE_OID=$(git rev-parse PRE) &&\n \ttest_commit POST &&\n \tPOST_OID=$(git rev-parse POST)\n '\n@@ -120,10 +121,10 @@ test_expect_success 'interleaving hook calls succeed' '\n \tEOF\n \n \tcat >expect <<-EOF &&\n-\t\thooks/update refs/tags/PRE 0000000000000000000000000000000000000000 63ac8e7bcdb882293465435909f54a96de17d4f7\n+\t\thooks/update refs/tags/PRE $ZERO_OID $PRE_OID\n \t\thooks/reference-transaction prepared\n \t\thooks/reference-transaction committed\n-\t\thooks/update refs/tags/POST 0000000000000000000000000000000000000000 99d53161c3a0a903b6561b9f6c0c665b3a476401\n+\t\thooks/update refs/tags/POST $ZERO_OID $POST_OID\n \t\thooks/reference-transaction prepared\n \t\thooks/reference-transaction committed\n \tEOF\n-- \n2.28.0.532.g36127779a4\n\n"},{"id":"403341","messageId":"20200811093346.GA33378@tanuki.pks.im","threadId":"54013","inReplyTo":"20200811065347.GA1486938@coredump.intra.peff.net","subject":"Re: What's cooking in git.git (Aug 2020, #02; Mon, 10)","fromName":"Patrick Steinhardt","fromEmail":"ps@pks.im","sentAt":"2020-08-11T09:33:46Z","receivedAt":"2020-08-11T09:33:50Z","isPatch":false,"sender":{"key":"ps@pks.im","avatar":"https://avatars.githubusercontent.com/u/4056630?v=4"},"body":"On Tue, Aug 11, 2020 at 02:53:47AM -0400, Jeff King wrote:\n> On Mon, Aug 10, 2020 at 12:44:26PM -0700, Junio C Hamano wrote:\n> \n> > * ps/ref-transaction-hook (2020-08-07) 1 commit\n> >   (merged to 'next' on 2020-08-10 at d8ad7cc8f6)\n> >  + refs: fix interleaving hook calls with reference-transaction hook\n> > \n> >  The logic to find the ref transaction hook script attempted to\n> >  cache the path to the found hook without realizing that it needed\n> >  to keep a copied value, as the API it used returned a transitory\n> >  buffer space.  This has been corrected.\n> > \n> >  Will merge to 'master'.\n> >  to be followed by a removal of the caching feature, which does not\n> >  seem to help even as a negative cache.\n> \n> I noticed this had a CI problem. Fix is below.\n> \n> -- >8 --\n> Subject: [PATCH] t1416: avoid hard-coded sha1 ids\n> \n> The test added by e5256c82e5 (refs: fix interleaving hook calls with\n> reference-transaction hook, 2020-08-07) uses hard-coded sha1 object ids\n> in its expected output. This causes it to fail when run with\n> GIT_TEST_DEFAULT_HASH=sha256.\n> \n> Let's make use of the oid variables we define earlier, as the rest of\n> the nearby tests do.\n> \n> Signed-off-by: Jeff King <peff@peff.net>\n> ---\n>  t/t1416-ref-transaction-hooks.sh | 5 +++--\n>  1 file changed, 3 insertions(+), 2 deletions(-)\n> \n> diff --git a/t/t1416-ref-transaction-hooks.sh b/t/t1416-ref-transaction-hooks.sh\n> index d4d19194bf..f6e741c6c0 100755\n> --- a/t/t1416-ref-transaction-hooks.sh\n> +++ b/t/t1416-ref-transaction-hooks.sh\n> @@ -7,6 +7,7 @@ test_description='reference transaction hooks'\n>  test_expect_success setup '\n>  \tmkdir -p .git/hooks &&\n>  \ttest_commit PRE &&\n> +\tPRE_OID=$(git rev-parse PRE) &&\n>  \ttest_commit POST &&\n>  \tPOST_OID=$(git rev-parse POST)\n>  '\n> @@ -120,10 +121,10 @@ test_expect_success 'interleaving hook calls succeed' '\n>  \tEOF\n>  \n>  \tcat >expect <<-EOF &&\n> -\t\thooks/update refs/tags/PRE 0000000000000000000000000000000000000000 63ac8e7bcdb882293465435909f54a96de17d4f7\n> +\t\thooks/update refs/tags/PRE $ZERO_OID $PRE_OID\n>  \t\thooks/reference-transaction prepared\n>  \t\thooks/reference-transaction committed\n> -\t\thooks/update refs/tags/POST 0000000000000000000000000000000000000000 99d53161c3a0a903b6561b9f6c0c665b3a476401\n> +\t\thooks/update refs/tags/POST $ZERO_OID $POST_OID\n>  \t\thooks/reference-transaction prepared\n>  \t\thooks/reference-transaction committed\n>  \tEOF\n> -- \n> 2.28.0.532.g36127779a4\n\nReviewed-by: Patrick Steinhardt <ps@pks.im>\n\nThanks for cleaning up after me!\n\nPatrick\n"},{"id":"403342","messageId":"20200811095942.GG8085@camp.crustytoothpaste.net","threadId":"54013","inReplyTo":"20200811065347.GA1486938@coredump.intra.peff.net","subject":"Re: What's cooking in git.git (Aug 2020, #02; Mon, 10)","fromName":"brian m. carlson","fromEmail":"sandals@crustytoothpaste.net","sentAt":"2020-08-11T09:59:42Z","receivedAt":"2020-08-11T10:00:20Z","isPatch":false,"sender":{"key":"sandals@crustytoothpaste.net","avatar":"https://avatars.githubusercontent.com/u/497054?v=4"},"body":"On 2020-08-11 at 06:53:47, Jeff King wrote:\n> diff --git a/t/t1416-ref-transaction-hooks.sh b/t/t1416-ref-transaction-hooks.sh\n> index d4d19194bf..f6e741c6c0 100755\n> --- a/t/t1416-ref-transaction-hooks.sh\n> +++ b/t/t1416-ref-transaction-hooks.sh\n> @@ -7,6 +7,7 @@ test_description='reference transaction hooks'\n>  test_expect_success setup '\n>  \tmkdir -p .git/hooks &&\n>  \ttest_commit PRE &&\n> +\tPRE_OID=$(git rev-parse PRE) &&\n>  \ttest_commit POST &&\n>  \tPOST_OID=$(git rev-parse POST)\n>  '\n> @@ -120,10 +121,10 @@ test_expect_success 'interleaving hook calls succeed' '\n>  \tEOF\n>  \n>  \tcat >expect <<-EOF &&\n> -\t\thooks/update refs/tags/PRE 0000000000000000000000000000000000000000 63ac8e7bcdb882293465435909f54a96de17d4f7\n> +\t\thooks/update refs/tags/PRE $ZERO_OID $PRE_OID\n>  \t\thooks/reference-transaction prepared\n>  \t\thooks/reference-transaction committed\n> -\t\thooks/update refs/tags/POST 0000000000000000000000000000000000000000 99d53161c3a0a903b6561b9f6c0c665b3a476401\n> +\t\thooks/update refs/tags/POST $ZERO_OID $POST_OID\n>  \t\thooks/reference-transaction prepared\n>  \t\thooks/reference-transaction committed\n>  \tEOF\n\nYeah, this looks reasonable.  I'm glad we've got CI working on the\nSHA-256 code now to catch these kinds of things.\n-- \nbrian m. carlson: Houston, Texas, US\n"},{"id":"403343","messageId":"874kp9biho.fsf@osv.gnss.ru","threadId":"54013","inReplyTo":"xmqqsgcub811.fsf@gitster.c.googlers.com","subject":"Re: What's cooking in git.git (Aug 2020, #02; Mon, 10)","fromName":"Sergey Organov","fromEmail":"sorganov@gmail.com","sentAt":"2020-08-11T10:10:43Z","receivedAt":"2020-08-11T10:10:50Z","isPatch":false,"sender":{"key":"sorganov@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8501568?v=4"},"body":"Junio C Hamano <gitster@pobox.com> writes:\n\n[...]\n\n> * jk/log-fp-implies-m (2020-07-29) 7 commits\n>   (merged to 'next' on 2020-08-03 at 39fefa6b82)\n>  + doc/git-log: clarify handling of merge commit diffs\n>  + doc/git-log: move \"-t\" into diff-options list\n>  + doc/git-log: drop \"-r\" diff option\n>  + doc/git-log: move \"Diff Formatting\" from rev-list-options\n>  + log: enable \"-m\" automatically with \"--first-parent\"\n>  + revision: add \"--no-diff-merges\" option to counteract \"-m\"\n>  + log: drop \"--cc implies -m\" logic\n>\n>  Originally merged to 'next' on 2020-08-01\n>\n>  \"git log --first-parent -p\" showed patches only for single-parent\n>  commits on the first-parent chain; the \"--first-parent\" option has\n>  been made to imply \"-m\".  Use \"--no-diff-merges\" to restore the\n>  previous behaviour to omit patches for merge commits.\n>\n>  On hold a bit.\n>  cf. <20200804200018.GB2014743@coredump.intra.peff.net>\n\nI believe I've submitted resolution for that.\n\n<20200805220832.3800-1-sorganov@gmail.com>\n\nDo you expect anything from me to push all this further?\n\nThanks,\n-- Sergey\n"},{"id":"403354","messageId":"20200811134855.GA19871@syl.lan","threadId":"54013","inReplyTo":"xmqqsgcub811.fsf@gitster.c.googlers.com","subject":"Re: What's cooking in git.git (Aug 2020, #02; Mon, 10)","fromName":"Taylor Blau","fromEmail":"me@ttaylorr.com","sentAt":"2020-08-11T13:48:55Z","receivedAt":"2020-08-11T13:49:01Z","isPatch":false,"sender":{"key":"me@ttaylorr.com","avatar":"https://avatars.githubusercontent.com/u/301000140?v=4"},"body":"On Mon, Aug 10, 2020 at 12:44:26PM -0700, Junio C Hamano wrote:\n> Here are the topics that have been cooking.  Commits prefixed with '-' are\n> only in 'seen' (formerly 'pu'---proposed updates) while commits prefixed\n> with '+' are in 'next'.  The ones marked with '.' do not appear in any of\n> the integration branches, but I am still holding onto them.\n\nI noticed that you picked up 'tb/bloom-improvements' in your fork, but\nthat it didn't make it into the list of cooking topics below. I suspect\nthat this is because of [1], where I got bitten by trying to sneak in an\nextra 'bitmap_free()' at the last moment.\n\nI have a fixup in [2] which needs to be squashed in at that point in the\nseries, along with dropping a '&' in the last patch to squelch a\ncompiler error introduced by the fixup.\n\nDo you want me to resend the whole series, or would you rather apply the\nfixups when queuing? Either is fine for me.\n\n> * tb/upload-pack-filters (2020-08-05) 4 commits\n>   (merged to 'next' on 2020-08-05 at 918e7092fe)\n>  + t5616: use test_i18ngrep for upload-pack errors\n>   (merged to 'next' on 2020-08-04 at 3ae57cc90c)\n>  + upload-pack.c: introduce 'uploadpackfilter.tree.maxDepth'\n>  + upload-pack.c: allow banning certain object filter(s)\n>  + list_objects_filter_options: introduce 'list_object_filter_config_name'\n>\n>  The component to respond to \"git fetch\" request is made more\n>  configurable to selectively allow or reject object filtering\n>  specification used for partial cloning.\n>\n>  Will merge to 'master'.\n>  cf. <20200804003722.GA2726931@coredump.intra.peff.net>\n\nThanks, I'm glad that this made it in.\n\nThanks,\nTaylor\n\n[1]: https://lore.kernel.org/git/xmqqbljoolhy.fsf@gitster.c.googlers.com/\n[2]: https://lore.kernel.org/git/20200811134807.GA26715@syl.lan/\n"},{"id":"403356","messageId":"CAHd-oW7Wd8oSaMhPFeRcEeKTJ-k_hC7b6e28efhXT5LFu1E_Uw@mail.gmail.com","threadId":"54013","inReplyTo":"xmqqsgcub811.fsf@gitster.c.googlers.com","subject":"Re: What's cooking in git.git (Aug 2020, #02; Mon, 10)","fromName":"Matheus Tavares Bernardino","fromEmail":"matheus.bernardino@usp.br","sentAt":"2020-08-11T14:27:04Z","receivedAt":"2020-08-11T14:27:22Z","isPatch":false,"sender":{"key":"matheus.tavb@gmail.com","avatar":"https://avatars.githubusercontent.com/u/12701583?v=4"},"body":"Hi, Junio\n\nOn Mon, Aug 10, 2020 at 4:44 PM Junio C Hamano <gitster@pobox.com> wrote:\n>\n> * mt/hash-to-hex-thread-safety (2020-06-26) 2 commits\n>  - hex: make hash_to_hex_algop() and friends thread-safe\n>  - compat/win32/pthread: add pthread_once()\n>\n>  hash_to_hex() used a set of rotating static buffers, which was not\n>  safe to use in a threaded environment.  This has been made safer by\n>  using thread-local storage.\n>\n>  Expecting a reroll.\n>  cf. <CAHd-oW6A2aBHg80R9kyifvF7thwzam5EYYoN9d2TaBcHJrcKWw@mail.gmail.com>\n\nI think we can drop this one. Dscho suggested using a different\napproach for this conversion, with Coccinelle [1]. I won't be able to\nlook at it right now, but if someone wants to give it a try, [2] would\nbe a good starting point.\n\n[1]: https://lore.kernel.org/git/nycvar.QRO.7.76.6.2007161214270.54@tvgsbejvaqbjf.bet/\n[2]: https://lore.kernel.org/git/a35a9334-6db2-d8e3-8ce5-15f15a9005e1@web.de/\n"},{"id":"403381","messageId":"xmqqbljh9egf.fsf@gitster.c.googlers.com","threadId":"54013","inReplyTo":"CAHd-oW7Wd8oSaMhPFeRcEeKTJ-k_hC7b6e28efhXT5LFu1E_Uw@mail.gmail.com","subject":"Re: What's cooking in git.git (Aug 2020, #02; Mon, 10)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2020-08-11T19:20:48Z","receivedAt":"2020-08-11T19:20:56Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Matheus Tavares Bernardino <matheus.bernardino@usp.br> writes:\n\n> Hi, Junio\n>\n> On Mon, Aug 10, 2020 at 4:44 PM Junio C Hamano <gitster@pobox.com> wrote:\n>>\n>> * mt/hash-to-hex-thread-safety (2020-06-26) 2 commits\n>>  - hex: make hash_to_hex_algop() and friends thread-safe\n>>  - compat/win32/pthread: add pthread_once()\n>>\n>>  hash_to_hex() used a set of rotating static buffers, which was not\n>>  safe to use in a threaded environment.  This has been made safer by\n>>  using thread-local storage.\n>>\n>>  Expecting a reroll.\n>>  cf. <CAHd-oW6A2aBHg80R9kyifvF7thwzam5EYYoN9d2TaBcHJrcKWw@mail.gmail.com>\n>\n> I think we can drop this one.\n\nOK.  Thanks.\n"},{"id":"403382","messageId":"xmqq7du59eb4.fsf@gitster.c.googlers.com","threadId":"54013","inReplyTo":"874kp9biho.fsf@osv.gnss.ru","subject":"Re: What's cooking in git.git (Aug 2020, #02; Mon, 10)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2020-08-11T19:23:59Z","receivedAt":"2020-08-11T19:24:08Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Sergey Organov <sorganov@gmail.com> writes:\n\n> Junio C Hamano <gitster@pobox.com> writes:\n>\n> [...]\n>\n>> * jk/log-fp-implies-m (2020-07-29) 7 commits\n>>   (merged to 'next' on 2020-08-03 at 39fefa6b82)\n>>  + doc/git-log: clarify handling of merge commit diffs\n>>  + doc/git-log: move \"-t\" into diff-options list\n>>  + doc/git-log: drop \"-r\" diff option\n>>  + doc/git-log: move \"Diff Formatting\" from rev-list-options\n>>  + log: enable \"-m\" automatically with \"--first-parent\"\n>>  + revision: add \"--no-diff-merges\" option to counteract \"-m\"\n>>  + log: drop \"--cc implies -m\" logic\n>>\n>>  Originally merged to 'next' on 2020-08-01\n>>\n>>  \"git log --first-parent -p\" showed patches only for single-parent\n>>  commits on the first-parent chain; the \"--first-parent\" option has\n>>  been made to imply \"-m\".  Use \"--no-diff-merges\" to restore the\n>>  previous behaviour to omit patches for merge commits.\n>>\n>>  On hold a bit.\n>>  cf. <20200804200018.GB2014743@coredump.intra.peff.net>\n>\n> I believe I've submitted resolution for that.\n>\n> <20200805220832.3800-1-sorganov@gmail.com>\n>\n> Do you expect anything from me to push all this further?\n\nI was waiting for a reroll of the \"resolution\" with just one extra\ntest I mentioned in <xmqq3650n2rc.fsf@gitster.c.googlers.com> in the\nthread.  With that everything would be good to go, I would think.\n\n"},{"id":"403383","messageId":"xmqq364t9ead.fsf@gitster.c.googlers.com","threadId":"54013","inReplyTo":"CAPig+cRtS3iMp3zLRKBOvEchrDPrCmozndbFpWtjiosk5nqGew@mail.gmail.com","subject":"Re: What's cooking in git.git (Aug 2020, #02; Mon, 10)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2020-08-11T19:24:26Z","receivedAt":"2020-08-11T19:24:34Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Eric Sunshine <sunshine@sunshineco.com> writes:\n\n> On Mon, Aug 10, 2020 at 3:45 PM Junio C Hamano <gitster@pobox.com> wrote:\n>> * es/init-no-separate-git-dir-in-bare (2020-08-10) 1 commit\n>> - init: disallow --separate-git-dir with bare repository\n>>\n>> \"git init --separate-git-dir\" can be used in an existing repository\n>> with a working tree to move its .git/ directory away from the\n>> working tree. Even though this re-init feature makes no sense in\n>> an existing bare repository, it was not erroring out. Now it does.\n>\n> This description is a bit misleading since it focuses only on existing\n> repositories, however, --separate-git-dir is intended for use with\n> both new and existing repositories. So, perhaps the description could\n> be rewritten something like this:\n>\n>     The purpose of \"git init --separate-git-dir\" is to initialize a\n>     new project with the repository separate from the working tree,\n>     or, in the case of an existing project, to move the repository\n>     (the .git/ directory) out of the working tree. It does not make\n>     sense to use --separate-git-dir with a bare repository for which\n>     there is no working tree, so disallow its use with bare\n>     repositories.\n\nThanks.\n"},{"id":"403385","messageId":"87364t563r.fsf@osv.gnss.ru","threadId":"54013","inReplyTo":"xmqq7du59eb4.fsf@gitster.c.googlers.com","subject":"Re: What's cooking in git.git (Aug 2020, #02; Mon, 10)","fromName":"Sergey Organov","fromEmail":"sorganov@gmail.com","sentAt":"2020-08-11T19:34:48Z","receivedAt":"2020-08-11T19:34:54Z","isPatch":false,"sender":{"key":"sorganov@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8501568?v=4"},"body":"Junio C Hamano <gitster@pobox.com> writes:\n\n> Sergey Organov <sorganov@gmail.com> writes:\n>\n>> Junio C Hamano <gitster@pobox.com> writes:\n>>\n>> [...]\n>>\n>>> * jk/log-fp-implies-m (2020-07-29) 7 commits\n>>>   (merged to 'next' on 2020-08-03 at 39fefa6b82)\n>>>  + doc/git-log: clarify handling of merge commit diffs\n>>>  + doc/git-log: move \"-t\" into diff-options list\n>>>  + doc/git-log: drop \"-r\" diff option\n>>>  + doc/git-log: move \"Diff Formatting\" from rev-list-options\n>>>  + log: enable \"-m\" automatically with \"--first-parent\"\n>>>  + revision: add \"--no-diff-merges\" option to counteract \"-m\"\n>>>  + log: drop \"--cc implies -m\" logic\n>>>\n>>>  Originally merged to 'next' on 2020-08-01\n>>>\n>>>  \"git log --first-parent -p\" showed patches only for single-parent\n>>>  commits on the first-parent chain; the \"--first-parent\" option has\n>>>  been made to imply \"-m\".  Use \"--no-diff-merges\" to restore the\n>>>  previous behaviour to omit patches for merge commits.\n>>>\n>>>  On hold a bit.\n>>>  cf. <20200804200018.GB2014743@coredump.intra.peff.net>\n>>\n>> I believe I've submitted resolution for that.\n>>\n>> <20200805220832.3800-1-sorganov@gmail.com>\n>>\n>> Do you expect anything from me to push all this further?\n>\n> I was waiting for a reroll of the \"resolution\" with just one extra\n> test I mentioned in <xmqq3650n2rc.fsf@gitster.c.googlers.com> in the\n> thread.  With that everything would be good to go, I would think.\n\nAnd I asked you to postpone that in <875z9v7b0k.fsf@osv.gnss.ru> for\nwhich I got no answer. I repeat it here for convenience:\n\n\"\nFor now, can we leave these patches as-is and then add additional tests\non top, please? I'm afraid that adding them now will force me to rewrite\ncurrent descriptions, as the result won't be minimal anymore, and I\nfeel uneasy about writing commit messages for these permuted tests\nanyway.\n\"\n\nThanks,\n-- Sergey\n"},{"id":"403386","messageId":"87tux93pku.fsf@osv.gnss.ru","threadId":"54013","inReplyTo":"87364t563r.fsf@osv.gnss.ru","subject":"Re: What's cooking in git.git (Aug 2020, #02; Mon, 10)","fromName":"Sergey Organov","fromEmail":"sorganov@gmail.com","sentAt":"2020-08-11T20:17:05Z","receivedAt":"2020-08-11T20:17:12Z","isPatch":false,"sender":{"key":"sorganov@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8501568?v=4"},"body":"Sergey Organov <sorganov@gmail.com> writes:\n\n> Junio C Hamano <gitster@pobox.com> writes:\n>\n>> Sergey Organov <sorganov@gmail.com> writes:\n>>\n>>> Junio C Hamano <gitster@pobox.com> writes:\n>>>\n>>> [...]\n>>>\n>>>> * jk/log-fp-implies-m (2020-07-29) 7 commits\n>>>>   (merged to 'next' on 2020-08-03 at 39fefa6b82)\n>>>>  + doc/git-log: clarify handling of merge commit diffs\n>>>>  + doc/git-log: move \"-t\" into diff-options list\n>>>>  + doc/git-log: drop \"-r\" diff option\n>>>>  + doc/git-log: move \"Diff Formatting\" from rev-list-options\n>>>>  + log: enable \"-m\" automatically with \"--first-parent\"\n>>>>  + revision: add \"--no-diff-merges\" option to counteract \"-m\"\n>>>>  + log: drop \"--cc implies -m\" logic\n>>>>\n>>>>  Originally merged to 'next' on 2020-08-01\n>>>>\n>>>>  \"git log --first-parent -p\" showed patches only for single-parent\n>>>>  commits on the first-parent chain; the \"--first-parent\" option has\n>>>>  been made to imply \"-m\".  Use \"--no-diff-merges\" to restore the\n>>>>  previous behaviour to omit patches for merge commits.\n>>>>\n>>>>  On hold a bit.\n>>>>  cf. <20200804200018.GB2014743@coredump.intra.peff.net>\n>>>\n>>> I believe I've submitted resolution for that.\n>>>\n>>> <20200805220832.3800-1-sorganov@gmail.com>\n>>>\n>>> Do you expect anything from me to push all this further?\n>>\n>> I was waiting for a reroll of the \"resolution\" with just one extra\n>> test I mentioned in <xmqq3650n2rc.fsf@gitster.c.googlers.com> in the\n>> thread.  With that everything would be good to go, I would think.\n\nHere is the patch you've asked for, separately, as I honestly believe it\ndoes not belong to the series in question themselves. This is on top of\nthe series.\n\n-- >8 --\nSubject: [PATCH] t/t4013: add permutation test for --diff-merges vs --first-parent\n\nSigned-off-by: Sergey Organov <sorganov@gmail.com>\n---\n t/t4013-diff-various.sh                       |  1 +\n ...--first-parent_--diff-merges=off_-p_master | 78 +++++++++++++++++++\n 2 files changed, 79 insertions(+)\n create mode 100644 t/t4013/diff.log_--first-parent_--diff-merges=off_-p_master\n\ndiff --git a/t/t4013-diff-various.sh b/t/t4013-diff-various.sh\nindex 86fb11cecc61..7f7f1058cb86 100755\n--- a/t/t4013-diff-various.sh\n+++ b/t/t4013-diff-various.sh\n@@ -299,6 +299,7 @@ log --root -c --patch-with-stat --summary master\n log --root --cc --patch-with-stat --summary master\n log --no-diff-merges -p --first-parent master\n log --diff-merges=off -p --first-parent master\n+log --first-parent --diff-merges=off -p master\n log -p --first-parent master\n log -m -p --first-parent master\n log -m -p master\ndiff --git a/t/t4013/diff.log_--first-parent_--diff-merges=off_-p_master b/t/t4013/diff.log_--first-parent_--diff-merges=off_-p_master\nnew file mode 100644\nindex 000000000000..ad2e6d3be22c\n--- /dev/null\n+++ b/t/t4013/diff.log_--first-parent_--diff-merges=off_-p_master\n@@ -0,0 +1,78 @@\n+$ git log --first-parent --diff-merges=off -p master\n+commit 80e25ffa65bcdbe82ef654b4d06dbbde7945c37f\n+Merge: 9a6d494 c7a2ab9\n+Author: A U Thor <author@example.com>\n+Date:   Mon Jun 26 00:04:00 2006 +0000\n+\n+    Merge branch 'side' into master\n+\n+commit 9a6d4949b6b76956d9d5e26f2791ec2ceff5fdc0\n+Author: A U Thor <author@example.com>\n+Date:   Mon Jun 26 00:02:00 2006 +0000\n+\n+    Third\n+\n+diff --git a/dir/sub b/dir/sub\n+index 8422d40..cead32e 100644\n+--- a/dir/sub\n++++ b/dir/sub\n+@@ -2,3 +2,5 @@ A\n+ B\n+ C\n+ D\n++E\n++F\n+diff --git a/file1 b/file1\n+new file mode 100644\n+index 0000000..b1e6722\n+--- /dev/null\n++++ b/file1\n+@@ -0,0 +1,3 @@\n++A\n++B\n++C\n+\n+commit 1bde4ae5f36c8d9abe3a0fce0c6aab3c4a12fe44\n+Author: A U Thor <author@example.com>\n+Date:   Mon Jun 26 00:01:00 2006 +0000\n+\n+    Second\n+    \n+    This is the second commit.\n+\n+diff --git a/dir/sub b/dir/sub\n+index 35d242b..8422d40 100644\n+--- a/dir/sub\n++++ b/dir/sub\n+@@ -1,2 +1,4 @@\n+ A\n+ B\n++C\n++D\n+diff --git a/file0 b/file0\n+index 01e79c3..b414108 100644\n+--- a/file0\n++++ b/file0\n+@@ -1,3 +1,6 @@\n+ 1\n+ 2\n+ 3\n++4\n++5\n++6\n+diff --git a/file2 b/file2\n+deleted file mode 100644\n+index 01e79c3..0000000\n+--- a/file2\n++++ /dev/null\n+@@ -1,3 +0,0 @@\n+-1\n+-2\n+-3\n+\n+commit 444ac553ac7612cc88969031b02b3767fb8a353a\n+Author: A U Thor <author@example.com>\n+Date:   Mon Jun 26 00:00:00 2006 +0000\n+\n+    Initial\n+$\n-- \n2.20.1\n"},{"id":"403387","messageId":"xmqqy2ml7wrd.fsf@gitster.c.googlers.com","threadId":"54013","inReplyTo":"87364t563r.fsf@osv.gnss.ru","subject":"Re: What's cooking in git.git (Aug 2020, #02; Mon, 10)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2020-08-11T20:28:22Z","receivedAt":"2020-08-11T20:28:27Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Sergey Organov <sorganov@gmail.com> writes:\n\n> And I asked you to postpone that in <875z9v7b0k.fsf@osv.gnss.ru> for\n> which I got no answer. I repeat it here for convenience:\n>\n> \"\n> For now, can we leave these patches as-is and then add additional tests\n> on top, please? I'm afraid that adding them now will force me to rewrite\n> current descriptions, as the result won't be minimal anymore, and I\n> feel uneasy about writing commit messages for these permuted tests\n> anyway.\n> \"\n\nI was expecting just one case I mentioned in the first message and\nnothing else, which is crucial to highlight how --first-parent's\n\"implication\" works with explicit --diff-merges.\n\nFull \"permutated tests\" was what you brought up in the discussion,\nbut I already said it was overkill to your response before you said\nthe above.  The \"minimal\" would be the posted patch plus a missing\n\"first-parent comes before --diff-merges=off\" test filled in.\n"},{"id":"403412","messageId":"87lfik50xy.fsf@osv.gnss.ru","threadId":"54013","inReplyTo":"xmqqy2ml7wrd.fsf@gitster.c.googlers.com","subject":"Re: What's cooking in git.git (Aug 2020, #02; Mon, 10)","fromName":"Sergey Organov","fromEmail":"sorganov@gmail.com","sentAt":"2020-08-11T21:26:17Z","receivedAt":"2020-08-11T21:26:23Z","isPatch":false,"sender":{"key":"sorganov@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8501568?v=4"},"body":"Junio C Hamano <gitster@pobox.com> writes:\n\n> Sergey Organov <sorganov@gmail.com> writes:\n>\n>> And I asked you to postpone that in <875z9v7b0k.fsf@osv.gnss.ru> for\n>> which I got no answer. I repeat it here for convenience:\n>>\n>> \"\n>> For now, can we leave these patches as-is and then add additional tests\n>> on top, please? I'm afraid that adding them now will force me to rewrite\n>> current descriptions, as the result won't be minimal anymore, and I\n>> feel uneasy about writing commit messages for these permuted tests\n>> anyway.\n>> \"\n>\n> I was expecting just one case I mentioned in the first message and\n> nothing else, which is crucial to highlight how --first-parent's\n> \"implication\" works with explicit --diff-merges.\n>\n> Full \"permutated tests\" was what you brought up in the discussion,\n> but I already said it was overkill to your response before you said\n> the above.  The \"minimal\" would be the posted patch plus a missing\n> \"first-parent comes before --diff-merges=off\" test filled in.\n\nI've sent the patch before I saw this, please see\n<87tux93pku.fsf@osv.gnss.ru>.\n\nIs it OK with you?\n\nThanks,\n-- Sergey\n"},{"id":"403415","messageId":"xmqqh7t89870.fsf@gitster.c.googlers.com","threadId":"54013","inReplyTo":"87tux93pku.fsf@osv.gnss.ru","subject":"Re: What's cooking in git.git (Aug 2020, #02; Mon, 10)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2020-08-11T21:36:03Z","receivedAt":"2020-08-11T21:36:11Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Sergey Organov <sorganov@gmail.com> writes:\n\n> Here is the patch you've asked for, separately, as I honestly believe it\n> does not belong to the series in question themselves. This is on top of\n> the series.\n\nWell, the tip commit's title being \"add test for --diff-merges=off\",\ntesting it standalone as well as in different ways to combine with\nanother option that potentially interacts with it would make the\ncommit complete, I would think.\n\nThe sample output shows that the explicit --diff-merges=off told\n\"git log\" not to show any patch for merge commits, which is what we\nwant to see, so it looks good.\n\nThanks.\n\n"},{"id":"403417","messageId":"xmqqd03w96hk.fsf@gitster.c.googlers.com","threadId":"54013","inReplyTo":"87lfik50xy.fsf@osv.gnss.ru","subject":"Re: What's cooking in git.git (Aug 2020, #02; Mon, 10)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2020-08-11T22:12:55Z","receivedAt":"2020-08-11T22:13:03Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Sergey Organov <sorganov@gmail.com> writes:\n\n> I've sent the patch before I saw this, please see\n> <87tux93pku.fsf@osv.gnss.ru>.\n\nThanks, looking good.\n"},{"id":"403436","messageId":"87eeoce0uh.fsf@osv.gnss.ru","threadId":"54013","inReplyTo":"xmqqh7t89870.fsf@gitster.c.googlers.com","subject":"Re: What's cooking in git.git (Aug 2020, #02; Mon, 10)","fromName":"Sergey Organov","fromEmail":"sorganov@gmail.com","sentAt":"2020-08-12T08:15:50Z","receivedAt":"2020-08-12T08:15:57Z","isPatch":false,"sender":{"key":"sorganov@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8501568?v=4"},"body":"Junio C Hamano <gitster@pobox.com> writes:\n\n> Sergey Organov <sorganov@gmail.com> writes:\n>\n>> Here is the patch you've asked for, separately, as I honestly believe it\n>> does not belong to the series in question themselves. This is on top of\n>> the series.\n>\n> Well, the tip commit's title being \"add test for --diff-merges=off\",\n> testing it standalone as well as in different ways to combine with\n> another option that potentially interacts with it would make the\n> commit complete, I would think.\n\nMy point is that this set of commits is essentially \"make\n--diff-merges=on a synonym for --no-diff-merges\", so it should only\ninclude the same test(s) that exist for --no-diff-merges.\n\nIf new tests are to be added, they should have been added before the\nseries, or after them, with their own reasoning, I think. Yeah, not a\nbig deal, just a bit of perfectionism here.\n\nThanks,\n-- Sergey\n"}]}