What's cooking (2023/07 #01)

1 files changed, 128 insertions, 148 deletions

diff --git a/whats-cooking.txt b/whats-cooking.txt
index a46d70b003..2301945f7c 100644
--- a/whats-cooking.txt
+++ b/whats-cooking.txt
@@ -1,10 +1,10 @@
 To: git@vger.kernel.org
-Subject: What's cooking in git.git (Jun 2023, #08; Fri, 30)
-X-master-at: 9748a6820043d5815bee770ffa51647e0adc2cf0
-X-next-at: 5b50783d6bcd27d89ccdda9a4ecfff0d938a98eb
+Subject: What's cooking in git.git (Jul 2023, #01; Wed, 5)
+X-master-at: a646b86cd10282de2ceb64ef33b5412e4fb2a54c
+X-next-at: a8219df2e13870c2093740ee0a3038d600051289
 Bcc: lwn@lwn.net, gitster@pobox.com
 
-What's cooking in git.git (Jun 2023, #08; Fri, 30)
+What's cooking in git.git (Jul 2023, #01; Wed, 5)
 --------------------------------------------------
 
 Here are the topics that have been cooking in my tree.  Commits
@@ -48,127 +48,79 @@ Release tarballs are available at:
 --------------------------------------------------
 [Graduated to 'master']
 
-* ds/remove-idx-before-pack (2023-06-20) 1 commit
-  (merged to 'next' on 2023-06-23 at fa97bf0e41)
- + packfile: delete .idx files before .pack files
-
- We create .pack and then .idx, we consider only packfiles that have
- .idx usable (those with only .pack are not ready yet), so we should
- remove .idx before removing .pack for consistency.
- source: <pull.1547.git.1687287675248.gitgitgadget@gmail.com>
-
-
-* en/header-split-cache-h-part-3 (2023-06-21) 28 commits
-  (merged to 'next' on 2023-06-23 at 84ad22bf36)
- + fsmonitor-ll.h: split this header out of fsmonitor.h
- + hash-ll, hashmap: move oidhash() to hash-ll
- + object-store-ll.h: split this header out of object-store.h
- + khash: name the structs that khash declares
- + merge-ll: rename from ll-merge
- + git-compat-util.h: remove unneccessary include of wildmatch.h
- + builtin.h: remove unneccessary includes
- + list-objects-filter-options.h: remove unneccessary include
- + diff.h: remove unnecessary include of oidset.h
- + repository: remove unnecessary include of path.h
- + log-tree: replace include of revision.h with simple forward declaration
- + cache.h: remove this no-longer-used header
- + read-cache*.h: move declarations for read-cache.c functions from cache.h
- + repository.h: move declaration of the_index from cache.h
- + merge.h: move declarations for merge.c from cache.h
- + diff.h: move declaration for global in diff.c from cache.h
- + preload-index.h: move declarations for preload-index.c from elsewhere
- + sparse-index.h: move declarations for sparse-index.c from cache.h
- + name-hash.h: move declarations for name-hash.c from cache.h
- + run-command.h: move declarations for run-command.c from cache.h
- + statinfo: move stat_{data,validity} functions from cache/read-cache
- + read-cache: move shared add/checkout/commit code
- + add: modify add_files_to_cache() to avoid globals
- + read-cache: move shared commit and ls-files code
- + setup: adopt shared init-db & clone code
- + init-db, clone: change unnecessary global into passed parameter
- + init-db: remove unnecessary global variable
- + init-db: document existing bug with core.bare in template config
-
- Header files cleanup.
- source: <pull.1525.v3.git.1684218848.gitgitgadget@gmail.com>
-
-
-* jc/abort-ll-merge-with-a-signal (2023-06-23) 2 commits
-  (merged to 'next' on 2023-06-24 at 685eb5d25c)
- + t6406: skip "external merge driver getting killed by a signal" test on Windows
-  (merged to 'next' on 2023-06-23 at 9c9c37e95e)
- + ll-merge: killing the external merge driver aborts the merge
-
- When the external merge driver is killed by a signal, its output
- should not be trusted as a resolution with conflicts that is
- proposed by the driver, but the code did.
- source: <xmqq4jmzc91e.fsf_-_@gitster.g>
-
-
-* tb/gc-recent-object-hook (2023-06-24) 1 commit
-  (merged to 'next' on 2023-06-24 at e8c295841b)
- + t7701: make annotated tag unreachable
+* bc/more-git-var (2023-06-27) 8 commits
+  (merged to 'next' on 2023-06-27 at ea14687e91)
+ + var: add config file locations
+ + var: add attributes files locations
+ + attr: expose and rename accessor functions
+ + var: adjust memory allocation for strings
+ + var: format variable structure with C99 initializers
+ + var: add support for listing the shell
+ + t: add a function to check executable bit
+ + var: mark unused parameters in git_var callbacks
 
