{"thread":{"id":"61391","subject":"[ANNOUNCE] Git v2.45.0","startedAt":"2024-04-29T17:12:33Z","lastAt":"2024-05-02T21:21:06Z","messageCount":12,"participants":["Junio C Hamano","rsbecker@nexbridge.com","Torsten Bögershausen","SZEDER Gábor"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"493660","messageId":"xmqq8r0ww0sj.fsf@gitster.g","threadId":"61391","inReplyTo":null,"subject":"[ANNOUNCE] Git v2.45.0","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2024-04-29T17:12:28Z","receivedAt":"2024-04-29T17:12:33Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"The latest feature release Git v2.45.0 is now available at the\nusual places.  It is comprised of 540 non-merge commits since\nv2.44.0, contributed by 96 people, 38 of which are new faces [*].\n\nThe tarballs are found at:\n\n    https://www.kernel.org/pub/software/scm/git/\n\nThe following public repositories all have a copy of the 'v2.45.0'\ntag and the 'master' branch that the tag points at:\n\n  url = https://git.kernel.org/pub/scm/git/git\n  url = https://kernel.googlesource.com/pub/scm/git/git\n  url = git://repo.or.cz/alt-git.git\n  url = https://github.com/gitster/git\n\nNew contributors whose contributions weren't in v2.44.0 are as follows.\nWelcome to the Git development community!\n\n  Ahelenia Ziemiańska, Angelo Dureghello, Aryan Gupta, Benjamin\n  Flesch, Bo Anderson, Brian C Tracy, Brian Lyles, Bruno Haible,\n  Chuck Lever, Dario Gjorgjevski, Dirk Gouders, Eugenio Gigante,\n  Florian Schmidt, Haritha D, Harmen Stoppels, Jean-Rémy\n  Falleri, Jiamu Sun, Jonas Wunderlich, Jonathan Davies, Julio\n  Bacellari, Kipras Melnikovas, Kisaragi Hiu, Marcel Röthke,\n  Matthew Rollings, Max Gautier, mirth hickford, Paweł Dominiak,\n  Pi Fisher, Ralph Seichter, Richard Macklin, shejialuo, Steven\n  Jeuris, Thalia Archibald, Tiago Pascoal, Vincenzo Mezzela,\n  Vũ Tiến Hưng, Xing Xin, and Yehezkel Bernat.\n\nReturning contributors who helped this release are as follows.\nThanks for your continued support.\n\n  Alexander Shopov, Arkadii Yakovets, Bagas Sanjaya, Beat Bolli,\n  brian m. carlson, Chandra Pratap, Christian Couder, Derrick\n  Stolee, Đoàn Trần Công Danh, Dragan Simic, Elijah Newren,\n  Emir SARI, Eric Sunshine, Eric W. Biederman, Ghanshyam Thakkar,\n  Han Young, Jakub Wilk, Jean-Noël Avila, Jeff Hostetler, Jeff\n  King, Jiang Xin, Johannes Schindelin, Johannes Sixt, John Cai,\n  Josh Steadmon, Josh Triplett, Junio C Hamano, Justin Tobler,\n  Karthik Nayak, Kate Golovanova, Kristoffer Haugsbakk, Kyle\n  Lippincott, Kyle Meyer, Linus Arver, Lumynous, Manlio Perillo,\n  Matthias Aßhauer, Matthias Rüster, M Hickford, Michael Lohmann,\n  Michael Osipov, Mike Hommey, Orgad Shaneh, Patrick Steinhardt,\n  Peter Hutterer, Peter Krefting, Philippe Blain, Phillip Wood,\n  Ralf Thielow, René Scharfe, Rubén Justo, Sergey Organov,\n  SZEDER Gábor, Taylor Blau, Teng Long, Ville Skyttä, Yasushi\n  SHOJI, and Yi-Jyun Pan.\n\n[*] We are counting not just the authorship contribution but issue\n    reporting, mentoring, helping and reviewing that are recorded in\n    the commit trailers.\n\n----------------------------------------------------------------\n\nGit v2.45 Release Notes\n=======================\n\nBackward Compatibility Notes\n\nUI, Workflows & Features\n\n * Integrate the reftable code into the refs framework as a backend.\n   With \"git init --ref-format=reftable\", hopefully it would be a lot\n   more efficient to manage a repository with many references.\n\n * \"git checkout -p\" and friends learned that that \"@\" is a synonym\n   for \"HEAD\".\n\n * Variants of vimdiff learned to honor mergetool.<variant>.layout\n   settings.\n\n * \"git reflog\" learned a \"list\" subcommand that enumerates known reflogs.\n\n * When a merge conflicted at a submodule, merge-ort backend used to\n   unconditionally give a lengthy message to suggest how to resolve\n   it.  Now the message can be squelched as an advice message.\n\n * \"git for-each-ref\" learned \"--include-root-refs\" option to show\n   even the stuff outside the 'refs/' hierarchy.\n\n * \"git rev-list --missing=print\" has learned to optionally take\n   \"--allow-missing-tips\", which allows the objects at the starting\n   points to be missing.\n\n * \"git merge-tree\" has learned that the three trees involved in the\n   3-way merge only need to be trees, not necessarily commits.\n\n * \"git log --merge\" learned to pay attention to CHERRY_PICK_HEAD and\n   other kinds of *_HEAD pseudorefs.\n\n * Platform specific tweaks for OS/390 has been added to\n   config.mak.uname.\n\n * Users with safe.bareRepository=explicit can still work from within\n   $GIT_DIR of a seconary worktree (which resides at .git/worktrees/$name/)\n   of the primary worktree without explicitly specifying the $GIT_DIR\n   environment variable or the --git-dir=<path> option.\n\n * The output format for dates \"iso-strict\" has been tweaked to show\n   a time in the Zulu timezone with \"Z\" suffix, instead of \"+00:00\".\n\n * \"git diff\" and friends learned two extra configuration variables,\n   diff.srcPrefix and diff.dstPrefix.\n\n * The status.showUntrackedFiles configuration variable had a name\n   that tempts users to set a Boolean value expressed in our usual\n   \"false\", \"off\", and \"0\", but it only took \"no\".  This has been\n   corrected so \"true\" and its synonyms are taken as \"normal\", while\n   \"false\" and its synonyms are taken as \"no\".\n\n * Remove an ancient and not well maintained Hg-to-git migration\n   script from contrib/.\n\n * Hints that suggest what to do after resolving conflicts can now be\n   squelched by disabling advice.mergeConflict.\n\n * Allow git-cherry-pick(1) to automatically drop redundant commits via\n   a new `--empty` option, similar to the `--empty` options for\n   git-rebase(1) and git-am(1). Includes a soft deprecation of\n   `--keep-redundant-commits` as well as some related docs changes and\n   sequencer code cleanup.\n\n * \"git config\" learned \"--comment=<message>\" option to leave a\n   comment immediately after the \"variable = value\" on the same line\n   in the configuration file.\n\n * core.commentChar used to be limited to a single byte, but has been\n   updated to allow an arbitrary multi-byte sequence.\n\n * \"git add -p\" and other \"interactive hunk selection\" UI has learned to\n   skip showing the hunk immediately after it has already been shown, and\n   an additional action to explicitly ask to reshow the current hunk.\n\n * \"git pack-refs\" learned the \"--auto\" option, which defers the decision of\n   whether and how to pack to the ref backend. This is used by the reftable\n   backend to avoid repacking of an already-optimal ref database. The new mode\n   is triggered from \"git gc --auto\".\n\n * \"git add -u <pathspec>\" and \"git commit [-i] <pathspec>\" did not\n   diagnose a pathspec element that did not match any files in certain\n   situations, unlike \"git add <pathspec>\" did.\n\n * The userdiff patterns for C# has been updated.\n\n * Git writes a \"waiting for your editor\" message on an incomplete\n   line after launching an editor, and then append another error\n   message on the same line if the editor errors out.  It now clears\n   the \"waiting for...\" line before giving the error message.\n\n * The filename used for rejected hunks \"git apply --reject\" creates\n   was limited to PATH_MAX, which has been lifted.\n\n * When \"git bisect\" reports the commit it determined to be the\n   culprit, we used to show it in a format that does not honor common\n   UI tweaks, like log.date and log.decorate.  The code has been\n   taught to use \"git show\" to follow more customizations.\n\n\nPerformance, Internal Implementation, Development Support etc.\n\n * The code to iterate over refs with the reftable backend has seen\n   some optimization.\n\n * More tests that are marked as \"ref-files only\" have been updated to\n   improve test coverage of reftable backend.\n\n * Some parts of command line completion script (in contrib/) have\n   been micro-optimized.\n\n * The way placeholders are to be marked-up in documentation have been\n   specified; use \"_<placeholder>_\" to typeset the word inside a pair\n   of <angle-brackets> emphasized.\n\n * \"git --no-lazy-fetch cmd\" allows to run \"cmd\" while disabling lazy\n   fetching of objects from the promisor remote, which may be handy\n   for debugging.\n\n * The implementation in \"git clean\" that makes \"-n\" and \"-i\" ignore\n   clean.requireForce has been simplified, together with the\n   documentation.\n\n * Uses of xwrite() helper have been audited and updated for better\n   error checking and simpler code.\n\n * Some trace2 events that lacked def_param have learned to show it,\n   enriching the output.\n\n * The parse-options code that deals with abbreviated long option\n   names have been cleaned up.\n\n * The code in reftable backend that creates new table files works\n   better with the tempfile framework to avoid leaving cruft after a\n   failure.\n\n * The reftable code has its own custom binary search function whose\n   comparison callback has an unusual interface, which caused the\n   binary search to degenerate into a linear search, which has been\n   corrected.\n\n * The code to iterate over reflogs in the reftable has been optimized\n   to reduce memory allocation and deallocation.\n\n * Work to support a repository that work with both SHA-1 and SHA-256\n   hash algorithms has started.\n\n * A new fuzz target that exercises config parsing code has been\n   added.\n\n * Fix the way recently added tests interpolate variables defined\n   outside them, and document the best practice to help future\n   developers.\n\n * Introduce an experimental protocol for contributors to propose the\n   topic description to be used in the \"What's cooking\" report, the\n   merge commit message for the topic, and in the release notes and\n   document it in the SubmittingPatches document.\n\n * The t/README file now gives a hint on running individual tests in\n   the \"t/\" directory with \"make t<num>-*.sh t<num>-*.sh\".\n   (merge 8d383806fc pb/test-scripts-are-build-targets later to maint).\n\n * The \"hint:\" messages given by the advice mechanism, when given a\n   message with a blank line, left a line with trailing whitespace,\n   which has been cleansed.\n\n * Documentation rules has been explicitly described how to mark-up\n   literal parts and a few manual pages have been updated as examples.\n\n * The .editorconfig file has been taught that a Makefile uses HT\n   indentation.\n\n * t-prio-queue test has been cleaned up by using C99 compound\n   literals; this is meant to also serve as a weather-balloon to smoke\n   out folks with compilers who have trouble compiling code that uses\n   the feature.\n\n * Windows binary used to decide the use of unix-domain socket at\n   build time, but it learned to make the decision at runtime instead.\n\n * The \"shared repository\" test in the t0610 reftable test failed\n   under restrictive umask setting (e.g. 007), which has been\n   corrected.\n\n * Document and apply workaround for a buggy version of dash that\n   mishandles \"local var=val\" construct.\n\n * The codepaths that reach date_mode_from_type() have been updated to\n   pass \"struct date_mode\" by value to make them thread safe.\n\n * The strategy to compact multiple tables of reftables after many\n   operations accumulate many entries has been improved to avoid\n   accumulating too many tables uncollected.\n\n * The code to iterate over reftable blocks has seen some optimization\n   to reduce memory allocation and deallocation.\n\n * The way \"git fast-import\" handles paths described in its input has\n   been tightened up and more clearly documented.\n\n * The cvsimport tests required that the platform understands\n   traditional timezone notations like CST6CDT, which has been\n   updated to work on those systems as long as they understand\n   POSIX notation with explicit tz transition dates.\n\n * The code to format trailers have been cleaned up.\n\n\nFixes since v2.44\n-----------------\n\n * \"git apply\" on a filesystem without filemode support have learned\n   to take a hint from what is in the index for the path, even when\n   not working with the \"--index\" or \"--cached\" option, when checking\n   the executable bit match what is required by the preimage in the\n   patch.\n   (merge 45b625142d cp/apply-core-filemode later to maint).\n\n * \"git column\" has been taught to reject negative padding value, as\n   it would lead to nonsense behaviour including division by zero.\n   (merge 76fb807faa kh/column-reject-negative-padding later to maint).\n\n * \"git am --help\" now tells readers what actions are available in\n   \"git am --whitespace=<action>\", in addition to saying that the\n   option is passed through to the underlying \"git apply\".\n   (merge a171dac734 jc/am-whitespace-doc later to maint).\n\n * \"git tag --column\" failed to check the exit status of its \"git\n   column\" invocation, which has been corrected.\n   (merge 92e66478fc rj/tag-column-fix later to maint).\n\n * Credential helper based on libsecret (in contrib/) has been updated\n   to handle an empty password correctly.\n   (merge 8f1f2023b7 mh/libsecret-empty-password-fix later to maint).\n\n * \"git difftool --dir-diff\" learned to honor the \"--trust-exit-code\"\n   option; it used to always exit with 0 and signalled success.\n   (merge eb84c8b6ce ps/difftool-dir-diff-exit-code later to maint).\n\n * The code incorrectly attempted to use textconv cache when asked,\n   even when we are not running in a repository, which has been\n   corrected.\n   (merge affe355fe7 jk/textconv-cache-outside-repo-fix later to maint).\n\n * Remove an empty file that shouldn't have been added in the first\n   place.\n   (merge 4f66942215 js/remove-cruft-files later to maint).\n\n * The logic to access reflog entries by date and number had ugly\n   corner cases at the boundaries, which have been cleaned up.\n   (merge 5edd126720 jk/reflog-special-cases-fix later to maint).\n\n * An error message from \"git upload-pack\", which responds to \"git\n   fetch\" requests, had a trailing NUL in it, which has been\n   corrected.\n   (merge 3f4c7a0805 sg/upload-pack-error-message-fix later to maint).\n\n * Clarify wording in the CodingGuidelines that requires <git-compat-util.h>\n   to be the first header file.\n   (merge 4e89f0e07c jc/doc-compat-util later to maint).\n\n * \"git commit -v --cleanup=scissors\" used to add the scissors line\n   twice in the log message buffer, which has been corrected.\n   (merge e90cc075cc jt/commit-redundant-scissors-fix later to maint).\n\n * A custom remote helper no longer cannot access the newly created\n   repository during \"git clone\", which is a regression in Git 2.44.\n   This has been corrected.\n   (merge 199f44cb2e ps/remote-helper-repo-initialization-fix later to maint).\n\n * Various parts of upload-pack have been updated to bound the resource\n   consumption relative to the size of the repository to protect from\n   abusive clients.\n   (merge 6cd05e768b jk/upload-pack-bounded-resources later to maint).\n\n * The upload-pack program, when talking over v2, accepted the\n   packfile-uris protocol extension from the client, even if it did\n   not advertise the capability, which has been corrected.\n   (merge a922bfa3b5 jk/upload-pack-v2-capability-cleanup later to maint).\n\n * Make sure failure return from merge_bases_many() is properly caught.\n   (merge 25fd20eb44 js/merge-base-with-missing-commit later to maint).\n\n * FSMonitor client code was confused when FSEvents were given in a\n   different case on a case-insensitive filesystem, which has been\n   corrected.\n   (merge 29c139ce78 jh/fsmonitor-icase-corner-case-fix later to maint).\n\n * The \"core.commentChar\" configuration variable only allows an ASCII\n   character, which was not clearly documented, which has been\n   corrected.\n   (merge fb7c556f58 kh/doc-commentchar-is-a-byte later to maint).\n\n * With release 2.44 we got rid of all uses of test_i18ngrep and there\n   is no in-flight topic that adds a new use of it.  Make a call to\n   test_i18ngrep a hard failure, so that we can remove it at the end\n   of this release cycle.\n   (merge 381a83dfa3 jc/test-i18ngrep later to maint).\n\n * The command line completion script (in contrib/) learned to\n   complete \"git reflog\" better.\n   (merge 1284f9cc11 rj/complete-reflog later to maint).\n\n * The logic to complete the command line arguments to \"git worktree\"\n   subcommand (in contrib/) has been updated to correctly honor things\n   like \"git -C dir\" etc.\n   (merge 3574816d98 rj/complete-worktree-paths-fix later to maint).\n\n * When git refuses to create a branch because the proposed branch\n   name is not a valid refname, an advice message is given to refer\n   the user to exact naming rules.\n   (merge 8fbd903e58 kh/branch-ref-syntax-advice later to maint).\n\n * Code simplification by getting rid of code that sets an environment\n   variable that is no longer used.\n   (merge 72a8d3f027 pw/rebase-i-ignore-cherry-pick-help-environment later to maint).\n\n * The code to find the effective end of log messages can fall into an\n   endless loop, which has been corrected.\n   (merge 2541cba2d6 fs/find-end-of-log-message-fix later to maint).\n\n * Mark-up used in the documentation has been improved for\n   consistency.\n   (merge 45d5ed3e50 ja/doc-markup-fixes later to maint).\n\n * The status.showUntrackedFiles configuration variable was\n   incorrectly documented to accept \"false\", which has been corrected.\n\n * Leaks from \"git restore\" have been plugged.\n   (merge 2f64da0790 rj/restore-plug-leaks later to maint).\n\n * \"git bugreport --no-suffix\" was not supported and instead\n   segfaulted, which has been corrected.\n   (merge b3b57c69da js/bugreport-no-suffix-fix later to maint).\n\n * The documentation for \"%(trailers[:options])\" placeholder in the\n   \"--pretty\" option of commands in the \"git log\" family has been\n   updated.\n   (merge bff85a338c bl/doc-key-val-sep-fix later to maint).\n\n * \"git checkout --conflict=bad\" reported a bad conflictStyle as if it\n   were given to a configuration variable; it has been corrected to\n   report that the command line option is bad.\n   (merge 5a99c1ac1a pw/checkout-conflict-errorfix later to maint).\n\n * Code clean-up in the \"git log\" machinery that implements custom log\n   message formatting.\n   (merge 1c10b8e5b0 jk/pretty-subject-cleanup later to maint).\n\n * \"git config\" corrupted literal HT characters written in the\n   configuration file as part of a value, which has been corrected.\n   (merge e6895c3f97 ds/config-internal-whitespace-fix later to maint).\n\n * A unit test for reftable code tried to enumerate all files in a\n   directory after reftable operations and expected to see nothing but\n   the files it wanted to leave there, but was fooled by .nfs* cruft\n   files left, which has been corrected.\n   (merge 0068aa7946 ps/reftable-unit-test-nfs-workaround later to maint).\n\n * The implementation and documentation of \"object-format\" option\n   exchange between the Git itself and its remote helpers did not\n   quite match, which has been corrected.\n\n * The \"--pretty=<shortHand>\" option of the commands in the \"git log\"\n   family, defined as \"[pretty] shortHand = <expansion>\" should have\n   been looked up case insensitively, but was not, which has been\n   corrected.\n   (merge f999d5188b bl/pretty-shorthand-config-fix later to maint).\n\n * \"git apply\" failed to extract the filename the patch applied to,\n   when the change was about an empty file created in or deleted from\n   a directory whose name ends with a SP, which has been corrected.\n   (merge 776ffd1a30 jc/apply-parse-diff-git-header-names-fix later to maint).\n\n * Update a more recent tutorial doc.\n   (merge 95ab557b4b dg/myfirstobjectwalk-updates later to maint).\n\n * The test script had an incomplete and ineffective attempt to avoid\n   clobbering the testing user's real crontab (and its equivalents),\n   which has been completed.\n   (merge 73cb87773b es/test-cron-safety later to maint).\n\n * Use advice_if_enabled() API to rewrite a simple pattern to\n   call advise() after checking advice_enabled().\n   (merge 6412d01527 rj/use-adv-if-enabled later to maint).\n\n * Another \"set -u\" fix for the bash prompt (in contrib/) script.\n   (merge d7805bc743 vs/complete-with-set-u-fix later to maint).\n\n * \"git checkout/switch --detach foo\", after switching to the detached\n   HEAD state, gave the tracking information for the 'foo' branch,\n   which was pointless.\n\n * \"git apply\" has been updated to lift the hardcoded pathname length\n   limit, which in turn allowed a mksnpath() function that is no\n   longer used.\n   (merge 708f7e0590 rs/apply-lift-path-length-limit later to maint).\n\n * A file descriptor leak in an error codepath, used when \"git apply\n   --reject\" fails to create the *.rej file, has been corrected.\n   (merge 2b1f456adf rs/apply-reject-fd-leakfix later to maint).\n\n * A config parser callback function fell through instead of returning\n   after recognising and processing a variable, wasting cycles, which\n   has been corrected.\n   (merge a816ccd642 ds/fetch-config-parse-microfix later to maint).\n\n * Fix was added to work around a regression in libcURL 8.7.0 (which has\n   already been fixed in their tip of the tree).\n   (merge 92a209bf24 jk/libcurl-8.7-regression-workaround later to maint).\n\n * The variable that holds the value read from the core.excludefile\n   configuration variable used to leak, which has been corrected.\n   (merge 0e0fefb29f jc/unleak-core-excludesfile later to maint).\n\n * vreportf(), which is used by error() and friends, has been taught\n   to give the error message printf-format string when its vsnprintf()\n   call fails, instead of showing nothing useful to identify the\n   nature of the error.\n   (merge c63adab961 rs/usage-fallback-to-show-message-format later to maint).\n\n * Adjust to an upcoming changes to GNU make that breaks our Makefiles.\n   (merge 227b8fd902 tb/make-indent-conditional-with-non-spaces later to maint).\n\n * Git 2.44 introduced a regression that makes the updated code to\n   barf in repositories with multi-pack index written by older\n   versions of Git, which has been corrected.\n\n * When .git/rr-cache/ rerere database gets corrupted or rerere is fed to\n   work on a file with conflicted hunks resolved incompletely, the rerere\n   machinery got confused and segfaulted, which has been corrected.\n   (merge 167395bb47 mr/rerere-crash-fix later to maint).\n\n * The \"receive-pack\" program (which responds to \"git push\") was not\n   converted to run \"git maintenance --auto\" when other codepaths that\n   used to run \"git gc --auto\" were updated, which has been corrected.\n   (merge 7bf3057d9c ps/run-auto-maintenance-in-receive-pack later to maint).\n\n * Other code cleanup, docfix, build fix, etc.\n   (merge f0e578c69c rs/use-xstrncmpz later to maint).\n   (merge 83e6eb7d7a ba/credential-test-clean-fix later to maint).\n   (merge 64562d784d jb/doc-interactive-singlekey-do-not-need-perl later to maint).\n   (merge c431a235e2 cp/t9146-use-test-path-helpers later to maint).\n   (merge 82d75402d5 ds/doc-send-email-capitalization later to maint).\n   (merge 41bff66e35 jc/doc-add-placeholder-fix later to maint).\n   (merge 6835f0efe9 jw/remote-doc-typofix later to maint).\n   (merge 244001aa20 hs/rebase-not-in-progress later to maint).\n   (merge 2ca6c07db2 jc/no-include-of-compat-util-from-headers later to maint).\n   (merge 87bd7fbb9c rs/fetch-simplify-with-starts-with later to maint).\n   (merge f39addd0d9 rs/name-rev-with-mempool later to maint).\n   (merge 9a97b43e03 rs/submodule-prefix-simplify later to maint).\n   (merge 40b8076462 ak/rebase-autosquash later to maint).\n   (merge 3223204456 eg/add-uflags later to maint).\n   (merge 5f78d52dce es/config-doc-sort-sections later to maint).\n   (merge 781fb7b4c2 as/option-names-in-messages later to maint).\n   (merge 51d41dc243 jk/doc-remote-helpers-markup-fix later to maint).\n   (merge e1aaf309db pb/ci-win-artifact-names-fix later to maint).\n   (merge ad538c61da jc/index-pack-fsck-levels later to maint).\n   (merge 67471bc704 ja/doc-formatting-fix later to maint).\n   (merge 86f9ce7dd6 bl/doc-config-fixes later to maint).\n   (merge 0d527842b7 az/grep-group-error-message-update later to maint).\n   (merge 7c43bdf07b rs/strbuf-expand-bad-format later to maint).\n   (merge 8b68b48d5c ds/typofix-core-config-doc later to maint).\n   (merge 39bb692152 rs/imap-send-use-xsnprintf later to maint).\n   (merge 8d320cec60 jc/t2104-style-fixes later to maint).\n   (merge b4454d5a7b pw/t3428-cleanup later to maint).\n   (merge 84a7c33a4b pf/commitish-committish later to maint).\n   (merge 8882ee9d68 la/mailmap-entry later to maint).\n   (merge 44bdba2fa6 rs/no-openssl-compilation-fix-on-macos later to maint).\n   (merge f412d72c19 yb/replay-doc-linkfix later to maint).\n   (merge 5da40be8d7 xx/rfc2822-date-format-in-doc later to maint).\n\n----------------------------------------------------------------\n\nChanges since v2.44.0 are as follows:\n\nAhelenia Ziemiańska (1):\n      grep: improve errors for unmatched ( and )\n\nAlexander Shopov (5):\n      transport-helper.c: trivial fix of error message\n      builtin/remote.c: trivial fix of error message\n      builtin/clone.c: trivial fix of message\n      revision.c: trivial fix to message\n      l10n: bg.po: Updated Bulgarian translation (5652t)\n\nArkadii Yakovets (1):\n      l10n: uk: v2.45 update\n\nAryan Gupta (1):\n      tests: modernize the test script t0010-racy-git.sh\n\nBagas Sanjaya (1):\n      l10n: po-id for 2.45\n\nBeat Bolli (25):\n      completion: use awk for filtering the config entries\n      date: make \"iso-strict\" conforming for the UTC timezone\n      t0006: add more tests with a negative TZ offset\n      doc: avoid redundant use of cat\n      contrib/subtree/t: avoid redundant use of cat\n      t/lib-cvs.sh: avoid redundant use of cat\n      t/annotate-tests.sh: avoid redundant use of cat\n      t/perf: avoid redundant use of cat\n      t/t0*: avoid redundant uses of cat\n      t/t1*: avoid redundant uses of cat\n      t/t3*: avoid redundant uses of cat\n      t/t4*: avoid redundant uses of cat\n      t/t5*: avoid redundant uses of cat\n      t/t6*: avoid redundant uses of cat\n      t/t7*: avoid redundant use of cat\n      t/t8*: avoid redundant use of cat\n      t/t9*: avoid redundant uses of cat\n      t/t1*: merge a \"grep | sed\" pipeline\n      t/t3*: merge a \"grep | awk\" pipeline\n      t/t4*: merge a \"grep | sed\" pipeline\n      t/t5*: merge a \"grep | sed\" pipeline\n      t/t8*: merge \"grep | sed\" pipelines\n      t/t9*: merge \"grep | sed\" pipelines\n      contrib/coverage-diff: avoid redundant pipelines\n      git-quiltimport: avoid an unnecessary subshell\n\nBo Anderson (5):\n      t/lib-credential: clean additional credential\n      osxkeychain: replace deprecated SecKeychain API\n      osxkeychain: erase all matching credentials\n      osxkeychain: erase matching passwords only\n      osxkeychain: store new attributes\n\nBrian C Tracy (1):\n      fuzz: add fuzzer for config parsing\n\nBrian Lyles (13):\n      docs: clarify file options in git-config `--edit`\n      docs: fix typo in git-config `--default`\n      docs: correct trailer `key_value_separator` description\n      docs: adjust trailer `separator` and `key_value_separator` language\n      pretty: update tests to use `test_config`\n      pretty: find pretty formats case-insensitively\n      docs: address inaccurate `--empty` default with `--exec`\n      docs: clean up `--empty` formatting in git-rebase(1) and git-am(1)\n      rebase: update `--empty=ask` to `--empty=stop`\n      sequencer: handle unborn branch with `--allow-empty`\n      sequencer: do not require `allow_empty` for redundant commit options\n      cherry-pick: enforce `--keep-redundant-commits` incompatibility\n      cherry-pick: add `--empty` for more robust redundant commit handling\n\nChandra Pratap (2):\n      apply: ignore working tree filemode when !core.filemode\n      t9146: replace test -d/-e/-f with appropriate test_path_is_* function\n\nChristian Couder (5):\n      revision: clarify a 'return NULL' in get_reference()\n      oidset: refactor oidset_insert_from_set()\n      t6022: fix 'test' style and 'even though' typo\n      rev-list: allow missing tips with --missing=[print|allow*]\n      revision: fix --missing=[print|allow*] for annotated tags\n\nDerrick Stolee (1):\n      fetch: return when parsing submodule.recurse\n\nDirk Gouders (6):\n      Documentation/user-manual.txt: example for generating object hashes\n      MyFirstObjectWalk: use additional arg in config_fn_t\n      MyFirstObjectWalk: fix misspelled \"builtins/\"\n      MyFirstObjectWalk: fix filtered object walk\n      MyFirstObjectWalk: fix description for counting omitted objects\n      MyFirstObjectWalk: add stderr to pipe processing\n\nDragan Simic (8):\n      documentation: send-email: use camel case consistently\n      config: minor addition of whitespace\n      config: really keep value-internal whitespace verbatim\n      t1300: add more tests for whitespace and inline comments\n      config.txt: describe handling of whitespace further\n      grep docs: describe --recurse-submodules further and improve formatting a bit\n      grep docs: describe --no-index further and improve formatting a bit\n      config: fix some small capitalization issues, as spotted\n\nEmir SARI (1):\n      l10n: tr: Update Turkish translations\n\nEric Sunshine (2):\n      docs: sort configuration variable groupings alphabetically\n      test-lib: fix non-functioning GIT_TEST_MAINT_SCHEDULER fallback\n\nEric W. Biederman (23):\n      object-file-convert: stubs for converting from one object format to another\n      oid-array: teach oid-array to handle multiple kinds of oids\n      object-names: support input of oids in any supported hash\n      repository: add a compatibility hash algorithm\n      loose: compatibilty short name support\n      object-file: update the loose object map when writing loose objects\n      object-file: add a compat_oid_in parameter to write_object_file_flags\n      commit: convert mergetag before computing the signature of a commit\n      commit: export add_header_signature to support handling signatures on tags\n      tag: sign both hashes\n      object: factor out parse_mode out of fast-import and tree-walk into in object.h\n      object-file-convert: don't leak when converting tag objects\n      object-file-convert: convert commits that embed signed tags\n      object-file: update object_info_extended to reencode objects\n      rev-parse: add an --output-object-format parameter\n      builtin/cat-file: let the oid determine the output algorithm\n      tree-walk: init_tree_desc take an oid to get the hash algorithm\n      object-file: handle compat objects in check_object_signature\n      builtin/ls-tree: let the oid determine the output algorithm\n      test-lib: compute the compatibility hash so tests may use it\n      t1006: rename sha1 to oid\n      t1006: test oid compatibility with cat-file\n      t1016-compatObjectFormat: add tests to verify the conversion between objects\n\nEugenio Gigante (1):\n      add: use unsigned type for collection of bits\n\nFlorian Schmidt (1):\n      wt-status: don't find scissors line beyond buf len\n\nGhanshyam Thakkar (5):\n      add-patch: classify '@' as a synonym for 'HEAD'\n      add -p tests: remove PERL prerequisites\n      setup: remove unnecessary variable\n      builtin/commit: error out when passing untracked path with -i\n      builtin/add: error out when passing untracked path with -u\n\nHaritha D (1):\n      build: support z/OS (OS/390).\n\nHarmen Stoppels (1):\n      rebase: make warning less passive aggressive\n\nJakub Wilk (1):\n      git-remote.txt: fix typo\n\nJean-Noël Avila (18):\n      doc: git-rev-parse: enforce command-line description syntax\n      doc: close unclosed angle-bracket of a placeholder in git-clone doc\n      doc: end sentences with full-stop\n      doc: clarify the format of placeholders\n      doc: git-init: format verbatim parts\n      doc: git-init: format placeholders\n      doc: git-init: rework definition lists\n      doc: git-init: rework config item init.templateDir\n      doc: git-clone: format verbatim words\n      doc: git-clone: format placeholders\n      doc: format alternatives in synopsis\n      doc: fix some placeholders formating\n      doc: rework CodingGuidelines with new formatting rules\n      doc: allow literal and emphasis format in doc vs help tests\n      doc: git-init: apply new documentation formatting guidelines\n      doc: git-clone: apply new documentation formatting guidelines\n      doc: git-clone: do not autoreference the manpage in itself\n      l10n: fr: v2.45.0\n\nJeff Hostetler (17):\n      name-hash: add index_dir_find()\n      t7527: add case-insensitve test for FSMonitor\n      fsmonitor: refactor refresh callback on directory events\n      fsmonitor: clarify handling of directory events in callback helper\n      fsmonitor: refactor refresh callback for non-directory events\n      dir: create untracked_cache_invalidate_trimmed_path()\n      fsmonitor: refactor untracked-cache invalidation\n      fsmonitor: move untracked-cache invalidation into helper functions\n      fsmonitor: return invalidated cache-entry count on directory event\n      fsmonitor: remove custom loop from non-directory path handler\n      fsmonitor: return invalidated cache-entry count on non-directory event\n      fsmonitor: trace the new invalidated cache-entry count\n      fsmonitor: refactor bit invalidation in refresh callback\n      fsmonitor: support case-insensitive events\n      t0211: demonstrate missing 'def_param' events for certain commands\n      trace2: avoid emitting 'def_param' set more than once\n      trace2: emit 'def_param' set with 'cmd_name' event\n\nJeff King (51):\n      t0303: check that helper_test_clean removes all credentials\n      userdiff: skip textconv caching when not in a repository\n      Revert \"refs: allow @{n} to work with n-sized reflog\"\n      get_oid_basic(): special-case ref@{n} for oldest reflog entry\n      read_ref_at(): special-case ref@{0} for an empty reflog\n      upload-pack: drop separate v2 \"haves\" array\n      upload-pack: switch deepen-not list to an oid_array\n      upload-pack: use oidset for deepen_not list\n      upload-pack: use a strmap for want-ref lines\n      upload-pack: accept only a single packfile-uri line\n      upload-pack: always turn off save_commit_buffer\n      upload-pack: use PARSE_OBJECT_SKIP_HASH_CHECK in more places\n      upload-pack: free tree buffers after parsing\n      upload-pack: use repository struct to get config\n      upload-pack: centralize setup of sideband-all config\n      upload-pack: use existing config mechanism for advertisement\n      upload-pack: only accept packfile-uris if we advertised it\n      doc/gitremote-helpers: fix missing single-quote\n      config: forbid newline as core.commentChar\n      strbuf: simplify comment-handling in add_lines() helper\n      strbuf: avoid static variables in strbuf_add_commented_lines()\n      commit: refactor base-case of adjust_comment_line_char()\n      strbuf: avoid shadowing global comment_line_char name\n      environment: store comment_line_char as a string\n      strbuf: accept a comment string for strbuf_stripspace()\n      strbuf: accept a comment string for strbuf_commented_addf()\n      strbuf: accept a comment string for strbuf_add_commented_lines()\n      prefer comment_line_str to comment_line_char for printing\n      find multi-byte comment chars in NUL-terminated strings\n      find multi-byte comment chars in unterminated buffers\n      sequencer: handle multi-byte comment characters when writing todo list\n      wt-status: drop custom comment-char stringification\n      environment: drop comment_line_char compatibility macro\n      config: allow multi-byte core.commentChar\n      shortlog: stop setting pp.print_email_subject\n      pretty: split oneline and email subject printing\n      pretty: drop print_email_subject flag\n      log: do not set up extra_headers for non-email formats\n      format-patch: return an allocated string from log_write_email_headers()\n      format-patch: simplify after-subject MIME header handling\n      doc/gitremote-helpers: fix more missing single-quotes\n      transport-helper: use write helpers more consistently\n      transport-helper: drop \"object-format <algo>\" option\n      transport-helper: send \"true\" value for object-format option\n      contrib: drop hg-to-git script\n      format-patch: fix leak of empty header string\n      rebase: use child_process_clear() to clean\n      config: add core.commentString\n      http: reset POSTFIELDSIZE when clearing curl handle\n      INSTALL: bump libcurl version to 7.21.3\n      remote-curl: add Transfer-Encoding header only for older curl\n\nJiamu Sun (1):\n      bugreport.c: fix a crash in `git bugreport` with `--no-suffix` option\n\nJiang Xin (1):\n      l10n: TEAMS: retire l10n teams no update in 1 year\n\nJohannes Schindelin (22):\n      merge-tree: accept 3 trees as arguments\n      merge-tree: fail with a non-zero exit code on missing tree objects\n      merge-ort: do check `parse_tree()`'s return value\n      t4301: verify that merge-tree fails on missing blob objects\n      Always check `parse_tree*()`'s return value\n      cache-tree: avoid an unnecessary check\n      fill_tree_descriptor(): mark error message for translation\n      neue: remove a bogus empty file\n      commit-reach(paint_down_to_common): plug two memory leaks\n      commit-reach(repo_in_merge_bases_many): optionally expect missing commits\n      commit-reach(repo_in_merge_bases_many): report missing commits\n      commit-reach(paint_down_to_common): prepare for handling shallow commits\n      commit-reach(paint_down_to_common): start reporting errors\n      commit-reach(merge_bases_many): pass on \"missing commits\" errors\n      commit-reach(get_merge_bases_many_0): pass on \"missing commits\" errors\n      commit-reach(repo_get_merge_bases): pass on \"missing commits\" errors\n      commit-reach(get_octopus_merge_bases): pass on \"missing commits\" errors\n      commit-reach(repo_get_merge_bases_many): pass on \"missing commits\" errors\n      commit-reach(repo_get_merge_bases_many_dirty): pass on errors\n      merge-recursive: prepare for `merge_submodule()` to report errors\n      merge-ort/merge-recursive: do report errors in `merge_submodule()`\n      merge-tree: fix argument type of the `--merge-base` option\n\nJohn Cai (1):\n      t5300: fix test_with_bad_commit()\n\nJonas Wunderlich (1):\n      doc: status.showUntrackedFiles does not take \"false\"\n\nJosh Triplett (2):\n      commit: avoid redundant scissor line with --cleanup=scissors -v\n      commit: unify logic to avoid multiple scissors lines when merging\n\nJulio Bacellari (1):\n      doc: remove outdated information about interactive.singleKey\n\nJunio C Hamano (64):\n      apply: correctly reverse patch's pre- and post-image mode bits\n      apply: code simplification\n      t9210: do not rely on lazy fetching to fail\n      git: --no-lazy-fetch option\n      doc: add shortcut to \"am --whitespace=<action>\"\n      doc: apply the new placeholder rules to git-add documentation\n      compat: drop inclusion of <git-compat-util.h>\n      Start the 2.45 cycle\n      git: document GIT_NO_REPLACE_OBJECTS environment variable\n      doc: clarify the wording on <git-compat-util.h> requirement\n      git: extend --no-lazy-fetch to work across subprocesses\n      The second batch\n      The third batch\n      test_i18ngrep: hard deprecate and forbid its use\n      unpack: replace xwrite() loop with write_in_full()\n      sideband: avoid short write(2)\n      repack: check error writing to pack-objects subprocess\n      clean: further clean-up of implementation around \"--force\"\n      The fourth batch\n      The fifth batch\n      setup: notice more types of implicit bare repositories\n      The sixth batch\n      status: unify parsing of --untracked= and status.showUntrackedFiles\n      status: allow --untracked=false and friends\n      The seventh batch\n      The eighth batch\n      config: fix --comment formatting\n      config: allow tweaking whitespace between value and comment\n      diff.*Prefix: use camelCase in the doc and test titles\n      The ninth batch\n      apply: parse names out of \"diff --git\" more carefully\n      The tenth batch\n      The eleventh batch\n      SubmittingPatches: release-notes entry experiment\n      The twelfth batch\n      t4126: make sure a directory with SP at the end is usable\n      t4126: fix \"funny directory name\" test on Windows (again)\n      advice: omit trailing whitespace\n      checkout: omit \"tracking\" information on a detached HEAD\n      The thirteenth batch\n      t2104: style fixes\n      The fourteenth batch\n      revision: optionally record matches with pathspec elements\n      The fifteenth batch\n      CodingGuidelines: describe \"export VAR=VAL\" rule\n      CodingGuidelines: quote assigned value in 'local var=$val'\n      t: local VAR=\"VAL\" (quote positional parameters)\n      t: local VAR=\"VAL\" (quote command substitution)\n      t: local VAR=\"VAL\" (quote ${magic-reference})\n      t: teach lint that RHS of 'local VAR=VAL' needs to be quoted\n      t0610: local VAR=\"VAL\" fix\n      t1016: local VAR=\"VAL\" fix\n      config: do not leak excludes_file\n      Makefile(s): do not enforce \"all indents must be done with tab\"\n      The sixteenth batch\n      t2104: style fixes\n      The seventeenth batch\n      The eighteenth batch\n      The ninteenth batch\n      The twentieth batch\n      Git 2.45-rc0\n      A bit more topics before -rc1\n      Git 2.45-rc1\n      Git 2.45\n\nJustin Tobler (3):\n      reftable/stack: expose option to disable auto-compaction\n      reftable/stack: add env to disable autocompaction\n      reftable/stack: use geometric table compaction\n\nKarthik Nayak (7):\n      refs: introduce `is_pseudoref()` and `is_headref()`\n      refs: extract out `loose_fill_ref_dir_regular_file()`\n      refs: introduce `refs_for_each_include_root_refs()`\n      ref-filter: rename 'FILTER_REFS_ALL' to 'FILTER_REFS_REGULAR'\n      for-each-ref: add new option to include root refs\n      update-ref: use {old,new}-oid instead of {old,new}value\n      githooks: use {old,new}-oid instead of {old,new}-value\n\nKipras Melnikovas (1):\n      mergetools: vimdiff: use correct tool's name when reading mergetool config\n\nKristoffer Haugsbakk (9):\n      column: disallow negative padding\n      column: guard against negative padding\n      gitcli: drop mention of “non-dashed form”\n      config: document `core.commentChar` as ASCII-only\n      t3200: improve test style\n      advice: make all entries stylistically consistent\n      advice: use backticks for verbatim\n      advice: use double quotes for regular quoting\n      branch: advise about ref syntax rules\n\nLinus Arver (15):\n      trailer: free trailer_info _after_ all related usage\n      shortlog: add test for de-duplicating folded trailers\n      trailer: rename functions to use 'trailer'\n      trailer: reorder format_trailers_from_commit() parameters\n      trailer: move interpret_trailers() to interpret-trailers.c\n      trailer_info_get(): reorder parameters\n      format_trailers(): use strbuf instead of FILE\n      format_trailer_info(): move \"fast path\" to caller\n      format_trailers_from_commit(): indirectly call trailer_info_get()\n      format_trailer_info(): use trailer_item objects\n      format_trailer_info(): drop redundant unfold_value()\n      format_trailer_info(): append newline for non-trailer lines\n      trailer: begin formatting unification\n      trailer: finish formatting unification\n      mailmap: change primary address for Linus Arver\n\nM Hickford (1):\n      libsecret: retrieve empty password\n\nMarcel Röthke (1):\n      rerere: fix crashes due to unmatched opening conflict markers\n\nMatthias Aßhauer (1):\n      Win32: detect unix socket support at runtime\n\nMax Gautier (1):\n      editorconfig: add Makefiles to \"text files\"\n\nMichael Lohmann (2):\n      revision: ensure MERGE_HEAD is a ref in prepare_show_merge\n      revision: implement `git log --merge` also for rebase/cherry-pick/revert\n\nOrgad Shaneh (1):\n      docs: remove duplicate entry and fix typo in 2.45 changelog\n\nPatrick Steinhardt (99):\n      refs: introduce reftable backend\n      ci: add jobs to test with the reftable backend\n      refs/reftable: fix leak when copying reflog fails\n      reftable/record: introduce function to compare records by key\n      reftable/merged: allocation-less dropping of shadowed records\n      reftable/merged: skip comparison for records of the same subiter\n      reftable/pq: allocation-less comparison of entry keys\n      reftable/block: swap buffers instead of copying\n      reftable/record: don't try to reallocate ref record name\n      reftable/reader: add comments to `table_iter_next()`\n      t: move tests exercising the \"files\" backend\n      t0410: convert tests to use DEFAULT_REPO_FORMAT prereq\n      t1400: exercise reflog with gaps with reftable backend\n      t1404: make D/F conflict tests compatible with reftable backend\n      t1405: remove unneeded cleanup step\n      t2011: exercise D/F conflicts with HEAD with the reftable backend\n      t7003: ensure filter-branch prunes reflogs with the reftable backend\n      git-difftool--helper: honor `--trust-exit-code` with `--dir-diff`\n      dir-iterator: pass name to `prepare_next_entry_data()` directly\n      dir-iterator: support iteration in sorted order\n      refs/files: sort reflogs returned by the reflog iterator\n      refs/files: sort merged worktree and common reflogs\n      refs: always treat iterators as ordered\n      refs: drop unused params from the reflog iterator callback\n      refs: stop resolving ref corresponding to reflogs\n      builtin/reflog: introduce subcommand to list reflogs\n      builtin/clone: allow remote helpers to detect repo\n      refs/reftable: don't fail empty transactions in repo without HEAD\n      reftable/pq: use `size_t` to track iterator index\n      reftable/merged: make `merged_iter` structure private\n      reftable/merged: advance subiter on subsequent iteration\n      reftable/merged: make subiters own their records\n      reftable/merged: remove unnecessary null check for subiters\n      reftable/merged: handle subiter cleanup on close only\n      reftable/merged: circumvent pqueue with single subiter\n      reftable/merged: avoid duplicate pqueue emptiness check\n      reftable/record: reuse refname when decoding\n      reftable/record: reuse refname when copying\n      reftable/record: decode keys in place\n      reftable: allow inlining of a few functions\n      refs/reftable: precompute prefix length\n      refs/reftable: reload correct stack when creating reflog iter\n      reftable/record: convert old and new object IDs to arrays\n      reftable/record: avoid copying author info\n      reftable/record: reuse refnames when decoding log records\n      reftable/record: reuse message when decoding log records\n      reftable/record: use scratch buffer when decoding records\n      refs/reftable: track last log record name via strbuf\n      t0610: remove unused variable assignment\n      lockfile: report when rollback fails\n      reftable/stack: register new tables as tempfiles\n      reftable/stack: register lockfiles during compaction\n      reftable/stack: register compacted tables as tempfiles\n      reftable/record: fix memory leak when decoding object records\n      reftable/block: fix binary search over restart counter\n      t5601: exercise clones with \"includeIf.*.onbranch\"\n      reftable: fix tests being broken by NFS' delete-after-close semantics\n      t7800: improve test descriptions with empty arguments\n      t7800: use single quotes for test bodies\n      t/README: document how to loop around test cases\n      reftable/stack: fix error handling in `reftable_stack_init_addition()`\n      reftable/error: discern locked/outdated errors\n      reftable/stack: use error codes when locking fails during compaction\n      reftable/stack: gracefully handle failed auto-compaction due to locks\n      refs/reftable: print errors on compaction failure\n      t/helper: drop pack-refs wrapper\n      refs: move `struct pack_refs_opts` to where it's used\n      refs: remove `PACK_REFS_ALL` flag\n      refs/reftable: expose auto compaction via new flag\n      builtin/pack-refs: release allocated memory\n      builtin/pack-refs: introduce new \"--auto\" flag\n      builtin/gc: move `struct maintenance_run_opts`\n      t6500: extract objects with \"17\" prefix\n      builtin/gc: forward git-gc(1)'s `--auto` flag when packing refs\n      builtin/gc: pack refs when using `git maintenance run --auto`\n      reftable/basics: fix return type of `binsearch()` to be `size_t`\n      reftable/basics: improve `binsearch()` test\n      reftable/refname: refactor binary search over refnames\n      reftable/block: refactor binary search over restart points\n      reftable/block: fix error handling when searching restart points\n      reftable/record: extract function to decode key lengths\n      reftable/block: avoid decoding keys when searching restart points\n      t0610: make `--shared=` tests reusable\n      t0610: execute git-pack-refs(1) with specified umask\n      reftable/block: rename `block_reader_start()`\n      reftable/block: merge `block_iter_seek()` and `block_reader_seek()`\n      reftable/block: better grouping of functions\n      reftable/block: introduce `block_reader_release()`\n      reftable/block: move ownership of block reader into `struct table_iter`\n      reftable/reader: iterate to next block in place\n      reftable/block: reuse uncompressed blocks\n      reftable/block: open-code call to `uncompress2()`\n      reftable/block: reuse `zstream` state on inflation\n      reftable/block: avoid copying block iterators on seek\n      pack-bitmap: gracefully handle missing BTMP chunks\n      run-command: introduce function to prepare auto-maintenance process\n      builtin/receive-pack: convert to use git-maintenance(1)\n      docs: improve changelog entry for `git pack-refs --auto`\n      docs: address typos in Git v2.45 changelog\n\nPeter Hutterer (1):\n      diff: add diff.srcPrefix and diff.dstPrefix configuration variables\n\nPeter Krefting (2):\n      bisect: report the found commit with \"show\"\n      l10n: sv.po: Update Swedish translation\n\nPhilippe Blain (5):\n      merge-ort: turn submodule conflict suggestions into an advice\n      ci(github): make Windows test artifacts name unique\n      sequencer: allow disabling conflict advice\n      builtin/am: allow disabling conflict advice\n      t/README: mention test files are make targets\n\nPhillip Wood (9):\n      rebase -i: stop setting GIT_CHERRY_PICK_HELP\n      xdiff-interface: refactor parsing of merge.conflictstyle\n      merge-ll: introduce LL_MERGE_OPTIONS_INIT\n      merge options: add a conflict style member\n      checkout: cleanup --conflict=<style> parsing\n      checkout: fix interaction between --conflict and --merge\n      t3428: modernize test setup\n      t3428: use test_commit_message\n      t3428: restore coverage for \"apply\" backend\n\nPi Fisher (1):\n      typo: replace 'commitish' with 'committish'\n\nRalf Thielow (1):\n      l10n: Update German translation\n\nRalph Seichter (1):\n      config: add --comment option to add a comment\n\nRené Scharfe (32):\n      use xstrncmpz()\n      fetch: convert strncmp() with strlen() to starts_with()\n      mem-pool: add mem_pool_strfmt()\n      name-rev: use mem_pool_strfmt()\n      submodule: use strvec_pushf() for --submodule-prefix\n      t-ctype: allow NUL anywhere in the specification string\n      t-ctype: simplify EOF check\n      t-ctype: align output of i\n      t-ctype: avoid duplicating class names\n      parse-options: recognize abbreviated negated option with arg\n      parse-options: set arg of abbreviated option lazily\n      parse-options: factor out register_abbrev() and struct parsed_option\n      parse-options: detect ambiguous self-negation\n      parse-options: normalize arg and long_name before comparison\n      parse-options: rearrange long_name matching code\n      t-prio-queue: shorten array index message\n      t-prio-queue: check result array bounds\n      factor out strbuf_expand_bad_format()\n      cat-file: use strbuf_expand_bad_format()\n      midx: use strvec_pushf() for pack-objects base name\n      mem-pool: use st_add() in mem_pool_strvfmt()\n      imap-send: use xsnprintf to format command\n      t-prio-queue: simplify using compound literals\n      apply: avoid fixed-size buffer in create_one_file()\n      path: remove mksnpath()\n      apply: don't leak fd on fdopen() error\n      usage: report vsnprintf(3) failure\n      date: make DATE_MODE thread-safe\n      git-compat-util: fix NO_OPENSSL on current macOS\n      imap-send: increase command size limit\n      apply: avoid using fixed-size buffer in write_out_one_reject()\n      don't report vsnprintf(3) error as bug\n\nRichard Macklin (1):\n      rebase: fix typo in autosquash documentation\n\nRubén Justo (18):\n      tag: error when git-column fails\n      completion: fix __git_complete_worktree_paths\n      completion: reflog with implicit \"show\"\n      completion: reflog show <log-options>\n      completion: introduce __git_find_subcommand\n      completion: factor out __git_resolve_builtins\n      completion: reflog subcommands and options\n      checkout: plug some leaks in git-restore\n      add-patch: introduce 'p' in interactive-patch\n      add-patch: do not print hunks repeatedly\n      add: use advise_if_enabled for ADVICE_ADD_IGNORED_FILE\n      add: use advise_if_enabled for ADVICE_ADD_EMPTY_PATHSPEC\n      add: use advise_if_enabled for ADVICE_ADD_EMBEDDED_REPO\n      launch_editor: waiting message on error\n      apply: plug a leak in apply_data\n      add-interactive: plug a leak in get_untracked_files\n      add-patch: plug a leak handling the '/' command\n      add: plug a leak on interactive_add\n\nSZEDER Gábor (1):\n      upload-pack: don't send null character in abort message to the client\n\nSergey Organov (1):\n      clean: improve -n and -f implementation and documentation\n\nSteven Jeuris (1):\n      userdiff: better method/property matching for C#\n\nTaylor Blau (9):\n      Documentation/config/pack.txt: fix broken AsciiDoc mark-up\n      upload-pack: disallow object-info capability by default\n      midx-write: move writing-related functions from midx.c\n      midx-write.c: factor out common want_included_pack() routine\n      midx-write.c: check count of packs to repack after grouping\n      midx-write.c: use `--stdin-packs` when repacking\n      t/t7700-repack.sh: fix test breakages with `GIT_TEST_MULTI_PACK_INDEX=1 `\n      Makefile(s): avoid recipe prefix in conditional statements\n      Documentation/RelNotes/2.45.0.txt: fix typo\n\nTeng Long (1):\n      l10n: zh_CN: for git 2.45 rounds\n\nThalia Archibald (8):\n      fast-import: tighten path unquoting\n      fast-import: directly use strbufs for paths\n      fast-import: allow unquoted empty path for root\n      fast-import: remove dead strbuf\n      fast-import: improve documentation for path quoting\n      fast-import: document C-style escapes for paths\n      fast-import: forbid escaped NUL in paths\n      fast-import: make comments more precise\n\nVille Skyttä (2):\n      completion: fix prompt with unset SHOWCONFLICTSTATE in nounset mode\n      completion: protect prompt against unset SHOWUPSTREAM in nounset mode\n\nVincenzo Mezzela (1):\n      t7301: use test_path_is_(missing|file)\n\nVũ Tiến Hưng (2):\n      l10n: Update Vietnamese team contact\n      l10n: vi: Updated translation for 2.45\n\nXing Xin (1):\n      Documentation: fix typos describing date format\n\nYehezkel Bernat (1):\n      Documentation: fix linkgit reference\n\nYi-Jyun Pan (1):\n      l10n: zh-TW: Git 2.45\n\nbrian m. carlson (7):\n      loose: add a mapping between SHA-1 and SHA-256 for loose objects\n      commit: write commits for both hashes\n      cache: add a function to read an OID of a specific algorithm\n      object-file-convert: add a function to convert trees between algorithms\n      object-file-convert: convert tag objects when writing\n      object-file-convert: convert commit objects when writing\n      repository: implement extensions.compatObjectFormat\n\nshejialuo (1):\n      t9117: prefer test_path_* helper functions\n\nĐoàn Trần Công Danh (1):\n      t9604: Fix test for musl libc and new Debian\n\n"},{"id":"493834","messageId":"0be901da9bee$950bc830$bf235890$@nexbridge.com","threadId":"61391","inReplyTo":"xmqq8r0ww0sj.fsf@gitster.g","subject":"RE: [ANNOUNCE] Git v2.45.0","fromName":"","fromEmail":"rsbecker@nexbridge.com","sentAt":"2024-05-01T17:39:50Z","receivedAt":"2024-05-01T17:40:08Z","isPatch":false,"sender":{"key":"randall.becker@nexbridge.ca","avatar":"https://avatars.githubusercontent.com/u/28956764?v=4"},"body":"On Monday, April 29, 2024 1:12 PM, Junio C Hamano wrote:\n>The latest feature release Git v2.45.0 is now available at the\n>usual places.  It is comprised of 540 non-merge commits since\n>v2.44.0, contributed by 96 people, 38 of which are new faces [*].\n>\n>The tarballs are found at:\n>\n>    https://www.kernel.org/pub/software/scm/git/\n>\n>The following public repositories all have a copy of the 'v2.45.0'\n>tag and the 'master' branch that the tag points at:\n>\n>  url = https://git.kernel.org/pub/scm/git/git\n>  url = https://kernel.googlesource.com/pub/scm/git/git\n>  url = git://repo.or.cz/alt-git.git\n>  url = https://github.com/gitster/git\n\nThe HPE NonStop builds for git 2.45.0 are available at the ITUGLIB website at https://ituglib.connect-community.org/apps/Ituglib/SrchOpenSrcLib.xhtml.\n\nThanks to the entire git and ITUGLIB teams for making these happen.\n\nRegards,\nRandall\n\n--\nBrief whoami: NonStop&UNIX developer since approximately\nUNIX(421664400)\nNonStop(211288444200000000)\n-- In real life, I talk too much.\n\n\n\n"},{"id":"493872","messageId":"20240502055621.GA29945@tb-raspi4","threadId":"61391","inReplyTo":"xmqq8r0ww0sj.fsf@gitster.g","subject":"t4216-log-bloom.sh broken ?","fromName":"Torsten Bögershausen","fromEmail":"tboegi@web.de","sentAt":"2024-05-02T05:56:22Z","receivedAt":"2024-05-02T05:56:28Z","isPatch":false,"sender":{"key":"tboegi@web.de","avatar":"https://avatars.githubusercontent.com/u/7138363?v=4"},"body":"[]\n\nHighjacking the Git v2.45.0 announcement:\nThere are 4 test cases in t4216-log-bloom.sh, that do not pass on one\nMac here (they pass on another machine)\n\nI haven't digged further, but here are some log files,\nhopefully helpful to somebody.\n\n\nnot ok 141 - Bloom reader notices too-small data chunk\n#\n#               check_corrupt_graph BDAT clear 00000000 &&\n#               echo \"warning: ignoring too-small changed-path chunk\" \\\n#                       \"(4 < 12) in commit-graph file\" >expect.err &&\n#               test_cmp expect.err err\n#\nnot ok 142 - Bloom reader notices out-of-bounds filter offsets\n#\n#               check_corrupt_graph BIDX 12 FFFFFFFF &&\n#               # use grep to avoid depending on exact chunk size\n#               grep \"warning: ignoring out-of-range offset (4294967295) for changed-path filter at pos 3 of .git/objects/info/commit-graph\" err\n\n#\nnot ok 143 - Bloom reader notices too-small index chunk\n#\n#               # replace the index with a single entry, making most\n#               # lookups out-of-bounds\n#               check_corrupt_graph BIDX clear 00000000 &&\n#               echo \"warning: commit-graph changed-path index chunk\" \\\n#                       \"is too small\" >expect.err &&\n#               test_cmp expect.err err\n#\nnot ok 144 - Bloom reader notices out-of-order index offsets\n#\n#               # we do not know any real offsets, but we can pick\n#               # something plausible; we should not get to the point of\n#               # actually reading from the bogus offsets anyway.\n#               corrupt_graph BIDX 4 0000000c00000005 &&\n#               echo \"warning: ignoring decreasing changed-path index offsets\" \\\n#                       \"(12 > 5) for positions 1 and 2 of .git/objects/info/commit-graph\" >expect.err &&\n#               git -c core.commitGraph=false log -- A/B/file2 >expect.out &&\n#               git -c core.commitGraph=true log -- A/B/file2 >out 2>err &&\n#               test_cmp expect.out out &&\n#               test_cmp expect.err err\n#\n# failed 4 among 144 test(s)\n1..144\n\n\nok 140 - Bloom generation backfills empty commits\n\nexpecting success of 4216.141 'Bloom reader notices too-small data chunk':\n\tcheck_corrupt_graph BDAT clear 00000000 &&\n\techo \"warning: ignoring too-small changed-path chunk\" \\\n\t\t\"(4 < 12) in commit-graph file\" >expect.err &&\n\ttest_cmp expect.err err\n\n++ check_corrupt_graph BDAT clear 00000000\n++ corrupt_graph BDAT clear 00000000\n++ graph=.git/objects/info/commit-graph\n++ test_when_finished 'rm -rf .git/objects/info/commit-graph'\n++ test 0 = 0\n++ test_cleanup='{ rm -rf .git/objects/info/commit-graph\n\t\t} && (exit \"$eval_ret\"); eval_ret=$?; :'\n++ git commit-graph write --reachable --changed-paths\n++ corrupt_chunk_file .git/objects/info/commit-graph BDAT clear 00000000\n++ fn=.git/objects/info/commit-graph\n++ shift\n++ perl /Users/tb/NoBackup/projects/git/git.pu/t/lib-chunk/corrupt-chunk-file.pl BDAT clear 00000000\n++ command /usr/bin/perl /Users/tb/NoBackup/projects/git/git.pu/t/lib-chunk/corrupt-chunk-file.pl BDAT clear 00000000\n++ /usr/bin/perl /Users/tb/NoBackup/projects/git/git.pu/t/lib-chunk/corrupt-chunk-file.pl BDAT clear 00000000\n++ mv .git/objects/info/commit-graph.tmp .git/objects/info/commit-graph\noverride r--r--r--  tb/staff for .git/objects/info/commit-graph? (y/n [n]) not overwritten\n++ git -c core.commitGraph=false log -- A/B/file2\n++ git -c core.commitGraph=true log -- A/B/file2\n++ test_cmp expect.out out\n++ test 2 -ne 2\n++ eval 'diff -u' '\"$@\"'\n+++ diff -u expect.out out\n++ echo 'warning: ignoring too-small changed-path chunk' '(4 < 12) in commit-graph file'\n++ test_cmp expect.err err\n++ test 2 -ne 2\n++ eval 'diff -u' '\"$@\"'\n+++ diff -u expect.err err\n--- expect.err\t2024-05-02 05:51:10.000000000 +0000\n+++ err\t2024-05-02 05:51:10.000000000 +0000\n@@ -1 +0,0 @@\n-warning: ignoring too-small changed-path chunk (4 < 12) in commit-graph file\nerror: last command exited with $?=1\n++ rm -rf .git/objects/info/commit-graph\n++ exit 1\n++ eval_ret=1\n++ :\nnot ok 141 - Bloom reader notices too-small data chunk\n\n\nexpecting success of 4216.142 'Bloom reader notices out-of-bounds filter offsets':\n\tcheck_corrupt_graph BIDX 12 FFFFFFFF &&\n\t# use grep to avoid depending on exact chunk size\n\tgrep \"warning: ignoring out-of-range offset (4294967295) for changed-path filter at pos 3 of .git/objects/info/commit-graph\" err\n\n++ check_corrupt_graph BIDX 12 FFFFFFFF\n++ corrupt_graph BIDX 12 FFFFFFFF\n++ graph=.git/objects/info/commit-graph\n++ test_when_finished 'rm -rf .git/objects/info/commit-graph'\n++ test 0 = 0\n++ test_cleanup='{ rm -rf .git/objects/info/commit-graph\n\t\t} && (exit \"$eval_ret\"); eval_ret=$?; :'\n++ git commit-graph write --reachable --changed-paths\n++ corrupt_chunk_file .git/objects/info/commit-graph BIDX 12 FFFFFFFF\n++ fn=.git/objects/info/commit-graph\n++ shift\n++ perl /Users/tb/NoBackup/projects/git/git.pu/t/lib-chunk/corrupt-chunk-file.pl BIDX 12 FFFFFFFF\n++ command /usr/bin/perl /Users/tb/NoBackup/projects/git/git.pu/t/lib-chunk/corrupt-chunk-file.pl BIDX 12 FFFFFFFF\n++ /usr/bin/perl /Users/tb/NoBackup/projects/git/git.pu/t/lib-chunk/corrupt-chunk-file.pl BIDX 12 FFFFFFFF\n++ mv .git/objects/info/commit-graph.tmp .git/objects/info/commit-graph\noverride r--r--r--  tb/staff for .git/objects/info/commit-graph? (y/n [n]) not overwritten\n++ git -c core.commitGraph=false log -- A/B/file2\n++ git -c core.commitGraph=true log -- A/B/file2\n++ test_cmp expect.out out\n++ test 2 -ne 2\n++ eval 'diff -u' '\"$@\"'\n+++ diff -u expect.out out\n++ grep 'warning: ignoring out-of-range offset (4294967295) for changed-path filter at pos 3 of .git/objects/info/commit-graph' err\nerror: last command exited with $?=1\n++ rm -rf .git/objects/info/commit-graph\n++ exit 1\n++ eval_ret=1\n++ :\nnot ok 142 - Bloom reader notices out-of-bounds filter offsets\n\nexpecting success of 4216.143 'Bloom reader notices too-small index chunk':\n\t# replace the index with a single entry, making most\n\t# lookups out-of-bounds\n\tcheck_corrupt_graph BIDX clear 00000000 &&\n\techo \"warning: commit-graph changed-path index chunk\" \\\n\t\t\"is too small\" >expect.err &&\n\ttest_cmp expect.err err\n\n++ check_corrupt_graph BIDX clear 00000000\n++ corrupt_graph BIDX clear 00000000\n++ graph=.git/objects/info/commit-graph\n++ test_when_finished 'rm -rf .git/objects/info/commit-graph'\n++ test 0 = 0\n++ test_cleanup='{ rm -rf .git/objects/info/commit-graph\n\t\t} && (exit \"$eval_ret\"); eval_ret=$?; :'\n++ git commit-graph write --reachable --changed-paths\n++ corrupt_chunk_file .git/objects/info/commit-graph BIDX clear 00000000\n++ fn=.git/objects/info/commit-graph\n++ shift\n++ perl /Users/tb/NoBackup/projects/git/git.pu/t/lib-chunk/corrupt-chunk-file.pl BIDX clear 00000000\n++ command /usr/bin/perl /Users/tb/NoBackup/projects/git/git.pu/t/lib-chunk/corrupt-chunk-file.pl BIDX clear 00000000\n++ /usr/bin/perl /Users/tb/NoBackup/projects/git/git.pu/t/lib-chunk/corrupt-chunk-file.pl BIDX clear 00000000\n++ mv .git/objects/info/commit-graph.tmp .git/objects/info/commit-graph\noverride r--r--r--  tb/staff for .git/objects/info/commit-graph? (y/n [n]) not overwritten\n++ git -c core.commitGraph=false log -- A/B/file2\n++ git -c core.commitGraph=true log -- A/B/file2\n++ test_cmp expect.out out\n++ test 2 -ne 2\n++ eval 'diff -u' '\"$@\"'\n+++ diff -u expect.out out\n++ echo 'warning: commit-graph changed-path index chunk' 'is too small'\n++ test_cmp expect.err err\n++ test 2 -ne 2\n++ eval 'diff -u' '\"$@\"'\n+++ diff -u expect.err err\n--- expect.err\t2024-05-02 05:51:11.000000000 +0000\n+++ err\t2024-05-02 05:51:11.000000000 +0000\n@@ -1 +0,0 @@\n-warning: commit-graph changed-path index chunk is too small\nerror: last command exited with $?=1\n++ rm -rf .git/objects/info/commit-graph\n++ exit 1\n++ eval_ret=1\n++ :\nnot ok 143 - Bloom reader notices too-small index chunk\n\nexpecting success of 4216.144 'Bloom reader notices out-of-order index offsets':\n\t# we do not know any real offsets, but we can pick\n\t# something plausible; we should not get to the point of\n\t# actually reading from the bogus offsets anyway.\n\tcorrupt_graph BIDX 4 0000000c00000005 &&\n\techo \"warning: ignoring decreasing changed-path index offsets\" \\\n\t\t\"(12 > 5) for positions 1 and 2 of .git/objects/info/commit-graph\" >expect.err &&\n\tgit -c core.commitGraph=false log -- A/B/file2 >expect.out &&\n\tgit -c core.commitGraph=true log -- A/B/file2 >out 2>err &&\n\ttest_cmp expect.out out &&\n\ttest_cmp expect.err err\n\n++ corrupt_graph BIDX 4 0000000c00000005\n++ graph=.git/objects/info/commit-graph\n++ test_when_finished 'rm -rf .git/objects/info/commit-graph'\n++ test 0 = 0\n++ test_cleanup='{ rm -rf .git/objects/info/commit-graph\n\t\t} && (exit \"$eval_ret\"); eval_ret=$?; :'\n++ git commit-graph write --reachable --changed-paths\n++ corrupt_chunk_file .git/objects/info/commit-graph BIDX 4 0000000c00000005\n++ fn=.git/objects/info/commit-graph\n++ shift\n++ perl /Users/tb/NoBackup/projects/git/git.pu/t/lib-chunk/corrupt-chunk-file.pl BIDX 4 0000000c00000005\n++ command /usr/bin/perl /Users/tb/NoBackup/projects/git/git.pu/t/lib-chunk/corrupt-chunk-file.pl BIDX 4 0000000c00000005\n++ /usr/bin/perl /Users/tb/NoBackup/projects/git/git.pu/t/lib-chunk/corrupt-chunk-file.pl BIDX 4 0000000c00000005\n++ mv .git/objects/info/commit-graph.tmp .git/objects/info/commit-graph\noverride r--r--r--  tb/staff for .git/objects/info/commit-graph? (y/n [n]) not overwritten\n++ echo 'warning: ignoring decreasing changed-path index offsets' '(12 > 5) for positions 1 and 2 of .git/objects/info/commit-graph'\n++ git -c core.commitGraph=false log -- A/B/file2\n++ git -c core.commitGraph=true log -- A/B/file2\n++ test_cmp expect.out out\n++ test 2 -ne 2\n++ eval 'diff -u' '\"$@\"'\n+++ diff -u expect.out out\n++ test_cmp expect.err err\n++ test 2 -ne 2\n++ eval 'diff -u' '\"$@\"'\n+++ diff -u expect.err err\n--- expect.err\t2024-05-02 05:51:11.000000000 +0000\n+++ err\t2024-05-02 05:51:11.000000000 +0000\n@@ -1 +0,0 @@\n-warning: ignoring decreasing changed-path index offsets (12 > 5) for positions 1 and 2 of .git/objects/info/commit-graph\nerror: last command exited with $?=1\n++ rm -rf .git/objects/info/commit-graph\n++ exit 1\n++ eval_ret=1\n++ :\nnot ok 144 - Bloom reader notices out-of-order index offsets\n"},{"id":"493919","messageId":"xmqqsez0fbam.fsf@gitster.g","threadId":"61391","inReplyTo":"20240502055621.GA29945@tb-raspi4","subject":"Re: t4216-log-bloom.sh broken ?","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2024-05-02T16:06:41Z","receivedAt":"2024-05-02T16:06:44Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Torsten Bögershausen <tboegi@web.de> writes:\n\n> []\n>\n> Highjacking the Git v2.45.0 announcement:\n\nI am not sure why you want to do so.  You are perfectly capable of\nsending a new message to the list (and I know you have done so in\nthe past).  Why would linux-kernel@ list want to hear about the\nmacOS issues?\n\n> There are 4 test cases in t4216-log-bloom.sh, that do not pass on one\n> Mac here (they pass on another machine)\n\nAnother machine being another mac?\n\n> expecting success of 4216.141 'Bloom reader notices too-small data chunk':\n> \tcheck_corrupt_graph BDAT clear 00000000 &&\n> \techo \"warning: ignoring too-small changed-path chunk\" \\\n> \t\t\"(4 < 12) in commit-graph file\" >expect.err &&\n> \ttest_cmp expect.err err\n>\n> ++ check_corrupt_graph BDAT clear 00000000\n> ++ corrupt_graph BDAT clear 00000000\n> ++ graph=.git/objects/info/commit-graph\n> ++ test_when_finished 'rm -rf .git/objects/info/commit-graph'\n> ++ test 0 = 0\n> ++ test_cleanup='{ rm -rf .git/objects/info/commit-graph\n> \t\t} && (exit \"$eval_ret\"); eval_ret=$?; :'\n> ++ git commit-graph write --reachable --changed-paths\n> ++ corrupt_chunk_file .git/objects/info/commit-graph BDAT clear 00000000\n> ++ fn=.git/objects/info/commit-graph\n> ++ shift\n> ++ perl /Users/tb/NoBackup/projects/git/git.pu/t/lib-chunk/corrupt-chunk-file.pl BDAT clear 00000000\n> ++ command /usr/bin/perl /Users/tb/NoBackup/projects/git/git.pu/t/lib-chunk/corrupt-chunk-file.pl BDAT clear 00000000\n> ++ /usr/bin/perl /Users/tb/NoBackup/projects/git/git.pu/t/lib-chunk/corrupt-chunk-file.pl BDAT clear 00000000\n> ++ mv .git/objects/info/commit-graph.tmp .git/objects/info/commit-graph\n> override r--r--r--  tb/staff for .git/objects/info/commit-graph? (y/n [n]) not overwritten\n\nIs this failure preventing the later steps of the test work as\nexpected, I wonder.  Is there something curious with the permission\nbits of /Users/tb/NoBackup/projects/git/git.pu/ directory or its t/\nsubdirectory?  Is there something curious with the \"umask\" of the\nuser running the test?  Are they different from what you see on your\nother mac that does not exhibit the problems?\n\nThanks.\n\n\n"},{"id":"493920","messageId":"xmqqle4sfada.fsf@gitster.g","threadId":"61391","inReplyTo":"20240502055621.GA29945@tb-raspi4","subject":"Re: t4216-log-bloom.sh broken ?","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2024-05-02T16:26:41Z","receivedAt":"2024-05-02T16:26:48Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Torsten Bögershausen <tboegi@web.de> writes:\n\n> ++ check_corrupt_graph BDAT clear 00000000\n> ++ corrupt_graph BDAT clear 00000000\n> ++ graph=.git/objects/info/commit-graph\n> ++ test_when_finished 'rm -rf .git/objects/info/commit-graph'\n> ++ test 0 = 0\n> ++ test_cleanup='{ rm -rf .git/objects/info/commit-graph\n> \t\t} && (exit \"$eval_ret\"); eval_ret=$?; :'\n> ++ git commit-graph write --reachable --changed-paths\n> ++ corrupt_chunk_file .git/objects/info/commit-graph BDAT clear 00000000\n> ++ fn=.git/objects/info/commit-graph\n> ++ shift\n> ++ perl /Users/tb/NoBackup/projects/git/git.pu/t/lib-chunk/corrupt-chunk-file.pl BDAT clear 00000000\n> ++ command /usr/bin/perl /Users/tb/NoBackup/projects/git/git.pu/t/lib-chunk/corrupt-chunk-file.pl BDAT clear 00000000\n> ++ /usr/bin/perl /Users/tb/NoBackup/projects/git/git.pu/t/lib-chunk/corrupt-chunk-file.pl BDAT clear 00000000\n> ++ mv .git/objects/info/commit-graph.tmp .git/objects/info/commit-graph\n> override r--r--r--  tb/staff for .git/objects/info/commit-graph? (y/n [n]) not overwritten\n\nAnother probing question.  Do t5318, t5319, t5324, and t5328 pass?\n\nThe \"mv\" is part of the corrupt_chunk_file helper is used by this:\n\n    corrupt_graph () {\n            graph=.git/objects/info/commit-graph &&\n            test_when_finished \"rm -rf $graph\" &&\n            git commit-graph write --reachable --changed-paths &&\n            corrupt_chunk_file $graph \"$@\"\n    }\n\nand reads like this:\n\n    corrupt_chunk_file () {\n            fn=$1; shift\n            perl \"$TEST_DIRECTORY\"/lib-chunk/corrupt-chunk-file.pl \\\n                    \"$@\" <\"$fn\" >\"$fn.tmp\" &&\n            mv \"$fn.tmp\" \"$fn\"\n    }\n\nIf the final \"mv\" is what is failing in the above trace, the test\ntried to corrupt the objects/info/commit-graph file, expecting that\nlater steps will notice breakage.  But if the broken file written in\ncommit-graph.tmp failed to become the final commit-graph, later\nsteps will see healthy graph file and it is understandable that we\ndo not see any breakage that we expect, thus failing the test.\n\nIs \"mv\" somehow configured to always go interactive on that box and\non that box alone?  For example if you have ~/bin/mv that does\nsomething silly like\n\n\t#!/bin/bash\n\texec /bin/mv -i \"$@\n\nand have ~/bin early in $PATH, it may be sufficient to explain the\nerror we see (I am not claiming that is the only way to cause this\ntest failure; it is just one possible one).\n\n"},{"id":"493935","messageId":"20240502185903.GA11895@tb-raspi4","threadId":"61391","inReplyTo":"xmqqsez0fbam.fsf@gitster.g","subject":"Re: t4216-log-bloom.sh broken ?","fromName":"Torsten Bögershausen","fromEmail":"tboegi@web.de","sentAt":"2024-05-02T18:59:03Z","receivedAt":"2024-05-02T18:59:15Z","isPatch":false,"sender":{"key":"tboegi@web.de","avatar":"https://avatars.githubusercontent.com/u/7138363?v=4"},"body":"On Thu, May 02, 2024 at 09:06:41AM -0700, Junio C Hamano wrote:\n> Torsten Bögershausen <tboegi@web.de> writes:\n>\n> > There are 4 test cases in t4216-log-bloom.sh, that do not pass on one\n> > Mac here (they pass on another machine)\n>\n> Another machine being another mac?\n\nYes, different mac, different MacOs, different $PATH probably.\n\n>\n> > expecting success of 4216.141 'Bloom reader notices too-small data chunk':\n> > \tcheck_corrupt_graph BDAT clear 00000000 &&\n> > \techo \"warning: ignoring too-small changed-path chunk\" \\\n> > \t\t\"(4 < 12) in commit-graph file\" >expect.err &&\n> > \ttest_cmp expect.err err\n> >\n> > ++ check_corrupt_graph BDAT clear 00000000\n> > ++ corrupt_graph BDAT clear 00000000\n> > ++ graph=.git/objects/info/commit-graph\n> > ++ test_when_finished 'rm -rf .git/objects/info/commit-graph'\n> > ++ test 0 = 0\n> > ++ test_cleanup='{ rm -rf .git/objects/info/commit-graph\n> > \t\t} && (exit \"$eval_ret\"); eval_ret=$?; :'\n> > ++ git commit-graph write --reachable --changed-paths\n> > ++ corrupt_chunk_file .git/objects/info/commit-graph BDAT clear 00000000\n> > ++ fn=.git/objects/info/commit-graph\n> > ++ shift\n> > ++ perl /Users/tb/NoBackup/projects/git/git.pu/t/lib-chunk/corrupt-chunk-file.pl BDAT clear 00000000\n> > ++ command /usr/bin/perl /Users/tb/NoBackup/projects/git/git.pu/t/lib-chunk/corrupt-chunk-file.pl BDAT clear 00000000\n> > ++ /usr/bin/perl /Users/tb/NoBackup/projects/git/git.pu/t/lib-chunk/corrupt-chunk-file.pl BDAT clear 00000000\n> > ++ mv .git/objects/info/commit-graph.tmp .git/objects/info/commit-graph\n> > override r--r--r--  tb/staff for .git/objects/info/commit-graph? (y/n [n]) not overwritten\n>\n> Is this failure preventing the later steps of the test work as\n> expected, I wonder.  Is there something curious with the permission\n> bits of /Users/tb/NoBackup/projects/git/git.pu/ directory or its t/\n> subdirectory?  Is there something curious with the \"umask\" of the\n> user running the test?  Are they different from what you see on your\n> other mac that does not exhibit the problems?\n>\n> Thanks.\n>\n>\n\nmv is /bin/mv, that seems to be good:\n\n$ type mv\nmv is /bin/mv\n\n$ alias | grep mv\n\n$ which mv\n/bin/mv\n\n$ umask\n0022\n\nI don't know, why we see\nr--r--r--  tb/staff for .git/objects/info/commit-graph\n\nBut, the \"-r--r--r--\" may be part of the problem, here is another one:\n$ find . -name commit-graph -print0 | xargs -0 ls -l\n-r--r--r--  1 tb  staff  1792 May  2 12:12 ./trash directory.t5318-commit-graph/bare/objects/info/commit-graph\n(And some more 6 in total. All with -r--r--r--)\n\nWhich means, yes, t5318 does not pass either:\nt5318-commit-graph.sh not ok 101 - reader notices too-small oid fanout chunk\nt5318-commit-graph.sh not ok 102 - reader notices fanout/lookup table mismatch\nt5318-commit-graph.sh not ok 103 - reader notices out-of-bounds fanout\nt5318-commit-graph.sh not ok 104 - reader notices too-small commit data chunk\nt5318-commit-graph.sh not ok 105 - reader notices out-of-bounds extra edge\nt5318-commit-graph.sh not ok 106 - reader notices too-small generations chunk\n\nSame problem here:\n++ mv full/.git/objects/info/commit-graph.tmp full/.git/objects/info/commit-graph\noverride r--r--r--  tb/staff for full/.git/objects/info/commit-graph? (y/n [n]) not overwritten\n\nThe rest of the test suite passes, I see the same failures even under a\nfresh cloned repo.\n\nAny hints are appreciated.\n\n\n"},{"id":"493937","messageId":"20240502192622.GA1841@szeder.dev","threadId":"61391","inReplyTo":"20240502185903.GA11895@tb-raspi4","subject":"Re: t4216-log-bloom.sh broken ?","fromName":"SZEDER Gábor","fromEmail":"szeder.dev@gmail.com","sentAt":"2024-05-02T19:26:22Z","receivedAt":"2024-05-02T19:26:25Z","isPatch":false,"sender":{"key":"szeder.dev@gmail.com","avatar":"https://avatars.githubusercontent.com/u/116324?v=4"},"body":"On Thu, May 02, 2024 at 08:59:03PM +0200, Torsten Bögershausen wrote:\n> On Thu, May 02, 2024 at 09:06:41AM -0700, Junio C Hamano wrote:\n> > Torsten Bögershausen <tboegi@web.de> writes:\n> >\n> > > There are 4 test cases in t4216-log-bloom.sh, that do not pass on one\n> > > Mac here (they pass on another machine)\n> >\n> > Another machine being another mac?\n> \n> Yes, different mac, different MacOs, different $PATH probably.\n> \n> >\n> > > expecting success of 4216.141 'Bloom reader notices too-small data chunk':\n> > > \tcheck_corrupt_graph BDAT clear 00000000 &&\n> > > \techo \"warning: ignoring too-small changed-path chunk\" \\\n> > > \t\t\"(4 < 12) in commit-graph file\" >expect.err &&\n> > > \ttest_cmp expect.err err\n> > >\n> > > ++ check_corrupt_graph BDAT clear 00000000\n> > > ++ corrupt_graph BDAT clear 00000000\n> > > ++ graph=.git/objects/info/commit-graph\n> > > ++ test_when_finished 'rm -rf .git/objects/info/commit-graph'\n> > > ++ test 0 = 0\n> > > ++ test_cleanup='{ rm -rf .git/objects/info/commit-graph\n> > > \t\t} && (exit \"$eval_ret\"); eval_ret=$?; :'\n> > > ++ git commit-graph write --reachable --changed-paths\n> > > ++ corrupt_chunk_file .git/objects/info/commit-graph BDAT clear 00000000\n> > > ++ fn=.git/objects/info/commit-graph\n> > > ++ shift\n> > > ++ perl /Users/tb/NoBackup/projects/git/git.pu/t/lib-chunk/corrupt-chunk-file.pl BDAT clear 00000000\n> > > ++ command /usr/bin/perl /Users/tb/NoBackup/projects/git/git.pu/t/lib-chunk/corrupt-chunk-file.pl BDAT clear 00000000\n> > > ++ /usr/bin/perl /Users/tb/NoBackup/projects/git/git.pu/t/lib-chunk/corrupt-chunk-file.pl BDAT clear 00000000\n> > > ++ mv .git/objects/info/commit-graph.tmp .git/objects/info/commit-graph\n> > > override r--r--r--  tb/staff for .git/objects/info/commit-graph? (y/n [n]) not overwritten\n> >\n> > Is this failure preventing the later steps of the test work as\n> > expected, I wonder.  Is there something curious with the permission\n> > bits of /Users/tb/NoBackup/projects/git/git.pu/ directory or its t/\n> > subdirectory?  Is there something curious with the \"umask\" of the\n> > user running the test?  Are they different from what you see on your\n> > other mac that does not exhibit the problems?\n> >\n> > Thanks.\n> >\n> >\n> \n> mv is /bin/mv, that seems to be good:\n> \n> $ type mv\n> mv is /bin/mv\n> \n> $ alias | grep mv\n> \n> $ which mv\n> /bin/mv\n> \n> $ umask\n> 0022\n> \n> I don't know, why we see\n> r--r--r--  tb/staff for .git/objects/info/commit-graph\n> \n> But, the \"-r--r--r--\" may be part of the problem, here is another one:\n> $ find . -name commit-graph -print0 | xargs -0 ls -l\n> -r--r--r--  1 tb  staff  1792 May  2 12:12 ./trash directory.t5318-commit-graph/bare/objects/info/commit-graph\n> (And some more 6 in total. All with -r--r--r--)\n> \n> Which means, yes, t5318 does not pass either:\n> t5318-commit-graph.sh not ok 101 - reader notices too-small oid fanout chunk\n> t5318-commit-graph.sh not ok 102 - reader notices fanout/lookup table mismatch\n> t5318-commit-graph.sh not ok 103 - reader notices out-of-bounds fanout\n> t5318-commit-graph.sh not ok 104 - reader notices too-small commit data chunk\n> t5318-commit-graph.sh not ok 105 - reader notices out-of-bounds extra edge\n> t5318-commit-graph.sh not ok 106 - reader notices too-small generations chunk\n> \n> Same problem here:\n> ++ mv full/.git/objects/info/commit-graph.tmp full/.git/objects/info/commit-graph\n> override r--r--r--  tb/staff for full/.git/objects/info/commit-graph? (y/n [n]) not overwritten\n\n'mv' in macOS doesn't conform to POSIX, and asks for confirmation when\nthe destination exists (is read-only?) even without '-i' and even when\nits stdin is not a terminal, which it won't get as its stdin in our\ntest suite is redirected from /dev/null.\n\nThis is a recurring issue, see e.g.:\n\n  https://public-inbox.org/git/20180616143513.10086-1-szeder.dev@gmail.com/\n  c20d4d702f (t1450: use \"mv -f\" within loose object directory, 2017-01-24)\n\n'mv -f' did the trick in the past.\n\n"},{"id":"493944","messageId":"xmqq7cgcdm4f.fsf@gitster.g","threadId":"61391","inReplyTo":"20240502192622.GA1841@szeder.dev","subject":"Re: t4216-log-bloom.sh broken ?","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2024-05-02T19:55:44Z","receivedAt":"2024-05-02T19:55:56Z","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>> Yes, different mac, different MacOs, different $PATH probably.\n>> ...\n>\n> 'mv' in macOS doesn't conform to POSIX, and asks for confirmation when\n> the destination exists (is read-only?) even without '-i' and even when\n> its stdin is not a terminal, which it won't get as its stdin in our\n> test suite is redirected from /dev/null.\n>\n> This is a recurring issue, see e.g.:\n>\n>   https://public-inbox.org/git/20180616143513.10086-1-szeder.dev@gmail.com/\n>   c20d4d702f (t1450: use \"mv -f\" within loose object directory, 2017-01-24)\n>\n> 'mv -f' did the trick in the past.\n\nHmph, while that is a good data point to explain why this one fails,\nit is a very unsatisfactory workaround, compared to a world where we\ndo not have to worry about such a broken mv (perhaps by noticing a\nbroken macOS /bin/mv and automatically doing mv () { mv -f \"$@\" }\nfor them).\n\nI am curious what other differences Torsten will find out between\ngood macs and bad ones.  Perhaps we can narrow down the bad apples?\n\n"},{"id":"493945","messageId":"xmqqwmocc73f.fsf@gitster.g","threadId":"61391","inReplyTo":"xmqq7cgcdm4f.fsf@gitster.g","subject":"Re: t4216-log-bloom.sh broken ?","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2024-05-02T20:05:40Z","receivedAt":"2024-05-02T20:05:45Z","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> ... it is a very unsatisfactory workaround, compared to a world where we\n> do not have to worry about such a broken mv (perhaps by noticing a\n> broken macOS /bin/mv and automatically doing mv () { mv -f \"$@\" }\n> for them).\n>\n> I am curious what other differences Torsten will find out between\n> good macs and bad ones.  Perhaps we can narrow down the bad apples?\n\nIn any case, while we are waiting, I did a few \"grep\":\n\n    $ git grep 'mv \\(.*\\)\\.tmp \\1' t\n    t/lib-t6000.sh:\tmv sed.script.tmp sed.script\n    t/t7508-status.sh:\trm \"$1\" && mv \"$1\".tmp \"$1\"\n    t/t8011-blame-split-file.sh:\tmv one.tmp one &&\n    t/t8011-blame-split-file.sh:\tmv two.tmp two &&\n    t/t9400-git-cvsserver-server.sh:\tmv merge.tmp merge &&\n    t/t9400-git-cvsserver-server.sh:\tmv merge.tmp merge &&\n    t/t9802-git-p4-filetype.sh:\t\tmv empty-symlink,v.tmp empty-symlink,v\n\n    $ git grep 'mv \"\\(.*\\)\\.tmp\" \"\\1\"' t\n    t/lib-chunk.sh:\tmv \"$fn.tmp\" \"$fn\"\n    t/t3404-rebase-interactive.sh:\tmv \"$1.tmp\" \"$1\"\n    t/t5515-fetch-merge-logic.sh:\tmv \"$file.tmp\" \"$file\"\n    t/t7600-merge.sh:) >\"$1.tmp\" && mv \"$1.tmp\" \"$1\"\n    t/t9001-send-email.sh:\tmv \"$1.tmp\" \"$1\"\n    t/t9001-send-email.sh:\tmv \"$1.tmp\" \"$1\"\n\n    $ git grep 'mv \\(.*\\)+ \\1' t\n    t/t4025-hunk-header.sh:\tmv file+ file\n    t/t6402-merge-rename.sh:\tmv A+ A &&\n    t/t6402-merge-rename.sh:\tmv A+ A &&\n    t/t6432-merge-recursive-space-options.sh:\tmv text.txt+ text.txt &&\n    t/t6432-merge-recursive-space-options.sh:\tmv text.txt+ text.txt &&\n\nThere is nothing other than the one we found in lib-chunk.sh that\nwork on files under .git/objects/ that are made read-only, so we\nshould be OK with a workaround like the attached.\n\n t/lib-chunk.sh | 3 ++-\n 1 file changed, 2 insertions(+), 1 deletion(-)\n\ndiff --git c/t/lib-chunk.sh w/t/lib-chunk.sh\nindex a7cd9c3c6d..9f01df190b 100644\n--- c/t/lib-chunk.sh\n+++ w/t/lib-chunk.sh\n@@ -13,5 +13,6 @@ corrupt_chunk_file () {\n \tfn=$1; shift\n \tperl \"$TEST_DIRECTORY\"/lib-chunk/corrupt-chunk-file.pl \\\n \t\t\"$@\" <\"$fn\" >\"$fn.tmp\" &&\n-\tmv \"$fn.tmp\" \"$fn\"\n+\t# some vintages of macOS 'mv' fails to overwrite a read-only file.\n+\tmv -f \"$fn.tmp\" \"$fn\"\n }\n"},{"id":"493946","messageId":"xmqqsez0c6ma.fsf@gitster.g","threadId":"61391","inReplyTo":"xmqq7cgcdm4f.fsf@gitster.g","subject":"Re: t4216-log-bloom.sh broken ?","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2024-05-02T20:15:57Z","receivedAt":"2024-05-02T20:16:00Z","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> I am curious what other differences Torsten will find out between\n> good macs and bad ones.  Perhaps we can narrow down the bad apples?\n\nSo, \"no, your 'mv' is broken\" seems to be the answer to the question\non the Subject line, and it is rather well-known, it seems.\n\n----- >8 --------- >8 --------- >8 --------- >8 -----\nSubject: [PATCH] t/lib-chunk: work around broken \"mv\" on some vintage of macOS\n\nWhen the destination is read-only, \"mv\" on some version of macOS\nasks whether to replace the destination even though in the test its\nstdin is not a terminal (and thus doesn't conform to POSIX[1]).\n\nThe helper to corrupt a chunk-file is designed to work on the\nfiles like commit-graph and multi-pack-index files that are\ngenerally read-only, so use \"mv -f\" to work around this issue.\n\nHelped-by: SZEDER Gábor <szeder.dev@gmail.com>\nSigned-off-by: Junio C Hamano <gitster@pobox.com>\n---\n t/lib-chunk.sh | 3 ++-\n 1 file changed, 2 insertions(+), 1 deletion(-)\n\ndiff --git a/t/lib-chunk.sh b/t/lib-chunk.sh\nindex a7cd9c3c6d..9f01df190b 100644\n--- a/t/lib-chunk.sh\n+++ b/t/lib-chunk.sh\n@@ -13,5 +13,6 @@ corrupt_chunk_file () {\n \tfn=$1; shift\n \tperl \"$TEST_DIRECTORY\"/lib-chunk/corrupt-chunk-file.pl \\\n \t\t\"$@\" <\"$fn\" >\"$fn.tmp\" &&\n-\tmv \"$fn.tmp\" \"$fn\"\n+\t# some vintages of macOS 'mv' fails to overwrite a read-only file.\n+\tmv -f \"$fn.tmp\" \"$fn\"\n }\n-- \n2.45.0-31-gd4cc1ec35f\n\n\n"},{"id":"493949","messageId":"20240502205121.GB11895@tb-raspi4","threadId":"61391","inReplyTo":"xmqqsez0c6ma.fsf@gitster.g","subject":"Re: t4216-log-bloom.sh broken ?","fromName":"Torsten Bögershausen","fromEmail":"tboegi@web.de","sentAt":"2024-05-02T20:51:21Z","receivedAt":"2024-05-02T20:51:28Z","isPatch":false,"sender":{"key":"tboegi@web.de","avatar":"https://avatars.githubusercontent.com/u/7138363?v=4"},"body":"On Thu, May 02, 2024 at 01:15:57PM -0700, Junio C Hamano wrote:\n> Junio C Hamano <gitster@pobox.com> writes:\n>\n> > I am curious what other differences Torsten will find out between\n> > good macs and bad ones.  Perhaps we can narrow down the bad apples?\n>\n> So, \"no, your 'mv' is broken\" seems to be the answer to the question\n> on the Subject line, and it is rather well-known, it seems.\n\n[]\n\n>  \t\t\"$@\" <\"$fn\" >\"$fn.tmp\" &&\n> -\tmv \"$fn.tmp\" \"$fn\"\n> +\t# some vintages of macOS 'mv' fails to overwrite a read-only file.\n> +\tmv -f \"$fn.tmp\" \"$fn\"\n\nThanks folks for digging - the patch makes both test cases pass.\nSome official information:\n\n$ man mv\n\non the both machines says\n[snip]\nIf the destination path does not have a mode which permits writing,\nmv prompts the user for confirmation as specified for the -i option.\n\nTo summarize what I have at hand right now:\nMacOs 11.7.10 fails t4216 and t5318, both pass under MacOs 13.6.6\n\nBut why doesn't `mv` not hang then ?\n\n"},{"id":"493951","messageId":"xmqq4jbfdi6h.fsf@gitster.g","threadId":"61391","inReplyTo":"20240502205121.GB11895@tb-raspi4","subject":"Re: t4216-log-bloom.sh broken ?","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2024-05-02T21:20:54Z","receivedAt":"2024-05-02T21:21:06Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Torsten Bögershausen <tboegi@web.de> writes:\n\n> Thanks folks for digging - the patch makes both test cases pass.\n> Some official information:\n>\n> $ man mv\n>\n> on the both machines says\n> [snip]\n> If the destination path does not have a mode which permits writing,\n> mv prompts the user for confirmation as specified for the -i option.\n>\n> To summarize what I have at hand right now:\n> MacOs 11.7.10 fails t4216 and t5318, both pass under MacOs 13.6.6\n>\n> But why doesn't `mv` not hang then ?\n\nIt is a sensible choice for the command to assume a \"safer\" no and\ncontinuing rather than hanging, so that part is understandable.\n\nSo presumably 13.6.6 has a working \"mv\" but their \"man mv\" is not\nupdated to how the command actually works?\n\nIn any case, let me declare victory and queue the patch I sent\nearlier.\n\nThanks.\n"}]}