From: Phillip Wood <phillip.wood123@gmail•com>
To: Siddharth Asthana <siddharthasthana31@gmail•com>, git@vger•kernel.org
Cc: gitster@pobox•com, christian.couder@gmail•com, ps@pks•im,
newren@gmail•com, code@khaugsbakk•name, rybak.a.v@gmail•com,
karthik.188@gmail•com, jltobler@gmail•com, toon@iotcl•com,
johncai86@gmail•com, johannes.schindelin@gmx•de
Subject: Re: [PATCH v2 1/1] replay: make atomic ref updates the default behavior
Date: Tue, 30 Sep 2025 11:05:20 +0100 [thread overview]
Message-ID: <9052eccc-1121-442f-ad51-4fe9217024a0@gmail.com> (raw)
In-Reply-To: <20250926230838.35870-2-siddharthasthana31@gmail.com>
Hi Siddharth
On 27/09/2025 00:08, Siddharth Asthana wrote:
> The git replay command currently outputs update commands that must be
> piped to git update-ref --stdin to actually update references:
>
> git replay --onto main topic1..topic2 | git update-ref --stdin
>
> This design has significant limitations for server-side operations. The
> two-command pipeline creates coordination complexity, provides no atomic
> transaction guarantees by default
Are you sure that's true? Maybe I'm missing something but my reading of
builtin/update-ref.c is that it when "--stdin" is given it starts a ref
transaction, reads the commands from stdin and applies them to that
transaction and then commits the transaction which will make the updates
atomic.
> , and complicates automation in bare
> repository environments where git replay is primarily used.
How does it complicate automation in bare repositories?
Christian has given detailed feedback on the rest of the commit message
so I'll not comment on it further.
> diff --git a/Documentation/git-replay.adoc b/Documentation/git-replay.adoc
> index 0b12bf8aa4..e104e0bc03 100644
> --- a/Documentation/git-replay.adoc
> +++ b/Documentation/git-replay.adoc
> @@ -9,16 +9,16 @@ git-replay - EXPERIMENTAL: Replay commits on a new base, works with bare repos t
> SYNOPSIS
> --------
> [verse]
> -(EXPERIMENTAL!) 'git replay' ([--contained] --onto <newbase> | --advance <branch>) <revision-range>...
> +(EXPERIMENTAL!) 'git replay' ([--contained] --onto <newbase> | --advance <branch>) [--output-commands | --allow-partial] <revision-range>...
Please wrap this very long line
> @@ -42,6 +42,20 @@ When `--advance` is specified, the update-ref command(s) in the output
> will update the branch passed as an argument to `--advance` to point at
> the new commits (in other words, this mimics a cherry-pick operation).
>
> +--output-commands::
> + Output update-ref commands instead of updating refs directly.
> + When this option is used, the output can be piped to `git update-ref --stdin`
> + for successive, relatively slow, ref updates. This is equivalent to the
> + old default behavior.
> +
> +--allow-partial::
> + Allow some ref updates to succeed even if others fail. By default,
> + ref updates are atomic (all succeed or all fail). With this option,
> + failed updates are reported as warnings rather than causing the entire
> + command to fail. The command exits with code 0 only if all updates
> + succeed; any failures result in exit code 1. Cannot be used with
> + `--output-commands`.
Rather than having two incompatible options perhaps we could have a
single "--update-refs=(yes|print|allow-partial-updates)" argument. I
think the name "--allow-partial" is rather ambiguous as it does not say
what it is allowing to be partial.
> +static int add_ref_to_transaction(struct ref_transaction *transaction,
> + const char *refname,
> + const struct object_id *new_oid,
> + const struct object_id *old_oid,
> + struct strbuf *err)
> +{
> + return ref_transaction_update(transaction, refname, new_oid, old_oid,
> + NULL, NULL, 0, "git replay", err);
> +}
I'm not sure this function adds much value. I think it would be better
to instead have a helper function that updates refs or prints the ref
updates so that we do not duplicate that code in the two places below.
> @@ -434,10 +481,18 @@ int cmd_replay(int argc,
> if (decoration->type == DECORATION_REF_LOCAL &&
> (contained || strset_contains(update_refs,
> decoration->name))) {
> - printf("update %s %s %s\n",
> - decoration->name,
> - oid_to_hex(&last_commit->object.oid),
> - oid_to_hex(&commit->object.oid));
> + if (output_commands) {
> + printf("update %s %s %s\n",
> + decoration->name,
> + oid_to_hex(&last_commit->object.oid),
> + oid_to_hex(&commit->object.oid));
> + } else if (add_ref_to_transaction(transaction, decoration->name,
> + &last_commit->object.oid,
> + &commit->object.oid,
> + &transaction_err) < 0) {
> + ret = error(_("failed to add ref update to transaction: %s"), transaction_err.buf);
> + goto cleanup;
> + }
> }
The lines here are very long due to the indentation, having a separate
function to update the refs or print the ref updates would be much more
readable.
> decoration = decoration->next;
> }
> @@ -445,10 +500,33 @@ int cmd_replay(int argc,
>
> /* In --advance mode, advance the target ref */
> if (result.clean == 1 && advance_name) {
> - printf("update %s %s %s\n",
> - advance_name,
> - oid_to_hex(&last_commit->object.oid),
> - oid_to_hex(&onto->object.oid));
> + if (output_commands) {
> + printf("update %s %s %s\n",
> + advance_name,
> + oid_to_hex(&last_commit->object.oid),
> + oid_to_hex(&onto->object.oid));
> + } else if (add_ref_to_transaction(transaction, advance_name,
> + &last_commit->object.oid,
> + &onto->object.oid,
> + &transaction_err) < 0) {
> + ret = error(_("failed to add ref update to transaction: %s"), transaction_err.buf);
> + goto cleanup;
> + }
> + }
Putting the code to update the refs or print the ref updates into a
single function would avoid this duplication and over-long lines.
Thanks
Phillip
> + /* Commit the ref transaction if we have one */
> + if (transaction && result.clean == 1) {
> + if (ref_transaction_commit(transaction, &transaction_err)) {
> + if (allow_partial) {
> + warning(_("some ref updates failed: %s"), transaction_err.buf);
> + ref_transaction_for_each_rejected_update(transaction,
> + print_rejected_update, NULL);
> + ret = 0; /* Set failure even with allow_partial */
> + } else {
> + ret = error(_("failed to update refs: %s"), transaction_err.buf);
> + goto cleanup;
> + }
> + }
> }
>
> merge_finalize(&merge_opt, &result);
> @@ -457,9 +535,17 @@ int cmd_replay(int argc,
> strset_clear(update_refs);
> free(update_refs);
> }
> - ret = result.clean;
> +
> + /* Handle empty ranges: if no commits were processed, treat as success */
> + if (!commits_processed)
> + ret = 1; /* Success - no commits to replay is not an error */
> + else
> + ret = result.clean;
>
> cleanup:
> + if (transaction)
> + ref_transaction_free(transaction);
> + strbuf_release(&transaction_err);
> release_revisions(&revs);
> free(advance_name);
>
> diff --git a/t/t3650-replay-basics.sh b/t/t3650-replay-basics.sh
> index 58b3759935..8b4301e227 100755
> --- a/t/t3650-replay-basics.sh
> +++ b/t/t3650-replay-basics.sh
> @@ -52,7 +52,7 @@ test_expect_success 'setup bare' '
> '
>
> test_expect_success 'using replay to rebase two branches, one on top of other' '
> - git replay --onto main topic1..topic2 >result &&
> + git replay --output-commands --onto main topic1..topic2 >result &&
>
> test_line_count = 1 result &&
>
> @@ -67,9 +67,30 @@ test_expect_success 'using replay to rebase two branches, one on top of other' '
> test_cmp expect result
> '
>
> +test_expect_success 'using replay with default atomic behavior (no output)' '
> + # Create a test branch that wont interfere with others
> + git branch atomic-test topic2 &&
> + git rev-parse atomic-test >atomic-test-old &&
> +
> + # Default behavior: atomic ref updates (no output)
> + git replay --onto main topic1..atomic-test >output &&
> + test_must_be_empty output &&
> +
> + # Verify the branch was updated
> + git rev-parse atomic-test >atomic-test-new &&
> + ! test_cmp atomic-test-old atomic-test-new &&
> +
> + # Verify the history is correct
> + git log --format=%s atomic-test >actual &&
> + test_write_lines E D M L B A >expect &&
> + test_cmp expect actual
> +'
> +
> test_expect_success 'using replay on bare repo to rebase two branches, one on top of other' '
> - git -C bare replay --onto main topic1..topic2 >result-bare &&
> - test_cmp expect result-bare
> + git -C bare replay --output-commands --onto main topic1..topic2 >result-bare &&
> +
> + # The result should match what we got from the regular repo
> + test_cmp result result-bare
> '
>
> test_expect_success 'using replay to rebase with a conflict' '
> @@ -86,7 +107,7 @@ test_expect_success 'using replay to perform basic cherry-pick' '
> # 2nd field of result is refs/heads/main vs. refs/heads/topic2
> # 4th field of result is hash for main instead of hash for topic2
>
> - git replay --advance main topic1..topic2 >result &&
> + git replay --output-commands --advance main topic1..topic2 >result &&
>
> test_line_count = 1 result &&
>
> @@ -102,7 +123,7 @@ test_expect_success 'using replay to perform basic cherry-pick' '
> '
>
> test_expect_success 'using replay on bare repo to perform basic cherry-pick' '
> - git -C bare replay --advance main topic1..topic2 >result-bare &&
> + git -C bare replay --output-commands --advance main topic1..topic2 >result-bare &&
> test_cmp expect result-bare
> '
>
> @@ -115,7 +136,7 @@ test_expect_success 'replay fails when both --advance and --onto are omitted' '
> '
>
> test_expect_success 'using replay to also rebase a contained branch' '
> - git replay --contained --onto main main..topic3 >result &&
> + git replay --output-commands --contained --onto main main..topic3 >result &&
>
> test_line_count = 2 result &&
> cut -f 3 -d " " result >new-branch-tips &&
> @@ -139,12 +160,12 @@ test_expect_success 'using replay to also rebase a contained branch' '
> '
>
> test_expect_success 'using replay on bare repo to also rebase a contained branch' '
> - git -C bare replay --contained --onto main main..topic3 >result-bare &&
> + git -C bare replay --output-commands --contained --onto main main..topic3 >result-bare &&
> test_cmp expect result-bare
> '
>
> test_expect_success 'using replay to rebase multiple divergent branches' '
> - git replay --onto main ^topic1 topic2 topic4 >result &&
> + git replay --output-commands --onto main ^topic1 topic2 topic4 >result &&
>
> test_line_count = 2 result &&
> cut -f 3 -d " " result >new-branch-tips &&
> @@ -168,7 +189,7 @@ test_expect_success 'using replay to rebase multiple divergent branches' '
> '
>
> test_expect_success 'using replay on bare repo to rebase multiple divergent branches, including contained ones' '
> - git -C bare replay --contained --onto main ^main topic2 topic3 topic4 >result &&
> + git -C bare replay --output-commands --contained --onto main ^main topic2 topic3 topic4 >result &&
>
> test_line_count = 4 result &&
> cut -f 3 -d " " result >new-branch-tips &&
> @@ -217,4 +238,131 @@ test_expect_success 'merge.directoryRenames=false' '
> --onto rename-onto rename-onto..rename-from
> '
>
> +# Tests for new default atomic behavior and options> > +test_expect_success 'replay default behavior should not produce
output when successful' '
> + git replay --onto main topic1..topic3 >output &&
> + test_must_be_empty output
> +'
> +
> +test_expect_success 'replay with --output-commands produces traditional output' '
> + git replay --output-commands --onto main topic1..topic3 >output &&
> + test_line_count = 1 output &&
> + grep "^update refs/heads/topic3 " output
> +'
> +
> +test_expect_success 'replay with --allow-partial should not produce output when successful' '
> + git replay --allow-partial --onto main topic1..topic3 >output &&
> + test_must_be_empty output
> +'
> +
> +test_expect_success 'replay fails when --output-commands and --allow-partial are used together' '
> + test_must_fail git replay --output-commands --allow-partial --onto main topic1..topic2 2>error &&
> + grep "cannot be used together" error
> +'
> +
> +test_expect_success 'replay with --contained updates multiple branches atomically' '
> + # Create fresh test branches based on the original structure
> + # contained-topic1 should be contained within the range to contained-topic3
> + git branch contained-base main &&
> + git checkout -b contained-topic1 contained-base &&
> + test_commit ContainedC &&
> + git checkout -b contained-topic3 contained-topic1 &&
> + test_commit ContainedG &&
> + test_commit ContainedH &&
> + git checkout main &&
> +
> + # Store original states
> + git rev-parse contained-topic1 >contained-topic1-old &&
> + git rev-parse contained-topic3 >contained-topic3-old &&
> +
> + # Use --contained to update multiple branches - this should update both
> + git replay --contained --onto main contained-base..contained-topic3 &&
> +
> + # Verify both branches were updated
> + git rev-parse contained-topic1 >contained-topic1-new &&
> + git rev-parse contained-topic3 >contained-topic3-new &&
> + ! test_cmp contained-topic1-old contained-topic1-new &&
> + ! test_cmp contained-topic3-old contained-topic3-new
> +'
> +
> +test_expect_success 'replay atomic behavior: all refs updated or none' '
> + # Store original state
> + git rev-parse topic4 >topic4-old &&
> +
> + # Default atomic behavior
> + git replay --onto main main..topic4 &&
> +
> + # Verify ref was updated
> + git rev-parse topic4 >topic4-new &&
> + ! test_cmp topic4-old topic4-new &&
> +
> + # Verify no partial state
> + git log --format=%s topic4 >actual &&
> + test_write_lines J I M L B A >expect &&
> + test_cmp expect actual
> +'
> +
> +test_expect_success 'replay works correctly with bare repositories' '
> + # Test atomic behavior in bare repo (important for Gitaly)
> + git checkout -b bare-test topic1 &&
> + test_commit BareTest &&
> +
> + # Test with bare repo - replay the commits from main..bare-test to get the full history
> + git -C bare fetch .. bare-test:bare-test &&
> + git -C bare replay --onto main main..bare-test &&
> +
> + # Verify the bare repo was updated correctly (no output)
> + git -C bare log --format=%s bare-test >actual &&
> + test_write_lines BareTest F C M L B A >expect &&
> + test_cmp expect actual
> +'
> +
> +test_expect_success 'replay --allow-partial with no failures produces no output' '
> + git checkout -b partial-test topic1 &&
> + test_commit PartialTest &&
> +
> + # Should succeed silently even with partial mode
> + git replay --allow-partial --onto main topic1..partial-test >output &&
> + test_must_be_empty output
> +'
> +
> +test_expect_success 'replay maintains ref update consistency' '
> + # Test that traditional vs atomic produce equivalent results
> + git checkout -b method1-test topic2 &&
> + git checkout -b method2-test topic2 &&
> +
> + # Both methods should update refs to point to the same replayed commits
> + git replay --output-commands --onto main topic1..method1-test >update-commands &&
> + git update-ref --stdin <update-commands &&
> + git log --format=%s method1-test >traditional-result &&
> +
> + # Direct atomic method should produce same commit history
> + git replay --onto main topic1..method2-test &&
> + git log --format=%s method2-test >atomic-result &&
> +
> + # Both methods should produce identical commit histories
> + test_cmp traditional-result atomic-result
> +'
> +
> +test_expect_success 'replay error messages are helpful and clear' '
> + # Test that error messages are clear
> + test_must_fail git replay --output-commands --allow-partial --onto main topic1..topic2 2>error &&
> + grep "cannot be used together" error
> +'
> +
> +test_expect_success 'replay with empty range produces no output and no changes' '
> + # Create a test branch for empty range testing
> + git checkout -b empty-test topic1 &&
> + git rev-parse empty-test >empty-test-before &&
> +
> + # Empty range should succeed but do nothing
> + git replay --onto main empty-test..empty-test >output &&
> + test_must_be_empty output &&
> +
> + # Branch should be unchanged
> + git rev-parse empty-test >empty-test-after &&
> + test_cmp empty-test-before empty-test-after
> +'
> +
> test_done
next prev parent reply other threads:[~2025-09-30 10:05 UTC|newest]
Thread overview: 125+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-09-08 4:36 [PATCH 0/2] replay: add --update-refs option Siddharth Asthana
2025-09-08 4:36 ` [PATCH 1/2] " Siddharth Asthana
2025-09-08 9:54 ` Patrick Steinhardt
2025-09-09 6:58 ` Siddharth Asthana
2025-09-09 9:00 ` Patrick Steinhardt
2025-09-09 7:32 ` Elijah Newren
2025-09-10 17:58 ` Siddharth Asthana
2025-09-08 4:36 ` [PATCH 2/2] replay: document --update-refs and --batch options Siddharth Asthana
2025-09-08 6:00 ` Christian Couder
2025-09-09 6:36 ` Siddharth Asthana
2025-09-09 7:26 ` Christian Couder
2025-09-10 20:26 ` Siddharth Asthana
2025-09-08 14:40 ` Kristoffer Haugsbakk
2025-09-09 7:06 ` Siddharth Asthana
2025-09-09 19:20 ` Andrei Rybak
2025-09-10 20:28 ` Siddharth Asthana
2025-09-08 6:07 ` [PATCH 0/2] replay: add --update-refs option Christian Couder
2025-09-09 6:36 ` Siddharth Asthana
2025-09-08 14:33 ` Kristoffer Haugsbakk
2025-09-09 7:04 ` Siddharth Asthana
2025-09-09 7:13 ` Elijah Newren
2025-09-09 7:47 ` Christian Couder
2025-09-09 9:19 ` Elijah Newren
2025-09-09 16:44 ` Junio C Hamano
2025-09-09 19:52 ` Elijah Newren
2025-09-26 23:08 ` [PATCH v2 0/1] replay: make atomic ref updates the default behavior Siddharth Asthana
2025-09-26 23:08 ` [PATCH v2 1/1] " Siddharth Asthana
2025-09-30 8:23 ` Christian Couder
2025-10-02 22:16 ` Siddharth Asthana
2025-10-03 7:30 ` Christian Couder
2025-10-02 22:55 ` Elijah Newren
2025-10-03 7:05 ` Christian Couder
2025-09-30 10:05 ` Phillip Wood [this message]
2025-10-02 10:00 ` Karthik Nayak
2025-10-02 22:20 ` Siddharth Asthana
2025-10-02 22:20 ` Siddharth Asthana
2025-10-08 14:01 ` Phillip Wood
2025-10-08 20:09 ` Siddharth Asthana
2025-10-08 20:59 ` Elijah Newren
2025-10-08 21:16 ` Siddharth Asthana
2025-10-09 9:40 ` Phillip Wood
2025-10-02 16:32 ` Elijah Newren
2025-10-02 18:27 ` Junio C Hamano
2025-10-02 23:42 ` Siddharth Asthana
2025-10-02 23:27 ` Siddharth Asthana
2025-10-03 7:59 ` Christian Couder
2025-10-08 19:59 ` Siddharth Asthana
2025-10-03 19:48 ` Elijah Newren
2025-10-03 20:32 ` Junio C Hamano
2025-10-08 20:06 ` Siddharth Asthana
2025-10-08 20:59 ` Junio C Hamano
2025-10-08 21:10 ` Siddharth Asthana
2025-10-08 21:30 ` Elijah Newren
2025-10-08 20:05 ` Siddharth Asthana
2025-10-02 17:14 ` [PATCH v2 0/1] " Kristoffer Haugsbakk
2025-10-02 23:36 ` Siddharth Asthana
2025-10-03 19:05 ` Kristoffer Haugsbakk
2025-10-08 20:02 ` Siddharth Asthana
2025-10-08 20:56 ` Elijah Newren
2025-10-08 21:16 ` Kristoffer Haugsbakk
2025-10-08 21:18 ` Siddharth Asthana
2025-10-13 18:33 ` [PATCH v3 0/3] replay: make atomic ref updates the default Siddharth Asthana
2025-10-13 18:33 ` [PATCH v3 1/3] replay: use die_for_incompatible_opt2() for option validation Siddharth Asthana
2025-10-13 18:33 ` [PATCH v3 2/3] replay: make atomic ref updates the default behavior Siddharth Asthana
2025-10-13 22:05 ` Junio C Hamano
2025-10-15 5:01 ` Siddharth Asthana
2025-10-13 18:33 ` [PATCH v3 3/3] replay: add replay.defaultAction config option Siddharth Asthana
2025-10-13 19:39 ` [PATCH v3 0/3] replay: make atomic ref updates the default Junio C Hamano
2025-10-15 4:57 ` Siddharth Asthana
2025-10-15 10:33 ` Christian Couder
2025-10-15 14:45 ` Junio C Hamano
2025-10-22 18:50 ` [PATCH v4 " Siddharth Asthana
2025-10-22 18:50 ` [PATCH v4 1/3] replay: use die_for_incompatible_opt2() for option validation Siddharth Asthana
2025-10-22 18:50 ` [PATCH v4 2/3] replay: make atomic ref updates the default behavior Siddharth Asthana
2025-10-22 21:19 ` Junio C Hamano
2025-10-28 19:03 ` Siddharth Asthana
2025-10-24 10:37 ` Christian Couder
2025-10-24 15:23 ` Junio C Hamano
2025-10-28 20:18 ` Siddharth Asthana
2025-10-28 19:39 ` Siddharth Asthana
2025-10-22 18:50 ` [PATCH v4 3/3] replay: add replay.refAction config option Siddharth Asthana
2025-10-24 11:01 ` Christian Couder
2025-10-24 15:30 ` Junio C Hamano
2025-10-28 20:08 ` Siddharth Asthana
2025-10-28 19:26 ` Siddharth Asthana
2025-10-24 13:28 ` Phillip Wood
2025-10-24 13:36 ` Phillip Wood
2025-10-28 19:47 ` Siddharth Asthana
2025-10-28 19:46 ` Siddharth Asthana
2025-10-23 18:47 ` [PATCH v4 0/3] replay: make atomic ref updates the default Junio C Hamano
2025-10-25 16:57 ` Junio C Hamano
2025-10-28 20:19 ` Siddharth Asthana
2025-10-24 9:39 ` Christian Couder
2025-10-28 21:46 ` [PATCH v5 " Siddharth Asthana
2025-10-28 21:46 ` [PATCH v5 1/3] replay: use die_for_incompatible_opt2() for option validation Siddharth Asthana
2025-10-28 21:46 ` [PATCH v5 2/3] replay: make atomic ref updates the default behavior Siddharth Asthana
2025-10-28 21:46 ` [PATCH v5 3/3] replay: add replay.refAction config option Siddharth Asthana
2025-10-29 16:19 ` Christian Couder
2025-10-29 17:00 ` Siddharth Asthana
2025-10-30 19:19 ` [PATCH v6 0/3] replay: make atomic ref updates the default Siddharth Asthana
2025-10-30 19:19 ` [PATCH v6 1/3] replay: use die_for_incompatible_opt2() for option validation Siddharth Asthana
2025-10-31 18:47 ` Elijah Newren
2025-11-05 18:39 ` Siddharth Asthana
2025-10-30 19:19 ` [PATCH v6 2/3] replay: make atomic ref updates the default behavior Siddharth Asthana
2025-10-31 18:49 ` Elijah Newren
2025-10-31 19:59 ` Junio C Hamano
2025-11-05 19:07 ` Siddharth Asthana
2025-11-03 16:25 ` Phillip Wood
2025-11-03 19:32 ` Siddharth Asthana
2025-11-04 16:15 ` Phillip Wood
2025-10-30 19:19 ` [PATCH v6 3/3] replay: add replay.refAction config option Siddharth Asthana
2025-10-31 7:08 ` Christian Couder
2025-11-05 19:03 ` Siddharth Asthana
2025-10-31 18:49 ` Elijah Newren
2025-11-05 19:10 ` Siddharth Asthana
2025-10-31 18:51 ` [PATCH v6 0/3] replay: make atomic ref updates the default Elijah Newren
2025-11-05 19:15 ` [PATCH v7 " Siddharth Asthana
2025-11-05 19:15 ` [PATCH v7 1/3] replay: use die_for_incompatible_opt2() for option validation Siddharth Asthana
2025-11-05 19:16 ` [PATCH v7 2/3] replay: make atomic ref updates the default behavior Siddharth Asthana
2025-11-05 19:16 ` [PATCH v7 3/3] replay: add replay.refAction config option Siddharth Asthana
2025-11-06 19:32 ` [PATCH v7 0/3] replay: make atomic ref updates the default Elijah Newren
2025-11-08 13:22 ` Siddharth Asthana
2025-11-08 17:11 ` Elijah Newren
2025-11-07 15:48 ` Phillip Wood
2025-11-08 13:23 ` Siddharth Asthana
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=9052eccc-1121-442f-ad51-4fe9217024a0@gmail.com \
--to=phillip.wood123@gmail$(echo .)com \
--cc=christian.couder@gmail$(echo .)com \
--cc=code@khaugsbakk$(echo .)name \
--cc=git@vger$(echo .)kernel.org \
--cc=gitster@pobox$(echo .)com \
--cc=jltobler@gmail$(echo .)com \
--cc=johannes.schindelin@gmx$(echo .)de \
--cc=johncai86@gmail$(echo .)com \
--cc=karthik.188@gmail$(echo .)com \
--cc=newren@gmail$(echo .)com \
--cc=ps@pks$(echo .)im \
--cc=rybak.a.v@gmail$(echo .)com \
--cc=siddharthasthana31@gmail$(echo .)com \
--cc=toon@iotcl$(echo .)com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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