b4 cherry-pick behavior

b4 cherry-pick behavior

[Rob Herring]

I've found that using -P_ and -P<n> have different behavior. The
former will not pick up any trailers from the cover letter while the
latter will. If that's desired behavior, can we make '-t' override
that or something? In my workflow when reading a thread, I have the
message-id readily available, but not the patch number unless I add
some parsing of the subject.

Here's an example. Note the "Analyzing ... messages" counts.

$ b4 am -P_ 20250423-dt-memory-region-v2-v2-3-2fbd6ebd3c88@kernel.org
Grabbing thread from
lore.kernel.org/all/20250423-dt-memory-region-v2-v2-3-2fbd6ebd3c88@kernel.org/t.mbox.gz
Analyzing 1 messages in the thread
Looking for additional code-review trailers on lore.kernel.org
Analyzing 18 code-review messages
Checking attestation on all messages, may take a moment...
---
  ✓ [PATCH v2 3/4] devres: Export devm_ioremap_resource_wc()
    ✓ Signed: DKIM/kernel.org
---
Total patches: 1 (cherrypicked:
<20250423-dt-memory-region-v2-v2-3-2fbd6ebd3c88@kernel.org>)
---
 Link: https://lore.kernel.org/r/20250423-dt-memory-region-v2-v2-3-2fbd6ebd3c88@kernel.org
 Base: not specified
       git am ./v2_20250423_robh_devres_export_devm_ioremap_resource_wc.mbx

$ b4 am -P3 20250423-dt-memory-region-v2-v2-3-2fbd6ebd3c88@kernel.org
Analyzing 15 messages in the thread
Looking for additional code-review trailers on lore.kernel.org
Analyzing 4 code-review messages
Checking attestation on all messages, may take a moment...
---
  ✓ [PATCH v2 3/4] devres: Export devm_ioremap_resource_wc()
    + Acked-by: Arnaud Pouliquen <arnaud.pouliquen@foss.st.com> (✓
DKIM/foss.st.com)
    + Tested-by: Tanmay Shah <tanmay.shah@amd.com> (✓ DKIM/amd.com)
  ---
  ✓ Signed: DKIM/kernel.org
---
Total patches: 1 (cherrypicked: 3)
---
Cover: ./v2_20250423_robh_of_common_memory_region_parsing.cover
 Link: https://lore.kernel.org/r/20250423-dt-memory-region-v2-v2-0-2fbd6ebd3c88@kernel.org
 Base: using specified base-commit 0af2f6be1b4281385b618cb86ad946eded089ac8
       git checkout -b v2_20250423_robh_kernel_org
0af2f6be1b4281385b618cb86ad946eded089ac8
       git am ./v2_20250423_robh_of_common_memory_region_parsing.mbx

Re: b4 cherry-pick behavior

[Konstantin Ryabitsev]

On Mon, May 05, 2025 at 04:42:16PM -0500, Rob Herring wrote:
> I've found that using -P_ and -P<n> have different behavior. The
> former will not pick up any trailers from the cover letter while the
> latter will. If that's desired behavior, can we make '-t' override
> that or something? In my workflow when reading a thread, I have the
> message-id readily available, but not the patch number unless I add
> some parsing of the subject.

Sorry for the delay -- I'm finally looking at this and I can confirm that this
is unintended behaviour. I'll fix it so -P_ and -P3 in this and similar cases
return identical results.

bugspray tag me

-K

Re: b4 cherry-pick behavior

[Bugspray Bot]

Hello:

This conversation is now tracked by Kernel.org Bugzilla:
https://bugzilla.kernel.org/show_bug.cgi?id=220247

There is no need to do anything else, just keep talking.
-- 
Deet-doot-dot, I am a bot.
Kernel.org Bugzilla (bugspray 0.1-dev)