- Test update.
- source: <259b1b559114ab1a9a0bd7f1ad29a4cba2612ae0.1687617197.git.me@ttaylorr.com>
+ Add more "git var" for toolsmiths to learn various locations Git is
+ configured with either via the configuration or hardcoded defaults.
+ source: <20230627161902.754472-1-sandals@crustytoothpaste.net>
 
---------------------------------------------------
-[New Topics]
 
-* jk/cherry-pick-revert-status (2023-06-27) 1 commit
-  (merged to 'next' on 2023-06-28 at 4a0d88ef7a)
- + fix cherry-pick/revert status when doing multiple commits
-
- During a cherry-pick or revert session that works on multiple
- commits, "git status" did not give correct information, which has
- been corrected.
+* ps/revision-stdin-with-options (2023-06-15) 3 commits
+  (merged to 'next' on 2023-06-26 at eda3e4d0b5)
+ + revision: handle pseudo-opts in `--stdin` mode
+ + revision: small readability improvement for reading from stdin
+ + revision: reorder `read_revisions_from_stdin()`
 
- Will merge to 'master'.
- source: <20230627224230.1951135-1-jacob.e.keller@intel.com>
+ The set-up code for the get_revision() API now allows feeding
+ options like --all and --not in the --stdin mode.
+ source: <cover.1686839572.git.ps@pks.im>
 
+--------------------------------------------------
+[New Topics]
 
-* ks/t4205-test-describe-with-abbrev-fix (2023-06-29) 1 commit
-  (merged to 'next' on 2023-06-29 at 5fc309dc75)
- + t4205: correctly test %(describe:abbrev=...)
+* cw/compat-util-header-cleanup (2023-07-05) 6 commits
+ - git-compat-util: move alloc macros to git-compat-util.h
+ - treewide: remove unnecessary includes for wrapper.h
+ - kwset: move translation table from ctype
+ - sane-ctype.h: create header for sane-ctype macros
+ - git-compat-util: move wrapper.c funcs to its header
+ - git-compat-util: move strbuf.c funcs to its header
 
- Test update.
+ Further shuffling of declarations across header files to streamline
+ file dependencies.
 
- Will merge to 'master'.
- source: <20230629133841.18784-2-five231003@gmail.com>
+ Will merge to 'next'?
+ source: <20230705170812.3833103-1-calvinwan@google.com>
 
 
-* jk/fsck-indices-in-worktrees (2023-06-29) 1 commit
-  (merged to 'next' on 2023-06-29 at 9d245b3681)
- + fsck: avoid misleading variable name
+* jk/imap-send-unused-variable-cleanup (2023-07-05) 3 commits
+ - imap-send: drop unused fields from imap_cmd_cb
+ - imap-send: drop unused parameter from imap_cmd_cb callback
+ - imap-send: use server conf argument in setup_curl()
 
- Code clarification.
+ "imap-send" codepaths got cleaned up to get rid of unused
+ parameters.
 
- Will merge to 'master'.
- source: <20230629181333.87465-1-ericsunshine@charter.net>
+ Will merge to 'next'.
+ source: <20230703063247.GA3524302@coredump.intra.peff.net>
 
 
-* js/empty-index-fixes (2023-06-29) 3 commits
-  (merged to 'next' on 2023-06-29 at cdfd131900)
- + commit -a -m: allow the top-level tree to become empty again
- + split-index: accept that a base index can be empty
- + do_read_index(): always mark index as initialized unless erroring out
+* ma/t0091-fixup (2023-07-05) 1 commit
+ - t0091-bugreport.sh: actually verify some content of report
 
