{"thread":{"id":"61449","subject":"What's cooking in git.git (May 2024, #04; Thu, 9)","startedAt":"2024-05-09T22:37:58Z","lastAt":"2024-05-10T18:40:43Z","messageCount":7,"participants":["Junio C Hamano","Torsten Bögershausen","Patrick Steinhardt","Justin Tobler"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"494384","messageId":"xmqqfruqsjb6.fsf@gitster.g","threadId":"61449","inReplyTo":null,"subject":"What's cooking in git.git (May 2024, #04; Thu, 9)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2024-05-09T22:37:49Z","receivedAt":"2024-05-09T22:37:58Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Here are the topics that have been cooking in my tree. Commits\nprefixed with '+' are in 'next' (being in 'next' is a sign that a\ntopic is stable enough to be used and are candidate to be in a\nfuture release). Commits prefixed with '-' are only in 'seen', and\naren't considered \"accepted\" at all and may be annotated with an URL\nto a message that raises issues but they are no means exhaustive. A\ntopic without enough support may be discarded after a long period of\nno activity (of course they can be resubmit when new interests\narise).\n\nCopies of the source code to Git live in many repositories, and the\nfollowing is a list of the ones I push into or their mirrors. Some\nrepositories have only a subset of branches.\n\nWith maint, master, next, seen, todo:\n\n\tgit://git.kernel.org/pub/scm/git/git.git/\n\tgit://repo.or.cz/alt-git.git/\n\thttps://kernel.googlesource.com/pub/scm/git/git/\n\thttps://github.com/git/git/\n\thttps://gitlab.com/git-scm/git/\n\nWith all the integration branches and topics broken out:\n\n\thttps://github.com/gitster/git/\n\nEven though the preformatted documentation in HTML and man format\nare not sources, they are published in these repositories for\nconvenience (replace \"htmldocs\" with \"manpages\" for the manual\npages):\n\n\tgit://git.kernel.org/pub/scm/git/git-htmldocs.git/\n\thttps://github.com/gitster/git-htmldocs.git/\n\nRelease tarballs are available at:\n\n\thttps://www.kernel.org/pub/software/scm/git/\n\n--------------------------------------------------\n[Graduated to 'master']\n\n* bb/rgb-12-bit-colors (2024-05-02) 3 commits\n (merged to 'next' on 2024-05-02 at 7fe29c98d7)\n + color: add support for 12-bit RGB colors\n + t/t4026-color: add test coverage for invalid RGB colors\n + t/t4026-color: remove an extra double quote character\n\n The color parsing code learned to handle 12-bit RGB colors, spelled\n as \"#RGB\" (in addition to \"#RRGGBB\" that is already supported).\n source: <20240502110331.6347-1-dev+git@drbeat.li>\n\n\n* bc/credential-scheme-enhancement (2024-04-16) 16 commits\n (merged to 'next' on 2024-05-01 at 789ec5bd35)\n + credential: add method for querying capabilities\n + credential-cache: implement authtype capability\n + t: add credential tests for authtype\n + credential: add support for multistage credential rounds\n + t5563: refactor for multi-stage authentication\n + docs: set a limit on credential line length\n + credential: enable state capability\n + credential: add an argument to keep state\n + http: add support for authtype and credential\n + docs: indicate new credential protocol fields\n + credential: add a field called \"ephemeral\"\n + credential: gate new fields on capability\n + credential: add a field for pre-encoded credentials\n + http: use new headers for each object request\n + remote-curl: reset headers on new request\n + credential: add an authtype field\n\n The credential helper protocol, together with the HTTP layer, have\n been enhanced to support authentication schemes different from\n username & password pair, like Bearer and NTLM.\n source: <20240417000240.3611948-1-sandals@crustytoothpaste.net>\n\n\n* bc/zsh-compatibility (2024-04-26) 2 commits\n (merged to 'next' on 2024-05-02 at 10222b2472)\n + vimdiff: make script and tests work with zsh\n + t4046: avoid continue in &&-chain for zsh\n\n zsh can pretend to be a normal shell pretty well except for some\n glitches that we tickle in some of our scripts. Work them around\n so that \"vimdiff\" and our test suite works well enough with it.\n source: <20240426221154.2194139-1-sandals@crustytoothpaste.net>\n\n\n* dk/zsh-git-repo-path-fix (2024-04-30) 1 commit\n (merged to 'next' on 2024-05-02 at 0446d8e63a)\n + completion: zsh: stop leaking local cache variable\n\n Command line completion support for zsh (in contrib/) has been\n updated to stop exposing internal state to end-user shell\n interaction.\n source: \n\n\n* jt/doc-submitting-rerolled-series (2024-04-25) 1 commit\n (merged to 'next' on 2024-05-01 at 1c5865f11c)\n + doc: clarify practices for submitting updated patch versions\n\n Developer doc update.\n source: <20240425213404.133660-1-jltobler@gmail.com>\n\n\n* ps/ci-test-with-jgit (2024-04-12) 13 commits\n (merged to 'next' on 2024-05-01 at 35e293e618)\n + t0612: add tests to exercise Git/JGit reftable compatibility\n + t0610: fix non-portable variable assignment\n + t06xx: always execute backend-specific tests\n + ci: install JGit dependency\n + ci: make Perforce binaries executable for all users\n + ci: merge scripts which install dependencies\n + ci: fix setup of custom path for GitLab CI\n + ci: merge custom PATH directories\n + ci: convert \"install-dependencies.sh\" to use \"/bin/sh\"\n + ci: drop duplicate package installation for \"linux-gcc-default\"\n + ci: skip sudo when we are already root\n + ci: expose distro name in dockerized GitHub jobs\n + ci: rename \"runs_on_pool\" to \"distro\"\n\n Tests to ensure interoperability between reftable written by jgit\n and our code have been added and enabled in CI.\n source: \n\n\n* ps/reftable-write-optim (2024-04-08) 11 commits\n (merged to 'next' on 2024-04-30 at 0667e3c05b)\n + reftable/block: reuse compressed array\n + reftable/block: reuse zstream when writing log blocks\n + reftable/writer: reset `last_key` instead of releasing it\n + reftable/writer: unify releasing memory\n + reftable/writer: refactorings for `writer_flush_nonempty_block()`\n + reftable/writer: refactorings for `writer_add_record()`\n + refs/reftable: don't recompute committer ident\n + reftable: remove name checks\n + refs/reftable: skip duplicate name checks\n + refs/reftable: perform explicit D/F check when writing symrefs\n + refs/reftable: fix D/F conflict error message on ref copy\n\n Code to write out reftable has seen some optimization and\n simplification.\n source: \n\n\n* ps/the-index-is-no-more (2024-04-18) 6 commits\n (merged to 'next' on 2024-05-01 at 7b9a0ab760)\n + repository: drop `initialize_the_repository()`\n + repository: drop `the_index` variable\n + builtin/clone: stop using `the_index`\n + repository: initialize index in `repo_init()`\n + builtin: stop using `the_index`\n + t/helper: stop using `the_index`\n (this branch is used by ps/undecided-is-not-necessarily-sha1.)\n\n The singleton index_state instance \"the_index\" has been eliminated\n by always instantiating \"the_repository\" and replacing references\n to \"the_index\" with references to its .index member.\n source: \n\n\n* rh/complete-symbolic-ref (2024-04-25) 3 commits\n (merged to 'next' on 2024-05-01 at 7020ecebe7)\n + completion: add docs on how to add subcommand completions\n + completion: improve docs for using __git_complete\n + completion: add 'symbolic-ref'\n\n Command line completion script (in contrib/) learned to complete\n \"git symbolic-ref\" a bit better (you need to enable plumbing\n commands to be completed with GIT_COMPLETION_SHOW_ALL_COMMANDS).\n source: <20240425101845.708554-3-rhi@pengutronix.de>\n\n\n* rj/add-p-typo-reaction (2024-04-30) 2 commits\n (merged to 'next' on 2024-05-01 at 3016062ebf)\n + add-patch: response to unknown command\n + add-patch: do not show UI messages on stderr\n\n When the user responds to a prompt given by \"git add -p\" with an\n unsupported command, list of available commands were given, which\n was too much if the user knew what they wanted to type but merely\n made a typo. Now the user gets a much shorter error message.\n source: <952a9514-3cf1-4601-8f0d-db57adc750c3@gmail.com>\n\n\n* rs/diff-parseopts-cleanup (2024-05-01) 1 commit\n (merged to 'next' on 2024-05-02 at cd9779c175)\n + diff-lib: stop calling diff_setup_done() in do_diff_cache()\n\n Code clean-up to remove code that is now a noop.\n source: <0e96c5a3-7b4d-4c6b-875e-d80e3eb07a00@web.de>\n\n--------------------------------------------------\n[New Topics]\n\n* vd/doc-merge-tree-x-option (2024-05-07) 1 commit\n (merged to 'next' on 2024-05-08 at 42637b8bdf)\n + Documentation/git-merge-tree.txt: document -X\n\n Doc update.\n\n Will merge to 'master'.\n source: \n\n\n* fa/p4-error (2024-05-08) 1 commit\n - git-p4: show Perforce error to the user\n\n P4 update.\n\n Will merge to 'next'.\n source: \n\n\n* jc/doc-manpages-l10n (2024-05-09) 1 commit\n - SubmittingPatches: advertise git-manpages-l10n project a bit\n\n The SubmittingPatches document now refers folks to manpages\n translation project.\n\n Comments?\n source: \n\n\n* jk/ci-macos-gcc13-fix (2024-05-09) 3 commits\n - ci: stop installing \"gcc-13\" for osx-gcc\n - ci: avoid bare \"gcc\" for osx-gcc job\n - ci: drop mention of BREW_INSTALL_PACKAGES variable\n\n CI fix.\n\n Will merge to 'next'.\n source: <20240509162219.GA1707955@coredump.intra.peff.net>\n\n\n* ps/ci-fuzzers-at-gitlab-fix (2024-05-09) 3 commits\n - gitlab-ci: fix installing dependencies for fuzz smoke tests\n - Merge branch 'ps/ci-python-2-deprecation' into ps/ci-fuzzers-at-gitlab-fix\n - Merge branch 'ps/ci-enable-minimal-fuzzers-at-gitlab' into ps/ci-fuzzers-at-gitlab-fix\n (this branch uses ps/ci-enable-minimal-fuzzers-at-gitlab and ps/ci-python-2-deprecation.)\n\n CI fix.\n\n Will merge to 'next'.\n source: <5a92a4aa9edd9653df71b284f07461c7906f97e2.1715241343.git.ps@pks.im>\n\n\n* ps/reftable-reusable-iterator (2024-05-08) 13 commits\n - reftable/merged: adapt interface to allow reuse of iterators\n - reftable/stack: provide convenience functions to create iterators\n - reftable/reader: adapt interface to allow reuse of iterators\n - reftable/generic: adapt interface to allow reuse of iterators\n - reftable/generic: move seeking of records into the iterator\n - reftable/merged: simplify indices for subiterators\n - reftable/merged: split up initialization and seeking of records\n - reftable/reader: set up the reader when initializing table iterator\n - reftable/reader: inline `reader_seek_internal()`\n - reftable/reader: separate concerns of table iter and reftable reader\n - reftable/reader: unify indexed and linear seeking\n - reftable/reader: avoid copying index iterator\n - reftable/block: use `size_t` to track restart point index\n\n source: \n\n\n* tb/precompose-getcwd (2024-05-09) 1 commit\n - macOS: ls-files path fails if path of workdir is NFD\n\n We forgot to normalize the result of getcwd() to NFC on macOS where\n all other paths are normalized, which has been corrected.\n\n Will merge to 'next'.\n source: <20240509161110.12121-1-tboegi@web.de>\n\n\n* jk/ci-test-with-jgit-fix (2024-05-09) 1 commit\n - ci: update coverity runs_on_pool reference\n\n CI fix.\n\n Will merge to 'next'.\n source: <20240509161506.GA1707588@coredump.intra.peff.net>\n\n\n* jc/patch-flow-updates (2024-05-09) 2 commits\n - SubmittingPatches: extend the \"flow\" section\n - SubmittingPatches: move the patch-flow section earlier\n\n Doc updates.\n\n Comments?\n source: <20240509211318.641896-1-gitster@pobox.com>\n\n--------------------------------------------------\n[Cooking]\n\n* it/refs-name-conflict (2024-05-06) 1 commit\n - refs: return conflict error when checking packed refs\n\n Comments?\n cf. \n source: \n\n\n* jc/format-patch-more-aggressive-range-diff (2024-05-06) 1 commit\n - format-patch: run range-diff with larger creation-factor\n\n The default \"creation-factor\" used by \"git format-patch\" has been\n raised to make it more aggressively find matching commits.\n\n Comments?\n source: \n\n\n* jp/tag-trailer (2024-05-07) 3 commits\n (merged to 'next' on 2024-05-07 at 646013793d)\n + builtin/tag: add --trailer option\n + builtin/commit: refactor --trailer logic\n + builtin/commit: use ARGV macro to collect trailers\n\n \"git tag\" learned the \"--trailer\" option to futz with the trailers\n in the same way as \"git commit\" does.\n\n Will merge to 'master'.\n source: \n\n\n* ow/refspec-glossary-update (2024-05-06) 1 commit\n (merged to 'next' on 2024-05-07 at 7d55ebb0c8)\n + Documentation: Mention that refspecs are explained elsewhere\n\n Doc update.\n\n Will merge to 'master'.\n source: <20240506182317.13477-1-oystwa@gmail.com>\n\n\n* ps/ci-enable-minimal-fuzzers-at-gitlab (2024-05-06) 1 commit\n (merged to 'next' on 2024-05-07 at a76024bd9f)\n + gitlab-ci: add smoke test for fuzzers\n (this branch is used by ps/ci-fuzzers-at-gitlab-fix.)\n\n CI update.\n\n Will merge to 'master'.\n source: <01fb94999f8e2014ba4d09ce7451a4f5d315ee72.1714371146.git.ps@pks.im>\n\n\n* ps/ci-python-2-deprecation (2024-05-06) 1 commit\n (merged to 'next' on 2024-05-07 at 0e11e94f99)\n + ci: fix Python dependency on Ubuntu 24.04\n (this branch is used by ps/ci-fuzzers-at-gitlab-fix.)\n\n Unbreak CI jobs so that we do not attempt to use Python 2 that has\n been removed from the platform.\n\n Will merge to 'master'.\n source: \n\n\n* rs/external-diff-with-exit-code (2024-05-06) 2 commits\n (merged to 'next' on 2024-05-08 at ad5fee3cbf)\n + diff: fix --exit-code with external diff\n + diff: report unmerged paths as changes in run_diff_cmd()\n\n The \"--exit-code\" option of \"git diff\" command learned to work with\n the \"--ext-diff\" option.\n\n Will merge to 'master'.\n source: <82561c70-ec33-41bf-b036-52310ffc1926@web.de>\n\n\n* jc/test-workaround-broken-mv (2024-05-02) 1 commit\n (merged to 'next' on 2024-05-06 at 7e009c6827)\n + t/lib-chunk: work around broken \"mv\" on some vintage of macOS\n\n Tests that try to corrupt in-repository files in chunked format did\n not work well on macOS due to its broken \"mv\", which has been\n worked around.\n\n Will merge to 'master'.\n source: \n\n\n* jt/port-ci-whitespace-check-to-gitlab (2024-05-03) 5 commits\n (merged to 'next' on 2024-05-08 at 774a29dde4)\n + gitlab-ci: add whitespace error check\n + ci: make the whitespace report optional\n + ci: separate whitespace check script\n + github-ci: fix link to whitespace error\n + ci: pre-collapse GitLab CI sections\n\n The \"whitespace check\" task that was enabled for GitHub Actions CI\n has been ported to GitLab CI.\n\n Will merge to 'master'.\n source: <20240503172110.181326-1-jltobler@gmail.com>\n\n\n* jc/no-default-attr-tree-in-bare (2024-05-03) 1 commit\n (merged to 'next' on 2024-05-08 at a34819b16d)\n + stop using HEAD for attributes in bare repository by default\n (this branch is used by ps/undecided-is-not-necessarily-sha1.)\n\n Git 2.43 started using the tree of HEAD as the source of attributes\n in a bare repository, which has severe performance implications.\n For now, revert the change, without ripping out a more explicit\n support for the attr.tree configuration variable.\n\n Will merge to 'master'.\n source: \n\n\n* jl/git-no-advice (2024-05-07) 4 commits\n (merged to 'next' on 2024-05-08 at c9e7e0866e)\n + t0018: two small fixes\n (merged to 'next' on 2024-05-06 at 95cc03ef96)\n + advice: add --no-advice global option\n + doc: add spacing around paginate options\n + doc: clean up usage documentation for --no-* opts\n\n A new global \"--no-advice\" option can be used to disable all advice\n messages, which is meant to be used only in scripts.\n\n Will merge to 'master'.\n source: <20240503071706.78109-1-james@jamesliu.io>\n\n\n* kn/ref-transaction-symref (2024-05-07) 8 commits\n - refs: remove `create_symref` and associated dead code\n - refs: rename `refs_create_symref()` to `refs_update_symref()`\n - refs: use transaction in `refs_create_symref()`\n - refs: add support for transactional symref updates\n - refs: move `original_update_refname` to 'refs.c'\n - refs: support symrefs in 'reference-transaction' hook\n - files-backend: extract out `create_symref_lock()`\n - refs: accept symref values in `ref_transaction_update()`\n\n Updates to symbolic refs can now be made as a part of ref\n transaction.\n\n Will merge to 'next'.\n source: <20240507125859.132116-1-knayak@gitlab.com>\n\n\n* ma/win32-unix-domain-socket (2024-05-03) 1 commit\n (merged to 'next' on 2024-05-06 at 799a546c96)\n + win32: fix building with NO_UNIX_SOCKETS\n\n Build fix.\n\n Will merge to 'master'.\n source: <20240503091427.2808390-1-mh@glandium.org>\n\n\n* ps/config-subcommands (2024-05-06) 14 commits\n (merged to 'next' on 2024-05-07 at f10c689a30)\n + builtin/config: display subcommand help\n + builtin/config: introduce \"edit\" subcommand\n + builtin/config: introduce \"remove-section\" subcommand\n + builtin/config: introduce \"rename-section\" subcommand\n + builtin/config: introduce \"unset\" subcommand\n + builtin/config: introduce \"set\" subcommand\n + builtin/config: introduce \"get\" subcommand\n + builtin/config: introduce \"list\" subcommand\n + builtin/config: pull out function to handle `--null`\n + builtin/config: pull out function to handle config location\n + builtin/config: use `OPT_CMDMODE()` to specify modes\n + builtin/config: move \"fixed-value\" option to correct group\n + builtin/config: move option array around\n + config: clarify memory ownership when preparing comment strings\n\n The operation mode options (like \"--get\") the \"git config\" command\n uses have been deprecated and replaced with subcommands (like \"git\n config get\").\n\n Will merge to 'master'.\n source: \n\n\n* ps/refs-without-the-repository (2024-05-07) 5 commits\n (merged to 'next' on 2024-05-08 at 2479b9de46)\n + refs: remove functions without ref store\n + cocci: apply rules to rewrite callers of \"refs\" interfaces\n + cocci: introduce rules to transform \"refs\" to pass ref store\n + refs: add `exclude_patterns` parameter to `for_each_fullref_in()`\n + refs: introduce missing functions that accept a `struct ref_store`\n\n The refs API lost functions that implicitly assumes to work on the\n primary ref_store by forcing the callers to pass a ref_store as an\n argument.\n\n Will merge to 'master'.\n source: \n\n\n* ps/reftable-write-options (2024-05-03) 11 commits\n - refs/reftable: allow configuring geometric factor\n - reftable: make the compaction factor configurable\n - refs/reftable: allow disabling writing the object index\n - refs/reftable: allow configuring restart interval\n - reftable: use `uint16_t` to track restart interval\n - refs/reftable: allow configuring block size\n - reftable/dump: support dumping a table's block structure\n - reftable/writer: improve error when passed an invalid block size\n - reftable/writer: drop static variable used to initialize strbuf\n - reftable: consistently pass write opts as value\n - reftable: consistently refer to `reftable_write_options` as `opts`\n\n The knobs to tweak how reftable files are written have been made\n available as configuration variables.\n\n Needs review.\n source: \n\n\n* tb/attr-limits (2024-05-03) 1 commit\n (merged to 'next' on 2024-05-06 at b98150cc71)\n + attr.c: move ATTR_MAX_FILE_SIZE check into read_attr_from_buf()\n\n The maximum size of attribute files is enforced more consistently.\n\n Will merge to 'master'.\n source: <28f6267709db78ba526d7ed9fc4a734674697c70.1714763555.git.me@ttaylorr.com>\n\n\n* ds/scalar-reconfigure-all-fix (2024-05-07) 1 commit\n (merged to 'next' on 2024-05-08 at eca398f4a5)\n + scalar: avoid segfault in reconfigure --all\n\n Scalar fix.\n\n Will merge to 'master'.\n source: \n\n\n* ps/pseudo-ref-terminology (2024-05-02) 10 commits\n - refs: refuse to write pseudorefs\n - ref-filter: properly distinuish pseudo and root refs\n - refs: pseudorefs are no refs\n - refs: root refs can be symbolic refs\n - refs: classify HEAD as a root ref\n - refs: refname `is_special_ref()` to `is_pseudo_ref()`\n - refs: rename `is_pseudoref()` to `is_root_ref()`\n - Documentation/glossary: define root refs as refs\n - Documentation/glossary: clarify limitations of pseudorefs\n - Documentation/glossary: redefine pseudorefs as special refs\n\n Terminology to call various ref-like things are getting\n straightened out.\n\n Comments?\n source: \n\n\n* jc/rev-parse-fatal-doc (2024-05-01) 1 commit\n - rev-parse: document how --is-* options work outside a repository\n\n Doc update.\n\n Comments?\n source: \n\n\n* ps/undecided-is-not-necessarily-sha1 (2024-05-06) 15 commits\n (merged to 'next' on 2024-05-08 at 9f8e894685)\n + repository: stop setting SHA1 as the default object hash\n + oss-fuzz/commit-graph: set up hash algorithm\n + builtin/shortlog: don't set up revisions without repo\n + builtin/diff: explicitly set hash algo when there is no repo\n + builtin/bundle: abort \"verify\" early when there is no repository\n + builtin/blame: don't access potentially unitialized `the_hash_algo`\n + builtin/rev-parse: allow shortening to more than 40 hex characters\n + remote-curl: fix parsing of detached SHA256 heads\n + attr: fix BUG() when parsing attrs outside of repo\n + attr: don't recompute default attribute source\n + parse-options-cb: only abbreviate hashes when hash algo is known\n + path: move `validate_headref()` to its only user\n + path: harden validation of HEAD with non-standard hashes\n + Merge branch 'ps/the-index-is-no-more' into ps/undecided-is-not-necessarily-sha1\n + Merge branch 'jc/no-default-attr-tree-in-bare' into ps/undecided-is-not-necessarily-sha1\n (this branch uses jc/no-default-attr-tree-in-bare.)\n\n Before discovering the repository details, We used to assume SHA-1\n as the \"default\" hash function, which has been corrected. Hopefully\n this will smoke out codepaths that rely on such an unwarranted\n assumptions.\n\n Will merge to 'master'.\n source: \n\n\n* pw/rebase-i-error-message (2024-04-08) 2 commits\n - rebase -i: improve error message when picking merge\n - rebase -i: pass struct replay_opts to parse_insn_line()\n\n When the user adds to \"git rebase -i\" instruction to \"pick\" a merge\n commit, the error experience is not pleasant. Such an error is now\n caught earlier in the process that parses the todo list.\n\n Expecting a reroll.\n cf. <88bc0787-e7ae-49e5-99e8-97f6c55ea8c6@gmail.com>\n source: \n\n\n* ds/send-email-per-message-block (2024-04-10) 2 commits\n - send-email: make it easy to discern the messages for each patch\n - send-email: move newline characters out of a few translatable strings\n\n \"git send-email\" learned to separate its reports on each message it\n sends out with an extra blank line in between.\n\n Comments?\n source: \n\n\n* ew/khash-to-khashl (2024-03-28) 3 commits\n - khashl: fix ensemble lookups on empty table\n - treewide: switch to khashl for memory savings\n - list-objects-filter: use kh_size API\n\n The hashtable library \"khash.h\" has been replaced with \"khashl.h\"\n that has better memory usage characteristics.\n\n Needs review.\n cf. \n source: <20240328101356.300374-1-e@80x24.org>\n\n\n* tb/pseudo-merge-reachability-bitmap (2024-04-30) 23 commits\n - t/perf: implement performace tests for pseudo-merge bitmaps\n - pseudo-merge: implement support for finding existing merges\n - ewah: `bitmap_equals_ewah()`\n - pack-bitmap: extra trace2 information\n - pack-bitmap.c: use pseudo-merges during traversal\n - t/test-lib-functions.sh: support `--date` in `test_commit_bulk()`\n - pack-bitmap: implement test helpers for pseudo-merge\n - ewah: implement `ewah_bitmap_popcount()`\n - pseudo-merge: implement support for reading pseudo-merge commits\n - pack-bitmap.c: read pseudo-merge extension\n - pseudo-merge: scaffolding for reads\n - pack-bitmap: extract `read_bitmap()` function\n - pack-bitmap-write.c: write pseudo-merge table\n - pack-bitmap-write.c: select pseudo-merge commits\n - pseudo-merge: implement support for selecting pseudo-merge commits\n - pack-bitmap: make `bitmap_writer_push_bitmapped_commit()` public\n - pack-bitmap: implement `bitmap_writer_has_bitmapped_object_id()`\n - pack-bitmap-write: support storing pseudo-merge commits\n - pseudo-merge.ch: initial commit\n - pack-bitmap: move some initialization to `bitmap_writer_init()`\n - pack-bitmap: drop unused `max_bitmaps` parameter\n - ewah: implement `ewah_bitmap_is_subset()`\n - Documentation/technical: describe pseudo-merge bitmaps format\n\n The pack-bitmap machinery learned to write pseudo-merge bitmaps,\n which act as imaginary octopus merges covering un-bitmapped\n reference tips. This enhances bitmap coverage, and thus,\n performance, for repositories with many references using bitmaps.\n\n Needs review.\n source: \n\n\n* la/hide-trailer-info (2024-05-02) 11 commits\n - trailer unit tests: inspect iterator contents\n - trailer: document parse_trailers() usage\n - trailer: retire trailer_info_get() from API\n - trailer: make trailer_info struct private\n - trailer: make parse_trailers() return trailer_info pointer\n - interpret-trailers: access trailer_info with new helpers\n - sequencer: use the trailer iterator\n - trailer: teach iterator about non-trailer lines\n - trailer: add unit tests for trailer iterator\n - Makefile: sort UNIT_TEST_PROGRAMS\n - Merge branch 'la/format-trailer-info' into la/hide-trailer-info\n\n The trailer API has been reshuffled a bit.\n\n Waiting for a review response.\n cf. \n source: \n\n\n* ds/doc-config-reflow (2024-03-14) 1 commit\n - config.txt: perform some minor reformatting\n\n Reflow a paragraph in the documentation source without any effect\n to the formatted text.\n\n Will discard.\n source: <97bdaf075bf5a68554cca1731eca78aff2662907.1710444774.git.dsimic@manjaro.org>\n\n\n* ie/config-includeif-hostname (2024-03-19) 2 commits\n - config: learn the \"hostname:\" includeIf condition\n - t: add a test helper for getting hostname\n\n The conditional inclusion mechanism for configuration files learned\n to switch on the hostname.\n\n Expecting a reroll.\n cf. <20240319210428.GC1159535@coredump.intra.peff.net>\n cf. <20240320001934.GA903718@coredump.intra.peff.net>\n source: <20240319183722.211300-1-ignacio@iencinas.com>\n\n\n* cw/git-std-lib (2024-02-28) 4 commits\n - SQUASH??? get rid of apparent debugging crufts\n - test-stdlib: show that git-std-lib is independent\n - git-std-lib: introduce Git Standard Library\n - pager: include stdint.h because uintmax_t is used\n\n Split libgit.a out to a separate git-std-lib tor easier reuse.\n\n Expecting a reroll.\n source: \n\n\n* js/unit-test-suite-runner (2024-05-06) 9 commits\n (merged to 'next' on 2024-05-07 at 3b376223e3)\n + cmake: let `test-tool` run the unit tests, too\n + ci: use test-tool as unit test runner on Windows\n + t/Makefile: run unit tests alongside shell tests\n + unit tests: add rule for running with test-tool\n + test-tool run-command testsuite: support unit tests\n + test-tool run-command testsuite: remove hardcoded filter\n + test-tool run-command testsuite: get shell from env\n + t0080: turn t-basic unit test into a helper\n + Merge branch 'jk/unit-tests-buildfix' into js/unit-test-suite-runner\n\n The \"test-tool\" has been taught to run testsuite tests in parallel,\n bypassing the need to use the \"prove\" tool.\n\n Will merge to 'master'.\n source: \n source: \n\n\n* bk/complete-dirname-for-am-and-format-patch (2024-01-12) 1 commit\n - completion: dir-type optargs for am, format-patch\n\n Command line completion support (in contrib/) has been\n updated for a few commands to complete directory names where a\n directory name is expected.\n\n Expecting a reroll.\n cf. <40c3a824-a961-490b-94d4-4eb23c8f713d@gmail.com>\n cf. <6683f24e-7e56-489d-be2d-8afe1fc38d2b@gmail.com>\n source: \n\n\n* bk/complete-send-email (2024-01-12) 1 commit\n - completion: don't complete revs when --no-format-patch\n\n Command line completion support (in contrib/) has been taught to\n avoid offering revision names as candidates to \"git send-email\" when\n the command is used to send pre-generated files.\n\n Expecting a reroll.\n cf. \n source: \n\n\n* tb/path-filter-fix (2024-01-31) 16 commits\n - bloom: introduce `deinit_bloom_filters()`\n - commit-graph: reuse existing Bloom filters where possible\n - object.h: fix mis-aligned flag bits table\n - commit-graph: new Bloom filter version that fixes murmur3\n - commit-graph: unconditionally load Bloom filters\n - bloom: prepare to discard incompatible Bloom filters\n - bloom: annotate filters with hash version\n - repo-settings: introduce commitgraph.changedPathsVersion\n - t4216: test changed path filters with high bit paths\n - t/helper/test-read-graph: implement `bloom-filters` mode\n - bloom.h: make `load_bloom_filter_from_graph()` public\n - t/helper/test-read-graph.c: extract `dump_graph_info()`\n - gitformat-commit-graph: describe version 2 of BDAT\n - commit-graph: ensure Bloom filters are read with consistent settings\n - revision.c: consult Bloom filters for root commits\n - t/t4216-log-bloom.sh: harden `test_bloom_filters_not_used()`\n\n The Bloom filter used for path limited history traversal was broken\n on systems whose \"char\" is unsigned; update the implementation and\n bump the format version to 2.\n\n Waiting for a final ack?\n cf. \n source: \n\n\n* jc/rerere-cleanup (2023-08-25) 4 commits\n - rerere: modernize use of empty strbuf\n - rerere: try_merge() should use LL_MERGE_ERROR when it means an error\n - rerere: fix comment on handle_file() helper\n - rerere: simplify check_one_conflict() helper function\n\n Code clean-up.\n\n Not ready to be reviewed yet.\n source: <20230824205456.1231371-1-gitster@pobox.com>\n\n--------------------------------------------------\n[Discarded]\n\n* js/cmake-with-test-tool (2024-04-30) 2 commits\n . cmake: let `test-tool` run the unit tests, too\n . Merge branch 'js/unit-test-suite-runner' into js/cmake-with-test-tool\n (this branch uses js/unit-test-suite-runner.)\n\n \"test-tool\" is now built in CMake build to also run the unit tests.\n\n Rolled into the base topic.\n source: \n"},{"id":"494385","messageId":"xmqq5xvmschw.fsf@gitster.g","threadId":"61449","inReplyTo":"xmqqfruqsjb6.fsf@gitster.g","subject":"Re: What's cooking in git.git (May 2024, #04; Thu, 9)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2024-05-10T01:04:59Z","receivedAt":"2024-05-10T01:05:03Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Junio C Hamano writes:\n\n> * cw/git-std-lib (2024-02-28) 4 commits\n> - SQUASH??? get rid of apparent debugging crufts\n> - test-stdlib: show that git-std-lib is independent\n> - git-std-lib: introduce Git Standard Library\n> - pager: include stdint.h because uintmax_t is used\n>\n> Split libgit.a out to a separate git-std-lib tor easier reuse.\n>\n> Expecting a reroll.\n> source: \n\nJust a heads-up. The topic is pushed out of 'seen', as it seems to\nbe hated by osx-* jobs when linking programs. As the topic has been\nexpecting a reroll for some time, it is not ultra urgent to fix\nthis, but before resurrecting it, we may want to look into it.\n\nhttps://github.com/git/git/actions/runs/9023837489\n\nis an example run where we see error messages like:\n\nld: warning: ignoring duplicate libraries: 'git-std-lib.a', 'libgit.a', 'reftable/libreftable.a', 'xdiff/lib.a'\n\nand\n\nld: Undefined symbols:\n _git_config_get_bool, referenced from:\n _precompose_string_if_needed in git-std-lib.a[11](precompose_utf8.o)\n\nin both clang and gcc jobs.\n\nWe did not see these breakages before, and we do not see these\nbreakages in today's 'seen' with this topic excluded. I haven't\ntried to include this but excluding tb/precompose-getcwd topic\ninstead, which I suspect might also work.\n\nThanks.\n"},{"id":"494391","messageId":"20240510063230.GA27291@tb-raspi4","threadId":"61449","inReplyTo":"xmqq5xvmschw.fsf@gitster.g","subject":"Re: What's cooking in git.git (May 2024, #04; Thu, 9)","fromName":"Torsten Bögershausen","fromEmail":"tboegi@web.de","sentAt":"2024-05-10T06:32:30Z","receivedAt":"2024-05-10T06:32:45Z","isPatch":false,"sender":{"key":"tboegi@web.de","avatar":"https://avatars.githubusercontent.com/u/7138363?v=4"},"body":"On Thu, May 09, 2024 at 06:04:59PM -0700, Junio C Hamano wrote:\n> Junio C Hamano writes:\n>\n> > * cw/git-std-lib (2024-02-28) 4 commits\n> > - SQUASH??? get rid of apparent debugging crufts\n> > - test-stdlib: show that git-std-lib is independent\n> > - git-std-lib: introduce Git Standard Library\n> > - pager: include stdint.h because uintmax_t is used\n> >\n> > Split libgit.a out to a separate git-std-lib tor easier reuse.\n> >\n> > Expecting a reroll.\n> > source: \n>\n> Just a heads-up. The topic is pushed out of 'seen', as it seems to\n> be hated by osx-* jobs when linking programs. As the topic has been\n> expecting a reroll for some time, it is not ultra urgent to fix\n> this, but before resurrecting it, we may want to look into it.\n>\n> https://github.com/git/git/actions/runs/9023837489\n>\n> is an example run where we see error messages like:\n>\n> ld: warning: ignoring duplicate libraries: 'git-std-lib.a', 'libgit.a', 'reftable/libreftable.a', 'xdiff/lib.a'\n>\n> and\n>\n> ld: Undefined symbols:\n> _git_config_get_bool, referenced from:\n> _precompose_string_if_needed in git-std-lib.a[11](precompose_utf8.o)\n>\n> in both clang and gcc jobs.\n>\n> We did not see these breakages before, and we do not see these\n> breakages in today's 'seen' with this topic excluded. I haven't\n> tried to include this but excluding tb/precompose-getcwd topic\n> instead, which I suspect might also work.\n\nI think that including tb/precompose-getcwd should be fine.\nAs a side note: Thanks for cleaning that up.\n\nI tried to reproduce the breakage, but could not.\nThe github log says:\n2024-05-09T21:33:21.8856440Z ++ CI_COMMIT=0c244ffcb1ec0d507d71e244817a3d1972acde59\n(But that commit is probably gone).\n\nTrying to compile\n commit 0ebed321d488910d7b34953df0e3bc9d5d74ba70 (HEAD, junio/cw/git-std-lib)\n Author: Junio C Hamano \n Date: Wed Feb 28 12:42:51 2024 -0800\n SQUASH??? get rid of apparent debugging crufts\n\nDid not bring up the problems here, compiling worked.\n\nThe problem on the github run is that\n LINK t/unit-tests/bin/t-ctype\nfailed because of missing symbols which should be provided by\ncompat/precompose_utf8.c\n\nI am not an expert on the libification effort.\nSomeone could wildly speculate that this very version of \"seen\"\ndid have merge conflicts, that led to compat/precompose_utf8.o missing\nsomewhere in the Makefile(s)\n\n"},{"id":"494397","messageId":"Zj3SsbHLp-gH_Jey@tanuki","threadId":"61449","inReplyTo":"xmqqfruqsjb6.fsf@gitster.g","subject":"Re: What's cooking in git.git (May 2024, #04; Thu, 9)","fromName":"Patrick Steinhardt","fromEmail":"ps@pks.im","sentAt":"2024-05-10T07:54:25Z","receivedAt":"2024-05-10T07:54:31Z","isPatch":false,"sender":{"key":"ps@pks.im","avatar":"https://avatars.githubusercontent.com/u/4056630?v=4"},"body":"On Thu, May 09, 2024 at 03:37:49PM -0700, Junio C Hamano wrote:\n> * ps/reftable-write-options (2024-05-03) 11 commits\n> - refs/reftable: allow configuring geometric factor\n> - reftable: make the compaction factor configurable\n> - refs/reftable: allow disabling writing the object index\n> - refs/reftable: allow configuring restart interval\n> - reftable: use `uint16_t` to track restart interval\n> - refs/reftable: allow configuring block size\n> - reftable/dump: support dumping a table's block structure\n> - reftable/writer: improve error when passed an invalid block size\n> - reftable/writer: drop static variable used to initialize strbuf\n> - reftable: consistently pass write opts as value\n> - reftable: consistently refer to `reftable_write_options` as `opts`\n> \n> The knobs to tweak how reftable files are written have been made\n> available as configuration variables.\n> \n> Needs review.\n> source: \n\nJust in case you missed it, there was a review by Justin already [1].\nIt's okay if you want to wait for more reviews though.\n\nPatrick\n\n[1]: https://lore.kernel.org/git/2rozmhwf2jpyqmy5vtf2t664ko7ztinbzdmwirrqxspjwzgj7t@jctxour2o73r/\n"},{"id":"494468","messageId":"xmqq5xvl7k97.fsf@gitster.g","threadId":"61449","inReplyTo":"Zj3SsbHLp-gH_Jey@tanuki","subject":"Re: What's cooking in git.git (May 2024, #04; Thu, 9)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2024-05-10T15:35:16Z","receivedAt":"2024-05-10T15:35:27Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Patrick Steinhardt writes:\n\n> Just in case you missed it, there was a review by Justin already [1].\n\n\"I read it and have no comments\" is somewhere between \"an invitation\nto miss\" and \"it does not highlight anything, either positive or\nnegative. Was the topic that uninteresting and uninspiring without\nanything noteworthy?\".\n\nThere were mentions of a new document for reviewers, similar to\nSubmittingPatches is for contributors, and it would make a good\nsection to document recommended ways for reviewers to demonstrate\nthat they understand (1) the area, (2) the goal of the patches, and\n(3) the implementation presented, better.\n"},{"id":"494473","messageId":"Zj5I9J-l24Pk-10q@framework","threadId":"61449","inReplyTo":"xmqq5xvl7k97.fsf@gitster.g","subject":"Re: What's cooking in git.git (May 2024, #04; Thu, 9)","fromName":"Patrick Steinhardt","fromEmail":"ps@pks.im","sentAt":"2024-05-10T16:19:00Z","receivedAt":"2024-05-10T16:19:06Z","isPatch":false,"sender":{"key":"ps@pks.im","avatar":"https://avatars.githubusercontent.com/u/4056630?v=4"},"body":"On Fri, May 10, 2024 at 08:35:16AM -0700, Junio C Hamano wrote:\n> Patrick Steinhardt writes:\n> \n> > Just in case you missed it, there was a review by Justin already [1].\n> \n> \"I read it and have no comments\" is somewhere between \"an invitation\n> to miss\" and \"it does not highlight anything, either positive or\n> negative. Was the topic that uninteresting and uninspiring without\n> anything noteworthy?\".\n\nI figured that this might've been it.\n\n> There were mentions of a new document for reviewers, similar to\n> SubmittingPatches is for contributors, and it would make a good\n> section to document recommended ways for reviewers to demonstrate\n> that they understand (1) the area, (2) the goal of the patches, and\n> (3) the implementation presented, better.\n\nRight, that's good feedback indeed. Ask questions, reexplain what\nhappens with your own words to both double check your understanding and\ndemonstrate it to others, ask for alternative ways to implement\nsomething. These are all ways to ACK a patch series that otherwise looks\ngood to you already.\n\nThanks!\n\nPatrick\n"},{"id":"494487","messageId":"262znkm7vg77nzp5fadhfuqkcs6wapsuzd5dtvffoopszzoyia@uohyfeerbncl","threadId":"61449","inReplyTo":"Zj5I9J-l24Pk-10q@framework","subject":"Re: What's cooking in git.git (May 2024, #04; Thu, 9)","fromName":"Justin Tobler","fromEmail":"jltobler@gmail.com","sentAt":"2024-05-10T18:39:17Z","receivedAt":"2024-05-10T18:40:43Z","isPatch":false,"sender":{"key":"jltobler@gmail.com","avatar":"https://avatars.githubusercontent.com/u/53454972?v=4"},"body":"On 24/05/10 06:19PM, Patrick Steinhardt wrote:\n> On Fri, May 10, 2024 at 08:35:16AM -0700, Junio C Hamano wrote:\n\n> > \"I read it and have no comments\" is somewhere between \"an invitation\n> > to miss\" and \"it does not highlight anything, either positive or\n> > negative. Was the topic that uninteresting and uninspiring without\n> > anything noteworthy?\".\n> \n> I figured that this might've been it.\n> \n> > There were mentions of a new document for reviewers, similar to\n> > SubmittingPatches is for contributors, and it would make a good\n> > section to document recommended ways for reviewers to demonstrate\n> > that they understand (1) the area, (2) the goal of the patches, and\n> > (3) the implementation presented, better.\n> \n> Right, that's good feedback indeed. Ask questions, reexplain what\n> happens with your own words to both double check your understanding and\n> demonstrate it to others, ask for alternative ways to implement\n> something. These are all ways to ACK a patch series that otherwise looks\n> good to you already.\n\nNoted. Thanks both for the feedback! :)\n\n-Justin\n"}]}