b4 claims patches are missing

b4 claims patches are missing

[Bjorn Helgaas]

Not sure what happened here.  Mutt sees this, which *looks* like it
should work:

  - Mar  8  9:24 Arnd Bergmann   [PATCH 1/3] PCI: controller: al: select CONFIG_
  - Mar  8  9:24 Arnd Bergmann   ├─>[PATCH 2/3] [RESEND] PCI: controller: thunde
  - Mar  8  9:24 Arnd Bergmann   ├─>[PATCH 3/3] [RESEND] PCI: controller: avoid
  - Mar  8 17:11 Krzysztof Wilcz ├─>Re: [PATCH 1/3] PCI: controller: al: select

$ b4 --version
0.6.2

$ b4 am -om/ https://lore.kernel.org/r/20210308152501.2135937-1-arnd@kernel.org
Looking up https://lore.kernel.org/r/20210308152501.2135937-1-arnd%40kernel.org
Grabbing thread from lore.kernel.org/linux-pci
Analyzing 4 messages in the thread
---
Thread incomplete, attempting to backfill
Grabbing thread from lore.kernel.org/lkml
---
Writing m/20210308_arnd_pci_controller_al_select_config_pci_ecam.mbx
  ✓ [PATCH 1/3] PCI: controller: al: select CONFIG_PCI_ECAM
    + Reviewed-by: Krzysztof Wilczyński <kw@linux.com>
  ERROR: missing [2/1]!
  ERROR: missing [3/1]!
---
Total patches: 1
---
WARNING: Thread incomplete!
 Link: https://lore.kernel.org/r/20210308152501.2135937-1-arnd@kernel.org
 Base: not found
       git am m/20210308_arnd_pci_controller_al_select_config_pci_ecam.mbx

Re: b4 claims patches are missing

[Bjorn Helgaas]

On Wed, Mar 10, 2021 at 01:44:39PM -0600, Bjorn Helgaas wrote:
> Not sure what happened here.  Mutt sees this, which *looks* like it
> should work:
> 
>   - Mar  8  9:24 Arnd Bergmann   [PATCH 1/3] PCI: controller: al: select CONFIG_
>   - Mar  8  9:24 Arnd Bergmann   ├─>[PATCH 2/3] [RESEND] PCI: controller: thunde
>   - Mar  8  9:24 Arnd Bergmann   ├─>[PATCH 3/3] [RESEND] PCI: controller: avoid
>   - Mar  8 17:11 Krzysztof Wilcz ├─>Re: [PATCH 1/3] PCI: controller: al: select
> 
> $ b4 --version
> 0.6.2
> 
> $ b4 am -om/ https://lore.kernel.org/r/20210308152501.2135937-1-arnd@kernel.org
> Looking up https://lore.kernel.org/r/20210308152501.2135937-1-arnd%40kernel.org
> Grabbing thread from lore.kernel.org/linux-pci
> Analyzing 4 messages in the thread
> ---
> Thread incomplete, attempting to backfill
> Grabbing thread from lore.kernel.org/lkml
> ---
> Writing m/20210308_arnd_pci_controller_al_select_config_pci_ecam.mbx
>   ✓ [PATCH 1/3] PCI: controller: al: select CONFIG_PCI_ECAM
>     + Reviewed-by: Krzysztof Wilczyński <kw@linux.com>
>   ERROR: missing [2/1]!
>   ERROR: missing [3/1]!
> ---
> Total patches: 1
> ---
> WARNING: Thread incomplete!
>  Link: https://lore.kernel.org/r/20210308152501.2135937-1-arnd@kernel.org
>  Base: not found
>        git am m/20210308_arnd_pci_controller_al_select_config_pci_ecam.mbx

Huh, all three of those messages from Arnd seem to have the same
Message-ID:

  https://lore.kernel.org/linux-pci/20210308152501.2135937-1-arnd@kernel.org/

Re: b4 claims patches are missing

[Konstantin Ryabitsev]