- A few places failed to differenciate the case where the index is
- truly empty (nothing added) and we haven't yet read from the
- on-disk index file, which have been corrected.
+ "git bugreport" tests did not test what it wanted to test, which
+ has been corrected.
 
- Will merge to 'master'.
- source: <pull.1554.git.1688044991.gitgitgadget@gmail.com>
+ Will merge to 'next'.
+ source: <20230705183532.3057433-1-martin.agren@gmail.com>
 
 
-* vd/adjust-mfow-doc-to-updated-headers (2023-06-29) 1 commit
- - docs: include "trace.h" in MyFirstObjectWalk.txt
+* rs/packet-length-simplify (2023-07-04) 1 commit
+ - pkt-line: don't check string length in packet_length()
 
- Code snippets in a tutorial document no longer compiled after
- recent header shuffling, which have been corrected.
+ Code simplification.
 
  Will merge to 'next'?
- source: <20230629185238.58961-1-vinayakdev.sci@gmail.com>
+ source: <89d58db7-6a01-b3fa-54f0-19d5a3819eb3@web.de>
 
 --------------------------------------------------
 [Stalled]
@@ -307,6 +259,62 @@ Release tarballs are available at:
 --------------------------------------------------
 [Cooking]
 
+* jk/cherry-pick-revert-status (2023-06-27) 1 commit
+  (merged to 'next' on 2023-06-28 at 4a0d88ef7a)
+ + fix cherry-pick/revert status when doing multiple commits
+
+ During a cherry-pick or revert session that works on multiple
+ commits, "git status" did not give correct information, which has
+ been corrected.
+
+ Will merge to 'master'.
+ source: <20230627224230.1951135-1-jacob.e.keller@intel.com>
+
+
+* ks/t4205-test-describe-with-abbrev-fix (2023-06-29) 1 commit
+  (merged to 'next' on 2023-06-29 at 5fc309dc75)
+ + t4205: correctly test %(describe:abbrev=...)
+
+ Test update.
+
+ Will merge to 'master'.
+ source: <20230629133841.18784-2-five231003@gmail.com>
+
+
+* jk/fsck-indices-in-worktrees (2023-06-29) 1 commit
+  (merged to 'next' on 2023-06-29 at 9d245b3681)
+ + fsck: avoid misleading variable name
+
+ Code clarification.
+
+ Will merge to 'master'.
+ source: <20230629181333.87465-1-ericsunshine@charter.net>
+
+
+* js/empty-index-fixes (2023-06-29) 3 commits
+  (merged to 'next' on 2023-06-29 at cdfd131900)
+ + commit -a -m: allow the top-level tree to become empty again
+ + split-index: accept that a base index can be empty
+ + do_read_index(): always mark index as initialized unless erroring out
+
+ A few places failed to differenciate the case where the index is
+ truly empty (nothing added) and we haven't yet read from the
+ on-disk index file, which have been corrected.
+
+ Will merge to 'master'.
+ source: <pull.1554.git.1688044991.gitgitgadget@gmail.com>
+
+
+* vd/adjust-mfow-doc-to-updated-headers (2023-07-04) 1 commit
+ - docs: add necessary headers to Documentation/MFOW.txt
+
+ Code snippets in a tutorial document no longer compiled after
+ recent header shuffling, which have been corrected.
+
+ Will merge to 'next'?
+ source: <20230702151457.3227-1-vinayakdev.sci@gmail.com>
+
+
 * mh/credential-erase-improvements-more (2023-06-24) 2 commits
  - credential/wincred: erase matching creds only
  - credential/libsecret: erase matching creds only
@@ -357,36 +365,18 @@ Release tarballs are available at:
  source: <pull.1552.git.1687772253869.gitgitgadget@gmail.com>
 
 
-* pw/diff-no-index-from-named-pipes (2023-06-27) 3 commits
+* pw/diff-no-index-from-named-pipes (2023-07-05) 4 commits
  - diff --no-index: support reading from named pipes
  - t4054: test diff --no-index with stdin
  - diff --no-index: die on error reading stdin
+ - diff --no-index: refuse to compare stdin to a directory
 
  "git diff --no-index" learned to read from named pipes as if they
  were regular files, to allow "git diff <(process) <(substitution)"
  some shells support.
 
