git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: About my git workflow; maybe it's useful for others

From
Stefan Beller <sbeller@google.com>
Date
Apr 22, 2015, 20:38 UTC
Message-ID
<CAGZ79kb2z+CbHhqrB9xW85n_33V8=iYV3jGf9pCNDX181tm3JA@mail.gmail.com>
In-Reply-To
<CACnwZYcD7ryJqM+85wxka+ViqOfy51bOgyetUEdgY1pcQPJv=A@mail.gmail.com>
On Wed, Apr 22, 2015 at 12:57 PM, Thiago Farina <tfransosi@gmail.com> wrote:
Show 28 quoted lines
> On Wed, Apr 22, 2015 at 4:50 PM, Stefan Beller <sbeller@google.com> wrote:
>> On Wed, Apr 22, 2015 at 12:38 PM, Thiago Farina <tfransosi@gmail.com> wrote:
>>>>
>>> IMO, sending email is the easiest part.
>>>
>>> The hard begins when you have to edit your patch and resend with the
>>> reviewers' feedback incorporated. For me that is the most tricky and
>>> hard part to get right, specially when using GMail as an email client.
>>>
>>> How do you handle that part of the process?
>>
>> I try to have as much in git as possible.
>>
>> So when the reviews trickle in, I change my commits (in git) accordingly
>> via rebase and edit and lots of fixup commits. I use git notes
>> to keep track of changes from one version to another.
>>
>> Having the "changes of the changes" in the git notes, I am (in theory)
>> always able to kick out a new version of the patch series with
>>
>>    rm 00* # delete old patches
>>    git format-patch --notes --coverletter somebranch...HEAD
>>    edit 0000-cover-letter.patch
>>    git send-email 00* --to=mailing list --to=John@doe.org --cc=Max@Mustermann.de
>>
> Is that capable of keeping the next patch set in the same thread that
> started when you sent the initial patch? Otherwise things get
> disconnected.

When typing it out quickly I forgot the --in-reply-to=<identifier> option for the git send-email command. The identifier needs to be looked u0p manually, which is still a pain point in my workflow.

>
> --
> Thiago Farina
Previous: Thiago Farina
Message 5 of 5 in “About my git workflow; maybe it's useful for others”
  1. Stefan BellerDec 30, 2014
  2. Thiago FarinaApr 22, 2015
  3. Stefan BellerApr 22, 2015
  4. Thiago FarinaApr 22, 2015
  5. Stefan BellerApr 22, 2015

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.