{"thread":{"id":"53805","subject":"What's cooking in git.git (Jul 2020, #01; Mon, 6)","startedAt":"2020-07-07T05:57:19Z","lastAt":"2020-07-10T21:38:37Z","messageCount":15,"participants":["Junio C Hamano","Derrick Stolee","Taylor Blau","SZEDER Gábor","Johannes Schindelin"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"401069","messageId":"xmqqh7uj7tqn.fsf@gitster.c.googlers.com","threadId":"53805","inReplyTo":null,"subject":"What's cooking in git.git (Jul 2020, #01; Mon, 6)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2020-07-07T05:57:04Z","receivedAt":"2020-07-07T05:57:19Z","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\nWe are about to tag the -rc0 \"preview\" and enter pre-release freeze\nperiod.  The change to let users name the initial branch that gets\nprepared in a newly created repository has now been merged to be\nincluded in the upcoming release.\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* ak/commit-graph-to-slab (2020-06-17) 4 commits\n  (merged to 'next' on 2020-06-25 at 9a10ae18d3)\n + commit-graph: minimize commit_graph_data_slab access\n + commit: move members graph_pos, generation to a slab\n + commit-graph: introduce commit_graph_data_slab\n + object: drop parsed_object_pool->commit_count\n\n A few fields in \"struct commit\" that do not have to always be\n present have been moved to commit slabs.\n\n\n* bc/http-push-flagsfix (2020-06-23) 1 commit\n  (merged to 'next' on 2020-06-25 at 1dabb1f61b)\n + http-push: ensure unforced pushes fail when data would be lost\n\n The code to push changes over \"dumb\" HTTP had a bad interaction\n with the commit reachability code due to incorrect allocation of\n object flag bits, which has been corrected.\n\n\n* bc/sha-256-cvs-svn-updates (2020-06-22) 14 commits\n  (merged to 'next' on 2020-06-25 at e0d1c87fcf)\n + git-cvsexportcommit: port to SHA-256\n + git-cvsimport: port to SHA-256\n + git-cvsserver: port to SHA-256\n + git-svn: set the OID length based on hash algorithm\n + perl: make SVN code hash independent\n + perl: make Git::IndexInfo work with SHA-256\n + perl: create and switch variables for hash constants\n + t/lib-git-svn: make hash size independent\n + t9101: make hash independent\n + t9104: make hash size independent\n + t9100: make test work with SHA-256\n + t9108: make test hash independent\n + t9168: make test hash independent\n + t9109: make test hash independent\n (this branch uses bc/sha-256-part-2.)\n\n CVS/SVN interface have been prepared for SHA-256 transition\n\n\n* bc/sha-256-part-2 (2020-06-19) 44 commits\n  (merged to 'next' on 2020-06-22 at dc153f366d)\n + remote-testgit: adapt for object-format\n + bundle: detect hash algorithm when reading refs\n + t5300: pass --object-format to git index-pack\n + t5704: send object-format capability with SHA-256\n + t5703: use object-format serve option\n + t5702: offer an object-format capability in the test\n + t/helper: initialize the repository for test-sha1-array\n + remote-curl: avoid truncating refs with ls-remote\n + t1050: pass algorithm to index-pack when outside repo\n + builtin/index-pack: add option to specify hash algorithm\n + remote-curl: detect algorithm for dumb HTTP by size\n + builtin/ls-remote: initialize repository based on fetch\n + t5500: make hash independent\n + serve: advertise object-format capability for protocol v2\n + connect: parse v2 refs with correct hash algorithm\n + connect: pass full packet reader when parsing v2 refs\n + Documentation/technical: document object-format for protocol v2\n + t1302: expect repo format version 1 for SHA-256\n + builtin/show-index: provide options to determine hash algo\n + t5302: modernize test formatting\n + packfile: compute and use the index CRC offset\n + t3200: mark assertion with SHA1 prerequisite\n + setup: set the_repository's hash algo when checking format\n + fetch-pack: parse and advertise the object-format capability\n + t5562: pass object-format in synthesized test data\n + builtin/clone: initialize hash algorithm properly\n + remote-curl: implement object-format extensions\n + transport-helper: implement object-format extensions\n + docs: update remote helper docs for object-format extensions\n + builtin/receive-pack: detect when the server doesn't support our hash\n + connect: detect algorithm when fetching refs\n + fetch-pack: detect when the server doesn't support our hash\n + connect: make parse_feature_value extern\n + send-pack: detect when the server doesn't support our hash\n + connect: add function to detect supported v1 hash functions\n + transport: add a hash algorithm member\n + pkt-line: add a member for hash algorithm\n + connect: add function to fetch value of a v2 server capability\n + connect: add function to parse multiple v1 capability values\n + remote: advertise the object-format capability on the server side\n + wrapper: add function to compare strings with different NUL termination\n + connect: have ref processing code take struct packet_reader\n + Documentation: document v1 protocol object-format capability\n + t1050: match object ID paths in a hash-insensitive way\n (this branch is used by bc/sha-256-cvs-svn-updates.)\n\n SHA-256 migration work continues.\n\n\n* cb/is-descendant-of (2020-06-23) 1 commit\n  (merged to 'next' on 2020-06-25 at ca0eaddf5e)\n + commit-reach: avoid is_descendant_of() shim\n\n Code clean-up.\n\n\n* dl/test-must-fail-fixes-5 (2020-06-24) 4 commits\n  (merged to 'next' on 2020-06-29 at efe53a7588)\n + lib-submodule-update: pass 'test_must_fail' as an argument\n + lib-submodule-update: prepend \"git\" to $command\n + lib-submodule-update: consolidate --recurse-submodules\n + lib-submodule-update: add space after function name\n\n The effort to avoid using test_must_fail on non-git command continues.\n\n\n* en/sparse-status (2020-06-22) 3 commits\n  (merged to 'next' on 2020-06-25 at 404403a075)\n + git-prompt: include sparsity state as well\n + git-prompt: document how in-progress operations affect the prompt\n + wt-status: show sparse checkout status as well\n\n \"git status\" learned to report the status of sparse checkout.\n\n\n* es/get-worktrees-unsort (2020-06-22) 2 commits\n  (merged to 'next' on 2020-06-25 at 732d93a36d)\n + worktree: drop get_worktrees() unused 'flags' argument\n + worktree: drop get_worktrees() special-purpose sorting option\n\n API cleanup for get_worktrees()\n\n\n* es/worktree-code-cleanup (2020-06-24) 1 commit\n  (merged to 'next' on 2020-06-29 at 91ffd85a31)\n + worktree: avoid dead-code in conditional\n\n Code cleanup.\n\n\n* jk/fast-export-anonym-alt (2020-06-25) 11 commits\n  (merged to 'next' on 2020-06-29 at 8a8d83e2ca)\n + fast-export: use local array to store anonymized oid\n + fast-export: anonymize \"master\" refname\n + fast-export: allow seeding the anonymized mapping\n + fast-export: add a \"data\" callback parameter to anonymize_str()\n + fast-export: move global \"idents\" anonymize hashmap into function\n + fast-export: use a flex array to store anonymized entries\n + fast-export: stop storing lengths in anonymized hashmaps\n + fast-export: tighten anonymize_mem() interface to handle only strings\n + fast-export: store anonymized oids as hex strings\n + fast-export: use xmemdupz() for anonymizing oids\n + t9351: derive anonymized tree checks from original repo\n\n \"git fast-export --anonymize\" learned to take customized mapping to\n allow its users to tweak its output more usable for debugging.\n\n\n* jl/complete-git-prune (2020-06-22) 1 commit\n  (merged to 'next' on 2020-06-25 at 64f30b8b41)\n + bash-completion: add git-prune into bash completion\n\n Add \"git prune\" to the completion (in contrib/), which could be\n typed by end-users from the command line.\n\n\n* js/default-branch-name (2020-06-30) 10 commits\n  (merged to 'next' on 2020-07-01 at c38d766581)\n + contrib: subtree: adjust test to change in fmt-merge-msg\n  (merged to 'next' on 2020-06-25 at 8f962f9219)\n + testsvn: respect `init.defaultBranch`\n + remote: use the configured default branch name when appropriate\n + clone: use configured default branch name when appropriate\n + init: allow setting the default for the initial branch name via the config\n + init: allow specifying the initial branch name for the new repository\n + docs: add missing diamond brackets\n + submodule: fall back to remote's HEAD for missing remote.<name>.branch\n + send-pack/transport-helper: avoid mentioning a particular branch\n + fmt-merge-msg: stop treating `master` specially\n\n The name of the primary branch in existing repositories, and the\n default name used for the first branch in newly created\n repositories, is made configurable, so that we can eventually wean\n ourselves off of the hardcoded 'master'.\n\n\n* js/diff-files-i-t-a-fix-for-difftool (2020-07-01) 2 commits\n  (merged to 'next' on 2020-07-01 at cf5d18635c)\n + difftool -d: ensure that intent-to-add files are handled correctly\n + diff-files --raw: show correct post-image of intent-to-add files\n\n \"git difftool\" has trouble dealing with paths added to the index\n with the intent-to-add bit.\n\n\n* js/pu-to-seen (2020-06-25) 3 commits\n  (merged to 'next' on 2020-06-25 at 06c9e5a885)\n + tests: reference `seen` wherever `pu` was referenced\n + docs: adjust the technical overview for the rename `pu` -> `seen`\n + docs: adjust for the recent rename of `pu` to `seen`\n\n The documentation and some tests have been adjusted for the recent\n renaming of \"pu\" branch to \"seen\".\n\n\n* mk/pb-pretty-email-without-domain-part-fix (2020-06-23) 1 commit\n  (merged to 'next' on 2020-06-25 at 81a3aca46b)\n + doc: fix author vs. committer copy/paste error\n\n Docfix.\n\n\n* ps/ref-transaction-hook (2020-06-19) 1 commit\n  (merged to 'next' on 2020-06-22 at 3a23dcdbdc)\n + refs: implement reference transaction hook\n\n A new hook.\n\n\n* rs/pack-bits-in-object-better (2020-06-24) 1 commit\n  (merged to 'next' on 2020-06-25 at b3cb296341)\n + revision: reallocate TOPO_WALK object flags\n\n By renumbering object flag bits, \"struct object\" managed to lose\n bloated inter-field padding.\n\n--------------------------------------------------\n[New Topics]\n\n* bw/fail-cloning-into-non-empty (2020-07-06) 1 commit\n - git clone: don't clone into non-empty directory\n\n \"git clone --separate-git-dir=$elsewhere\" used to stomp on the\n contents of the existing directory $elsewhere, which has been\n taught to fail when $elsewhere is not an empty directory.\n\n Expecting a reroll.\n with improved commit log message.\n cf. <xmqqv9j08dxo.fsf@gitster.c.googlers.com>\n\n\n* cc/cat-file-usage-update (2020-07-01) 1 commit\n  (merged to 'next' on 2020-07-06 at 270769a009)\n + cat-file: add missing [=<format>] to usage/synopsis\n\n Doc/usage update.\n\n Will merge to 'master'.\n\n\n* cc/pretty-contents-size (2020-07-06) 2 commits\n - ref-filter: add support for %(contents:size)\n - Documentation: clarify %(contents:XXXX) doc\n\n \"git for-each-ref --format=<>\" learned %(contents:size).\n\n Expecting a reroll.\n with clarified semantics and tests with various object types\n cf. <xmqq5zb09v4e.fsf@gitster.c.googlers.com>\n\n\n* ls/mergetool-meld-auto-merge (2020-07-06) 1 commit\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 with final fixes and readability improvements.\n cf. <xmqqzh8c8eda.fsf@gitster.c.googlers.com>\n\n\n* pb/log-rev-list-doc (2020-07-06) 4 commits\n - rev-list-description.txt: fix Asciidoc syntax\n - git-log.txt: include rev-list-description.txt\n - git-rev-list.txt: move description to separate file\n - git-log.txt: add links to 'rev-list' and 'diff' docs\n\n \"git help log\" has been enhanced by sharing more material from the\n documentation for the underlying \"git rev-list\" command.\n\n Expecting a reroll.\n with reordering the last one to make it preparatory clean-up.\n cf. <xmqqimf087pq.fsf@gitster.c.googlers.com>\n cf. <xmqqeepo87hz.fsf@gitster.c.googlers.com>\n\n\n* rs/line-log-until (2020-07-06) 1 commit\n  (merged to 'next' on 2020-07-06 at 84d6bc67fa)\n + revision: disable min_age optimization with line-log\n\n \"git log -Lx,y:path --before=date\" lost track of where the range\n should be because it didn't take the changes made by the youngest\n commits that are omitted from the output into account.\n\n Will merge to 'master'.\n\n\n* ta/wait-on-aliased-commands-upon-signal (2020-07-06) 2 commits\n - Wait for child on signal death for aliases to externals\n - Wait for child on signal death for aliases to builtins\n\n When an aliased command, whose output is piped to a pager by git,\n gets killed by a signal, the pager got into a funny state, which\n has been corrected (again).\n\n Expecting a reroll.\n with improved log messages.\n cf. <20200706211403.GB85133@coredump.intra.peff.net>\n\n\n* tb/upload-pack-filters (2020-07-06) 4 commits\n - upload-pack.c: introduce 'uploadpack.filter.tree.maxDepth'\n - upload-pack.c: pass 'struct list_objects_filter_options *'\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 Waiting for reviews.\n cf. <cover.1593720075.git.me@ttaylorr.com>\n\n\n* vs/completion-with-set-u (2020-07-01) 1 commit\n  (merged to 'next' on 2020-07-06 at 72b1eca511)\n + completion: nounset mode fixes\n\n The command line completion support (in contrib/) used to be\n prepared to work with \"set -u\" but recent changes got a bit more\n sloppy.  This has been corrected.\n\n Will merge to 'master'.\n\n--------------------------------------------------\n[Stalled]\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-04-23) 12 commits\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--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: fix `cmd_*()` function switch default return\n\n Rewrite of the remainder of \"git bisect\" script in C continues.\n\n Expecting a response to reviews.\n cf. <nycvar.QRO.7.76.6.2005230007260.56@tvgsbejvaqbjf.bet>\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* 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* 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* ra/send-email-in-reply-to-from-command-line-wins (2020-07-01) 1 commit\n  (merged to 'next' on 2020-07-06 at 3892a1ab88)\n + send-email: restore --in-reply-to superseding behavior\n\n \"git send-email --in-reply-to=<msg>\" did not use the In-Reply-To:\n header with the value given from the command line, and let it be\n overridden by the value on In-Reply-To: header in the messages\n being sent out (if exists).\n\n Will merge to 'master'.\n\n\n* ds/commit-graph-bloom-updates (2020-07-01) 10 commits\n  (merged to 'next' on 2020-07-06 at 177e6b362e)\n + commit-graph: check all leading directories in changed path Bloom filters\n + revision: empty pathspecs should not use Bloom filters\n + revision.c: fix whitespace\n + commit-graph: check chunk sizes after writing\n + commit-graph: simplify chunk writes into loop\n + commit-graph: unify the signatures of all write_graph_chunk_*() functions\n + commit-graph: persist existence of changed-paths\n + bloom: fix logic in get_bloom_filter()\n + commit-graph: change test to die on parse, not load\n + commit-graph: place bloom_settings in context\n (this branch uses sg/commit-graph-cleanups.)\n\n Updates to the changed-paths bloom filter.\n\n Will merge to 'master'.\n\n\n* ss/cmake-build (2020-06-26) 8 commits\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 CMake support to build with MSVC for Windows bypassing the Makefile.\n\n Waiting for a (hopefully final) review.\n\n\n* sg/commit-graph-cleanups (2020-06-08) 10 commits\n + commit-graph: simplify write_commit_graph_file() #2\n + commit-graph: simplify write_commit_graph_file() #1\n + commit-graph: simplify parse_commit_graph() #2\n + commit-graph: simplify parse_commit_graph() #1\n + commit-graph: clean up #includes\n + diff.h: drop diff_tree_oid() & friends' return value\n + commit-slab: add a function to deep free entries on the slab\n + commit-graph-format.txt: all multi-byte numbers are in network byte order\n + commit-graph: fix parsing the Chunk Lookup table\n + tree-walk.c: don't match submodule entries for 'submod/anything'\n (this branch is used by ds/commit-graph-bloom-updates.)\n\n The changed-path Bloom filter is improved using ideas from an\n independent implementation.\n\n Under review.\n cf. <20200627155610.GN2898@szeder.dev>\n cf. <20200627163335.GO2898@szeder.dev>\n cf. <20200627155348.GM2898@szeder.dev>\n\n\n* es/config-hooks (2020-05-21) 4 commits\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 What's the status of this one?  Abandoned?\n\n\n* pw/rebase-i-more-options (2020-06-26) 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* 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 Needs review.\n\n\n* hn/reftable (2020-07-06) 4 commits\n - Treat BISECT_HEAD as a pseudo ref\n - checkout: add '\\n' to reflog message\n - t3432: use git-reflog to inspect the reflog for HEAD\n - lib-t6000.sh: write tag using git-update-ref\n\n Preliminary clean-up of the refs API in preparation for adding a\n new refs backend \"reftable\".\n\n Expecting a reroll.\n The second one should be dropped, and possibly replaced with a\n patch that moves message normalization from backends to the more\n generic layer.\n cf. <xmqqmu4ca31g.fsf@gitster.c.googlers.com>\n\n--------------------------------------------------\n[Discarded]\n\n* jc/credential-store-file-format-doc (2020-04-27) 1 commit\n . credential-store: document the file format a bit more\n\n Now has become a part of Carlo's credential-store fix patches.\n\n\n* js/ci-skip-on-github-workflow (2020-05-02) 1 commit\n . ci: respect the [skip ci] convention in our GitHub workflow \"CI/PR\"\n\n Allow contributors to mark a branch/push that it does not have to\n be built via GitHub actions, in a way similar to how Travis lets\n them mark the commits with an embedded \"[skip ci]\" string.\n\n Superseded by dd/ci-only-on-selective-branches topic.\n\n\n* dd/ci-only-on-selective-branches (2020-05-05) 2 commits\n - CI: limit GitHub Actions to designated branches\n - SubmittingPatches: advertise GitHub Actions CI\n\n Instead of always building all branches of all forks of our project\n at GitHub via GitHub Actions, only build when branches with known\n and specific names are updated, and also a pull request.\n\n Superseded by jk/ci-only-on-selected-branches\n\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"},{"id":"401096","messageId":"90cf9ac2-1ab0-1c9b-706b-6c5176ab47dd@gmail.com","threadId":"53805","inReplyTo":"xmqqh7uj7tqn.fsf@gitster.c.googlers.com","subject":"sg/commit-graph-cleanups (was Re: What's cooking in git.git (Jul 2020, #01; Mon, 6))","fromName":"Derrick Stolee","fromEmail":"stolee@gmail.com","sentAt":"2020-07-07T13:29:29Z","receivedAt":"2020-07-07T13:29:32Z","isPatch":false,"sender":{"key":"stolee@gmail.com","avatar":"https://avatars.githubusercontent.com/u/570044?v=4"},"body":"On 7/7/2020 1:57 AM, Junio C Hamano wrote:\n> * ds/commit-graph-bloom-updates (2020-07-01) 10 commits\n>   (merged to 'next' on 2020-07-06 at 177e6b362e)\n>  + commit-graph: check all leading directories in changed path Bloom filters\n>  + revision: empty pathspecs should not use Bloom filters\n>  + revision.c: fix whitespace\n>  + commit-graph: check chunk sizes after writing\n>  + commit-graph: simplify chunk writes into loop\n>  + commit-graph: unify the signatures of all write_graph_chunk_*() functions\n>  + commit-graph: persist existence of changed-paths\n>  + bloom: fix logic in get_bloom_filter()\n>  + commit-graph: change test to die on parse, not load\n>  + commit-graph: place bloom_settings in context\n>  (this branch uses sg/commit-graph-cleanups.)\n> \n>  Updates to the changed-paths bloom filter.\n> \n>  Will merge to 'master'.\n> \n> * sg/commit-graph-cleanups (2020-06-08) 10 commits\n>  + commit-graph: simplify write_commit_graph_file() #2\n>  + commit-graph: simplify write_commit_graph_file() #1\n>  + commit-graph: simplify parse_commit_graph() #2\n>  + commit-graph: simplify parse_commit_graph() #1\n>  + commit-graph: clean up #includes\n>  + diff.h: drop diff_tree_oid() & friends' return value\n>  + commit-slab: add a function to deep free entries on the slab\n>  + commit-graph-format.txt: all multi-byte numbers are in network byte order\n>  + commit-graph: fix parsing the Chunk Lookup table\n>  + tree-walk.c: don't match submodule entries for 'submod/anything'\n>  (this branch is used by ds/commit-graph-bloom-updates.)\n> \n>  The changed-path Bloom filter is improved using ideas from an\n>  independent implementation.\n> \n>  Under review.\n>  cf. <20200627155610.GN2898@szeder.dev>\n>  cf. <20200627163335.GO2898@szeder.dev>\n>  cf. <20200627155348.GM2898@szeder.dev>\n\nI see these are both in 'next' (thanks!).\n\nPerhaps the \"Under review\" status for sg/commit-graph-cleanups\nshould be modified to match ds/commit-graph-bloom-updates?\n\nThanks,\n-Stolee\n"},{"id":"401123","messageId":"xmqq4kqj744z.fsf@gitster.c.googlers.com","threadId":"53805","inReplyTo":"90cf9ac2-1ab0-1c9b-706b-6c5176ab47dd@gmail.com","subject":"Re: sg/commit-graph-cleanups (was Re: What's cooking in git.git (Jul 2020, #01; Mon, 6))","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2020-07-07T15:10:04Z","receivedAt":"2020-07-07T15:10:13Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Derrick Stolee <stolee@gmail.com> writes:\n\n> Perhaps the \"Under review\" status for sg/commit-graph-cleanups\n> should be modified to match ds/commit-graph-bloom-updates?\n\nThanks.  Will do.\n"},{"id":"401124","messageId":"xmqqzh8b5pgv.fsf@gitster.c.googlers.com","threadId":"53805","inReplyTo":"90cf9ac2-1ab0-1c9b-706b-6c5176ab47dd@gmail.com","subject":"Re: sg/commit-graph-cleanups (was Re: What's cooking in git.git (Jul 2020, #01; Mon, 6))","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2020-07-07T15:12:16Z","receivedAt":"2020-07-07T15:12:22Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Derrick Stolee <stolee@gmail.com> writes:\n\n> On 7/7/2020 1:57 AM, Junio C Hamano wrote:\n>> * ds/commit-graph-bloom-updates (2020-07-01) 10 commits\n>>   (merged to 'next' on 2020-07-06 at 177e6b362e)\n>>  + commit-graph: check all leading directories in changed path Bloom filters\n>>  + revision: empty pathspecs should not use Bloom filters\n>>  + revision.c: fix whitespace\n>>  + commit-graph: check chunk sizes after writing\n>>  + commit-graph: simplify chunk writes into loop\n>>  + commit-graph: unify the signatures of all write_graph_chunk_*() functions\n>>  + commit-graph: persist existence of changed-paths\n>>  + bloom: fix logic in get_bloom_filter()\n>>  + commit-graph: change test to die on parse, not load\n>>  + commit-graph: place bloom_settings in context\n>>  (this branch uses sg/commit-graph-cleanups.)\n>> \n>>  Updates to the changed-paths bloom filter.\n>> \n>>  Will merge to 'master'.\n>> \n>> * sg/commit-graph-cleanups (2020-06-08) 10 commits\n>>  + commit-graph: simplify write_commit_graph_file() #2\n>>  + commit-graph: simplify write_commit_graph_file() #1\n>>  + commit-graph: simplify parse_commit_graph() #2\n>>  + commit-graph: simplify parse_commit_graph() #1\n>>  + commit-graph: clean up #includes\n>>  + diff.h: drop diff_tree_oid() & friends' return value\n>>  + commit-slab: add a function to deep free entries on the slab\n>>  + commit-graph-format.txt: all multi-byte numbers are in network byte order\n>>  + commit-graph: fix parsing the Chunk Lookup table\n>>  + tree-walk.c: don't match submodule entries for 'submod/anything'\n>>  (this branch is used by ds/commit-graph-bloom-updates.)\n>> \n>>  The changed-path Bloom filter is improved using ideas from an\n>>  independent implementation.\n>> \n>>  Under review.\n>>  cf. <20200627155610.GN2898@szeder.dev>\n>>  cf. <20200627163335.GO2898@szeder.dev>\n>>  cf. <20200627155348.GM2898@szeder.dev>\n>\n> I see these are both in 'next' (thanks!).\n>\n> Perhaps the \"Under review\" status for sg/commit-graph-cleanups\n> should be modified to match ds/commit-graph-bloom-updates?\n\nOops, I am not sure what happened here.  There I thought were issues\npointed out at least on the latter series but was I hallucinating?\n\nFor now, as -rc0 is so close, I'm tempted to mark both of them to be\ncooked in 'next' during the rc period.\n\nThanks.\n"},{"id":"401129","messageId":"7ce9163b-eafe-20a8-bd8e-7042807c98f0@gmail.com","threadId":"53805","inReplyTo":"xmqqzh8b5pgv.fsf@gitster.c.googlers.com","subject":"Re: sg/commit-graph-cleanups (was Re: What's cooking in git.git (Jul 2020, #01; Mon, 6))","fromName":"Derrick Stolee","fromEmail":"stolee@gmail.com","sentAt":"2020-07-07T16:00:12Z","receivedAt":"2020-07-07T16:00:15Z","isPatch":false,"sender":{"key":"stolee@gmail.com","avatar":"https://avatars.githubusercontent.com/u/570044?v=4"},"body":"On 7/7/2020 11:12 AM, Junio C Hamano wrote:\n> Derrick Stolee <stolee@gmail.com> writes:\n> \n>> On 7/7/2020 1:57 AM, Junio C Hamano wrote:\n>>> * ds/commit-graph-bloom-updates (2020-07-01) 10 commits\n>>>   (merged to 'next' on 2020-07-06 at 177e6b362e)\n>>>  + commit-graph: check all leading directories in changed path Bloom filters\n>>>  + revision: empty pathspecs should not use Bloom filters\n>>>  + revision.c: fix whitespace\n>>>  + commit-graph: check chunk sizes after writing\n>>>  + commit-graph: simplify chunk writes into loop\n>>>  + commit-graph: unify the signatures of all write_graph_chunk_*() functions\n>>>  + commit-graph: persist existence of changed-paths\n>>>  + bloom: fix logic in get_bloom_filter()\n>>>  + commit-graph: change test to die on parse, not load\n>>>  + commit-graph: place bloom_settings in context\n>>>  (this branch uses sg/commit-graph-cleanups.)\n>>>\n>>>  Updates to the changed-paths bloom filter.\n>>>\n>>>  Will merge to 'master'.\n>>>\n>>> * sg/commit-graph-cleanups (2020-06-08) 10 commits\n>>>  + commit-graph: simplify write_commit_graph_file() #2\n>>>  + commit-graph: simplify write_commit_graph_file() #1\n>>>  + commit-graph: simplify parse_commit_graph() #2\n>>>  + commit-graph: simplify parse_commit_graph() #1\n>>>  + commit-graph: clean up #includes\n>>>  + diff.h: drop diff_tree_oid() & friends' return value\n>>>  + commit-slab: add a function to deep free entries on the slab\n>>>  + commit-graph-format.txt: all multi-byte numbers are in network byte order\n>>>  + commit-graph: fix parsing the Chunk Lookup table\n>>>  + tree-walk.c: don't match submodule entries for 'submod/anything'\n>>>  (this branch is used by ds/commit-graph-bloom-updates.)\n>>>\n>>>  The changed-path Bloom filter is improved using ideas from an\n>>>  independent implementation.\n>>>\n>>>  Under review.\n>>>  cf. <20200627155610.GN2898@szeder.dev>\n>>>  cf. <20200627163335.GO2898@szeder.dev>\n>>>  cf. <20200627155348.GM2898@szeder.dev>\n>>\n>> I see these are both in 'next' (thanks!).\n>>\n>> Perhaps the \"Under review\" status for sg/commit-graph-cleanups\n>> should be modified to match ds/commit-graph-bloom-updates?\n> \n> Oops, I am not sure what happened here.  There I thought were issues\n> pointed out at least on the latter series but was I hallucinating?\n\nI believe I have resolved all the open comments in both series.\n\nLooking back, I suppose I never responded on the feedback to\n\"commit-slab: add a function to deep free entries on the slab\" that\nthe deep free is never called. The intention is to add a free\neventually, but at the current point it is not appropriate.\n\n> For now, as -rc0 is so close, I'm tempted to mark both of them to be\n> cooked in 'next' during the rc period.\n\nThis is completely acceptable. No need to rush this into the RC. I'm\nhappy to let this cook and do more small improvements on top in the\n2.29 cycle, if necessary.\n\nThanks,\n-Stolee\n"},{"id":"401131","messageId":"xmqqimez5l5q.fsf@gitster.c.googlers.com","threadId":"53805","inReplyTo":"7ce9163b-eafe-20a8-bd8e-7042807c98f0@gmail.com","subject":"Re: sg/commit-graph-cleanups (was Re: What's cooking in git.git (Jul 2020, #01; Mon, 6))","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2020-07-07T16:45:21Z","receivedAt":"2020-07-07T16:45:29Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Derrick Stolee <stolee@gmail.com> writes:\n\n>> Oops, I am not sure what happened here.  There I thought were issues\n>> pointed out at least on the latter series but was I hallucinating?\n>\n> I believe I have resolved all the open comments in both series.\n\nOK, then I do not have to be worried.\n\nLet's merge it down before -rc0.\n\nThanks.\n\n\n"},{"id":"401132","messageId":"20200707164953.GA36941@syl.lan","threadId":"53805","inReplyTo":"xmqqimez5l5q.fsf@gitster.c.googlers.com","subject":"Re: sg/commit-graph-cleanups (was Re: What's cooking in git.git (Jul 2020, #01; Mon, 6))","fromName":"Taylor Blau","fromEmail":"me@ttaylorr.com","sentAt":"2020-07-07T16:49:53Z","receivedAt":"2020-07-07T16:49:58Z","isPatch":false,"sender":{"key":"me@ttaylorr.com","avatar":"https://avatars.githubusercontent.com/u/301000140?v=4"},"body":"On Tue, Jul 07, 2020 at 09:45:21AM -0700, Junio C Hamano wrote:\n> Derrick Stolee <stolee@gmail.com> writes:\n>\n> >> Oops, I am not sure what happened here.  There I thought were issues\n> >> pointed out at least on the latter series but was I hallucinating?\n> >\n> > I believe I have resolved all the open comments in both series.\n>\n> OK, then I do not have to be worried.\n>\n> Let's merge it down before -rc0.\n\nStolee and I did find a few unrelated bugs with Bloom filters in the\npast day or two while starting to use them at GitHub. The biggest is\nthat Bloom filters are not used if the top-most incremental layer does\nnot have Bloom data.\n\nI'm not sure if this is a show-stopping bug or not, but if it is, I'd be\nhappy to send a patch or two on top before -rc0.\n\nThe other bug is less serious, which is that we do reuse Bloom data from\nprevious graphs even when writing a new graph with `--split=replace`.\nThe fix is more involved, so I'm less inclined to rush it before -rc0.\n\nLet me know your thoughts. Thanks.\n\n> Thanks.\n\nTaylor\n"},{"id":"401136","messageId":"20200707172123.GD11341@szeder.dev","threadId":"53805","inReplyTo":"xmqqimez5l5q.fsf@gitster.c.googlers.com","subject":"Re: sg/commit-graph-cleanups (was Re: What's cooking in git.git (Jul 2020, #01; Mon, 6))","fromName":"SZEDER Gábor","fromEmail":"szeder.dev@gmail.com","sentAt":"2020-07-07T17:21:23Z","receivedAt":"2020-07-07T17:21:30Z","isPatch":false,"sender":{"key":"szeder.dev@gmail.com","avatar":"https://avatars.githubusercontent.com/u/116324?v=4"},"body":"On Tue, Jul 07, 2020 at 09:45:21AM -0700, Junio C Hamano wrote:\n> Derrick Stolee <stolee@gmail.com> writes:\n> \n> >> Oops, I am not sure what happened here.  There I thought were issues\n> >> pointed out at least on the latter series but was I hallucinating?\n> >\n> > I believe I have resolved all the open comments in both series.\n> \n> OK, then I do not have to be worried.\n> \n> Let's merge it down before -rc0.\n\nI think it was merged to next too soon.  I haven't even done with\nreporting all the issues that were already in v2.27.0, let alone\nlooking through these followup series...\n\n"},{"id":"401143","messageId":"xmqqa70b5h3w.fsf@gitster.c.googlers.com","threadId":"53805","inReplyTo":"20200707172123.GD11341@szeder.dev","subject":"Re: sg/commit-graph-cleanups (was Re: What's cooking in git.git (Jul 2020, #01; Mon, 6))","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2020-07-07T18:12:51Z","receivedAt":"2020-07-07T18:12:59Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"SZEDER Gábor <szeder.dev@gmail.com> writes:\n\n> On Tue, Jul 07, 2020 at 09:45:21AM -0700, Junio C Hamano wrote:\n>> Derrick Stolee <stolee@gmail.com> writes:\n>> \n>> >> Oops, I am not sure what happened here.  There I thought were issues\n>> >> pointed out at least on the latter series but was I hallucinating?\n>> >\n>> > I believe I have resolved all the open comments in both series.\n>> \n>> OK, then I do not have to be worried.\n>> \n>> Let's merge it down before -rc0.\n>\n> I think it was merged to next too soon.  I haven't even done with\n> reporting all the issues that were already in v2.27.0, let alone\n> looking through these followup series...\n\nOh, so what's your preference?  Reverting all the commit-graph stuff\nthat is not in 2.26 may be tempting but is not practically feasible\nthis late in the cycle.  It sounded like Derrick was OK to leave\nthem in 'next', and I am fine with keeping them out of the upcoming\nrelease.\n"},{"id":"401186","messageId":"nycvar.QRO.7.76.6.2007081630180.50@tvgsbejvaqbjf.bet","threadId":"53805","inReplyTo":"xmqqh7uj7tqn.fsf@gitster.c.googlers.com","subject":"mr/bisect-in-c-2, was Re: What's cooking in git.git (Jul 2020, #01; Mon, 6)","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2020-07-08T14:31:16Z","receivedAt":"2020-07-08T14:31:30Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi Junio,\n\nOn Mon, 6 Jul 2020, Junio C Hamano wrote:\n\n> * mr/bisect-in-c-2 (2020-04-23) 12 commits\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--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: fix `cmd_*()` function switch default return\n>\n>  Rewrite of the remainder of \"git bisect\" script in C continues.\n>\n>  Expecting a response to reviews.\n>  cf. <nycvar.QRO.7.76.6.2005230007260.56@tvgsbejvaqbjf.bet>\n\nMaybe you could change this to\n\n   Needs more reviews.\n   cf. <20200701133504.18360-1-mirucam@gmail.com>\n\n?\n\nThanks,\nDscho\n"},{"id":"401189","messageId":"xmqq36622eru.fsf@gitster.c.googlers.com","threadId":"53805","inReplyTo":"nycvar.QRO.7.76.6.2007081630180.50@tvgsbejvaqbjf.bet","subject":"Re: mr/bisect-in-c-2, was Re: What's cooking in git.git (Jul 2020, #01; Mon, 6)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2020-07-08T15:43:49Z","receivedAt":"2020-07-08T15:43:56Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:\n\n>>  Rewrite of the remainder of \"git bisect\" script in C continues.\n>>\n>>  Expecting a response to reviews.\n>>  cf. <nycvar.QRO.7.76.6.2005230007260.56@tvgsbejvaqbjf.bet>\n>\n> Maybe you could change this to\n>\n>    Needs more reviews.\n>    cf. <20200701133504.18360-1-mirucam@gmail.com>\n>\n> ?\n\nThanks, absolutely.\n"},{"id":"401205","messageId":"xmqqy2ntzjoi.fsf@gitster.c.googlers.com","threadId":"53805","inReplyTo":"xmqq36622eru.fsf@gitster.c.googlers.com","subject":"Re: mr/bisect-in-c-2, was Re: What's cooking in git.git (Jul 2020, #01; Mon, 6)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2020-07-08T23:11:25Z","receivedAt":"2020-07-08T23:11:33Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Junio C Hamano <gitster@pobox.com> writes:\n\n> Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:\n>\n>>>  Rewrite of the remainder of \"git bisect\" script in C continues.\n>>>\n>>>  Expecting a response to reviews.\n>>>  cf. <nycvar.QRO.7.76.6.2005230007260.56@tvgsbejvaqbjf.bet>\n>>\n>> Maybe you could change this to\n>>\n>>    Needs more reviews.\n>>    cf. <20200701133504.18360-1-mirucam@gmail.com>\n>>\n>> ?\n>\n> Thanks, absolutely.\n\nActually, the state was before that.  It would have been \"Reroll exists\"\nwith the same reference.\n\nIn any case, the latest one broke build of 'seen' and I had to\nexclude it from the day's integration.  Now it should be listed\nas \"Needs more work\" or something.\n\n\n\n"},{"id":"401234","messageId":"nycvar.QRO.7.76.6.2007091119020.50@tvgsbejvaqbjf.bet","threadId":"53805","inReplyTo":"xmqqy2ntzjoi.fsf@gitster.c.googlers.com","subject":"Re: mr/bisect-in-c-2, was Re: What's cooking in git.git (Jul 2020, #01; Mon, 6)","fromName":"Johannes Schindelin","fromEmail":"johannes.schindelin@gmx.de","sentAt":"2020-07-09T09:20:23Z","receivedAt":"2020-07-09T09:20:29Z","isPatch":false,"sender":{"key":"johannes.schindelin@gmx.de","avatar":"https://avatars.githubusercontent.com/u/127790?v=4"},"body":"Hi Junio,\n\nOn Wed, 8 Jul 2020, Junio C Hamano wrote:\n\n> Junio C Hamano <gitster@pobox.com> writes:\n>\n> > Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:\n> >\n> >>>  Rewrite of the remainder of \"git bisect\" script in C continues.\n> >>>\n> >>>  Expecting a response to reviews.\n> >>>  cf. <nycvar.QRO.7.76.6.2005230007260.56@tvgsbejvaqbjf.bet>\n> >>\n> >> Maybe you could change this to\n> >>\n> >>    Needs more reviews.\n> >>    cf. <20200701133504.18360-1-mirucam@gmail.com>\n> >>\n> >> ?\n> >\n> > Thanks, absolutely.\n>\n> Actually, the state was before that.  It would have been \"Reroll exists\"\n> with the same reference.\n>\n> In any case, the latest one broke build of 'seen' and I had to\n> exclude it from the day's integration.  Now it should be listed\n> as \"Needs more work\" or something.\n\nI missed that it broke... sorry, these days I don't get to watch the CI\nbuilds as diligently as I used to.\n\nI'll Miriam so she sees this.\n\nCiao,\nDscho\n"},{"id":"401385","messageId":"20200710211342.GE11341@szeder.dev","threadId":"53805","inReplyTo":"xmqqa70b5h3w.fsf@gitster.c.googlers.com","subject":"Re: sg/commit-graph-cleanups (was Re: What's cooking in git.git (Jul 2020, #01; Mon, 6))","fromName":"SZEDER Gábor","fromEmail":"szeder.dev@gmail.com","sentAt":"2020-07-10T21:13:42Z","receivedAt":"2020-07-10T21:13:48Z","isPatch":false,"sender":{"key":"szeder.dev@gmail.com","avatar":"https://avatars.githubusercontent.com/u/116324?v=4"},"body":"On Tue, Jul 07, 2020 at 11:12:51AM -0700, Junio C Hamano wrote:\n> SZEDER Gábor <szeder.dev@gmail.com> writes:\n> \n> > On Tue, Jul 07, 2020 at 09:45:21AM -0700, Junio C Hamano wrote:\n> >> Derrick Stolee <stolee@gmail.com> writes:\n> >> \n> >> >> Oops, I am not sure what happened here.  There I thought were issues\n> >> >> pointed out at least on the latter series but was I hallucinating?\n> >> >\n> >> > I believe I have resolved all the open comments in both series.\n> >> \n> >> OK, then I do not have to be worried.\n> >> \n> >> Let's merge it down before -rc0.\n> >\n> > I think it was merged to next too soon.  I haven't even done with\n> > reporting all the issues that were already in v2.27.0, let alone\n> > looking through these followup series...\n> \n> Oh, so what's your preference?  Reverting all the commit-graph stuff\n> that is not in 2.26 may be tempting but is not practically feasible\n> this late in the cycle.  It sounded like Derrick was OK to leave\n> them in 'next', and I am fine with keeping them out of the upcoming\n> release.\n\nYeah, I think it's sensible to leave them out from the upcoming\nrelease, because that would give us the possibility (and me the time)\nto go over them once more.  Glancing through it, I see that Derrick\nkept most of my commit messages as-is, but some of them contain\nforward references to later patches in my modified path Bloom filters\nseries that don't make sense on their own.  There are also some\nmissing tests, a missing \"Reported-by: me\", missing updates to the\nformat specs, and even a missing(?) patch.\n\n"},{"id":"401386","messageId":"xmqqeepjrqy1.fsf@gitster.c.googlers.com","threadId":"53805","inReplyTo":"20200710211342.GE11341@szeder.dev","subject":"Re: sg/commit-graph-cleanups (was Re: What's cooking in git.git (Jul 2020, #01; Mon, 6))","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2020-07-10T21:38:30Z","receivedAt":"2020-07-10T21:38:37Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"SZEDER Gábor <szeder.dev@gmail.com> writes:\n\n> On Tue, Jul 07, 2020 at 11:12:51AM -0700, Junio C Hamano wrote:\n>> SZEDER Gábor <szeder.dev@gmail.com> writes:\n>> \n>> > On Tue, Jul 07, 2020 at 09:45:21AM -0700, Junio C Hamano wrote:\n>> >> Derrick Stolee <stolee@gmail.com> writes:\n>> >> \n>> >> >> Oops, I am not sure what happened here.  There I thought were issues\n>> >> >> pointed out at least on the latter series but was I hallucinating?\n>> >> >\n>> >> > I believe I have resolved all the open comments in both series.\n>> >> \n>> >> OK, then I do not have to be worried.\n>> >> \n>> >> Let's merge it down before -rc0.\n>> >\n>> > I think it was merged to next too soon.  I haven't even done with\n>> > reporting all the issues that were already in v2.27.0, let alone\n>> > looking through these followup series...\n>> \n>> Oh, so what's your preference?  Reverting all the commit-graph stuff\n>> that is not in 2.26 may be tempting but is not practically feasible\n>> this late in the cycle.  It sounded like Derrick was OK to leave\n>> them in 'next', and I am fine with keeping them out of the upcoming\n>> release.\n>\n> Yeah, I think it's sensible to leave them out from the upcoming\n> release, because that would give us the possibility (and me the time)\n> to go over them once more.  Glancing through it, I see that Derrick\n> kept most of my commit messages as-is, but some of them contain\n> forward references to later patches in my modified path Bloom filters\n> series that don't make sense on their own.  There are also some\n> missing tests, a missing \"Reported-by: me\", missing updates to the\n> format specs, and even a missing(?) patch.\n\nOK, thanks.\n"}]}