* What's cooking in git.git (Jun 2023, #05; Tue, 20)
@ 2023-06-21 0:04 Junio C Hamano
2023-06-21 8:55 ` bug in en/header-split-cache-h-part-3, was " Jeff King
0 siblings, 1 reply; 8+ messages in thread
From: Junio C Hamano @ 2023-06-21 0:04 UTC (permalink / raw
To: git
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
future release). Commits prefixed with '-' are only in 'seen', and
aren't considered "accepted" at all and may be annotated with an URL
to a message that raises issues but they are no means exhaustive. A
topic without enough support may be discarded after a long period of
no activity (of course they can be resubmit when new interests
arise).
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.
With maint, master, next, seen, todo:
git://git.kernel.org/pub/scm/git/git.git/
git://repo.or.cz/alt-git.git/
https://kernel.googlesource.com/pub/scm/git/git/
https://github.com/git/git/
https://gitlab.com/git-vcs/git/
With all the integration branches and topics broken out:
https://github.com/gitster/git/
Even though the preformatted documentation in HTML and man format
are not sources, they are published in these repositories for
convenience (replace "htmldocs" with "manpages" for the manual
pages):
git://git.kernel.org/pub/scm/git/git-htmldocs.git/
https://github.com/gitster/git-htmldocs.git/
Release tarballs are available at:
https://www.kernel.org/pub/software/scm/git/
--------------------------------------------------
[Graduated to 'master']
* jc/test-modernization (2023-05-19) 20 commits
(merged to 'next' on 2023-06-15 at 811ac3f7ad)
+ t7101-reset-empty-subdirs: modernize test format
+ t6050-replace: modernize test format
+ t5306-pack-nobase: modernize test format
+ t5303-pack-corruption-resilience: modernize test format
+ t5301-sliding-window: modernize test format
+ t5300-pack-object: modernize test format
+ t4206-log-follow-harder-copies: modernize test format
+ t4202-log: modernize test format
+ t4004-diff-rename-symlink: modernize test format
+ t4003-diff-rename-1: modernize test format
+ t4002-diff-basic: modernize test format
+ t3903-stash: modernize test format
+ t3700-add: modernize test format
+ t3500-cherry: modernize test format
+ t1006-cat-file: modernize test format
+ t1002-read-tree-m-u-2way: modernize test format
+ t1001-read-tree-m-2way: modernize test format
+ t3210-pack-refs: modernize test format
+ t0030-stripspace: modernize test format
+ t0000-basic: modernize test format
(this branch is used by jc/test-modernization-2.)
Originally merged to 'next' on 2023-06-13
source: <pull.1513.git.git.1684440205.gitgitgadget@gmail.com>
* jc/test-modernization-2 (2023-05-23) 10 commits
(merged to 'next' on 2023-06-15 at 01df673682)
+ t9400-git-cvsserver-server: modernize test format
+ t9200-git-cvsexportcommit: modernize test format
+ t9104-git-svn-follow-parent: modernize test format
+ t9100-git-svn-basic: modernize test format
+ t7700-repack: modernize test format
+ t7600-merge: modernize test format
+ t7508-status: modernize test format
+ t7201-co: modernize test format
+ t7111-reset-table: modernize test format
+ t7110-reset-merge: modernize test format
(this branch uses jc/test-modernization.)
Originally merged to 'next' on 2023-06-13
Test style updates.
source: <pull.1514.git.git.1684599239.gitgitgadget@gmail.com>
* jk/ci-use-clang-for-sanitizer-jobs (2023-06-03) 3 commits
(merged to 'next' on 2023-06-15 at 268459fc6a)
+ ci: drop linux-clang job
+ ci: run ASan/UBSan in a single job
+ ci: use clang for ASan/UBSan checks
Originally merged to 'next' on 2023-06-12
Clang's sanitizer implementation seems to work better than GCC's.
source: <20230601180220.GA4167745@coredump.intra.peff.net>
* jk/log-follow-with-non-literal-pathspec (2023-06-03) 3 commits
(merged to 'next' on 2023-06-15 at 6848e0b8e6)
+ diff: detect pathspec magic not supported by --follow
+ diff: factor out --follow pathspec check
+ pathspec: factor out magic-to-name function
Originally merged to 'next' on 2023-06-13
"git [-c log.follow=true] log [--follow] ':(glob)f**'" used to barf.
source: <20230601173724.GA4158369@coredump.intra.peff.net>
* kh/use-default-notes-doc (2023-06-06) 2 commits
(merged to 'next' on 2023-06-15 at f25ced157e)
+ notes: move the documentation to the struct
+ notes: update documentation for `use_default_notes`
Originally merged to 'next' on 2023-06-13
Doc update.
source: <cover.1685958731.git.code@khaugsbakk.name>
* la/doc-interpret-trailers (2023-06-14) 9 commits
(merged to 'next' on 2023-06-15 at 6ff4a2b913)
+ doc: trailer: add more examples in DESCRIPTION
+ doc: trailer: mention 'key' in DESCRIPTION
+ doc: trailer.<token>.command: emphasize deprecation
+ doc: trailer: use angle brackets for <token> and <value>
+ doc: trailer: remove redundant phrasing
+ doc: trailer: examples: avoid the word "message" by itself
+ doc: trailer: drop "commit message part" phrasing
+ doc: trailer: swap verb order
+ doc: trailer: fix grammar
Originally merged to 'next' on 2023-06-14
Doc update.
source: <pull.1506.v4.git.git.1686797630.gitgitgadget@gmail.com>
* mh/commit-reach-get-reachable-plug-leak (2023-06-04) 1 commit
(merged to 'next' on 2023-06-15 at 62e19d66fb)
+ commit-reach: fix memory leak in get_reachable_subset()
Originally merged to 'next' on 2023-06-12
Plug memory leak.
source: <20230603002819.1122129-1-mh@glandium.org>
* pb/complete-and-document-auto-merge-and-friends (2023-05-23) 6 commits
(merged to 'next' on 2023-06-15 at ebe3216719)
+ completion: complete AUTO_MERGE
+ Documentation: document AUTO_MERGE
+ git-merge.txt: modernize word choice in "True merge" section
+ completion: complete REVERT_HEAD and BISECT_HEAD
+ revisions.txt: document more special refs
+ revisions.txt: use description list for special refs
Originally merged to 'next' on 2023-06-13
Document more pseudo-refs and teach the command line completion
machinery to complete AUTO_MERGE.
source: <pull.1515.v2.git.1684783741.gitgitgadget@gmail.com>
* ps/fetch-cleanups (2023-05-17) 9 commits
(merged to 'next' on 2023-06-15 at 33878153a5)
+ fetch: use `fetch_config` to store "submodule.fetchJobs" value
+ fetch: use `fetch_config` to store "fetch.parallel" value
+ fetch: use `fetch_config` to store "fetch.recurseSubmodules" value
+ fetch: use `fetch_config` to store "fetch.showForcedUpdates" value
+ fetch: use `fetch_config` to store "fetch.pruneTags" value
+ fetch: use `fetch_config` to store "fetch.prune" value
+ fetch: pass through `fetch_config` directly
+ fetch: drop unneeded NULL-check for `remote_ref`
+ fetch: drop unused DISPLAY_FORMAT_UNKNOWN enum value
Originally merged to 'next' on 2023-06-12
Code clean-up.
cf. <20230519002128.GD2442034@coredump.intra.peff.net>
source: <cover.1684324059.git.ps@pks.im>
* tb/submodule-null-deref-fix (2023-05-25) 1 commit
(merged to 'next' on 2023-06-15 at 8cb6a7710a)
+ builtin/submodule--helper.c: handle missing submodule URLs
Originally merged to 'next' on 2023-06-13
"git submodule" code trusted the data coming from the config (and
the in-tree .gitmodules file) too much without validating, leading
to NULL dereference if the user mucks with a repository (e.g.
submodule.<name>.url is removed). This has been corrected.
source: <ae6cf3fa461b85e346f034371dae56a2790dfa20.1684957882.git.me@ttaylorr.com>
* tl/quote-problematic-arg-for-clarity (2023-06-03) 1 commit
(merged to 'next' on 2023-06-15 at 4e52ad2fb3)
+ surround %s with quotes when failed to lookup commit
Originally merged to 'next' on 2023-06-12
Error message fix.
source: <1f7c62a8870433792076fae30d6c4dc4b61a00d8.1685366301.git.dyroneteng@gmail.com>
* tz/test-fix-pthreads-prereq (2023-05-26) 1 commit
(merged to 'next' on 2023-06-15 at 2d2446ff97)
+ trace2 tests: fix PTHREADS prereq
Originally merged to 'next' on 2023-06-12
Test fix.
source: <20230525031218.3554586-1-tmz@pobox.com>
* tz/test-ssh-verifytime-fix (2023-05-26) 1 commit
(merged to 'next' on 2023-06-15 at dd97dd32f3)
+ t/lib-gpg: fix ssh-keygen -Y check-novalidate with openssh-9.0
Originally merged to 'next' on 2023-06-12
Test fix.
source: <20230525031026.3554406-1-tmz@pobox.com>
* vd/worktree-config-is-per-repository (2023-05-26) 3 commits
(merged to 'next' on 2023-06-15 at 6301770cd6)
+ repository: move 'repository_format_worktree_config' to repo scope
+ config: pass 'repo' directly to 'config_with_options()'
+ config: use gitdir to get worktree config
Originally merged to 'next' on 2023-06-13
The value of config.worktree is per-repository, but has been kept
in a singleton global variable per process. This has been OK as
most Git operations interacted with a single repository at a time,
but not right for operations like recursive "grep" that want to
access multiple repositories from a single process without forking.
The global variable has been eliminated and made into a member in
the per-repository data structure.
cf. <3145f4f3-7bd4-8a1b-4943-11b7d22b60c6@github.com>
cf. <kl6lr0qwno2q.fsf@chooglen-macbookpro.roam.corp.google.com>
source: <pull.1536.v2.git.1685064781.gitgitgadget@gmail.com>
--------------------------------------------------
[New Topics]
* gc/discover-not-setup (2023-06-16) 1 commit
(merged to 'next' on 2023-06-16 at 27d1cd2a1b)
+ setup.c: don't setup in discover_git_directory()
discover_git_directory() no longer touches the_repository.
Will merge to 'master'.
source: <pull.1526.git.git.1686771358484.gitgitgadget@gmail.com>
* mh/credential-erase-improvements (2023-06-15) 2 commits
(merged to 'next' on 2023-06-16 at 5a3ed25303)
+ credential: erase all matching credentials
+ credential: avoid erasing distinct password
Will merge to 'master'.
cf. <20230615212116.GA39325@coredump.intra.peff.net>
source: <pull.1525.v4.git.git.1686856773.gitgitgadget@gmail.com>
* cc/repack-sift-filtered-objects-to-separate-pack (2023-06-14) 9 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 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>
* js/cmake-wo-cache-h (2023-06-15) 1 commit
(merged to 'next' on 2023-06-16 at b138273e7e)
+ cmake: adapt to `cache.h` being no more
(this branch uses en/header-split-cache-h-part-3.)
Build fix in en/header-split-cache-h-part-3 topic.
Will merge to 'master'.
source: <pull.1525.v3.git.1684218848.gitgitgadget@gmail.com>
* ps/revision-stdin-with-options (2023-06-15) 3 commits
- 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 'next'?
source: <cover.1686839572.git.ps@pks.im>
* rs/doc-ls-tree-hex-literal (2023-06-15) 1 commit
(merged to 'next' on 2023-06-16 at 472c7f55c9)
+ ls-tree: fix documentation of %x format placeholder
Doc update.
Will merge to 'master'.
source: <23e54256-9ad5-e978-d0b8-abb511232fd1@web.de>
* jk/commit-use-no-divider-with-interpret-trailers (2023-06-16) 1 commit
(merged to 'next' on 2023-06-20 at 08e5f2a6b5)
+ commit: pass --no-divider to interpret-trailers
When "git commit --trailer=..." invokes the interpret-trailers
machinery, it knows what it feeds to interpret-trailers is a full
log message without any patch, but failed to express that by
passing the "--no-divider" option, which has been corrected.
Will merge to 'master'.
source: <20230617042624.GA562686@coredump.intra.peff.net>
* jk/redact-h2h3-headers-fix (2023-06-17) 1 commit
(merged to 'next' on 2023-06-20 at c1247fd527)
+ http: handle both "h2" and "h2h3" in curl info lines
Curl library recently changed how http2 traces are shown and broke
the code to redact sensitive info header, which has been fixed.
Will merge to 'master'.
source: <20230617051559.GD562686@coredump.intra.peff.net>
* rs/strbuf-expand-step (2023-06-18) 5 commits
- strbuf: simplify strbuf_expand_literal_cb()
- replace strbuf_expand() with strbuf_expand_step()
- replace strbuf_expand_dict_cb() with strbuf_expand_step()
- strbuf: factor out strbuf_expand_step()
- pretty: factor out expand_separator()
Code clean-up around strbuf_expand() API.
source: <767baa64-20a6-daf2-d34b-d81f72363749@web.de>
--------------------------------------------------
[Stalled]
* ed/fsmonitor-windows-named-pipe (2023-03-24) 1 commit
- fsmonitor: handle differences between Windows named pipe functions
Fix fsmonitor on Windows when the filesystem path contains certain
characters.
Expecting a reroll.
cf. <b9cf67e4-22a7-2ff0-8310-9223bea10d6d@jeffhostetler.com>
source: <pull.1503.git.1679678090412.gitgitgadget@gmail.com>
* rn/sparse-diff-index (2023-04-10) 1 commit
- diff-index: enable sparse index
"git diff-index" command has been taught to work better with the
sparse index.
Expecting a reroll.
cf. <62821012-4fc3-5ad8-695c-70f7ab14a8c9@github.com>
source: <20230408112342.404318-1-nanth.raghul@gmail.com>
* es/recurse-submodules-option-is-a-bool (2023-04-10) 1 commit
- usage: clarify --recurse-submodules as a boolean
The "--[no-]recurse-submodules" option of "git checkout" and others
supported an undocumented syntax --recurse-submodules=<value> where
the value can spell a Boolean in various ways. The support for the
syntax is being dropped.
Expecting a reroll.
cf. <ZDSTFwMFO7vbj/du@google.com>
source: <ZDSTFwMFO7vbj/du@google.com>
* cb/checkout-same-branch-twice (2023-03-22) 2 commits
- SQUASH??? the test marked to expect failure passes from day one
- checkout/switch: disallow checking out same branch in multiple worktrees
"git checkout -B $branch" failed to protect against checking out
a branch that is checked out elsewhere, unlike "git branch -f" did.
Expecting a hopefully minor and final reroll.
cf. <CAPUEspj_Bh+LgYLnWfeBdcq_uV5Cbou-7H51GLFjzSa5Qzby9w@mail.gmail.com>
source: <20230120113553.24655-1-carenas@gmail.com>
* tk/pull-conflict-suggest-rebase-merge-not-rebase-true (2023-02-13) 1 commit
- pull: conflict hint pull.rebase suggestion should offer "merges" vs "true"
In an advice message after failed non-ff pull, we used to suggest
setting pull.rebase=true, but these days pull.rebase=merges may be
more inline with the original spirit of "rebuild your side on top
of theirs".
May want to discard.
This is too much of a departure from the existing practice.
cf. <CAMMLpeTPEoKVTbfc17w+Y9qn7jOGmQi_Ux0Y3sFW5QTgGWJ=SA@mail.gmail.com>
cf. <CABPp-BGqAxKnxDRVN4cYMteLp33hvto07R3=TJBT5WubJT4+Og@mail.gmail.com>
source: <pull.1474.git.1675614276549.gitgitgadget@gmail.com>
* ab/tag-object-type-errors (2023-05-10) 4 commits
- tag: don't emit potentially incorrect "object is a X, not a Y"
- tag: don't misreport type of tagged objects in errors
- object tests: add test for unexpected objects in tags
- Merge branch 'jk/parse-object-type-mismatch' into ab/tag-object-type-errors
Hardening checks around mismatched object types when one of those
objects is a tag.
source: <cover-v2-0.3-00000000000-20221230T011725Z-avarab@gmail.com>
* ad/test-record-count-when-harness-is-in-use (2022-12-25) 1 commit
- test-lib: allow storing counts with test harnesses
Allow summary results from tests to be written to t/test-results
directory even when a test harness like 'prove' is in use.
Expecting a reroll.
cf. <CABPp-BGoPuGCZw+9wCgdYyRR4Zf4y9Kun27GrQhtMdYWpOUsYQ@mail.gmail.com>
source: <20221224225200.1027806-1-adam@dinwoodie.org>
* so/diff-merges-more (2022-12-18) 5 commits
- diff-merges: improve --diff-merges documentation
- diff-merges: issue warning on lone '-m' option
- diff-merges: support list of values for --diff-merges
- diff-merges: implement log.diffMerges-m-imply-p config
- diff-merges: implement [no-]hide option and log.diffMergesHide config
Assorted updates to "--diff-merges=X" option.
May want to discard.
Breaking compatibility does not seem worth it.
source: <20221217132955.108542-1-sorganov@gmail.com>
* ab/imap-send-requires-curl (2023-02-02) 6 commits
- imap-send: correctly report "host" when using "tunnel"
- imap-send: remove old --no-curl codepath
- imap-send: make --curl no-optional
- imap-send: replace auto-probe libcurl with hard dependency
- imap-send doc: the imap.sslVerify is used with imap.tunnel
- imap-send: note "auth_method", not "host" on auth method failure
Give a hard dependency on cURL library to build "git imap-send",
and remove the code to interact with IMAP server without using cURL.
Expecting a reroll.
The 'tunnel' part is still iffy.
cf. <230203.86bkmabfjr.gmgdl@evledraar.gmail.com>
source: <cover-v2-0.6-00000000000-20230202T093706Z-avarab@gmail.com>
* cw/submodule-status-in-parallel (2023-03-02) 6 commits
- diff-lib: parallelize run_diff_files for submodules
- diff-lib: refactor out diff_change logic
- submodule: refactor is_submodule_modified()
- submodule: move status parsing into function
- submodule: rename strbuf variable
- run-command: add on_stderr_output_fn to run_processes_parallel_opts
"git submodule status" learned to run the comparison in submodule
repositories in parallel.
Expecting a reroll.
cf. <CAFySSZDk05m6gU5-V1R+y3YnQ5PPduVW54+_gjBwD0rmacsLsw@mail.gmail.com>
cf. <230307.865ybc273g.gmgdl@evledraar.gmail.com>
source: <20230302215237.1473444-1-calvinwan@google.com>
--------------------------------------------------
[Cooking]
* js/doc-unit-tests (2023-06-13) 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>
* jt/doc-use-octal-with-printf (2023-06-14) 1 commit
(merged to 'next' on 2023-06-16 at ebb72b9bd8)
+ CodingGuidelines: use octal escapes, not hex
Suggest to refrain from using hex literals that are non-portable
when writing printf(1) format strings.
Will merge to 'master'.
source: <20230614213145.475607-1-jonathantanmy@google.com>
* js/defeat-ignore-submodules-config-with-explicit-addition (2023-06-14) 1 commit
(merged to 'next' on 2023-06-16 at bfc01bd5e2)
+ diff-lib: honor override_submodule_config flag bit
Even when diff.ignoreSubmodules tells us to ignore submodule
changes, "git commit" with an index that already records changes to
submodules should include the submodule changes in the resulting
commit, but it did not.
Will merge to 'master'.
source: <20230614164856.3186012-2-sokcevic@google.com>
* as/dtype-compilation-fix (2023-06-12) 1 commit
(merged to 'next' on 2023-06-16 at 267100947e)
+ statinfo.h: move DTYPE defines from dir.h
Compilation fix for platforms without D_TYPE in struct dirent.
Will merge to 'master'.
source: <20230606205935.3183276-1-asedeno@google.com>
* la/docs-typofixes (2023-06-12) 1 commit
(merged to 'next' on 2023-06-16 at 56ddcb16e6)
+ docs: typofixes
Typofixes.
Will merge to 'master'.
source: <pull.1542.v2.git.1686166007816.gitgitgadget@gmail.com>
* mh/mingw-case-sensitive-build (2023-06-12) 1 commit
- mingw: use lowercase includes for some Windows headers
Names of MinGW header files are spelled in mixed case in some
source files, but the build host can be using case sensitive
filesystem with header files with their name spelled in all
lowercase.
Needs review.
source: <20230604211934.1365289-1-mh@glandium.org>
* pb/complete-diff-options (2023-06-12) 25 commits
- diff.c: mention completion above add_diff_options
- completion: complete --remerge-diff
- completion: complete --diff-merges, its options and --no-diff-merges
- completion: move --pickaxe-{all,regex} to __git_diff_common_options
- completion: complete --ws-error-highlight
- completion: complete --unified
- completion: complete --patch-with-raw
- completion: complete --output-indicator-{context,new,old}
- completion: complete --output
- completion: complete --no-stat
- completion: complete --no-relative
- completion: complete --line-prefix
- completion: complete --ita-invisible-in-index and --ita-visible-in-index
- completion: complete --irreversible-delete
- completion: complete --ignore-matching-lines
- completion: complete --function-context
- completion: complete --find-renames
- completion: complete --find-object
- completion: complete --find-copies
- completion: complete --default-prefix
- completion: complete --compact-summary
- completion: complete --combined-all-paths
- completion: complete --cc
- completion: complete --break-rewrites
- completion: add comments describing __git_diff_* globals
Completion updates.
Expecting a reroll.
cf. <5cba334c-4d75-0dac-20c6-9e3def1f224a@gmail.com>
source: <pull.1543.v2.git.1686574374.gitgitgadget@gmail.com>
* rj/leakfixes (2023-06-17) 11 commits
(merged to 'next' on 2023-06-17 at 49e54a88e2)
+ tests: mark as passing with SANITIZE=leak
+ config: fix a leak in git_config_copy_or_rename_section_in_file
+ branch: fix a leak in cmd_branch
+ branch: fix a leak in setup_tracking
+ rev-parse: fix a leak with --abbrev-ref
(merged to 'next' on 2023-06-16 at d9330ea230)
+ branch: fix a leak in setup_tracking
+ branch: fix a leak in check_tracking_branch
+ branch: fix a leak in inherit_tracking
+ branch: fix a leak in dwim_and_setup_tracking
+ remote: fix a leak in query_matches_negative_refspec
+ config: fix a leak in git_config_copy_or_rename_section_in_file
Leakfixes
Will merge to 'master'.
source: <71b49be1-eeef-2f60-38f1-cfcbe4c8252f@gmail.com>
source: <9eb931e7-dd18-8ce8-4d4a-cf3d85066189@gmail.com>
* rs/run-command-exec-error-on-noent (2023-06-12) 2 commits
(merged to 'next' on 2023-06-16 at 6bb4425334)
+ run-command: report exec error even on ENOENT
+ t1800: loosen matching of error message for bad shebang
Simplify error message when run-command fails to start a command.
Will merge to 'master'.
source: <14e1be60-3765-0ba5-00f5-2848bb57bf53@web.de>
* sl/worktree-sparse (2023-06-12) 1 commit
(merged to 'next' on 2023-06-16 at ca0a7edcc5)
+ worktree: integrate with sparse-index
"git worktree" learned to work better with sparse index feature.
Will merge to 'master'.
source: <20230606172633.669916-1-cheskaqiqi@gmail.com>
* tb/collect-pack-filenames-fix (2023-06-12) 1 commit
(merged to 'next' on 2023-06-20 at abcc6892c8)
+ builtin/repack.c: only collect fully-formed packs
Avoid breakage of "git pack-objects --cruft" due to inconsistency
between the way the code enumerates packfiles in the repository.
Will merge to 'master'.
source: <20230607101617.ges6tnMry4E52lDGld43QgtNUsIS4YQq6w-t71hEfkQ@z>
* tb/open-midx-bitmap-fallback (2023-06-12) 1 commit
(merged to 'next' on 2023-06-16 at 6546fee734)
+ pack-bitmap.c: gracefully degrade on failure to load MIDX'd pack
Gracefully deal with a stale MIDX file that lists a packfile that
no longer exists.
Will merge to 'master'.
source: <f123b68cb8a277fbf105b1789a84b9405a499b79.1686178854.git.me@ttaylorr.com>
* tz/lib-gpg-prereq-fix (2023-06-12) 1 commit
(merged to 'next' on 2023-06-16 at 19770e970a)
+ t/lib-gpg: require GPGSSH for GPGSSH_VERIFYTIME prereq
Test update.
Will merge to 'master'.
source: <20230606214707.55739-1-tmz@pobox.com>
* ps/cat-file-null-output (2023-06-12) 5 commits
(merged to 'next' on 2023-06-15 at e841ad2be5)
+ cat-file: add option '-Z' that delimits input and output with NUL
+ cat-file: simplify reading from standard input
+ strbuf: provide CRLF-aware helper to read until a specified delimiter
+ t1006: modernize test style to use `test_cmp`
+ t1006: don't strip timestamps from expected results
"git cat-file --batch" and friends learned "-Z" that uses NUL
delimiter for both input and output.
Will merge to 'master'.
source: <cover.1686028409.git.ps@pks.im>
* ds/add-i-color-configuration-fix (2023-06-12) 2 commits
(merged to 'next' on 2023-06-15 at 377b9f9a00)
+ add: test use of brackets when color is disabled
+ add: check color.ui for interactive add
The reimplemented "git add -i" did not honor color.ui configuration.
Will merge to 'master'.
source: <pull.1541.v2.git.1686061219078.gitgitgadget@gmail.com>
* ks/ref-filter-signature (2023-06-06) 2 commits
- ref-filter: add new "signature" atom
- t/lib-gpg: introduce new prereq GPG2
The "git for-each-ref" family of commands learned placeholders
related to GPG signature verification.
Needs review.
source: <20230604185815.15761-1-five231003@gmail.com>
* ds/disable-replace-refs (2023-06-12) 3 commits
(merged to 'next' on 2023-06-15 at e7baddb745)
+ repository: create read_replace_refs setting
+ replace-objects: create wrapper around setting
+ repository: create disable_replace_refs()
(this branch uses tb/pack-bitmap-traversal-with-boundary.)
Introduce a mechanism to disable replace refs globally and per
repository.
Will merge to 'master'.
source: <pull.1537.v3.git.1686057877.gitgitgadget@gmail.com>
* jt/path-filter-fix (2023-06-13) 4 commits
- commit-graph: new filter ver. that fixes murmur3
- repo-settings: introduce commitgraph.changedPathsVersion
- t4216: test changed path filters with high bit paths
- gitformat-commit-graph: describe version 2 of BDAT
The Bloom filter used for path limited history traversal was broken
on systems whose "char" is unsigned; update the implementation and
bump the format version to 2.
Expecting a reroll.
cf. <c7b66d2c-cdc3-1f0f-60a0-a2ee21c277bf@github.com>
source: <cover.1686677910.git.jonathantanmy@google.com>
* tk/cherry-pick-sequence-requires-clean-worktree (2023-06-01) 1 commit
- cherry-pick: refuse cherry-pick sequence if index is dirty
"git cherry-pick A" that replays a single commit stopped before
clobbering local modification, but "git cherry-pick A..B" did not,
which has been corrected.
Expecting a reroll.
cf. <999f12b2-38d6-f446-e763-4985116ad37d@gmail.com>
source: <pull.1535.v2.git.1685264889088.gitgitgadget@gmail.com>
* mh/credential-libsecret-attrs (2023-06-16) 1 commit
- credential/libsecret: store new attributes
The way authentication related data other than passwords (e.g.
oath token and password expiration data) are stored in libsecret
keyrings has been rethought.
Needs review.
source: <pull.1469.v5.git.git.1686945306242.gitgitgadget@gmail.com>
* ja/worktree-orphan (2023-05-17) 8 commits
(merged to 'next' on 2023-06-15 at 16bb552b79)
+ worktree add: emit warn when there is a bad HEAD
+ worktree add: extend DWIM to infer --orphan
+ worktree add: introduce "try --orphan" hint
+ worktree add: add --orphan flag
+ t2400: add tests to verify --quiet
+ t2400: refactor "worktree add" opt exclusion tests
+ t2400: cleanup created worktree in test
+ worktree add: include -B in usage docs
Originally merged to 'next' on 2023-06-14
'git worktree add' learned how to create a worktree based on an
orphaned branch with `--orphan`.
Will merge to 'master'.
source: <20230517214711.12467-1-jacobabel@nullpo.dev>
* tb/refs-exclusion-and-packed-refs (2023-06-20) 16 commits
- ls-refs.c: avoid enumerating hidden refs where possible
- upload-pack.c: avoid enumerating hidden refs where possible
- builtin/receive-pack.c: avoid enumerating hidden references
- refs.h: let `for_each_namespaced_ref()` take excluded patterns
- refs/packed-backend.c: ignore complicated hidden refs rules
- revision.h: store hidden refs in a `strvec`
- refs/packed-backend.c: add trace2 counters for jump list
- refs/packed-backend.c: implement jump lists to avoid excluded pattern(s)
- refs/packed-backend.c: refactor `find_reference_location()`
- refs: plumb `exclude_patterns` argument throughout
- builtin/for-each-ref.c: add `--exclude` option
- ref-filter.c: parameterize match functions over patterns
- ref-filter: add `ref_filter_clear()`
- ref-filter: clear reachable list pointers after freeing
- ref-filter.h: provide `REF_FILTER_INIT`
- refs.c: rename `ref_filter`
Enumerating refs in the packed-refs file, while excluding refs that
match certain patterns, has been optimized.
source: <cover.1687270849.git.me@ttaylorr.com>
* en/header-split-cache-h-part-3 (2023-05-16) 29 commits
(merged to 'next' on 2023-06-15 at 76ebce0b7a)
+ 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
+ Merge branch 'en/header-split-cache-h-part-2' into en/header-split-cache-h-part-3
(this branch is used by js/cmake-wo-cache-h.)
Originally merged to 'next' on 2023-06-13
Header files cleanup.
Will merge to 'master' together with its fix-up in js/cmake-wo-cache-h
source: <pull.1525.v3.git.1684218848.gitgitgadget@gmail.com>
* cc/git-replay (2023-06-03) 15 commits
- replay: stop assuming replayed branches do not diverge
- replay: add --contained to rebase contained branches
- replay: add --advance or 'cherry-pick' mode
- replay: disallow revision specific options and pathspecs
- replay: use standard revision ranges
- replay: make it a minimal server side command
- replay: remove HEAD related sanity check
- replay: remove progress and info output
- replay: add an important FIXME comment about gpg signing
- replay: don't simplify history
- replay: introduce pick_regular_commit()
- replay: die() instead of failing assert()
- replay: start using parse_options API
- replay: introduce new builtin
- t6429: remove switching aspects of fast-rebase
source: <20230602102533.876905-1-christian.couder@gmail.com>
* tb/gc-recent-object-hook (2023-06-12) 2 commits
(merged to 'next' on 2023-06-16 at 2d8d8ea2ef)
+ gc: introduce `gc.recentObjectsHook`
+ reachable.c: extract `obj_is_recent()`
"git pack-objects" learned to invoke a new hook program that
enumerates extra objects to be used as anchoring points to keep
otherwise unreachable objects in cruft packs.
Will merge to 'master'.
source: <cover.1686178684.git.me@ttaylorr.com>
* ob/revert-of-revert (2023-05-05) 1 commit
- sequencer: beautify subject of reverts of reverts
Instead of "Revert "Revert "original"", give "Reapply "original""
as the title for a revert of a revert.
Expecting a hopefully final reroll.
Looking much better, except for minor cosmetic issues.
source: <20230428083528.1699221-1-oswald.buddenhagen@gmx.de>
* cw/strbuf-cleanup (2023-06-12) 7 commits
- strbuf: remove global variable
- path: move related function to path
- object-name: move related functions to object-name
- credential-store: move related functions to credential-store file
- abspath: move related functions to abspath
- strbuf: clarify dependency
- strbuf: clarify API boundary
Move functions that are not about pure string manipulation out of
strbuf.[ch]
source: <20230606194720.2053551-1-calvinwan@google.com>
* tl/notes-separator (2023-04-28) 6 commits
- notes.c: introduce "--[no-]stripspace" option
- notes.c: append separator instead of insert by pos
- notes.c: introduce '--separator=<paragraph-break>' option
- t3321: add test cases about the notes stripspace behavior
- notes.c: use designated initializers for clarity
- notes.c: cleanup 'strbuf_grow' call in 'append_edit'
(this branch is used by jc/notes-separator-fix.)
'git notes append' was taught '--separator' to specify string to insert
between paragraphs.
Expecting a reroll.
cf. <20230615071314.GC1460739@coredump.intra.peff.net>
source: <cover.1682671758.git.dyroneteng@gmail.com>
* jc/notes-separator-fix (2023-06-13) 1 commit
- notes: do not access before the beginning of an array
(this branch uses tl/notes-separator.)
Fix to tl/notes-separator topic.
Will drop once tl/notes-separator gets updated.
source: <cover.1682671758.git.dyroneteng@gmail.com>
* pw/rebase-i-after-failure (2023-04-21) 6 commits
- rebase -i: fix adding failed command to the todo list
- rebase: fix rewritten list for failed pick
- rebase --continue: refuse to commit after failed command
- sequencer: factor out part of pick_commits()
- rebase -i: remove patch file after conflict resolution
- rebase -i: move unlink() calls
Various fixes to the behaviour of "rebase -i" when the command got
interrupted by conflicting changes.
Expecting a reroll.
cf. <xmqqsfcthrpb.fsf@gitster.g>
cf. <1fd54422-b66a-c2e4-7cd7-934ea01190ad@gmail.com>
source: <pull.1492.v2.git.1682089074.gitgitgadget@gmail.com>
* tb/pack-bitmap-traversal-with-boundary (2023-05-08) 3 commits
+ pack-bitmap.c: use commit boundary during bitmap traversal
+ pack-bitmap.c: extract `fill_in_bitmap()`
+ object: add object_array initializer helper function
(this branch is used by ds/disable-replace-refs.)
Originally merged to 'next' on 2023-06-15
The object traversal using reachability bitmap done by
"pack-object" has been tweaked to take advantage of the fact that
using "boundary" commits as representative of all the uninteresting
ones can save quite a lot of object enumeration.
Will merge to 'master'.
source: <cover.1683567065.git.me@ttaylorr.com>
--------------------------------------------------
[Discarded]
* mh/credential-password-expiry-libsecret (2023-05-05) 1 commit
. credential/libsecret: support password_expiry_utc
Originally merged to 'next' on 2023-05-09
The libsecret credential helper learns to handle the password
expiry time information.
Superseded by mh/credential-libsecret-attrs.
cf. <CAGJzqskMwOJkriH6serqdwAVYi+fftEL8ohJd-suP6v+OxB_bg@mail.gmail.com>
source: <pull.1469.v3.git.git.1683270298313.gitgitgadget@gmail.com>
* tb/pack-bitmap-index-seek (2023-03-20) 6 commits
. pack-bitmap.c: factor out `bitmap_index_seek_commit()`
. pack-bitmap.c: use `bitmap_index_seek()` where possible
. pack-bitmap.c: factor out manual `map_pos` manipulation
. pack-bitmap.c: drop unnecessary 'inline's
. pack-bitmap.c: hide bitmap internals in `read_be32()`
. pack-bitmap.c: hide bitmap internals in `read_u8()`
Clean-up the pack-bitmap codepath.
Retracted for now.
cf. <ZJCI6FHtbuOKPlV1@nand.local>
source: <cover.1679342296.git.me@ttaylorr.com>
^ permalink raw reply [flat|nested] 8+ messages in thread
* bug in en/header-split-cache-h-part-3, was Re: What's cooking in git.git (Jun 2023, #05; Tue, 20)
2023-06-21 0:04 What's cooking in git.git (Jun 2023, #05; Tue, 20) Junio C Hamano
@ 2023-06-21 8:55 ` Jeff King
2023-06-21 17:05 ` Junio C Hamano
0 siblings, 1 reply; 8+ messages in thread
From: Jeff King @ 2023-06-21 8:55 UTC (permalink / raw
To: Junio C Hamano; +Cc: Elijah Newren, git
On Tue, Jun 20, 2023 at 05:04:42PM -0700, Junio C Hamano wrote:
> * en/header-split-cache-h-part-3 (2023-05-16) 29 commits
> (merged to 'next' on 2023-06-15 at 76ebce0b7a)
> + 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
> + Merge branch 'en/header-split-cache-h-part-2' into en/header-split-cache-h-part-3
> (this branch is used by js/cmake-wo-cache-h.)
>
> Originally merged to 'next' on 2023-06-13
>
> Header files cleanup.
>
> Will merge to 'master' together with its fix-up in js/cmake-wo-cache-h
> source: <pull.1525.v3.git.1684218848.gitgitgadget@gmail.com>
I think this series has a bug with finding the correct templates dir. If
I check out 76ebce0b7a (Merge branch 'en/header-split-cache-h-part-3'
into next, 2023-06-15) and run:
make prefix=/tmp/foo install && /tmp/foo/bin/git init /tmp/bar
I get:
warning: templates not found in /usr/share/git-core/templates
Whereas if I go to 76ebce0b7a^, that warning does not appear (and
presumably the templates come from /tmp/foo/share, but I didn't check).
Our tests can't notice because they always specify the in-repo template
dir directly in the bin-wrappers script (and other users might not even
get the warning if they have another git installed in /usr; it would
just silently use the wrong template).
Side note: I'm using the merge along 'next' there because much to my
surprise, the tip of en/header-split-cache-h-part-3 does not build by
itself! It needs the evil merge result from ccd12a3d6c (Merge branch
'en/header-split-cache-h-part-2', 2023-05-09). I wonder if it got
applied on the wrong base. It does work when merged to 'next' (and
should with 'master' as well), but it hurts bisectability.
After rebasing on 'master' to make it buildable on its own, I bisected
my make/init command above, which shows the template problem appearing
in 3f85c72fad (setup: adopt shared init-db & clone code, 2023-05-16).
I guess the problem is the movement of the code from init-db.c to
setup.c, and we'd want something like this:
diff --git a/Makefile b/Makefile
index e440728c24..b09c8165d0 100644
--- a/Makefile
+++ b/Makefile
@@ -2742,8 +2742,8 @@ exec-cmd.sp exec-cmd.s exec-cmd.o: EXTRA_CPPFLAGS = \
'-DBINDIR="$(bindir_relative_SQ)"' \
'-DFALLBACK_RUNTIME_PREFIX="$(prefix_SQ)"'
-builtin/init-db.sp builtin/init-db.s builtin/init-db.o: GIT-PREFIX
-builtin/init-db.sp builtin/init-db.s builtin/init-db.o: EXTRA_CPPFLAGS = \
+setup.sp setup.s setup.o: GIT-PREFIX
+setup.sp setup.s setup.o: EXTRA_CPPFLAGS = \
-DDEFAULT_GIT_TEMPLATE_DIR='"$(template_dir_SQ)"'
config.sp config.s config.o: GIT-PREFIX
It does make me wonder if we should also just do this:
diff --git a/setup.c b/setup.c
index f8e1296766..6e7282e680 100644
--- a/setup.c
+++ b/setup.c
@@ -1718,10 +1718,6 @@ int daemonize(void)
#endif
}
-#ifndef DEFAULT_GIT_TEMPLATE_DIR
-#define DEFAULT_GIT_TEMPLATE_DIR "/usr/share/git-core/templates"
-#endif
-
#ifdef NO_TRUSTABLE_FILEMODE
#define TEST_FILEMODE 0
#else
Since the Makefile always provides that value, having a baked-in
fallback does not make much sense. And in this case it masked a real bug
which should have been a compilation error, and instead silently used
the wrong value.
So I think we'd at least want to fix the Makefile before graduating this
topic any further. But IMHO it would also be worth adjusting the topic's
start point so that we don't have a big chunk of commits which fail to
build in the final history.
-Peff
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: bug in en/header-split-cache-h-part-3, was Re: What's cooking in git.git (Jun 2023, #05; Tue, 20)
2023-06-21 8:55 ` bug in en/header-split-cache-h-part-3, was " Jeff King
@ 2023-06-21 17:05 ` Junio C Hamano
2023-06-21 20:26 ` Jeff King
0 siblings, 1 reply; 8+ messages in thread
From: Junio C Hamano @ 2023-06-21 17:05 UTC (permalink / raw
To: Jeff King; +Cc: Elijah Newren, git
Jeff King <peff@peff.net> writes:
> On Tue, Jun 20, 2023 at 05:04:42PM -0700, Junio C Hamano wrote:
>
>> * en/header-split-cache-h-part-3 (2023-05-16) 29 commits
>> ...
>> Will merge to 'master' together with its fix-up in js/cmake-wo-cache-h
>> source: <pull.1525.v3.git.1684218848.gitgitgadget@gmail.com>
>
> I think this series has a bug with finding the correct templates dir. If
> I check out 76ebce0b7a (Merge branch 'en/header-split-cache-h-part-3'
> into next, 2023-06-15) and run:
>
> make prefix=/tmp/foo install && /tmp/foo/bin/git init /tmp/bar
>
> I get:
>
> warning: templates not found in /usr/share/git-core/templates
>
> Whereas if I go to 76ebce0b7a^, that warning does not appear (and
> presumably the templates come from /tmp/foo/share, but I didn't check).
Ouch.
> Our tests can't notice because they always specify the in-repo template
> dir directly in the bin-wrappers script (and other users might not even
> get the warning if they have another git installed in /usr; it would
> just silently use the wrong template).
>
> Side note: I'm using the merge along 'next' there because much to my
> surprise, the tip of en/header-split-cache-h-part-3 does not build by
> itself! It needs the evil merge result from ccd12a3d6c (Merge branch
> 'en/header-split-cache-h-part-2', 2023-05-09). I wonder if it got
> applied on the wrong base. It does work when merged to 'next' (and
> should with 'master' as well), but it hurts bisectability.
>
> After rebasing on 'master' to make it buildable on its own, I bisected
> my make/init command above, which shows the template problem appearing
> in 3f85c72fad (setup: adopt shared init-db & clone code, 2023-05-16).
>
> I guess the problem is the movement of the code from init-db.c to
> setup.c, and we'd want something like this:
>
> diff --git a/Makefile b/Makefile
> index e440728c24..b09c8165d0 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -2742,8 +2742,8 @@ exec-cmd.sp exec-cmd.s exec-cmd.o: EXTRA_CPPFLAGS = \
> '-DBINDIR="$(bindir_relative_SQ)"' \
> '-DFALLBACK_RUNTIME_PREFIX="$(prefix_SQ)"'
>
> -builtin/init-db.sp builtin/init-db.s builtin/init-db.o: GIT-PREFIX
> -builtin/init-db.sp builtin/init-db.s builtin/init-db.o: EXTRA_CPPFLAGS = \
> +setup.sp setup.s setup.o: GIT-PREFIX
> +setup.sp setup.s setup.o: EXTRA_CPPFLAGS = \
> -DDEFAULT_GIT_TEMPLATE_DIR='"$(template_dir_SQ)"'
>
> config.sp config.s config.o: GIT-PREFIX
>
>
> It does make me wonder if we should also just do this:
>
> diff --git a/setup.c b/setup.c
> index f8e1296766..6e7282e680 100644
> --- a/setup.c
> +++ b/setup.c
> @@ -1718,10 +1718,6 @@ int daemonize(void)
> #endif
> }
>
> -#ifndef DEFAULT_GIT_TEMPLATE_DIR
> -#define DEFAULT_GIT_TEMPLATE_DIR "/usr/share/git-core/templates"
> -#endif
> -
> #ifdef NO_TRUSTABLE_FILEMODE
> #define TEST_FILEMODE 0
> #else
>
> Since the Makefile always provides that value, having a baked-in
> fallback does not make much sense. And in this case it masked a real bug
> which should have been a compilation error, and instead silently used
> the wrong value.
>
> So I think we'd at least want to fix the Makefile before graduating this
> topic any further. But IMHO it would also be worth adjusting the topic's
> start point so that we don't have a big chunk of commits which fail to
> build in the final history.
Hmph, meaning (1) revert the merge of the topic to 'next', (2)
rebase the topic on top of the current 'master', instead of 4bd872e0,
which was a merge of the prerequisite series into then-current
master, (3) apply the Makefile (plus setup.c) fix, and then (4)
merge the result back to 'next'?
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: bug in en/header-split-cache-h-part-3, was Re: What's cooking in git.git (Jun 2023, #05; Tue, 20)
2023-06-21 17:05 ` Junio C Hamano
@ 2023-06-21 20:26 ` Jeff King
2023-06-21 22:03 ` Junio C Hamano
0 siblings, 1 reply; 8+ messages in thread
From: Jeff King @ 2023-06-21 20:26 UTC (permalink / raw
To: Junio C Hamano; +Cc: Elijah Newren, git
On Wed, Jun 21, 2023 at 10:05:40AM -0700, Junio C Hamano wrote:
> > So I think we'd at least want to fix the Makefile before graduating this
> > topic any further. But IMHO it would also be worth adjusting the topic's
> > start point so that we don't have a big chunk of commits which fail to
> > build in the final history.
>
> Hmph, meaning (1) revert the merge of the topic to 'next', (2)
> rebase the topic on top of the current 'master', instead of 4bd872e0,
> which was a merge of the prerequisite series into then-current
> master, (3) apply the Makefile (plus setup.c) fix, and then (4)
> merge the result back to 'next'?
Yeah. I guess the real build problem is actually in the merge of split-2
(it conflicted with a simultaneous topic, hence the fix coming in the
merge). So another option to address that here would be to amend the
4bd872e0ed (Merge branch 'en/header-split-cache-h-part-2' into
en/header-split-cache-h-part-3, 2023-05-08) to include that fixup.
As for the others, I'd consider:
1. (optional) Drop the #ifndef at the very start of the series, before
we touch anything, with the rationale that it is not doing anything
and masks errors. I don't _think_ this can ever backfire, because
we unconditionally set DEFAULT_GIT_TEMPLATE_DIR (unlike some other
things like DEFAULT_PAGER, where the Makefile might leave it
unset). But we can also leave this out, or do it as a separate
topic, if we want to minimize changes / risk of screwing something
up.
2. Squash the Makefile fix into the "adopt shared init-db" patch
(currently 0d652b238).
And that would leave the result fully bisectable. But if we prefer to
keep the history closer to reality, I can prepare the Makefile thing as
a patch on top.
-Peff
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: bug in en/header-split-cache-h-part-3, was Re: What's cooking in git.git (Jun 2023, #05; Tue, 20)
2023-06-21 20:26 ` Jeff King
@ 2023-06-21 22:03 ` Junio C Hamano
2023-06-23 6:33 ` Elijah Newren
0 siblings, 1 reply; 8+ messages in thread
From: Junio C Hamano @ 2023-06-21 22:03 UTC (permalink / raw
To: Jeff King; +Cc: Elijah Newren, git
Jeff King <peff@peff.net> writes:
> Yeah. I guess the real build problem is actually in the merge of split-2
> (it conflicted with a simultaneous topic, hence the fix coming in the
> merge). So another option to address that here would be to amend the
> 4bd872e0ed (Merge branch 'en/header-split-cache-h-part-2' into
> en/header-split-cache-h-part-3, 2023-05-08) to include that fixup.
>
> As for the others, I'd consider:
>
> 1. (optional) Drop the #ifndef at the very start of the series, before
> we touch anything, with the rationale that it is not doing anything
> and masks errors. I don't _think_ this can ever backfire, because
> we unconditionally set DEFAULT_GIT_TEMPLATE_DIR (unlike some other
> things like DEFAULT_PAGER, where the Makefile might leave it
> unset). But we can also leave this out, or do it as a separate
> topic, if we want to minimize changes / risk of screwing something
> up.
>
> 2. Squash the Makefile fix into the "adopt shared init-db" patch
> (currently 0d652b238).
>
> And that would leave the result fully bisectable. But if we prefer to
> keep the history closer to reality, I can prepare the Makefile thing as
> a patch on top.
I've done the messiest, I guess ;-)
* revert merge of the part-3 topic and Dscho's cmake fix out of 'next'.
* rebase part-3 on top of the more recent 'master'.
* squash in the two hunks (including setup.c change) from you into
the "setup: adopt shared init-db & clone code" step.
* squash in Dscho's cmake fix into "cache.h: remove this
no-longer-used header" step.
The result is not in 'next' yet until I hear something from those
who have been involved in the topic, including Elijah and Dscho.
Thanks.
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: bug in en/header-split-cache-h-part-3, was Re: What's cooking in git.git (Jun 2023, #05; Tue, 20)
2023-06-21 22:03 ` Junio C Hamano
@ 2023-06-23 6:33 ` Elijah Newren
2023-06-23 16:38 ` Junio C Hamano
0 siblings, 1 reply; 8+ messages in thread
From: Elijah Newren @ 2023-06-23 6:33 UTC (permalink / raw
To: Junio C Hamano; +Cc: Jeff King, git
On Wed, Jun 21, 2023 at 3:03 PM Junio C Hamano <gitster@pobox.com> wrote:
>
> Jeff King <peff@peff.net> writes:
>
> > Yeah. I guess the real build problem is actually in the merge of split-2
> > (it conflicted with a simultaneous topic, hence the fix coming in the
> > merge). So another option to address that here would be to amend the
> > 4bd872e0ed (Merge branch 'en/header-split-cache-h-part-2' into
> > en/header-split-cache-h-part-3, 2023-05-08) to include that fixup.
> >
> > As for the others, I'd consider:
> >
> > 1. (optional) Drop the #ifndef at the very start of the series, before
> > we touch anything, with the rationale that it is not doing anything
> > and masks errors. I don't _think_ this can ever backfire, because
> > we unconditionally set DEFAULT_GIT_TEMPLATE_DIR (unlike some other
> > things like DEFAULT_PAGER, where the Makefile might leave it
> > unset). But we can also leave this out, or do it as a separate
> > topic, if we want to minimize changes / risk of screwing something
> > up.
> >
> > 2. Squash the Makefile fix into the "adopt shared init-db" patch
> > (currently 0d652b238).
> >
> > And that would leave the result fully bisectable. But if we prefer to
> > keep the history closer to reality, I can prepare the Makefile thing as
> > a patch on top.
>
> I've done the messiest, I guess ;-)
>
> * revert merge of the part-3 topic and Dscho's cmake fix out of 'next'.
>
> * rebase part-3 on top of the more recent 'master'.
>
> * squash in the two hunks (including setup.c change) from you into
> the "setup: adopt shared init-db & clone code" step.
>
> * squash in Dscho's cmake fix into "cache.h: remove this
> no-longer-used header" step.
>
> The result is not in 'next' yet until I hear something from those
> who have been involved in the topic, including Elijah and Dscho.
I did a range-diff to compare my original series to your newly rebased
one, and read through all the differences (including Dscho's and
Peff's suggested changes, as well as the various slight adjustments
due to rebasing). I also rebuilt every patch in your rebase of the
series to ensure they all build, and ran all tests on a couple of the
patches to verify the pass (I didn't run all tests for each patch,
because I did that for my original series and the differences in the
range-diff suggested I only needed to spot check all the tests).
Anyway, your rebase of en/header-split-cache-h-part-3, including
Dscho's and Peff's changes, all look good to me.
Thanks everyone!
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: bug in en/header-split-cache-h-part-3, was Re: What's cooking in git.git (Jun 2023, #05; Tue, 20)
2023-06-23 6:33 ` Elijah Newren
@ 2023-06-23 16:38 ` Junio C Hamano
2023-06-24 1:25 ` Jeff King
0 siblings, 1 reply; 8+ messages in thread
From: Junio C Hamano @ 2023-06-23 16:38 UTC (permalink / raw
To: Elijah Newren; +Cc: Jeff King, git
Elijah Newren <newren@gmail.com> writes:
> Anyway, your rebase of en/header-split-cache-h-part-3, including
> Dscho's and Peff's changes, all look good to me.
>
> Thanks everyone!
Thanks for confirming. Let's merge it down to 'next' then.
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: bug in en/header-split-cache-h-part-3, was Re: What's cooking in git.git (Jun 2023, #05; Tue, 20)
2023-06-23 16:38 ` Junio C Hamano
@ 2023-06-24 1:25 ` Jeff King
0 siblings, 0 replies; 8+ messages in thread
From: Jeff King @ 2023-06-24 1:25 UTC (permalink / raw
To: Junio C Hamano; +Cc: Elijah Newren, git
On Fri, Jun 23, 2023 at 09:38:07AM -0700, Junio C Hamano wrote:
> Elijah Newren <newren@gmail.com> writes:
>
> > Anyway, your rebase of en/header-split-cache-h-part-3, including
> > Dscho's and Peff's changes, all look good to me.
> >
> > Thanks everyone!
>
> Thanks for confirming. Let's merge it down to 'next' then.
I did a range-diff between en/header-split-cache-h-part-3{1} and the
current tip, and the changes look good to me. It might have been worth
mentioning the #ifndef changes in the commit message for e8cf8ef507
(setup: adopt shared init-db & clone code, 2023-05-16). But I see you
merged to 'next', and I don't think it's worth going back to change it
again.
It's nice when we can get everything in there, but I find that it's not
uncommon to sometimes dig extra details out of the list archive if I
don't understand part of a change when doing archaeology.
-Peff
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2023-06-24 1:25 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-06-21 0:04 What's cooking in git.git (Jun 2023, #05; Tue, 20) Junio C Hamano
2023-06-21 8:55 ` bug in en/header-split-cache-h-part-3, was " Jeff King
2023-06-21 17:05 ` Junio C Hamano
2023-06-21 20:26 ` Jeff King
2023-06-21 22:03 ` Junio C Hamano
2023-06-23 6:33 ` Elijah Newren
2023-06-23 16:38 ` Junio C Hamano
2023-06-24 1:25 ` Jeff King
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).