From: Kyle Meyer <kyle@kyleam.com>
To: Josh Steadmon <steadmon@google.com>
Cc: tools@linux.kernel.org
Subject: Re: b4 doesn't notice updated base-commit
Date: Mon, 18 Apr 2022 18:25:42 -0400 [thread overview]
Message-ID: <87tuaq81w9.fsf@kyleam.com> (raw)
In-Reply-To: <Yl2gMy2Fgpe0TUZ5@google.com>
Josh Steadmon writes:
> Hi, just noticed this minor bug in b4 v0.8.0: when fetching the latest
> version of a series, b4 seems to always advise that the base-commit is
> whatever was advertised in the first cover letter, even if the
> base-commit changes for later versions of the series. An example (from
> the Git list):
>
> ```
> $ b4 am -o - pull.1161.v4.git.1647843442911.gitgitgadget@gmail.com | git am
> Looking up https://lore.kernel.org/r/pull.1161.v4.git.1647843442911.gitgitgadget%40gmail.com
> [...]
> Base: dab1b7905d0b295f1acef9785bb2b9cbb0fdec84
> git checkout -b v4_20220228_gitgitgadget_gmail_com dab1b7905d0b295f1acef9785bb2b9cbb0fdec84
> ```
I can trigger the issue on my end too, but I suspect it's more specific
than "always [advising] that the base-commit is whatever was advertised
in the first cover letter".
For example, here's a thread that has a different base commit at v1 and
the latest version (v10), and the correct base is picked up
(b80121027d12 not 5d01301f2b86):
https://lore.kernel.org/git/pull.1212.git.git.1643915286.gitgitgadget@gmail.com/
Perhaps the problem with the thread you referenced has something to do
with v4 consisting of only one patch and no cover letter while the
previous iterations had multiple patches and a cover letter.
The cover letter written by 'b4 am' suggests it's getting stuck on the
cover letter for the v3:
$ b4 am pull.1161.v4.git.1647843442911.gitgitgadget@gmail.com
$ ls -1
v4_20220228_gitgitgadget_adding_new_branch_autosetupmerge_option_simple.cover
v4_20220228_gitgitgadget_adding_new_branch_autosetupmerge_option_simple.mbx
$ grep Subject v4_20220228_gitgitgadget_adding_new_branch_autosetupmerge_option_simple.cover
Subject: [PATCH v3 0/2] adding new branch.autosetupmerge option "simple"
prev parent reply other threads:[~2022-04-18 22:25 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-04-18 17:30 b4 doesn't notice updated base-commit Josh Steadmon
2022-04-18 22:25 ` Kyle Meyer [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=87tuaq81w9.fsf@kyleam.com \
--to=kyle@kyleam.com \
--cc=steadmon@google.com \
--cc=tools@linux.kernel.org \
/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