- Expecting a reroll.
- cf. <457ff920-865e-f018-1d79-f3cb1121d199@gmail.com>
- source: <cover.1687874975.git.phillip.wood@dunelm.org.uk>
-
-
-* bc/more-git-var (2023-06-27) 8 commits
-  (merged to 'next' on 2023-06-27 at ea14687e91)
- + var: add config file locations
- + var: add attributes files locations
- + attr: expose and rename accessor functions
- + var: adjust memory allocation for strings
- + var: format variable structure with C99 initializers
- + var: add support for listing the shell
- + t: add a function to check executable bit
- + var: mark unused parameters in git_var callbacks
-
- Add more "git var" for toolsmiths to learn various locations Git is
- configured with either via the configuration or hardcoded defaults.
-
- Will merge to 'master'.
- source: <20230627161902.754472-1-sandals@crustytoothpaste.net>
+ Will merge to 'next'.
+ source: <cover.1688586536.git.phillip.wood@dunelm.org.uk>
 
 
 * jc/doc-hash-object-types (2023-06-28) 1 commit
@@ -399,35 +389,22 @@ Release tarballs are available at:
  source: <pull.1533.v3.git.git.1688004473941.gitgitgadget@gmail.com>
 
 
-* cc/repack-sift-filtered-objects-to-separate-pack (2023-06-14) 9 commits
+* cc/repack-sift-filtered-objects-to-separate-pack (2023-07-05) 8 commits
  - gc: add `gc.repackFilterTo` config option
  - repack: implement `--filter-to` for storing filtered out objects
  - gc: add `gc.repackFilter` config option
  - repack: add `--filter=<filter-spec>` option
+ - repack: refactor finding pack prefix
  - repack: refactor finishing pack-objects command
- - repack: refactor piping an oid to a command
  - t/helper: add 'find-pack' test-tool
- - pack-objects: add `--print-filtered` to print omitted objects
  - pack-objects: allow `--filter` without `--stdout`
 
  "git repack" machinery learns to pay attention to the "--filter="
  option.
 
- Needs review.
- source: <20230614192541.1599256-1-christian.couder@gmail.com>
-
-
-* ps/revision-stdin-with-options (2023-06-15) 3 commits
-  (merged to 'next' on 2023-06-26 at eda3e4d0b5)
- + revision: handle pseudo-opts in `--stdin` mode
- + revision: small readability improvement for reading from stdin
- + revision: reorder `read_revisions_from_stdin()`
-
- The set-up code for the get_revision() API now allows feeding
- options like --all and --not in the --stdin mode.
-
- Will merge to 'master'.
- source: <cover.1686839572.git.ps@pks.im>
+ Comments?
+ cf. <xmqqh6qi1byn.fsf@gitster.g>
+ source: <20230705060812.2865188-1-christian.couder@gmail.com>
 
 
 * rs/strbuf-expand-step (2023-06-18) 5 commits
@@ -444,13 +421,13 @@ Release tarballs are available at:
  source: <767baa64-20a6-daf2-d34b-d81f72363749@web.de>
 
 
-* js/doc-unit-tests (2023-06-13) 1 commit
+* js/doc-unit-tests (2023-06-30) 1 commit
  - unit tests: Add a project plan document
 
  Process to add some form of low-level unit tests has started.
 
- Comments?  Filling in blanks?
- source: <8afdb215d7e10ca16a2ce8226b4127b3d8a2d971.1686352386.git.steadmon@google.com>
+ Still filling in blanks.
+ source: <0169ce6fb9ccafc089b74ae406db0d1a8ff8ac65.1688165272.git.steadmon@google.com>
 
 
 * mh/mingw-case-sensitive-build (2023-06-12) 1 commit
@@ -505,7 +482,8 @@ Release tarballs are available at:
  The "git for-each-ref" family of commands learned placeholders
  related to GPG signature verification.
 
- Needs review.
+ Will merge to 'next'.
+ cf. <CAP8UFD1CfUm+9PTXO0wBBdBs0U=YVte+EiPbDyUyg7T152aw2g@mail.gmail.com>
  source: <20230604185815.15761-1-five231003@gmail.com>
 
 
@@ -567,6 +545,8 @@ Release tarballs are available at:
 
  Enumerating refs in the packed-refs file, while excluding refs that
  match certain patterns, has been optimized.
+
+ Under discussion.
  source: <cover.1687270849.git.me@ttaylorr.com>