From: Phillip Wood <phillip.wood123@gmail.com>
To: Glen Choo via GitGitGadget <gitgitgadget@gmail.com>, git@vger.kernel.org
Cc: "Jonathan Tan" <jonathantanmy@google.com>,
"Ævar Arnfjörð Bjarmason" <avarab@gmail.com>,
"Emily Shaffer" <nasamuffin@google.com>,
"Jeff King" <peff@peff.net>, "Glen Choo" <chooglen@google.com>
Subject: Re: [PATCH v5 00/11] config: remove global state from config iteration
Date: Tue, 11 Jul 2023 19:41:28 +0100 [thread overview]
Message-ID: <00194850-b881-2b01-9b9b-fc7d7422e804@gmail.com> (raw)
In-Reply-To: <pull.1497.v5.git.git.1687980390.gitgitgadget@gmail.com>
Hi Glen
Sorry it has taken me so long to look at these patches. I see they're in
master now but I have just had a quick look through anyway and they
looked good. It would definitely be nice to fix the hacky handling of
"git checkout --conflict" at some point but that is a long standing
problem and not the fault of this series.
Best Wishes
Phillip
On 28/06/2023 20:26, Glen Choo via GitGitGadget wrote:
> As promised, this version addresses the comments on v3.
>
> = Changes since v4
>
> - Squash 6-7/12 since `test_must_fail` doesn't catch BUG()
> - Move a hunk to later in the series where it belongs
> - Replace a memcpy with `*a = *b`
>
> = Changes since v3
>
> - Rebase onto newer 'master'
> - Move the 'remove UNUSED from tr2_cfg_cb' hunk from 9/12 -> 8/12. It should
> have been there all along; v3 8/12 didn't build at all.
>
>
> Glen Choo (11):
> config: inline git_color_default_config
> urlmatch.h: use config_fn_t type
> config: add ctx arg to config_fn_t
> config.c: pass ctx in configsets
> config: pass ctx with config files
> config.c: pass ctx with CLI config
> trace2: plumb config kvi
> config: pass kvi to die_bad_number()
> config.c: remove config_reader from configsets
> config: add kvi.path, use it to evaluate includes
> config: pass source to config_parser_event_fn_t
>
> alias.c | 3 +-
> archive-tar.c | 5 +-
> archive-zip.c | 1 +
> builtin/add.c | 8 +-
> builtin/blame.c | 5 +-
> builtin/branch.c | 8 +-
> builtin/cat-file.c | 5 +-
> builtin/checkout.c | 12 +-
> builtin/clean.c | 9 +-
> builtin/clone.c | 11 +-
> builtin/column.c | 3 +-
> builtin/commit-graph.c | 3 +-
> builtin/commit.c | 20 +-
> builtin/config.c | 72 ++-
> builtin/difftool.c | 5 +-
> builtin/fetch.c | 13 +-
> builtin/fsmonitor--daemon.c | 11 +-
> builtin/grep.c | 12 +-
> builtin/help.c | 5 +-
> builtin/index-pack.c | 9 +-
> builtin/log.c | 12 +-
> builtin/merge.c | 7 +-
> builtin/multi-pack-index.c | 1 +
> builtin/pack-objects.c | 19 +-
> builtin/patch-id.c | 5 +-
> builtin/pull.c | 5 +-
> builtin/push.c | 5 +-
> builtin/read-tree.c | 5 +-
> builtin/rebase.c | 5 +-
> builtin/receive-pack.c | 15 +-
> builtin/reflog.c | 7 +-
> builtin/remote.c | 15 +-
> builtin/repack.c | 5 +-
> builtin/reset.c | 5 +-
> builtin/send-pack.c | 5 +-
> builtin/show-branch.c | 8 +-
> builtin/stash.c | 5 +-
> builtin/submodule--helper.c | 3 +-
> builtin/tag.c | 9 +-
> builtin/var.c | 5 +-
> builtin/worktree.c | 5 +-
> bundle-uri.c | 9 +-
> color.c | 8 -
> color.h | 6 +-
> compat/mingw.c | 3 +-
> compat/mingw.h | 4 +-
> config.c | 552 +++++++-----------
> config.h | 80 ++-
> connect.c | 4 +-
> .../coccinelle/config_fn_ctx.pending.cocci | 144 +++++
> contrib/coccinelle/git_config_number.cocci | 27 +
> convert.c | 4 +-
> credential.c | 1 +
> delta-islands.c | 4 +-
> diff.c | 19 +-
> diff.h | 7 +-
> fetch-pack.c | 5 +-
> fmt-merge-msg.c | 7 +-
> fmt-merge-msg.h | 3 +-
> fsck.c | 12 +-
> fsck.h | 4 +-
> git-compat-util.h | 2 +
> gpg-interface.c | 7 +-
> grep.c | 7 +-
> grep.h | 4 +-
> help.c | 9 +-
> http.c | 15 +-
> ident.c | 4 +-
> ident.h | 4 +-
> imap-send.c | 7 +-
> ll-merge.c | 1 +
> ls-refs.c | 1 +
> mailinfo.c | 5 +-
> notes-utils.c | 4 +-
> notes.c | 4 +-
> pager.c | 5 +-
> pretty.c | 1 +
> promisor-remote.c | 4 +-
> remote.c | 8 +-
> revision.c | 4 +-
> scalar.c | 4 +-
> sequencer.c | 29 +-
> setup.c | 18 +-
> submodule-config.c | 31 +-
> submodule-config.h | 3 +-
> t/helper/test-config.c | 24 +-
> t/helper/test-userdiff.c | 4 +-
> t/t1300-config.sh | 27 +
> trace2.c | 4 +-
> trace2.h | 3 +-
> trace2/tr2_cfg.c | 16 +-
> trace2/tr2_sysenv.c | 3 +-
> trace2/tr2_tgt.h | 4 +-
> trace2/tr2_tgt_event.c | 4 +-
> trace2/tr2_tgt_normal.c | 4 +-
> trace2/tr2_tgt_perf.c | 4 +-
> trailer.c | 2 +
> upload-pack.c | 18 +-
> urlmatch.c | 7 +-
> urlmatch.h | 8 +-
> worktree.c | 2 +-
> xdiff-interface.c | 5 +-
> xdiff-interface.h | 4 +-
> 103 files changed, 960 insertions(+), 638 deletions(-)
> create mode 100644 contrib/coccinelle/config_fn_ctx.pending.cocci
> create mode 100644 contrib/coccinelle/git_config_number.cocci
>
>
> base-commit: 6ff334181cfb6485d3ba50843038209a2a253907
> Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-1497%2Fchooglen%2Fconfig%2Fno-global-v5
> Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-1497/chooglen/config/no-global-v5
> Pull-Request: https://github.com/git/git/pull/1497
>
> Range-diff vs v4:
>
> 1: 7bfffb454c5 = 1: 7bfffb454c5 config: inline git_color_default_config
> 2: 739c519ce62 = 2: 739c519ce62 urlmatch.h: use config_fn_t type
> 3: a9a0a50f32a = 3: a9a0a50f32a config: add ctx arg to config_fn_t
> 4: 39b2e291f86 = 4: 39b2e291f86 config.c: pass ctx in configsets
> 5: bfc6d2833c5 = 5: bfc6d2833c5 config: pass ctx with config files
> 6: 897bdc759b5 < -: ----------- builtin/config.c: test misuse of format_config()
> 7: 33e4437737d ! 6: 7b24eefbcf3 config.c: pass ctx with CLI config
> @@ Commit message
> * git_config_parse_parameter() hasn't been setting config source
> information, so plumb "kvi" there too.
>
> - * "git config --get-urlmatch --show-scope" iterates config to collect
> - values, but then attempts to display the scope after config iteration.
> - Fix this by copying the "kvi" value in the collection phase so that it
> - can be read back later. This means that we can now support "git config
> - --get-urlmatch --show-origin" (we don't allow this combination of args
> - because of this bug), but that is left unchanged for now.
> + * Several sites in builtin/config.c have been calling current_config_*()
> + functions outside of config callbacks (indirectly, via the
> + format_config() helper), which means they're reading state that isn't
> + set correctly:
>
> - * "git config --default" doesn't have config source metadata when
> - displaying the default value. Fix this by treating the default value
> - as if it came from the command line (e.g. like we do with "git -c" or
> - "git config --file"), using kvi_from_param().
> + * "git config --get-urlmatch --show-scope" iterates config to collect
> + values, but then attempts to display the scope after config
> + iteration, causing the "unknown" scope to be shown instead of the
> + config file's scope. It's clear that this wasn't intended: we knew
> + that "--get-urlmatch" couldn't show config source metadata, which is
> + why "--show-origin" was marked incompatible with "--get-urlmatch"
> + when it was introduced [1]. It was most likely a mistake that we
> + allowed "--show-scope" to sneak through.
> +
> + Fix this by copying the "kvi" value in the collection phase so that
> + it can be read back later. This means that we can now support "git
> + config --get-urlmatch --show-origin", but that is left unchanged
> + for now.
> +
> + * "git config --default" doesn't have config source metadata when
> + displaying the default value, so "--show-scope" also results in
> + "unknown", and "--show-origin" results in a BUG(). Fix this by
> + treating the default value as if it came from the command line (e.g.
> + like we do with "git -c" or "git config --file"), using
> + kvi_from_param().
> +
> + [1] https://lore.kernel.org/git/20160205112001.GA13397@sigill.intra.peff.net/
>
> Signed-off-by: Glen Choo <chooglen@google.com>
>
> @@ config.c: static int configset_find_element(struct config_set *set, const char *
> const char *value)
> {
> @@ config.c: static int configset_add_value(struct config_reader *reader,
> - l_item->e = e;
> - l_item->value_index = e->value_list.nr - 1;
> -
> -- if (!reader->source)
> -- BUG("configset_add_value has no source");
> + if (!reader->source)
> + BUG("configset_add_value has no source");
> if (reader->source->name) {
> - kvi_from_source(reader->source, current_config_scope(), kv_info);
> + kvi_from_source(reader->source, kvi_p->scope, kv_info);
> @@ config.h: void git_global_config(char **user, char **xdg);
> * Match and parse a config key of the form:
>
> ## t/t1300-config.sh ##
> -@@ t/t1300-config.sh: test_expect_success 'urlmatch with --show-scope' '
> - EOF
> +@@ t/t1300-config.sh: test_expect_success 'urlmatch' '
> + test_cmp expect actual
> + '
>
> - cat >expect <<-EOF &&
> -- unknown http.cookiefile /tmp/cookie.txt
> -- unknown http.sslverify false
> ++test_expect_success 'urlmatch with --show-scope' '
> ++ cat >.git/config <<-\EOF &&
> ++ [http "https://weak.example.com"]
> ++ sslVerify = false
> ++ cookieFile = /tmp/cookie.txt
> ++ EOF
> ++
> ++ cat >expect <<-EOF &&
> + local http.cookiefile /tmp/cookie.txt
> + local http.sslverify false
> - EOF
> - git config --get-urlmatch --show-scope HTTP https://weak.example.com >actual &&
> - test_cmp expect actual
> ++ EOF
> ++ git config --get-urlmatch --show-scope HTTP https://weak.example.com >actual &&
> ++ test_cmp expect actual
> ++'
> ++
> + test_expect_success 'urlmatch favors more specific URLs' '
> + cat >.git/config <<-\EOF &&
> + [http "https://example.com/"]
> @@ t/t1300-config.sh: test_expect_success '--show-origin blob ref' '
> + test_cmp expect output
> '
>
> - test_expect_success '--show-origin with --default' '
> -- test_must_fail git config --show-origin --default foo some.key
> ++test_expect_success '--show-origin with --default' '
> + git config --show-origin --default foo some.key >actual &&
> + echo "command line: foo" >expect &&
> + test_cmp expect actual
> - '
> -
> ++'
> ++
> test_expect_success '--show-scope with --list' '
> + cat >expect <<-EOF &&
> + global user.global=true
> @@ t/t1300-config.sh: test_expect_success '--show-scope with --show-origin' '
> -
> - test_expect_success '--show-scope with --default' '
> - git config --show-scope --default foo some.key >actual &&
> -- echo "unknown foo" >expect &&
> -+ echo "command foo" >expect &&
> - test_cmp expect actual
> + test_cmp expect output
> '
>
> ++test_expect_success '--show-scope with --default' '
> ++ git config --show-scope --default foo some.key >actual &&
> ++ echo "command foo" >expect &&
> ++ test_cmp expect actual
> ++'
> ++
> + test_expect_success 'override global and system config' '
> + test_when_finished rm -f \"\$HOME\"/.gitconfig &&
> + cat >"$HOME"/.gitconfig <<-EOF &&
> 8: 9bd5f60282c = 7: 7d64dcbdade trace2: plumb config kvi
> 9: 114723ee4a7 = 8: 9e71c10ca0a config: pass kvi to die_bad_number()
> 10: 807057b6d7f ! 9: 4776600e790 config.c: remove config_reader from configsets
> @@ config.c: static int configset_add_value(const struct key_value_info *kvi_p,
> l_item->e = e;
> l_item->value_index = e->value_list.nr - 1;
>
> +- if (!reader->source)
> +- BUG("configset_add_value has no source");
> - if (reader->source->name) {
> - kvi_from_source(reader->source, kvi_p->scope, kv_info);
> - } else {
> - kvi_from_param(kv_info);
> - }
> -+ memcpy(kv_info, kvi_p, sizeof(struct key_value_info));
> ++ *kv_info = *kvi_p;
> si->util = kv_info;
>
> return 0;
> 11: 3f0f84df972 = 10: 2b33977aba6 config: add kvi.path, use it to evaluate includes
> 12: fe2f154fe8b = 11: 8347d3c9b80 config: pass source to config_parser_event_fn_t
>
prev parent reply other threads:[~2023-07-11 18:41 UTC|newest]
Thread overview: 115+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-04-21 19:13 [PATCH 00/14] [RFC] config: remove global state from config iteration Glen Choo via GitGitGadget
2023-04-21 19:13 ` [PATCH 01/14] config.c: introduce kvi_fn(), use it for configsets Glen Choo via GitGitGadget
2023-04-21 19:13 ` [PATCH 02/14] config.c: use kvi for CLI config Glen Choo via GitGitGadget
2023-05-01 11:06 ` Ævar Arnfjörð Bjarmason
2023-04-21 19:13 ` [PATCH 03/14] config: use kvi for config files Glen Choo via GitGitGadget
2023-04-21 19:13 ` [PATCH 04/14] config: add kvi.path, use it to evaluate includes Glen Choo via GitGitGadget
2023-04-21 19:13 ` [PATCH 05/14] config: pass source to config_parser_event_fn_t Glen Choo via GitGitGadget
2023-04-21 19:13 ` [PATCH 06/14] config: inline git_color_default_config Glen Choo via GitGitGadget
2023-04-21 19:13 ` [PATCH 07/14] urlmatch.h: use config_fn_t type Glen Choo via GitGitGadget
2023-04-21 19:13 ` [PATCH 08/14] (RFC-only) config: add kvi arg to config_fn_t Glen Choo via GitGitGadget
2023-04-21 19:13 ` [PATCH 09/14] (RFC-only) config: apply cocci to config_fn_t implementations Glen Choo via GitGitGadget
2023-04-21 19:13 ` [PATCH 10/14] (RFC-only) config: finish config_fn_t refactor Glen Choo via GitGitGadget
2023-05-01 11:19 ` Ævar Arnfjörð Bjarmason
2023-05-05 21:07 ` Jonathan Tan
2023-05-09 22:46 ` Glen Choo
2023-05-11 16:21 ` Jonathan Tan
2023-05-08 21:00 ` Glen Choo
2023-04-21 19:13 ` [PATCH 11/14] config: remove current_config_(line|name|origin_type) Glen Choo via GitGitGadget
2023-04-21 19:13 ` [PATCH 12/14] config: remove current_config_scope() Glen Choo via GitGitGadget
2023-04-21 19:13 ` [PATCH 13/14] config: pass kvi to die_bad_number() Glen Choo via GitGitGadget
2023-04-21 19:13 ` [PATCH 14/14] config: remove config_reader from configset_add_value Glen Choo via GitGitGadget
2023-05-30 18:41 ` [PATCH v2 00/14] [RFC] config: remove global state from config iteration Glen Choo via GitGitGadget
2023-05-30 18:41 ` [PATCH v2 01/14] config: inline git_color_default_config Glen Choo via GitGitGadget
2023-05-30 18:42 ` [PATCH v2 02/14] urlmatch.h: use config_fn_t type Glen Choo via GitGitGadget
2023-05-30 18:42 ` [PATCH v2 03/14] (RFC-only) config: add kvi arg to config_fn_t Glen Choo via GitGitGadget
2023-06-01 9:50 ` Phillip Wood
2023-06-01 16:22 ` Glen Choo
2023-06-02 9:54 ` Phillip Wood
2023-06-02 16:46 ` Glen Choo
2023-06-05 9:38 ` Phillip Wood
2023-06-09 23:19 ` Glen Choo
2023-05-30 18:42 ` [PATCH v2 04/14] (RFC-only) config: apply cocci to config_fn_t implementations Glen Choo via GitGitGadget
2023-05-30 18:42 ` [PATCH v2 05/14] (RFC-only) config: finish config_fn_t refactor Glen Choo via GitGitGadget
2023-06-01 22:17 ` Jonathan Tan
2023-05-30 18:42 ` [PATCH v2 06/14] config.c: pass kvi in configsets Glen Choo via GitGitGadget
2023-06-01 22:21 ` Jonathan Tan
2023-05-30 18:42 ` [PATCH v2 07/14] config: provide kvi with config files Glen Choo via GitGitGadget
2023-06-01 22:41 ` Jonathan Tan
2023-06-01 23:54 ` Jonathan Tan
2023-05-30 18:42 ` [PATCH v2 08/14] builtin/config.c: test misuse of format_config() Glen Choo via GitGitGadget
2023-05-30 18:42 ` [PATCH v2 09/14] config.c: provide kvi with CLI config Glen Choo via GitGitGadget
2023-06-01 23:35 ` Jonathan Tan
2023-06-02 17:26 ` Glen Choo
2023-05-30 18:42 ` [PATCH v2 10/14] trace2: plumb config kvi Glen Choo via GitGitGadget
2023-06-01 23:38 ` Jonathan Tan
2023-05-30 18:42 ` [PATCH v2 11/14] config: pass kvi to die_bad_number() Glen Choo via GitGitGadget
2023-06-01 23:48 ` Jonathan Tan
2023-06-02 17:23 ` Glen Choo
2023-05-30 18:42 ` [PATCH v2 12/14] config.c: remove config_reader from configsets Glen Choo via GitGitGadget
2023-05-30 18:42 ` [PATCH v2 13/14] config: add kvi.path, use it to evaluate includes Glen Choo via GitGitGadget
2023-06-02 0:06 ` Jonathan Tan
2023-05-30 18:42 ` [PATCH v2 14/14] config: pass source to config_parser_event_fn_t Glen Choo via GitGitGadget
2023-06-02 0:08 ` Jonathan Tan
2023-06-02 17:20 ` Glen Choo
2023-06-20 19:43 ` [PATCH v3 00/12] config: remove global state from config iteration Glen Choo via GitGitGadget
2023-06-20 19:43 ` [PATCH v3 01/12] config: inline git_color_default_config Glen Choo via GitGitGadget
2023-06-20 21:01 ` Junio C Hamano
2023-06-20 19:43 ` [PATCH v3 02/12] urlmatch.h: use config_fn_t type Glen Choo via GitGitGadget
2023-06-20 21:02 ` Junio C Hamano
2023-06-20 19:43 ` [PATCH v3 03/12] config: add ctx arg to config_fn_t Glen Choo via GitGitGadget
2023-06-20 19:43 ` [PATCH v3 04/12] config.c: pass ctx in configsets Glen Choo via GitGitGadget
2023-06-20 21:19 ` Junio C Hamano
2023-06-20 19:43 ` [PATCH v3 05/12] config: pass ctx with config files Glen Choo via GitGitGadget
2023-06-20 19:43 ` [PATCH v3 06/12] builtin/config.c: test misuse of format_config() Glen Choo via GitGitGadget
2023-06-20 21:35 ` Junio C Hamano
2023-06-20 23:06 ` Glen Choo
2023-06-23 20:32 ` Jonathan Tan
2023-06-24 1:31 ` Jeff King
2023-06-28 17:28 ` Glen Choo
2023-06-20 19:43 ` [PATCH v3 07/12] config.c: pass ctx with CLI config Glen Choo via GitGitGadget
2023-06-23 20:35 ` Jonathan Tan
2023-06-23 21:41 ` Glen Choo
2023-06-20 19:43 ` [PATCH v3 08/12] trace2: plumb config kvi Glen Choo via GitGitGadget
2023-06-23 20:40 ` Jonathan Tan
2023-06-20 19:43 ` [PATCH v3 09/12] config: pass kvi to die_bad_number() Glen Choo via GitGitGadget
2023-06-20 19:43 ` [PATCH v3 10/12] config.c: remove config_reader from configsets Glen Choo via GitGitGadget
2023-06-23 20:57 ` Jonathan Tan
2023-06-23 21:33 ` Junio C Hamano
2023-06-20 19:43 ` [PATCH v3 11/12] config: add kvi.path, use it to evaluate includes Glen Choo via GitGitGadget
2023-06-20 19:43 ` [PATCH v3 12/12] config: pass source to config_parser_event_fn_t Glen Choo via GitGitGadget
2023-06-20 21:46 ` Junio C Hamano
2023-06-21 21:46 ` [PATCH v3 00/12] config: remove global state from config iteration Junio C Hamano
2023-06-21 23:06 ` Glen Choo
2023-06-23 21:02 ` Jonathan Tan
2023-06-23 21:33 ` Junio C Hamano
2023-06-23 21:45 ` Glen Choo
2023-06-26 18:11 ` [PATCH v4 " Glen Choo via GitGitGadget
2023-06-26 18:11 ` [PATCH v4 01/12] config: inline git_color_default_config Glen Choo via GitGitGadget
2023-06-26 18:11 ` [PATCH v4 02/12] urlmatch.h: use config_fn_t type Glen Choo via GitGitGadget
2023-06-26 18:11 ` [PATCH v4 03/12] config: add ctx arg to config_fn_t Glen Choo via GitGitGadget
2023-06-26 18:11 ` [PATCH v4 04/12] config.c: pass ctx in configsets Glen Choo via GitGitGadget
2023-06-26 18:11 ` [PATCH v4 05/12] config: pass ctx with config files Glen Choo via GitGitGadget
2023-06-26 18:11 ` [PATCH v4 06/12] builtin/config.c: test misuse of format_config() Glen Choo via GitGitGadget
2023-06-26 18:11 ` [PATCH v4 07/12] config.c: pass ctx with CLI config Glen Choo via GitGitGadget
2023-06-26 18:11 ` [PATCH v4 08/12] trace2: plumb config kvi Glen Choo via GitGitGadget
2023-06-26 18:11 ` [PATCH v4 09/12] config: pass kvi to die_bad_number() Glen Choo via GitGitGadget
2023-06-26 18:11 ` [PATCH v4 10/12] config.c: remove config_reader from configsets Glen Choo via GitGitGadget
2023-06-26 18:11 ` [PATCH v4 11/12] config: add kvi.path, use it to evaluate includes Glen Choo via GitGitGadget
2023-06-26 18:11 ` [PATCH v4 12/12] config: pass source to config_parser_event_fn_t Glen Choo via GitGitGadget
2023-06-26 20:45 ` [PATCH v4 00/12] config: remove global state from config iteration Junio C Hamano
2023-06-28 19:26 ` [PATCH v5 00/11] " Glen Choo via GitGitGadget
2023-06-28 19:26 ` [PATCH v5 01/11] config: inline git_color_default_config Glen Choo via GitGitGadget
2023-06-28 19:26 ` [PATCH v5 02/11] urlmatch.h: use config_fn_t type Glen Choo via GitGitGadget
2023-06-28 19:26 ` [PATCH v5 03/11] config: add ctx arg to config_fn_t Glen Choo via GitGitGadget
2023-06-28 19:26 ` [PATCH v5 04/11] config.c: pass ctx in configsets Glen Choo via GitGitGadget
2023-06-28 19:26 ` [PATCH v5 05/11] config: pass ctx with config files Glen Choo via GitGitGadget
2023-06-28 19:26 ` [PATCH v5 06/11] config.c: pass ctx with CLI config Glen Choo via GitGitGadget
2023-06-28 19:26 ` [PATCH v5 07/11] trace2: plumb config kvi Glen Choo via GitGitGadget
2023-06-28 19:26 ` [PATCH v5 08/11] config: pass kvi to die_bad_number() Glen Choo via GitGitGadget
2023-06-28 19:26 ` [PATCH v5 09/11] config.c: remove config_reader from configsets Glen Choo via GitGitGadget
2023-06-28 19:26 ` [PATCH v5 10/11] config: add kvi.path, use it to evaluate includes Glen Choo via GitGitGadget
2023-06-28 19:26 ` [PATCH v5 11/11] config: pass source to config_parser_event_fn_t Glen Choo via GitGitGadget
2023-06-28 22:23 ` [PATCH v5 00/11] config: remove global state from config iteration Jonathan Tan
2023-06-28 22:47 ` Junio C Hamano
2023-07-11 18:41 ` Phillip Wood [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=00194850-b881-2b01-9b9b-fc7d7422e804@gmail.com \
--to=phillip.wood123@gmail.com \
--cc=avarab@gmail.com \
--cc=chooglen@google.com \
--cc=git@vger.kernel.org \
--cc=gitgitgadget@gmail.com \
--cc=jonathantanmy@google.com \
--cc=nasamuffin@google.com \
--cc=peff@peff.net \
--cc=phillip.wood@dunelm.org.uk \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).