On Tue, Apr 4, 2023 at 9:56 PM Linus Arver via GitGitGadget <gitgitgadget@xxxxxxxxx> wrote: > > From: Linus Arver <linusa@xxxxxxxxxx> > > The HTML version of MyFirstContribution [1] does not render the > asterisks (*) meant to be typed in as glob patterns by the user, because > they are being interpreted as bold text delimiters. Yes, they should be between backticks in order to be interpreted as literals. Acked-by: Felipe Contreras <felipe.contreras@xxxxxxxxx> > Documentation/MyFirstContribution.txt | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Documentation/MyFirstContribution.txt b/Documentation/MyFirstContribution.txt > index ccfd0cb5f3e..9c64bf58757 100644 > --- a/Documentation/MyFirstContribution.txt > +++ b/Documentation/MyFirstContribution.txt > @@ -1164,7 +1164,7 @@ After you run this command, `format-patch` will output the patches to the `psuh/ > directory, alongside the v1 patches. Using a single directory makes it easy to > refer to the old v1 patches while proofreading the v2 patches, but you will need > to be careful to send out only the v2 patches. We will use a pattern like > -"psuh/v2-*.patch" (not "psuh/*.patch", which would match v1 and v2 patches). > +`psuh/v2-*.patch` (not `psuh/*.patch`, which would match v1 and v2 patches). Small nit: with this change we would lose the quotes, which are helpful, I would rather do "`foo`". And for what it's worth I would revamp the whole section, something like this: --- a/Documentation/MyFirstContribution.txt +++ b/Documentation/MyFirstContribution.txt @@ -1136,18 +1136,18 @@ information on how to handle comments from reviewers. We'll reuse our `psuh` topic branch for v2. Before we make any changes, we'll mark the tip of our v1 branch for easy reference: ----- +.... $ git checkout psuh $ git branch psuh-v1 ----- +.... Refine your patch series by using `git rebase -i` to adjust commits based upon reviewer comments. Once the patch series is ready for submission, generate your patches again, but with some new flags: ----- +.... $ git format-patch -v2 --cover-letter -o psuh/ --range-diff master..psuh-v1 master.. ----- +.... The `--range-diff master..psuh-v1` parameter tells `format-patch` to include a range-diff between `psuh-v1` and `psuh` in the cover letter (see @@ -1157,53 +1157,53 @@ between your v1 and v2 patches. The `-v2` parameter tells `format-patch` to output your patches as version "2". For instance, you may notice that your v2 patches are all named like `v2-000n-my-commit-subject.patch`. `-v2` will also format -your patches by prefixing them with "[PATCH v2]" instead of "[PATCH]", -and your range-diff will be prefaced with "Range-diff against v1". +your patches by prefixing them with "`[PATCH v2]`" instead of "`[PATCH]`", +and your range-diff will be prefaced with "`Range-diff against v1`". After you run this command, `format-patch` will output the patches to the `psuh/` directory, alongside the v1 patches. Using a single directory makes it easy to refer to the old v1 patches while proofreading the v2 patches, but you will need to be careful to send out only the v2 patches. We will use a pattern like -"psuh/v2-*.patch" (not "psuh/*.patch", which would match v1 and v2 patches). +"`psuh/v2-*.patch`" (not "`psuh/*.patch`", which would match v1 and v2 patches). Edit your cover letter again. Now is a good time to mention what's different between your last version and now, if it's something significant. You do not need the exact same body in your second cover letter; focus on explaining to reviewers the changes you've made that may not be as visible. -You will also need to go and find the Message-Id of your previous cover letter. +You will also need to go and find the `Message-ID` of your previous cover letter. You can either note it when you send the first series, from the output of `git send-email`, or you can look it up on the https://lore.kernel.org/git[mailing list]. Find your cover letter in the -archives, click on it, then click "permalink" or "raw" to reveal the Message-Id +archives, click on it, then click "permalink" or "raw" to reveal the `Message-ID` header. It should match: ----- -Message-Id: <foo.12345.author@xxxxxxxxxxx> ----- +.... +Message-ID: <foo.12345.author@xxxxxxxxxxx> +.... -Your Message-Id is `<foo.12345.author@xxxxxxxxxxx>`. This example will be used -below as well; make sure to replace it with the correct Message-Id for your -**previous cover letter** - that is, if you're sending v2, use the Message-Id -from v1; if you're sending v3, use the Message-Id from v2. +Your `Message-ID` is `<foo.12345.author@xxxxxxxxxxx>`. This example will be used +below as well; make sure to replace it with the correct `Message-ID` for your +**previous cover letter** - that is, if you're sending v2, use the `Message-ID` +from v1; if you're sending v3, use the `Message-ID` from v2. While you're looking at the email, you should also note who is CC'd, as it's common practice in the mailing list to keep all CCs on a thread. You can add these CC lines directly to your cover letter with a line like so in the header (before the Subject line): ----- +.... CC: author@xxxxxxxxxxx, Othe R <other@xxxxxxxxxxx> ----- +.... Now send the emails again, paying close attention to which messages you pass in to the command: ----- +.... $ git send-email --to=target@xxxxxxxxxxx --in-reply-to="<foo.12345.author@xxxxxxxxxxx>" psuh/v2-*.patch ----- +.... [[single-patch]] === Bonus Chapter: One-Patch Changes -- Felipe Contreras
