From: Junio C Hamano Date: Wed, 8 Dec 2021 00:06:35 +0000 (-0800) Subject: What's cooking (2021/12 #02) X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=c161867edae5946a4fdf9a44f307e433d751da91;p=thirdparty%2Fgit.git What's cooking (2021/12 #02) --- diff --git a/whats-cooking.txt b/whats-cooking.txt index 190f7b75c7..a92a3567d8 100644 --- a/whats-cooking.txt +++ b/whats-cooking.txt @@ -1,36 +1,12 @@ To: git@vger.kernel.org Bcc: lwn@lwn.net -Subject: What's cooking in git.git (Dec 2021, #01; Fri, 3) +Subject: What's cooking in git.git (Dec 2021, #02; Tue, 7) X-master-at: abe6bb3905392d5eb6b01fa6e54d7e784e0522aa -X-next-at: a8338297339b1ab064e799c15c4fc56c122ef967 +X-next-at: 8cdad7bc9aee8e9403eb7f1ef605b37815892e18 -What's cooking in git.git (Dec 2021, #01; Fri, 3) +What's cooking in git.git (Dec 2021, #02; Tue, 7) -------------------------------------------------- -This is probably the first issue of the "What's cooking" report, in -which all topics have some sort of description. - -Even better, each and every topic is annotated with some sort of -verdict (like "Will discard", "Expecting a reroll", "Will merge to -'next'?" etc.). There is no topic in the "Undecided" category when -this report is fed to "cook -w" script (found in the 'todo' branch). - -A handful of topics are added to 'next', and we have about 40 topics -cooking there. For them, the future is automated---they spend about -a week in 'next', and if we find issues in them, we incrementally -improve them, or get merged to 'master'. - -Then we have about a dozen topics that are marked for 'next' (please -raise your hand if you have objection before a real damage is done), -and 8 more that may be ready for 'next' (please raise your hand if -you want to nudge them in either direction, for or against). - - - - - - - Here are the topics that have been cooking in my tree. Commits prefixed with '+' are in 'next' (being in 'next' is a sign that a topic is stable enough to be used and are candidate to be in a @@ -42,12 +18,6 @@ useful"). Do not read too much into a topic being in (or not in) 'seen'. The ones marked with '.' do not appear in any of the integration branches, but I am still holding onto them. -A few topics for regression fixes have been merged to 'next' and -will be merged to 'master' shortly. There are many topics listed as -"New" that are in 'seen', but I haven't read many of them (and I -shouldn't have anyway, before dealing with regression fixes) and -haven't gave any summary comments in this report, either, yet. - Copies of the source code to Git live in many repositories, and the following is a list of the ones I push into or their mirrors. Some repositories have only a subset of branches. @@ -79,6 +49,243 @@ Release tarballs are available at: -------------------------------------------------- [New Topics] +* je/http-better-error-output (2021-12-03) 1 commit + . http-backend: give a hint that web browser access is not supported + + When the http-backend program, which is the server-side component + for the smart HTTP transport, sends a "404 Not found" error, we + deliberately did not say anything to the requesting client. We now + send a message back to the browser to tell the user that they do + not want to visit the URL via their browser, instead of a totally + blank page. + + Expecting a reroll. + Breaks its self tests. + cf. <7r23s082-o3q0-479o-srqn-r45q778s5nq7@vanv.qr> + source: <20211202102855.23907-1-jengelh@inai.de> + + +* es/doc-stdout-vs-stderr (2021-12-04) 1 commit + - CodingGuidelines: document which output goes to stdout vs. stderr + + Coding guideline document has been updated to clarify what goes to + standard error i nour system. + + Will merge to 'next'. + source: <20211202223110.22062-1-sunshine@sunshineco.com> + + +* tb/pack-revindex-on-disk-cleanup (2021-12-04) 1 commit + - packfile: make `close_pack_revindex()` static + + Code clean-up. + + Will merge to 'next'. + source: + + +* ds/sparse-deep-pattern-checkout-fix (2021-12-06) 2 commits + - unpack-trees: use traverse_path instead of name + - t1092: add deeper changes during a checkout + + The sparse-index/sparse-checkout feature had a bug in its use of + the matching code to determine which path is in or outside the + sparse checkout patterns. + + Will merge to 'next'. + source: + + +* en/name-rev-shorter-output (2021-12-04) 1 commit + - name-rev: prefer shorter names over following merges + + "git name-rev" has been tweaked to give output that is shorter and + easier to understand. + + Will merge to 'next'? + source: + + +* en/rebase-x-wo-git-dir-env (2021-12-04) 1 commit + - sequencer: do not export GIT_DIR and GIT_WORK_TREE for 'exec' + + "git rebase -x" by mistake started exporting the GIT_DIR and + GIT_WORK_TREE environment variables when the command was rewritten + in C, which has been corrected. + + Will merge to 'next'. + source: + + +* es/worktree-chatty-to-stderr (2021-12-04) 2 commits + - git-worktree.txt: add missing `-v` to synopsis for `worktree list` + - worktree: send "chatty" messages to stderr + + "git worktree add" showed "Preparing worktree" message to the + standard output stream, but when it failed, the message from die() + went to the standard error stream. Depending on the order the + stdio streams are flushed at the program end, this resulted in + confusing output. It has been corrected by sending all the chatty + messages to the standard error stream. + + Will merge to 'next'. + source: <20211203034420.47447-1-sunshine@sunshineco.com> + + +* ja/i18n-similar-messages (2021-12-05) 10 commits + - i18n: ref-filter: factorize "%(foo) atom used without %(bar) atom" + - i18n: factorize "--foo outside a repository" + - i18n: refactor "unrecognized %(foo) argument" strings + - i18n: factorize "no directory given for --foo" + - i18n: factorize "--foo requires --bar" and the like + - i18n: tag.c factorize i18n strings + - i18n: standardize "cannot open" and "cannot read" + - i18n: turn "options are incompatible" into "are mutually exclusive" + - i18n: refactor "%s, %s and %s are mutually exclusive" + - i18n: refactor "foo and bar are mutually exclusive" + + Similar message templates have been consolidated so that + translators need to work on fewer number of messages. + + Needs review. + source: + + +* js/test-initial-branch-override-cleanup (2021-12-05) 1 commit + - tests: set GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME only when needed + + Many tests that used to need GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME + mechanism to force "git" to use 'master' as the default name for + the initial branch no longer need it; the use of the mechanism from + them have been removed. + + Will merge to 'next'. + source: + + +* ab/die-with-bug (2021-12-07) 4 commits + - object.c: use BUG(...) no die("BUG: ...") in lookup_object_by_type() + - pathspec: use BUG(...) not die("BUG:%s:%d....", , ) + - strbuf.h: use BUG(...) not die("BUG: ...") + - pack-objects: use BUG(...) not die("BUG: ...") + + Code clean-up. + + Will merge to 'next'. + source: + + +* ab/usage-die-message (2021-12-07) 6 commits + - config API: use get_error_routine(), not vreportf() + - usage.c + gc: add and use a die_message_errno() + - gc: return from cmd_gc(), don't call exit() + - usage.c API users: use die_message() for error() + exit 128 + - usage.c API users: use die_message() for "fatal :" + exit 128 + - usage.c: add a die_message() routine + + Code clean-up to hide vreportf() from public API. + + Needs review. + source: + + +* js/compat-util-msvc-flex-array (2021-12-06) 1 commit + - git-compat-util(msvc): C11 does not imply support for zero-sized arrays + + MSVC build started to break on flex arrays? + + Will merge to 'next'? + source: + + +* ab/common-main-cleanup (2021-12-07) 1 commit + - common-main.c: call exit(), don't return + + Code clean-up. + + Will merge to 'next'? + source: + + +* ab/fetch-set-upstream-while-detached (2021-12-07) 1 commit + - pull, fetch: fix segfault in --set-upstream option + + "git fetch --set-upstream" did not check if there is a current + branch, leading to a segfault when it is run on a detached HEAD, + which has been corrected. + + Will merge to 'next'. + source: + + +* ew/cbtree-remove-unused-and-broken-cb-unlink (2021-12-07) 1 commit + - cbtree: remove broken and unused cb_unlink + + Code clean-up. + + Will merge to 'next'. + source: <20211207183810.688481-1-e@80x24.org> + +-------------------------------------------------- +[Stalled] + +* ar/submodule-update (2021-10-13) 9 commits + . submodule--helper: rename helper functions + . submodule--helper: remove unused helpers + . submodule: move core cmd_update() logic to C + . submodule--helper: run update using child process struct + . submodule--helper: allow setting superprefix for init_submodule() + . submodule--helper: refactor get_submodule_displaypath() + . submodule--helper: rename helpers for update-clone + . submodule--helper: get remote names from any repository + . submodule--helper: split up ensure_core_worktree() + + Rewrite of "git submodule update" in C. + + Expecting a reroll? + cf. + source: <20211013051805.45662-10-raykar.ath@gmail.com> + + +* ms/customizable-ident-expansion (2021-09-01) 1 commit + - keyword expansion: make "$Id$" string configurable + + Instead of "$Id$", user-specified string (like $FreeBSD$) can be + used as an in-blob placeholder for keyword expansion. + + Will discard. + Stalled for too long. + cf. + cf. <211101.86fssf3bn3.gmgdl@evledraar.gmail.com> + source: + + +* pw/fix-some-issues-in-reset-head (2021-10-01) 12 commits + - rebase -m: don't fork git checkout + - rebase --apply: set ORIG_HEAD correctly + - rebase --apply: fix reflog + - reset_head(): take struct rebase_head_opts + - rebase: cleanup reset_head() calls + - reset_head(): make default_reflog_action optional + - reset_head(): factor out ref updates + - reset_head(): remove action parameter + - reset_head(): don't run checkout hook if there is an error + - reset_head(): fix checkout + - rebase: factor out checkout for up to date branch + - Merge branch 'pw/rebase-of-a-tag-fix' into pw/fix-some-issues-in-reset-head + + Fix "some issues" in a helper function reset_head(). + + Expecting a reroll. + cf. + This needs a lot better explanation, including what the issues are, + which codepaths the helper is used in and to do what, and tests to + protect the fixes. + source: + +-------------------------------------------------- +[Cooking] + * ab/cat-file (2021-11-29) 10 commits - cat-file: use GET_OID_ONLY_TO_DIE in --(textconv|filters) - object-name.c: don't have GET_OID_ONLY_TO_DIE imply *_QUIETLY @@ -101,7 +308,7 @@ Release tarballs are available at: source: -* ab/grep-patterntype (2021-11-29) 7 commits +* ab/grep-patterntype (2021-12-04) 7 commits - grep: simplify config parsing and option parsing - grep API: call grep_config() after grep_init() - grep.c: don't pass along NULL callback value @@ -112,9 +319,8 @@ Release tarballs are available at: Some code clean-up in the "git grep" machinery. - Need to pick up a reroll. - cf. - source: + Needs review. + source: * en/rebase-x-fix (2021-11-29) 1 commit @@ -142,7 +348,7 @@ Release tarballs are available at: source: -* jt/conditional-config-on-remote-url (2021-11-29) 2 commits +* jt/conditional-config-on-remote-url (2021-12-04) 2 commits - config: include file if remote URL matches a glob - config: make git_config_include() static @@ -150,9 +356,8 @@ Release tarballs are available at: "[includeIf ]" learns to base its decision on the URL of the remote repository the repository interacts with. - Need to pick up a reroll. - cf. - source: <3b3af0da983c65ac8282b14141495f6859cd575a.1638217387.git.jonathantanmy@google.com> + Needs review. + source: * tb/cruft-packs (2021-11-29) 17 commits @@ -210,87 +415,15 @@ Release tarballs are available at: source: -* jl/subtree-check-parents-argument-passing-fix (2021-12-01) 1 commit +* jl/subtree-check-parents-argument-passing-fix (2021-12-05) 1 commit - subtree: fix argument handling in check_parents Fix performance-releated bug in "git subtree" (in contrib/). Expecting a reroll. - cf. - source: - --------------------------------------------------- -[Stalled] - -* ar/submodule-update (2021-10-13) 9 commits - . submodule--helper: rename helper functions - . submodule--helper: remove unused helpers - . submodule: move core cmd_update() logic to C - . submodule--helper: run update using child process struct - . submodule--helper: allow setting superprefix for init_submodule() - . submodule--helper: refactor get_submodule_displaypath() - . submodule--helper: rename helpers for update-clone - . submodule--helper: get remote names from any repository - . submodule--helper: split up ensure_core_worktree() - - Rewrite of "git submodule update" in C. - - Expecting a reroll? - cf. - source: <20211013051805.45662-10-raykar.ath@gmail.com> - - -* cf/fetch-set-upstream-while-detached (2021-07-06) 1 commit - - fetch: fix segfault on --set-upstream while on a detached HEAD - - "git fetch --set-upstream" while on detached HEAD segfaulted - instead of noticing that such an operation did not make sense. - - Expecting a reroll. - A low hanging fruit to make this usable is an addition of a few tests; - any takers? - cf. - cf. - source: <20210706162238.575988-1-clemens@endorphin.org> - - -* ms/customizable-ident-expansion (2021-09-01) 1 commit - - keyword expansion: make "$Id$" string configurable - - Instead of "$Id$", user-specified string (like $FreeBSD$) can be - used as an in-blob placeholder for keyword expansion. - - Will discard. - Stalled for too long. - cf. - cf. <211101.86fssf3bn3.gmgdl@evledraar.gmail.com> - source: - - -* pw/fix-some-issues-in-reset-head (2021-10-01) 12 commits - - rebase -m: don't fork git checkout - - rebase --apply: set ORIG_HEAD correctly - - rebase --apply: fix reflog - - reset_head(): take struct rebase_head_opts - - rebase: cleanup reset_head() calls - - reset_head(): make default_reflog_action optional - - reset_head(): factor out ref updates - - reset_head(): remove action parameter - - reset_head(): don't run checkout hook if there is an error - - reset_head(): fix checkout - - rebase: factor out checkout for up to date branch - - Merge branch 'pw/rebase-of-a-tag-fix' into pw/fix-some-issues-in-reset-head + cf. + source: - Fix "some issues" in a helper function reset_head(). - - Will discard. - This needs a lot better explanation, including what the issues are, - which codepaths the helper is used in and to do what, and tests to - protect the fixes. - source: - --------------------------------------------------- -[Cooking] * ab/ambiguous-object-name (2021-11-26) 6 commits - object-name: re-use "struct strbuf" in show_ambiguous_object() @@ -344,7 +477,7 @@ Release tarballs are available at: source: -* hn/allow-bogus-oid-in-ref-tests (2021-12-02) 8 commits +* hn/allow-bogus-oid-in-ref-tests (2021-12-07) 8 commits - t1430: create valid symrefs using test-helper - t1430: remove refs using test-tool - refs: introduce REF_SKIP_REFNAME_VERIFICATION flag @@ -359,7 +492,7 @@ Release tarballs are available at: want to test Git in. Will merge to 'next'. - source: + source: * hn/t1404-df-limitation-is-ref-files-only (2021-11-29) 1 commit @@ -407,13 +540,13 @@ Release tarballs are available at: source: -* jc/grep-patterntype-default-doc (2021-11-29) 1 commit +* jc/grep-patterntype-default-doc (2021-12-05) 1 commit - grep: clarify what `grep.patternType=default` means Doc update. - Will merge to 'next'. - source: + Will merge to 'next'? + source: * ab/ci-updates (2021-11-23) 5 commits @@ -479,6 +612,7 @@ Release tarballs are available at: * hn/create-reflog-simplify (2021-11-22) 1 commit (merged to 'next' on 2021-12-02 at 87ff49b776) + refs: drop force_create argument of create_reflog API + (this branch is used by hn/reftable-coverity-fixes.) A small simplification of API. @@ -603,7 +737,7 @@ Release tarballs are available at: source: <20211118154317.639118-1-alexhenrie24@gmail.com> -* ds/fetch-pull-with-sparse-index (2021-11-29) 3 commits +* ds/fetch-pull-with-sparse-index (2021-12-06) 3 commits - ls-files: add --sparse option - fetch/pull: use the sparse index - Merge branch 'ld/sparse-diff-blame' into ds/fetch-pull-with-sparse-index @@ -612,8 +746,10 @@ Release tarballs are available at: "git fetch" and "git pull" are now declared sparse-index clean. Also "git ls-files" learns the "--sparse" option to help debugging. - Expecting a reroll? + Expecting a reroll. Discussion seems to have stalled, with a few loose ends questions. + Also since the branch that this depends on was rerolled, it may + benefit from a refresh. source: @@ -662,7 +798,11 @@ Release tarballs are available at: source: <5a9e49e7f1540d762374382e1769edf112cf093f.1637272706.git.steadmon@google.com> -* pw/xdiff-classify-record-in-histogram (2021-11-18) 3 commits +* pw/xdiff-classify-record-in-histogram (2021-12-04) 6 commits + (merged to 'next' on 2021-12-07 at 8cdad7bc9a) + + xdiff: drop unused flags parameter from recs_match + + xdiff: drop xpparam_t parameter from histogram cmp_recs() + + xdiff: drop CMP_ENV macro from xhistogram (merged to 'next' on 2021-12-02 at 5ccd12d09b) + xdiff: simplify comparison + xdiff: avoid unnecessary memory allocations @@ -671,7 +811,8 @@ Release tarballs are available at: "diff --histogram" optimization. Will merge to 'master'. - source: + source: + source: * rs/mergesort (2021-11-18) 1 commit @@ -684,7 +825,7 @@ Release tarballs are available at: source: <5eabbe1c-4c0f-559a-da21-423afec89e7e@web.de> -* xw/am-empty (2021-12-01) 3 commits +* xw/am-empty (2021-12-07) 3 commits - am: support --allow-empty to record specific empty patches - am: support --empty=