{"thread":{"id":"59733","subject":"What's cooking in git.git (May 2023, #04; Thu, 11)","startedAt":"2023-05-12T00:39:17Z","lastAt":"2023-05-12T20:52:21Z","messageCount":9,"participants":["Junio C Hamano","Felipe Contreras","Taylor Blau","Patrick Steinhardt","Elijah Newren","brian m. carlson"],"isPatch":false,"patchVersion":null,"patchTotal":null},"messages":[{"id":"477117","messageId":"xmqqo7mqs7rp.fsf@gitster.g","threadId":"59733","inReplyTo":null,"subject":"What's cooking in git.git (May 2023, #04; Thu, 11)","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2023-05-12T00:39:06Z","receivedAt":"2023-05-12T00:39:17Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Here are the topics that have been cooking in my tree.  Commits\nprefixed with '+' are in 'next' (being in 'next' is a sign that a\ntopic is stable enough to be used and are candidate to be in a future\nrelease).  Commits prefixed with '-' are only in 'seen', and aren't\nconsidered \"accepted\" at all and may be annotated with an URL to a\nmessage that raises issues but they are no means exhaustive.  A\ntopic without enough support may be discarded after a long period of\nno activity.\n\nCopies of the source code to Git live in many repositories, and the\nfollowing is a list of the ones I push into or their mirrors.  Some\nrepositories have only a subset of branches.\n\nWith maint, master, next, seen, todo:\n\n\tgit://git.kernel.org/pub/scm/git/git.git/\n\tgit://repo.or.cz/alt-git.git/\n\thttps://kernel.googlesource.com/pub/scm/git/git/\n\thttps://github.com/git/git/\n\thttps://gitlab.com/git-vcs/git/\n\nWith all the integration branches and topics broken out:\n\n\thttps://github.com/gitster/git/\n\nEven though the preformatted documentation in HTML and man format\nare not sources, they are published in these repositories for\nconvenience (replace \"htmldocs\" with \"manpages\" for the manual\npages):\n\n\tgit://git.kernel.org/pub/scm/git/git-htmldocs.git/\n\thttps://github.com/gitster/git-htmldocs.git/\n\nRelease tarballs are available at:\n\n\thttps://www.kernel.org/pub/software/scm/git/\n\n--------------------------------------------------\n[New Topics]\n\n* js/rebase-count-fixes (2023-05-10) 2 commits\n - rebase -r: fix the total number shown in the progress\n - rebase --update-refs: fix loops\n\n A few bugs in the sequencer machinery that results in miscounting\n the steps have been corrected.\n\n Waiting for responses to reviews.\n cf. <b481d212-89c5-5c8e-c99f-6df3909523c7@gmail.com>\n source: <pull.1531.git.1683759338.gitgitgadget@gmail.com>\n\n\n* ds/merge-tree-use-config (2023-05-10) 1 commit\n  (merged to 'next' on 2023-05-11 at e0dab53028)\n + merge-tree: load default git config\n\n Allow git forges to disable replace-refs feature while running \"git\n merge-tree\".\n\n Will merge to 'master'.\n source: <pull.1530.git.1683745654800.gitgitgadget@gmail.com>\n\n--------------------------------------------------\n[Graduated to 'master']\n\n* ah/doc-attributes-text (2023-05-03) 1 commit\n  (merged to 'next' on 2023-05-04 at 121c97817e)\n + docs: rewrite the documentation of the text and eol attributes\n\n Doc update to clarify how text and eol attributes interact to\n specify the end-of-line conversion.\n source: <20230503044656.221175-2-alexhenrie24@gmail.com>\n\n\n* ar/test-cleanup-unused-file-creation-part2 (2023-05-03) 6 commits\n  (merged to 'next' on 2023-05-03 at 185f0ec195)\n + t2019: don't create unused files\n + t1502: don't create unused files\n + t1450: don't create unused files\n + t1300: don't create unused files\n + t1300: fix config file syntax error descriptions\n + t0300: don't create unused file\n\n Test cleanup.\n cf. <CABPp-BFCBPTBc9savP13gNFmBo_WCnZHUv_Z8bWE6T6GZOrs-Q@mail.gmail.com>\n source: <20230417191044.909094-1-rybak.a.v@gmail.com>\n\n\n* fc/doc-drop-custom-callout-format (2023-05-03) 1 commit\n  (merged to 'next' on 2023-05-04 at c2fd654367)\n + doc: remove custom callouts format\n\n Our custom callout formatter is no longer used in the documentation\n formatting toolchain, as the upstream default ones give better\n output these days.\n source: <20230418011828.47851-1-felipe.contreras@gmail.com>\n\n\n* fc/doc-man-lift-title-length-limit (2023-05-03) 1 commit\n  (merged to 'next' on 2023-05-04 at d048eb1ef8)\n + doc: manpage: remove maximum title length\n\n The titles of manual pages used to be chomped at an unreasonably\n short limit, which has been removed.\n source: <20230503171658.8996-1-felipe.contreras@gmail.com>\n\n\n* hx/negotiator-non-recursive (2023-04-26) 2 commits\n  (merged to 'next' on 2023-05-02 at 64f97cc72d)\n + negotiator/skipping: fix some problems in mark_common()\n + negotiator/default: avoid stack overflow\n\n The implementation of the default \"negotiator\", used to find common\n ancestor over the network for object tranfer, used to be recursive;\n it was updated to be iterative to conserve stackspace usage.\n cf. <9ad5f246-e21f-0a13-1a53-1ae3307c3f0e@github.com>\n source: <cover.1682513384.git.hanxin.hx@bytedance.com>\n\n\n* ma/gittutorial-fixes (2023-04-20) 2 commits\n  (merged to 'next' on 2023-05-02 at bf96011972)\n + gittutorial: wrap literal examples in backticks\n + gittutorial: drop early mention of origin\n\n Doc fixes.\n source: <cover.1681579244.git.martin.agren@gmail.com>\n\n\n* mh/credential-oauth-refresh-token (2023-04-21) 1 commit\n  (merged to 'next' on 2023-05-04 at 9ade4aa024)\n + credential: new attribute oauth_refresh_token\n\n The credential subsystem learns to help OAuth framework.\n source: <pull.1394.v2.git.1682070479816.gitgitgadget@gmail.com>\n\n\n* mh/credential-password-expiry-wincred (2023-04-03) 1 commit\n  (merged to 'next' on 2023-05-06 at 51aae61f60)\n + credential/wincred: store password_expiry_utc\n\n Teach the recently invented \"password expiry time\" trait to the\n wincred credential helper.\n cf. <CAGJzqsm4LmpYE46v2=y4=A+Och44zaQyzTXQRteX-KNSzA_18g@mail.gmail.com>\n source: <pull.1477.v3.git.git.1680508028077.gitgitgadget@gmail.com>\n\n\n* mh/use-wincred-from-system (2023-03-27) 1 commit\n  (merged to 'next' on 2023-05-06 at a545c5d4aa)\n + credential/wincred: include wincred.h\n\n Code clean-up.\n cf. <8511e030-8167-715c-5ed4-1646e6e9ef85@gmx.de>\n cf. <CAGJzqsm4LmpYE46v2=y4=A+Och44zaQyzTXQRteX-KNSzA_18g@mail.gmail.com>\n source: <pull.1496.git.1679707396407.gitgitgadget@gmail.com>\n\n\n* ms/send-email-feed-header-to-validate-hook (2023-04-19) 2 commits\n  (merged to 'next' on 2023-05-03 at ba7079bcd4)\n + send-email: expose header information to git-send-email's sendemail-validate hook\n + send-email: refactor header generation functions\n\n \"git send-email\" learned to give the e-mail headers to the validate\n hook by passing an extra argument from the command line.\n source: <20230419202703.2911836-1-michael.strawbridge@amd.com>\n\n\n* rs/test-ctype-eof (2023-05-02) 1 commit\n  (merged to 'next' on 2023-05-02 at a19201a6a9)\n + test-ctype: check EOF\n\n ctype tests have been taught to test EOF, too.\n source: <1dbbd7ce-2b3f-6f88-f902-1d4ffe62e259@web.de>\n\n\n* sd/doc-gitignore-and-rm-cached (2023-05-03) 1 commit\n  (merged to 'next' on 2023-05-04 at a4b1b128d6)\n + docs: clarify git rm --cached function in gitignore note\n\n Doc update.\n source: <pull.862.v3.git.git.1683150400953.gitgitgadget@gmail.com>\n\n\n* tb/credential-long-lines (2023-05-01) 7 commits\n  (merged to 'next' on 2023-05-02 at f027002b92)\n + contrib/credential: embiggen fixed-size buffer in wincred\n + contrib/credential: avoid fixed-size buffer in libsecret\n + contrib/credential: .gitignore libsecret build artifacts\n + contrib/credential: remove 'gnome-keyring' credential helper\n + contrib/credential: avoid fixed-size buffer in osxkeychain\n + t/lib-credential.sh: ensure credential helpers handle long headers\n + credential.c: store \"wwwauth[]\" values in `credential_read()`\n\n The implementation of credential helpers used fgets() over fixed\n size buffers to read protocol messages, causing the remainder of\n the folded long line to trigger unexpected behaviour, which has\n been corrected.\n source: <cover.1682956419.git.me@ttaylorr.com>\n\n--------------------------------------------------\n[Stalled]\n\n* ed/fsmonitor-windows-named-pipe (2023-03-24) 1 commit\n - fsmonitor: handle differences between Windows named pipe functions\n\n Fix fsmonitor on Windows when the filesystem path contains certain\n characters.\n\n Expecting a reroll.\n cf. <b9cf67e4-22a7-2ff0-8310-9223bea10d6d@jeffhostetler.com>\n source: <pull.1503.git.1679678090412.gitgitgadget@gmail.com>\n\n\n* rn/sparse-diff-index (2023-04-10) 1 commit\n - diff-index: enable sparse index\n\n \"git diff-index\" command has been taught to work better with the\n sparse index.\n\n Expecting a reroll.\n cf. <62821012-4fc3-5ad8-695c-70f7ab14a8c9@github.com>\n source: <20230408112342.404318-1-nanth.raghul@gmail.com>\n\n\n* ja/worktree-orphan (2023-04-17) 8 commits\n - worktree add: emit warn when there is a bad HEAD\n - worktree add: extend DWIM to infer --orphan\n - worktree add: introduce \"try --orphan\" hint\n - worktree add: add --orphan flag\n - t2400: add tests to verify --quiet\n - t2400: refactor \"worktree add\" opt exclusion tests\n - t2400: print captured git output when finished\n - worktree add: include -B in usage docs\n\n 'git worktree add' learned how to create a worktree based on an\n orphaned branch with `--orphan`.\n\n Expecting a reroll.\n cf. <m7crhly2j3k76whydbtdwhvzghvoql436g3lqtst4v3ahv75aa@wq5zjepgrxz5>\n source: <20230417093255.31079-1-jacobabel@nullpo.dev>\n\n\n* es/recurse-submodules-option-is-a-bool (2023-04-10) 1 commit\n - usage: clarify --recurse-submodules as a boolean\n\n The \"--[no-]recurse-submodules\" option of \"git checkout\" and others\n supported an undocumented syntax --recurse-submodules=<value> where\n the value can spell a Boolean in various ways.  The support for the\n syntax is being dropped.\n\n Expecting a reroll.\n cf. <ZDSTFwMFO7vbj/du@google.com>\n source: <ZDSTFwMFO7vbj/du@google.com>\n\n\n* tb/pack-bitmap-index-seek (2023-03-20) 6 commits\n - pack-bitmap.c: factor out `bitmap_index_seek_commit()`\n - pack-bitmap.c: use `bitmap_index_seek()` where possible\n - pack-bitmap.c: factor out manual `map_pos` manipulation\n - pack-bitmap.c: drop unnecessary 'inline's\n - pack-bitmap.c: hide bitmap internals in `read_be32()`\n - pack-bitmap.c: hide bitmap internals in `read_u8()`\n\n Clean-up the pack-bitmap codepath.\n\n Kicked back to 'seen' out of 'next'.\n There is a BUG() on data errors that needs to be fixed.\n source: <cover.1679342296.git.me@ttaylorr.com>\n\n\n* cb/checkout-same-branch-twice (2023-03-22) 2 commits\n - SQUASH??? the test marked to expect failure passes from day one\n - checkout/switch: disallow checking out same branch in multiple worktrees\n\n \"git checkout -B $branch\" failed to protect against checking out\n a branch that is checked out elsewhere, unlike \"git branch -f\" did.\n\n Expecting a hopefully minor and final reroll.\n cf. <CAPUEspj_Bh+LgYLnWfeBdcq_uV5Cbou-7H51GLFjzSa5Qzby9w@mail.gmail.com>\n source: <20230120113553.24655-1-carenas@gmail.com>\n\n\n* tk/pull-conflict-suggest-rebase-merge-not-rebase-true (2023-02-13) 1 commit\n - pull: conflict hint pull.rebase suggestion should offer \"merges\" vs \"true\"\n\n In an advice message after failed non-ff pull, we used to suggest\n setting pull.rebase=true, but these days pull.rebase=merges may be\n more inline with the original spirit of \"rebuild your side on top\n of theirs\".\n\n May want to discard.\n This is too much of a departure from the existing practice.\n cf. <CAMMLpeTPEoKVTbfc17w+Y9qn7jOGmQi_Ux0Y3sFW5QTgGWJ=SA@mail.gmail.com>\n cf. <CABPp-BGqAxKnxDRVN4cYMteLp33hvto07R3=TJBT5WubJT4+Og@mail.gmail.com>\n source: <pull.1474.git.1675614276549.gitgitgadget@gmail.com>\n\n\n* ab/tag-object-type-errors (2023-05-10) 4 commits\n - tag: don't emit potentially incorrect \"object is a X, not a Y\"\n - tag: don't misreport type of tagged objects in errors\n - object tests: add test for unexpected objects in tags\n - Merge branch 'jk/parse-object-type-mismatch' into ab/tag-object-type-errors\n\n Hardening checks around mismatched object types when one of those\n objects is a tag.\n source: <cover-v2-0.3-00000000000-20221230T011725Z-avarab@gmail.com>\n\n\n* ad/test-record-count-when-harness-is-in-use (2022-12-25) 1 commit\n - test-lib: allow storing counts with test harnesses\n\n Allow summary results from tests to be written to t/test-results\n directory even when a test harness like 'prove' is in use.\n\n Expecting a reroll.\n cf. <CABPp-BGoPuGCZw+9wCgdYyRR4Zf4y9Kun27GrQhtMdYWpOUsYQ@mail.gmail.com>\n source: <20221224225200.1027806-1-adam@dinwoodie.org>\n\n\n* so/diff-merges-more (2022-12-18) 5 commits\n - diff-merges: improve --diff-merges documentation\n - diff-merges: issue warning on lone '-m' option\n - diff-merges: support list of values for --diff-merges\n - diff-merges: implement log.diffMerges-m-imply-p config\n - diff-merges: implement [no-]hide option and log.diffMergesHide config\n\n Assorted updates to \"--diff-merges=X\" option.\n\n May want to discard.\n Breaking compatibility does not seem worth it.\n source: <20221217132955.108542-1-sorganov@gmail.com>\n\n\n* ab/imap-send-requires-curl (2023-02-02) 6 commits\n - imap-send: correctly report \"host\" when using \"tunnel\"\n - imap-send: remove old --no-curl codepath\n - imap-send: make --curl no-optional\n - imap-send: replace auto-probe libcurl with hard dependency\n - imap-send doc: the imap.sslVerify is used with imap.tunnel\n - imap-send: note \"auth_method\", not \"host\" on auth method failure\n\n Give a hard dependency on cURL library to build \"git imap-send\",\n and remove the code to interact with IMAP server without using cURL.\n\n Expecting a reroll.\n The 'tunnel' part is still iffy.\n cf. <230203.86bkmabfjr.gmgdl@evledraar.gmail.com>\n source: <cover-v2-0.6-00000000000-20230202T093706Z-avarab@gmail.com>\n\n\n* tc/cat-file-z-use-cquote (2023-05-10) 1 commit\n - cat-file: quote-format name in error when using -z\n\n \"cat-file\" in the batch mode that is fed NUL-terminated pathnames\n learned to cquote them in its error output (otherwise, a funny\n pathname with LF in it would break the lines in the output stream).\n\n Will merge to 'next'?\n source: <20230510190116.795641-2-toon@iotcl.com>\n\n\n* cw/submodule-status-in-parallel (2023-03-02) 6 commits\n - diff-lib: parallelize run_diff_files for submodules\n - diff-lib: refactor out diff_change logic\n - submodule: refactor is_submodule_modified()\n - submodule: move status parsing into function\n - submodule: rename strbuf variable\n - run-command: add on_stderr_output_fn to run_processes_parallel_opts\n\n \"git submodule status\" learned to run the comparison in submodule\n repositories in parallel.\n\n Expecting a reroll.\n cf. <CAFySSZDk05m6gU5-V1R+y3YnQ5PPduVW54+_gjBwD0rmacsLsw@mail.gmail.com>\n cf. <230307.865ybc273g.gmgdl@evledraar.gmail.com>\n source: <20230302215237.1473444-1-calvinwan@google.com>\n\n--------------------------------------------------\n[Cooking]\n\n* jc/t9800-fix-use-of-show-s-raw (2023-05-06) 1 commit\n  (merged to 'next' on 2023-05-09 at f4d7a7d671)\n + t9800: correct misuse of 'show -s --raw' in a test\n\n A test fix.\n\n Will merge to 'master'.\n source: <xmqqild5rvvw.fsf@gitster.g>\n\n\n* tl/push-branches-is-an-alias-for-all (2023-05-06) 1 commit\n  (merged to 'next' on 2023-05-09 at 1d8c3e5dcf)\n + push: introduce '--branches' option\n\n \"git push --all\" gained an alias \"git push --branches\".\n\n Will merge to 'master'.\n source: <20230506113408.61529-2-tenglong.tl@alibaba-inc.com>\n\n\n* en/header-split-cache-h-part-3 (2023-05-08) 25 commits\n - fsmonitor-ll.h: split this header out of fsmonitor.h\n - hash-ll, hashmap: move oidhash() to hash-ll\n - git-compat-util.h: remove unneccessary include of wildmatch.h\n - builtin.h: remove unneccessary includes\n - list-objects-filter-options.h: remove unneccessary include\n - diff.h: remove unnecessary include of oidset.h\n - repository: remove unnecessary include of path.h\n - log-tree: replace include of revision.h with simple forward declaration\n - cache.h: remove this no-longer-used header\n - read-cache*.h: move declarations for read-cache.c functions from cache.h\n - repository.h: move declaration of the_index from cache.h\n - merge.h: move declarations for merge.c from cache.h\n - diff.h: move declaration for global in diff.c from cache.h\n - preload-index.h: move declarations for preload-index.c from elsewhere\n - sparse-index.h: move declarations for sparse-index.c from cache.h\n - name-hash.h: move declarations for name-hash.c from cache.h\n - run-command.h: move declarations for run-command.c from cache.h\n - statinfo: move stat_{data,validity} functions from cache/read-cache\n - read-cache: move shared add/checkout/commit code\n - add: modify add_files_to_cache() to avoid globals\n - read-cache: move shared commit and ls-files code\n - setup: adopt shared init-db & clone code\n - init-db, clone: change unnecessary global into passed parameter\n - init-db: remove unnecessary global variable & document existing bug\n - Merge branch 'en/header-split-cache-h-part-2' into en/header-split-cache-h-part-3\n\n Header files cleanup.\n\n Needs review.\n source: <pull.1525.git.1683431149.gitgitgadget@gmail.com>\n\n\n* js/subtree-fully-spelt-quiet-and-debug-options (2023-05-08) 1 commit\n  (merged to 'next' on 2023-05-10 at 2980db981e)\n + subtree: support long global flags\n\n \"git subtree\" (in contrib/) update.\n\n Will merge to 'master'.\n source: <pull.1514.git.1683428988693.gitgitgadget@gmail.com>\n\n\n* js/gitk-fixes-from-gfw (2023-05-08) 3 commits\n  (merged to 'next' on 2023-05-08 at adb334deec)\n + Merge gitk changes into js/gitk-fixes-from-gfw\n + gitk: escape file paths before piping to git log\n + gitk: prevent overly long command lines\n\n Gitk updates from GfW project.\n\n Will merge to 'master'.\n source: <pull.1469.git.1674559397.gitgitgadget@gmail.com>\n\n\n* la/doc-interpret-trailers (2023-05-11) 9 commits\n - doc: trailer: add more examples in DESCRIPTION\n - doc: trailer: mention 'key' in DESCRIPTION\n - doc: trailer.<token>.command: emphasize deprecation\n - doc: trailer: use angle brackets for <token> and <value>\n - doc: trailer: remove redundant phrasing\n - doc: trailer: examples: avoid the word \"message\" by itself\n - doc: trailer: drop \"commit message part\" phrasing\n - doc: trailer: swap verb order\n - doc: trailer: fix grammar\n\n Doc update.\n\n On hold. Looking good.\n Want to be rebased once kh/doc-interpret-trailers-updates lands.\n source: <pull.1506.v2.git.git.1683839975.gitgitgadget@gmail.com>\n\n\n* ar/test-cleanup-unused-file-creation (2023-05-09) 1 commit\n  (merged to 'next' on 2023-05-09 at 7734f3d690)\n + test: rev-parse-upstream: add missing cmp\n\n Test fix.\n\n Will merge to 'master'.\n source: <20230509000246.1760327-1-felipe.contreras@gmail.com>\n\n\n* cc/git-replay (2023-05-09) 16 commits\n - SQUASH???\n - replay: stop assuming replayed branches do not diverge\n - replay: add --contained to rebase contained branches\n - replay: add --advance or 'cherry-pick' mode\n - replay: disallow revision specific options and pathspecs\n - replay: use standard revision ranges\n - replay: make it a minimal server side command\n - replay: remove HEAD related sanity check\n - replay: remove progress and info output\n - replay: add an important FIXME comment about gpg signing\n - replay: don't simplify history\n - replay: introduce pick_regular_commit()\n - replay: die() instead of failing assert()\n - replay: start using parse_options API\n - replay: introduce new builtin\n - t6429: remove switching aspects of fast-rebase\n\n source: <20230509175347.1714141-1-christian.couder@gmail.com>\n\n\n* jc/pack-ref-exclude-include (2023-05-11) 4 commits\n - pack-refs: teach pack-refs --include option\n - revision: modify ref_exclusions to handle inclusions\n - pack-refs: teach --exclude option to exclude refs from being packed\n - docs: clarify git-pack-refs --all will pack all refs\n\n \"git pack-refs\" learns \"--include\" and \"--exclude\" to tweak the ref\n hierarchy to be packed using pattern matching.\n\n Needs review.\n source: <pull.1501.v3.git.git.1683828635.gitgitgadget@gmail.com>\n\n\n* jk/test-verbose-no-more (2023-05-08) 3 commits\n  (merged to 'next' on 2023-05-09 at 897bfd0a6b)\n + t: drop \"verbose\" helper function\n + t7001: use \"ls-files --format\" instead of \"cut\"\n + t7001: avoid git on upstream of pipe\n\n Retire \"verbose\" helper function from the test framework.\n\n Will merge to 'master'.\n source: <20230508185953.GA2108869@coredump.intra.peff.net>\n\n\n* tb/pack-extra-cruft-tips (2023-05-03) 1 commit\n - builtin/pack-objects.c: introduce `pack.extraCruftTips`\n\n \"git pack-objects\" learned to invoke a new hook program that\n enumerates extra objects to be used as anchoring points to keep\n otherwise unreachable objects in cruft packs.\n\n Under discussion\n cf. <20230505221921.GE3321533@coredump.intra.peff.net>\n source: <27a7f16aab35b5cac391d9831aadb0f2e2146313.1683151485.git.me@ttaylorr.com>\n\n\n* jc/diff-s-with-other-options (2023-05-05) 1 commit\n - diff: fix interaction between the \"-s\" option and other options\n (this branch uses jc/dirstat-plug-leaks.)\n\n The \"-s\" (silent, squelch) option of the \"diff\" family of commands\n did not interact with other options that specify the output format\n well.  This has been cleaned up so that it will clear all the\n formatting options given before.\n\n Will merge to 'next'?\n source: <20230505165952.335256-1-gitster@pobox.com>\n\n\n* fc/asciidoc-code-block-hack (2023-05-04) 2 commits\n - doc: manpage: remove ancient workaround\n - doc: git-svn: use listing instead of verse\n\n Documentation toolchain lost an ancient workaround that is no\n longer needed.\n\n Needs review.\n source: <20230504035517.329765-1-felipe.contreras@gmail.com>\n\n\n* fc/doc-revisions-markup-fix (2023-05-04) 2 commits\n - doc: revisions: add a bunch of missing quotes\n - doc: revisions: fix missing escape\n\n Doc cleanup.\n\n Needs review.\n source: <20230504163421.100400-1-felipe.contreras@gmail.com>\n\n\n* fc/doc-use-datestamp-in-commit (2023-05-05) 2 commits\n  (merged to 'next' on 2023-05-06 at 1eda43b03f)\n + doc-diff: drop SOURCE_DATE_EPOCH override\n + doc: doc-diff: specify date\n\n An earlier change broke \"doc-diff\", which has been corrected.\n\n Will merge to 'master'.\n source: <20230503232349.59997-1-felipe.contreras@gmail.com>\n source: <20230505211610.GA3197168@coredump.intra.peff.net>\n\n\n* jc/dirstat-plug-leaks (2023-05-05) 2 commits\n  (merged to 'next' on 2023-05-09 at 2df7cf06e4)\n + diff: plug leaks in dirstat\n + diff: refactor common tail part of dirstat computation\n (this branch is used by jc/diff-s-with-other-options.)\n\n \"git diff --dirstat\" leaked memory, which has been plugged.\n\n Will merge to 'master'.\n source: <20230505211917.2746751-1-gitster@pobox.com>\n\n\n* jc/name-rev-deprecate-stdin-further (2023-05-06) 1 commit\n  (merged to 'next' on 2023-05-09 at 0db4f08e32)\n + name-rev: make --stdin hidden\n\n The \"--stdin\" option of \"git name-rev\" has been replaced with\n the \"--annotate-stdin\" option more than a year ago.  We stop\n advertising it in the \"git name-rev -h\" output.\n\n Will merge to 'master'.\n source: <pull.1225.v3.git.git.1683346451239.gitgitgadget@gmail.com>\n\n\n* ob/revert-of-revert (2023-05-05) 1 commit\n - sequencer: beautify subject of reverts of reverts\n\n Instead of \"Revert \"Revert \"original\"\", give \"Reapply \"original\"\"\n as the title for a revert of a revert.\n\n Expecting a hopefully final reroll.\n Looking much better, except for minor cosmetic issues.\n source: <20230428083528.1699221-1-oswald.buddenhagen@gmx.de>\n\n\n* jc/attr-source-tree (2023-05-06) 1 commit\n  (merged to 'next' on 2023-05-10 at cb94f4fba6)\n + attr: teach \"--attr-source=<tree>\" global option to \"git\"\n\n \"git --attr-source=<tree> cmd $args\" is a new way to have any\n command to read attributes not from the working tree but from the\n given tree object.\n\n Will merge to 'master'.\n source: <pull.1470.v6.git.git.1683346530487.gitgitgadget@gmail.com>\n\n\n* mc/send-email-header-cmd (2023-05-01) 3 commits\n  (merged to 'next' on 2023-05-06 at 04514468b6)\n + send-email: detect empty blank lines in command output\n + send-email: add --header-cmd, --no-header-cmd options\n + send-email: extract execute_cmd from recipients_cmd\n\n \"git send-email\" learned \"--header-cmd=<cmd>\" that can inject\n arbitrary e-mail header lines to the outgoing messages.\n\n Will merge to 'master'.\n source: <20230501143848.19674-1-maxim.cournoyer@gmail.com>\n\n\n* bc/clone-empty-repo-via-protocol-v0 (2023-05-01) 1 commit\n - upload-pack: advertise capabilities when cloning empty repos\n\n Expecting a minor and hopefully final update.\n cf. <20230501224038.GA1174291@coredump.intra.peff.net>\n source: <20230501170018.1410567-2-sandals@crustytoothpaste.net>\n\n\n* ds/fsck-bitmap (2023-05-02) 2 commits\n  (merged to 'next' on 2023-05-09 at c2148408a2)\n + fsck: use local repository\n + fsck: verify checksums of all .bitmap files\n\n \"git fsck\" learned to detect bit-flip breakages in the reachability\n bitmap files.\n\n Will merge to 'master'.\n source: <pull.1526.v2.git.1683034042.gitgitgadget@gmail.com>\n\n\n* gc/trace-bare-repo-setup (2023-05-01) 1 commit\n  (merged to 'next' on 2023-05-06 at 72b977d2f9)\n + setup: trace bare repository setups\n\n The tracing mechanism learned to notice and report when\n auto-discovered bare repositories are being used, as allowing so\n without explicitly stating the user intends to do so (with setting\n GIT_DIR for example) can be used with social engineering as an\n attack vector.\n\n Will merge to 'master'.\n source: <e98be8e7f703fc741e06d9208545abc8c24d1a4a.1682962110.git.steadmon@google.com>\n\n\n* cw/strbuf-cleanup (2023-05-11) 7 commits\n - strbuf: remove global variable\n - strbuf: clarify dependency\n - path: move related function to path\n - object-name: move related functions to object-name\n - credential-store: move related functions to credential-store file\n - abspath: move related functions to abspath\n - strbuf: clarify API boundary\n\n Move functions that are not about pure string manipulation out of\n strbuf.[ch]\n\n Will merge to 'next'?\n source: <20230511194446.1492907-1-calvinwan@google.com>\n\n\n* jc/doc-clarify-git-default-hash-variable (2023-04-26) 1 commit\n  (merged to 'next' on 2023-05-06 at 7bae4db4cc)\n + doc: GIT_DEFAULT_HASH is and will be ignored during \"clone\"\n\n The documentation was misleading about the interaction between\n GIT_DEFAULT_HASH and \"git clone\", which has been clarified to\n stress that the variable is to be ignored by the command.\n\n Will merge to 'master'.\n source: <xmqqzg6uvfpo.fsf_-_@gitster.g>\n\n\n* tl/notes-separator (2023-04-28) 6 commits\n  (merged to 'next' on 2023-05-06 at 598ed19c9b)\n + notes.c: introduce \"--[no-]stripspace\" option\n + notes.c: append separator instead of insert by pos\n + notes.c: introduce '--separator=<paragraph-break>' option\n + t3321: add test cases about the notes stripspace behavior\n + notes.c: use designated initializers for clarity\n + notes.c: cleanup 'strbuf_grow' call in 'append_edit'\n\n 'git notes append' was taught '--separator' to specify string to insert\n between paragraphs.\n\n Will merge to 'master'.\n source: <cover.1682671758.git.dyroneteng@gmail.com>\n\n\n* pw/rebase-i-after-failure (2023-04-21) 6 commits\n - rebase -i: fix adding failed command to the todo list\n - rebase: fix rewritten list for failed pick\n - rebase --continue: refuse to commit after failed command\n - sequencer: factor out part of pick_commits()\n - rebase -i: remove patch file after conflict resolution\n - rebase -i: move unlink() calls\n\n Various fixes to the behaviour of \"rebase -i\" when the command got\n interrupted by conflicting changes.\n\n Expecting a reroll.\n cf. <xmqqsfcthrpb.fsf@gitster.g>\n cf. <1fd54422-b66a-c2e4-7cd7-934ea01190ad@gmail.com>\n source: <pull.1492.v2.git.1682089074.gitgitgadget@gmail.com>\n\n\n* sl/sparse-write-tree-part-2 (2023-05-08) 1 commit\n - t1092: update a write-tree test\n\n Fix-up to a topic already graduated to 'master'.\n\n Will merge to 'next'?\n source: <20230508202140.464363-1-cheskaqiqi@gmail.com>\n\n\n* tb/pack-bitmap-traversal-with-boundary (2023-05-08) 3 commits\n - pack-bitmap.c: use commit boundary during bitmap traversal\n - pack-bitmap.c: extract `fill_in_bitmap()`\n - object: add object_array initializer helper function\n\n The object traversal using reachability bitmap done by\n \"pack-object\" has been tweaked to take advantage of the fact that\n using \"boundary\" commits as representative of all the uninteresting\n ones can save quite a lot of object enumeration.\n source: <cover.1683567065.git.me@ttaylorr.com>\n\n\n* ar/config-count-tests-updates (2023-04-24) 3 commits\n  (merged to 'next' on 2023-05-06 at c585a56132)\n + t1300: add tests for missing keys\n + t1300: check stderr for \"ignores pairs\" tests\n + t1300: drop duplicate test\n\n Test updates.\n\n Will merge to 'master'.\n source: <20230423134649.431783-1-rybak.a.v@gmail.com>\n\n\n* ps/fetch-output-format (2023-05-10) 9 commits\n - fetch: introduce machine-parseable \"porcelain\" output format\n - fetch: move option related variables into main function\n - fetch: lift up parsing of \"fetch.output\" config variable\n - fetch: introduce `display_format` enum\n - fetch: refactor calculation of the display table width\n - fetch: print left-hand side when fetching HEAD:foo\n - fetch: add a test to exercise invalid output formats\n - fetch: split out tests for output format\n - fetch: fix `--no-recurse-submodules` with multi-remote fetches\n\n \"git fetch\" learned the \"--porcelain\" option that emits what it did\n in a machine-parseable format.\n\n Will merge to 'next'?\n source: <cover.1683721293.git.ps@pks.im>\n\n\n* sg/retire-unused-cocci (2023-04-20) 1 commit\n - cocci: remove 'unused.cocci'\n\n Retire a rather expensive-to-run Coccinelle check patch.\n\n Will merge to 'next'.\n source: <20230420205350.600760-1-szeder.dev@gmail.com>\n\n\n* gc/doc-cocci-updates (2023-04-27) 2 commits\n - cocci: codify authoring and reviewing practices\n - cocci: add headings to and reword README\n\n Update documentation regarding Coccinelle patches.\n\n Will merge to 'next'?\n source: <pull.1495.v2.git.git.1682634143.gitgitgadget@gmail.com>\n\n\n* kh/doc-interpret-trailers-updates (2023-05-01) 4 commits\n  (merged to 'next' on 2023-05-06 at 4e9a640131)\n + doc: interpret-trailers: fix example\n + doc: interpret-trailers: don’t use deprecated config\n + doc: interpret-trailers: use input redirection\n + doc: interpret-trailers: don’t use heredoc in examples\n\n Doc update.\n\n Will merge to 'master'.\n source: <cover.1682970213.git.code@khaugsbakk.name>\n\n\n* pb/complete-and-document-auto-merge-and-friends (2023-04-14) 5 commits\n  (merged to 'next' on 2023-04-20 at 2728a01622)\n + completion: complete AUTO_MERGE\n + Documentation: document AUTO_MERGE\n + git-merge.txt: modernize word choice in \"True merge\" section\n + completion: complete REVERT_HEAD and BISECT_HEAD\n + revisions.txt: document more special refs\n\n Document more pseudo-refs and teach the command line completion\n machinery to complete AUTO_MERGE.\n\n On hold, waiting for a fix-up.\n cf. <a10648d5-8cba-4c7c-a303-b5b0dcdd3310@gmail.com>\n source: <pull.1515.git.1681495119.gitgitgadget@gmail.com>\n\n\n* mh/credential-password-expiry-libsecret (2023-05-05) 1 commit\n  (merged to 'next' on 2023-05-09 at fd898e3756)\n + credential/libsecret: support password_expiry_utc\n\n The libsecret credential helper learns to handle the password\n expiry time information.\n\n Will merge to 'master'.\n source: <pull.1469.v3.git.git.1683270298313.gitgitgadget@gmail.com>\n\n\n* sl/diff-files-sparse (2023-05-09) 2 commits\n - diff-files: integrate with sparse index\n - t1092: add tests for `git diff-files`\n\n Teach \"diff-files\" not to expand sparse-index unless needed.\n\n Will merge to 'next'.\n cf. <f51a8d77-c480-f021-38c4-78a9d75cdd11@github.com>\n source: <20230509194241.469477-1-cheskaqiqi@gmail.com>\n\n\n* rj/branch-unborn-in-other-worktrees (2023-03-27) 5 commits\n  (merged to 'next' on 2023-05-06 at c61e9b0672)\n + branch: avoid unnecessary worktrees traversals\n + branch: rename orphan branches in any worktree\n + branch: description for orphan branch errors\n + branch: use get_worktrees() in copy_or_rename_branch()\n + branch: test for failures while renaming branches\n\n Error messages given when working on an unborn branch that is\n checked out in another worktree have been improved.\n\n Will merge to 'master'.\n source: <f8e6447e-5cd3-98fa-f567-39e1c60dacb0@gmail.com>\n\n--------------------------------------------------\n[Discarded]\n\n* ed/fsmonitor-inotify (2022-12-13) 6 commits\n . fsmonitor: update doc for Linux\n . fsmonitor: test updates\n . fsmonitor: enable fsmonitor for Linux\n . fsmonitor: implement filesystem change listener for Linux\n . fsmonitor: determine if filesystem is local or remote\n . fsmonitor: prepare to share code between Mac OS and Linux\n\n Bundled fsmonitor for Linux using inotify API.\n\n Seems to break asan job at CI when merged to 'seen'.\n cf. <xmqqzg7d8xn2.fsf@gitster.g>\n source: <pull.1352.v5.git.git.1670882286.gitgitgadget@gmail.com>\n\n\n* my/wildmatch-cleanups (2023-02-27) 5 commits\n . wildmatch: more cleanups after killing uchar\n . wildmatch: use char instead of uchar\n . wildmatch: remove NEGATE_CLASS(2) macros with trivial refactoring\n . wildmatch: remove IS*() macros\n . git-compat-util: add isblank() and isgraph()\n\n Code clean-up.\n\n Discarded without prejudice after more than 6 weeks of inactivity.\n source: <20230226115021.1681834-1-masahiroy@kernel.org>\n\n\n* cw/forbid-use-of-gitlink-outside-submodules (2023-02-28) 6 commits\n . add: reject nested repositories\n . tests: remove duplicate .gitmodules path\n . tests: use `git submodule add` and fix expected status\n . tests: use `git submodule add` and fix expected diffs\n . tests: Use `git submodule add` instead of `git add`\n . t4041, t4060: modernize test style\n\n Forbid \"git add dir\" from adding a gitlink to the index.\n\n Discarded without prejudice after more than 6 weeks of inactivity.\n source: <20230228185253.2356546-1-calvinwan@google.com>\n\n\n* nw/for-each-ref-signature (2023-03-13) 1 commit\n . ref-filter: add new \"signature\" atom\n\n \"git (for-each-ref|branch|tag) --format=<format>\" learns the \"%(signature)\"\n placeholder and friends.\n\n Breaks CI.\n cf. <xmqqpm9bosjw.fsf@gitster.g>\n source: <20230311210607.64927-2-nsengiyumvawilberforce@gmail.com>\n"},{"id":"477122","messageId":"CAMP44s0GAiFhoHFo4Ex464utf2Jm1F-LEdX2PZdUGEdp5x6Liw@mail.gmail.com","threadId":"59733","inReplyTo":"xmqqo7mqs7rp.fsf@gitster.g","subject":"Re: What's cooking in git.git (May 2023, #04; Thu, 11)","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2023-05-12T01:36:36Z","receivedAt":"2023-05-12T01:36:51Z","isPatch":false,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"Junio C Hamano wrote:\n> * ds/merge-tree-use-config (2023-05-10) 1 commit\n>   (merged to 'next' on 2023-05-11 at e0dab53028)\n>  + merge-tree: load default git config\n>\n>  Allow git forges to disable replace-refs feature while running \"git\n>  merge-tree\".\n>\n>  Will merge to 'master'.\n>  source: <pull.1530.git.1683745654800.gitgitgadget@gmail.com>\n\nWhy was this series merged after only 11 minutes of review window? Are patches\nfrom GitHub favored over all others?\n\nAs I explained here [1], this is clearly the wrong approach because it ends up\ncalling git_default_config *twice*.\n\nThe simplest fix is [2], which simply shuffles one line of code.\n\nThe proper fix is to reorganize the way merge_recursive_config() works, so\ngit_xmerge_config is called only once at the right time (before any\noption parsing). That not only fixes the reported bug, but also makes\nmerge.conflictStyle work for both real merges and trivial merges, along with\nfixing other issues.\n\n[1] https://lore.kernel.org/git/645d672447ebb_13d3fe294f@chronos.notmuch/\n[2] https://lore.kernel.org/git/20230511215608.1297686-1-felipe.contreras@gmail.com/\n[3] https://lore.kernel.org/git/20230511235951.2309462-1-felipe.contreras@gmail.com/\n\n-- \nFelipe Contreras\n"},{"id":"477125","messageId":"ZF2tDgngoBHZojLf@nand.local","threadId":"59733","inReplyTo":"CAMP44s0GAiFhoHFo4Ex464utf2Jm1F-LEdX2PZdUGEdp5x6Liw@mail.gmail.com","subject":"Re: What's cooking in git.git (May 2023, #04; Thu, 11)","fromName":"Taylor Blau","fromEmail":"me@ttaylorr.com","sentAt":"2023-05-12T03:05:50Z","receivedAt":"2023-05-12T03:06:09Z","isPatch":false,"sender":{"key":"me@ttaylorr.com","avatar":"https://avatars.githubusercontent.com/u/301000140?v=4"},"body":"On Thu, May 11, 2023 at 08:36:36PM -0500, Felipe Contreras wrote:\n> Junio C Hamano wrote:\n> > * ds/merge-tree-use-config (2023-05-10) 1 commit\n> >   (merged to 'next' on 2023-05-11 at e0dab53028)\n> >  + merge-tree: load default git config\n> >\n> >  Allow git forges to disable replace-refs feature while running \"git\n> >  merge-tree\".\n> >\n> >  Will merge to 'master'.\n> >  source: <pull.1530.git.1683745654800.gitgitgadget@gmail.com>\n>\n> Why was this series merged after only 11 minutes of review window? Are patches\n> from GitHub favored over all others?\n\nCertainly not.\n\nThe reason that this was merged quickly is because both of the first two\nreviewers had already seen the patch and reviewed it earlier on the\ngit-security list. The patch that Stolee sent was urgent enough to merit\na quick merge. But it was not so urgent as to do an embargoed release\nsince the client-side attack isn't interesting, rather it was important\nfor forges which may use merge-tree directly to perform merges.\n\nThanks,\nTaylor\n"},{"id":"477127","messageId":"CAMP44s0N7tLmpEbScVDDc=M2NG=vx+SoMLu4Vc_vRcS7dvLOgQ@mail.gmail.com","threadId":"59733","inReplyTo":"ZF2tDgngoBHZojLf@nand.local","subject":"Re: What's cooking in git.git (May 2023, #04; Thu, 11)","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2023-05-12T03:46:39Z","receivedAt":"2023-05-12T03:46:55Z","isPatch":false,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Thu, May 11, 2023 at 10:05 PM Taylor Blau <me@ttaylorr.com> wrote:\n> On Thu, May 11, 2023 at 08:36:36PM -0500, Felipe Contreras wrote:\n> > Junio C Hamano wrote:\n\n> > > * ds/merge-tree-use-config (2023-05-10) 1 commit\n> > >   (merged to 'next' on 2023-05-11 at e0dab53028)\n> > >  + merge-tree: load default git config\n> > >\n> > >  Allow git forges to disable replace-refs feature while running \"git\n> > >  merge-tree\".\n> > >\n> > >  Will merge to 'master'.\n> > >  source: <pull.1530.git.1683745654800.gitgitgadget@gmail.com>\n> >\n> > Why was this series merged after only 11 minutes of review window? Are patches\n> > from GitHub favored over all others?\n>\n> Certainly not.\n>\n> The reason that this was merged quickly is because both of the first two\n> reviewers had already seen the patch and reviewed it earlier on the\n> git-security list. The patch that Stolee sent was urgent enough to merit\n> a quick merge.\n\nThere's a quick review, and there is zero review.\n\nEven Derrick Stolee wanted more time for the public list to review the patch.\n\nIf the eyeballs of the public list are not wanted after a security\nreview, then why bother sending it here? Just merge it directly from\ngit-security.\n\nI don't think that's desirable though. I share the opinion of Linus\nTorvalds that security fixes are not special: they are just another\nfix. Therefore they should go through the same process as any other\npatch, because just like any other patch, they can introduce\nregressions, and benefit from more eyeballs.\n\nIf \"given enough eyeballs, all bugs are shallow\", I fail to see why we\nwould want less eyeballs for security fixes. I for one found two\nissues with the patch, my first comment was a bit more than an hour\nlater, and it's already merged.\n\nI don't think that's ideal.\n\nCheers.\n\n-- \nFelipe Contreras\n"},{"id":"477160","messageId":"ZF3nFaHjOxt4a3ee@ncase","threadId":"59733","inReplyTo":"xmqqo7mqs7rp.fsf@gitster.g","subject":"Re: ps/fetch-output-format (was: What's cooking in git.git (May 2023, #04; Thu, 11))","fromName":"Patrick Steinhardt","fromEmail":"ps@pks.im","sentAt":"2023-05-12T07:13:25Z","receivedAt":"2023-05-12T07:16:12Z","isPatch":false,"sender":{"key":"ps@pks.im","avatar":"https://avatars.githubusercontent.com/u/4056630?v=4"},"body":"On Thu, May 11, 2023 at 05:39:06PM -0700, Junio C Hamano wrote:\n> * ps/fetch-output-format (2023-05-10) 9 commits\n>  - fetch: introduce machine-parseable \"porcelain\" output format\n>  - fetch: move option related variables into main function\n>  - fetch: lift up parsing of \"fetch.output\" config variable\n>  - fetch: introduce `display_format` enum\n>  - fetch: refactor calculation of the display table width\n>  - fetch: print left-hand side when fetching HEAD:foo\n>  - fetch: add a test to exercise invalid output formats\n>  - fetch: split out tests for output format\n>  - fetch: fix `--no-recurse-submodules` with multi-remote fetches\n> \n>  \"git fetch\" learned the \"--porcelain\" option that emits what it did\n>  in a machine-parseable format.\n> \n>  Will merge to 'next'?\n>  source: <cover.1683721293.git.ps@pks.im>\n\nFrom my point of view the series is ready to be merged now, and Glen has\nalso given his Reviewed-by.\n\nWe may add below diff to \"fetch: introduce `display_format` enum\". No\nfurther changes would be required for any of the other patches as we\nalways explicitly initialize the value already and don't use the\n`UNKNOWN` value anywhere.\n\ndiff --git a/builtin/fetch.c b/builtin/fetch.c\nindex 462fc86b99..6583e31e3d 100644\n--- a/builtin/fetch.c\n+++ b/builtin/fetch.c\n@@ -49,7 +49,6 @@ enum {\n };\n \n enum display_format {\n-\tDISPLAY_FORMAT_UNKNOWN = 0,\n \tDISPLAY_FORMAT_FULL,\n \tDISPLAY_FORMAT_COMPACT,\n \tDISPLAY_FORMAT_PORCELAIN,\n\nI'm rather indifferent though and feel like re-sending all patches only\nto change this one line of code generates too much noise. But I'm happy\nto adjust in order to make your life easier.\n\nThanks!\n\nPatrick\n"},{"id":"477185","messageId":"CABPp-BFLOpsm6wg+CidEHaQhJ4FTnszfj8SUeBWxkFZgSeJ8Nw@mail.gmail.com","threadId":"59733","inReplyTo":"xmqqo7mqs7rp.fsf@gitster.g","subject":"tl/push-branches-is-an-alias-for-all (Was: Re: What's cooking in git.git (May 2023, #04; Thu, 11))","fromName":"Elijah Newren","fromEmail":"newren@gmail.com","sentAt":"2023-05-12T15:26:09Z","receivedAt":"2023-05-12T15:26:31Z","isPatch":false,"sender":{"key":"newren@gmail.com","avatar":"https://avatars.githubusercontent.com/u/5455730?v=4"},"body":"On Thu, May 11, 2023 at 6:22 PM Junio C Hamano <gitster@pobox.com> wrote:\n>\n> * tl/push-branches-is-an-alias-for-all (2023-05-06) 1 commit\n>   (merged to 'next' on 2023-05-09 at 1d8c3e5dcf)\n>  + push: introduce '--branches' option\n>\n>  \"git push --all\" gained an alias \"git push --branches\".\n>\n>  Will merge to 'master'.\n>  source: <20230506113408.61529-2-tenglong.tl@alibaba-inc.com>\n\nThis topic breaks the tests for me; it needs a trivial one-character\nfix: https://lore.kernel.org/git/pull.1532.git.1683904156670.gitgitgadget@gmail.com/\n"},{"id":"477196","messageId":"xmqq3541qxad.fsf@gitster.g","threadId":"59733","inReplyTo":"CABPp-BFLOpsm6wg+CidEHaQhJ4FTnszfj8SUeBWxkFZgSeJ8Nw@mail.gmail.com","subject":"Re: tl/push-branches-is-an-alias-for-all","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2023-05-12T17:23:06Z","receivedAt":"2023-05-12T17:23:20Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Elijah Newren <newren@gmail.com> writes:\n\n> On Thu, May 11, 2023 at 6:22 PM Junio C Hamano <gitster@pobox.com> wrote:\n>>\n>> * tl/push-branches-is-an-alias-for-all (2023-05-06) 1 commit\n>>   (merged to 'next' on 2023-05-09 at 1d8c3e5dcf)\n>>  + push: introduce '--branches' option\n>>\n>>  \"git push --all\" gained an alias \"git push --branches\".\n>>\n>>  Will merge to 'master'.\n>>  source: <20230506113408.61529-2-tenglong.tl@alibaba-inc.com>\n>\n> This topic breaks the tests for me; it needs a trivial one-character\n> fix: https://lore.kernel.org/git/pull.1532.git.1683904156670.gitgitgadget@gmail.com/\n\nI wonder what made this breakage hidden.  I usually run the tests\nafter accepting a series standalone, and I run the full test suite\n(excluding the ones that depend on things I do not have, like p4)\nwith bash and dash before pushing anything out, and I haven't seen\nGitHub Actions break due to this.\n\nAh, it is because t/Makefile essentially uses $(TEST_SHELL_PATH_SQ)\nto run $@, instead of directly doing \"./$@\", in both cases with and\nwithout probe.  This is very much deliberate in that we do not want\nto rely on and get affected by whichever shell choice the system\nadministrator made when installing \"/bin/sh\".\n\nI'll apply your patch primarily for consistency (a shell script file\nthat begins with \"#!bin/sh\" never is a good thing, and everybody\nelse begins with \"#!/bin/sh\") for now, but I think the justification\nof your patch is questionable in that we do not want our developers\nto rely on \"./t5583-push-branches.sh\" to work [*], as these scripts\nare designed to run under $(TEST_SHELL_PATH).\n\nThanks.\n\n\n[Footnote]\n\n * We could, for example, make t/t[0-9][0-9][0-9]-*.sh scripts\n   consistently use bogus shebang line to catch possible breakage to\n   t/Makefile that bypasses $(TEST_SHELL_PATH) in the future.  If\n   that happens, developers who expect ./t5583-push-branches.sh to\n   work will be broken.\n"},{"id":"477205","messageId":"xmqqmt29pcoq.fsf@gitster.g","threadId":"59733","inReplyTo":"ZF3nFaHjOxt4a3ee@ncase","subject":"Re: ps/fetch-output-format","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2023-05-12T19:33:25Z","receivedAt":"2023-05-12T19:33:37Z","isPatch":false,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Patrick Steinhardt <ps@pks.im> writes:\n\n> We may add below diff to \"fetch: introduce `display_format` enum\". No\n> further changes would be required for any of the other patches as we\n> always explicitly initialize the value already and don't use the\n> `UNKNOWN` value anywhere.\n\nYup, if I were writing this series from scratch I wouldn't have\nadded the UNKNOWN one myself, but it is so small thing that we can\nleave it for those who want to do trivial clean-ups after the dust\nsettles (it could even be one of future microprojects).\n\n>\n> diff --git a/builtin/fetch.c b/builtin/fetch.c\n> index 462fc86b99..6583e31e3d 100644\n> --- a/builtin/fetch.c\n> +++ b/builtin/fetch.c\n> @@ -49,7 +49,6 @@ enum {\n>  };\n>  \n>  enum display_format {\n> -\tDISPLAY_FORMAT_UNKNOWN = 0,\n>  \tDISPLAY_FORMAT_FULL,\n>  \tDISPLAY_FORMAT_COMPACT,\n>  \tDISPLAY_FORMAT_PORCELAIN,\n>\n> I'm rather indifferent though and feel like re-sending all patches only\n> to change this one line of code generates too much noise. But I'm happy\n> to adjust in order to make your life easier.\n>\n> Thanks!\n>\n> Patrick\n"},{"id":"477214","messageId":"ZF6m/ooy0OwsB3ku@tapette.crustytoothpaste.net","threadId":"59733","inReplyTo":"xmqqo7mqs7rp.fsf@gitster.g","subject":"Re: What's cooking in git.git (May 2023, #04; Thu, 11)","fromName":"brian m. carlson","fromEmail":"sandals@crustytoothpaste.net","sentAt":"2023-05-12T20:52:14Z","receivedAt":"2023-05-12T20:52:21Z","isPatch":false,"sender":{"key":"sandals@crustytoothpaste.net","avatar":"https://avatars.githubusercontent.com/u/497054?v=4"},"body":"On 2023-05-12 at 00:39:06, Junio C Hamano wrote:\n> * bc/clone-empty-repo-via-protocol-v0 (2023-05-01) 1 commit\n>  - upload-pack: advertise capabilities when cloning empty repos\n> \n>  Expecting a minor and hopefully final update.\n>  cf. <20230501224038.GA1174291@coredump.intra.peff.net>\n>  source: <20230501170018.1410567-2-sandals@crustytoothpaste.net>\n\nI'll try to get a v3 out early next week.\n-- \nbrian m. carlson (he/him or they/them)\nToronto, Ontario, CA\n"}]}