On Wed, Mar 10, 2021 at 01:44:37PM -0600, Bjorn Helgaas wrote:
> Not sure what happened here.  Mutt sees this, which *looks* like it
> should work:
> 
>   - Mar  8  9:24 Arnd Bergmann   [PATCH 1/3] PCI: controller: al: select CONFIG_
>   - Mar  8  9:24 Arnd Bergmann   ├─>[PATCH 2/3] [RESEND] PCI: controller: thunde
>   - Mar  8  9:24 Arnd Bergmann   ├─>[PATCH 3/3] [RESEND] PCI: controller: avoid
>   - Mar  8 17:11 Krzysztof Wilcz ├─>Re: [PATCH 1/3] PCI: controller: al: select

Hmm... Yeah, it's a bug due to "[PATCH X/N] [RESEND]". We try to deal with
lists that add extra bracketed junk to subjects like "[alsa-devel] [PATCH
X/N]", so we drop all bracketed prefixes until the last one. The debug output
shows what's happening:

	Analyzing 5 messages in the thread
	Looking at: [PATCH 1/3] PCI: controller: al: select CONFIG_PCI_ECAM
	  adding as patch
	Looking at: [RESEND] PCI: controller: thunder: fix compile testing
	  adding as patch
	Looking at: [RESEND] PCI: controller: avoid building empty drivers
	  adding as patch
	Looking at: Re: [PATCH 1/3] PCI: controller: al: select CONFIG_PCI_ECAM
	  adding to followups
	Looking at: Re: [PATCH 1/3] PCI: controller: al: select CONFIG_PCI_ECAM
	  adding to followups

We clearly need to be more careful when doing subject normalization. I'll try
to work around this. Thanks for the report.

For now, you can do "b4 mbox" that thread, edit subjects to remove the
[RESEND] bits and then "b4 am -m ./that.mbox".

Regards,
-K

Re: b4 claims patches are missing

[Bjorn Helgaas]

On Wed, Mar 10, 2021 at 01:50:44PM -0600, Bjorn Helgaas wrote:
> On Wed, Mar 10, 2021 at 01:44:39PM -0600, Bjorn Helgaas wrote:
> > Not sure what happened here.  Mutt sees this, which *looks* like it
> > should work:
> > 
> >   - Mar  8  9:24 Arnd Bergmann   [PATCH 1/3] PCI: controller: al: select CONFIG_
> >   - Mar  8  9:24 Arnd Bergmann   ├─>[PATCH 2/3] [RESEND] PCI: controller: thunde
> >   - Mar  8  9:24 Arnd Bergmann   ├─>[PATCH 3/3] [RESEND] PCI: controller: avoid
> >   - Mar  8 17:11 Krzysztof Wilcz ├─>Re: [PATCH 1/3] PCI: controller: al: select
> > 
> > $ b4 --version
> > 0.6.2
> > 
> > $ b4 am -om/ https://lore.kernel.org/r/20210308152501.2135937-1-arnd@kernel.org
> > Looking up https://lore.kernel.org/r/20210308152501.2135937-1-arnd%40kernel.org
> > Grabbing thread from lore.kernel.org/linux-pci
> > Analyzing 4 messages in the thread
> > ---
> > Thread incomplete, attempting to backfill
> > Grabbing thread from lore.kernel.org/lkml
> > ---
> > Writing m/20210308_arnd_pci_controller_al_select_config_pci_ecam.mbx
> >   ✓ [PATCH 1/3] PCI: controller: al: select CONFIG_PCI_ECAM
> >     + Reviewed-by: Krzysztof Wilczyński <kw@linux.com>
> >   ERROR: missing [2/1]!
> >   ERROR: missing [3/1]!
> > ---
> > Total patches: 1
> > ---
> > WARNING: Thread incomplete!
> >  Link: https://lore.kernel.org/r/20210308152501.2135937-1-arnd@kernel.org
> >  Base: not found
> >        git am m/20210308_arnd_pci_controller_al_select_config_pci_ecam.mbx
> 
> Huh, all three of those messages from Arnd seem to have the same
> Message-ID:
> 
>   https://lore.kernel.org/linux-pci/20210308152501.2135937-1-arnd@kernel.org/

Sorry, my mistake.  I obviously can't distinguish "Message-ID" from
"In-Reply-To".