Linux maintainer tooling and workflows
 help / color / Atom feed
* b4 claims patches are missing
@ 2021-03-10 19:44 Bjorn Helgaas
  2021-03-10 19:50 ` Bjorn Helgaas
  2021-03-10 19:57 ` Konstantin Ryabitsev
  0 siblings, 2 replies; 4+ messages in thread
From: Bjorn Helgaas @ 2021-03-10 19:44 UTC (permalink / raw)
  To: tools; +Cc: Arnd Bergmann, Krzysztof Wilczyński

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

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: b4 claims patches are missing
  2021-03-10 19:44 b4 claims patches are missing Bjorn Helgaas
@ 2021-03-10 19:50 ` Bjorn Helgaas
  2021-03-10 20:11   ` Bjorn Helgaas
  2021-03-10 19:57 ` Konstantin Ryabitsev
  1 sibling, 1 reply; 4+ messages in thread
From: Bjorn Helgaas @ 2021-03-10 19:50 UTC (permalink / raw)
  To: tools; +Cc: Arnd Bergmann, Krzysztof Wilczyński

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/

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: b4 claims patches are missing
  2021-03-10 19:44 b4 claims patches are missing Bjorn Helgaas
  2021-03-10 19:50 ` Bjorn Helgaas
@ 2021-03-10 19:57 ` Konstantin Ryabitsev
  1 sibling, 0 replies; 4+ messages in thread
From: Konstantin Ryabitsev @ 2021-03-10 19:57 UTC (permalink / raw)
  To: Bjorn Helgaas; +Cc: tools, Arnd Bergmann, Krzysztof Wilczyński

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

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: b4 claims patches are missing
  2021-03-10 19:50 ` Bjorn Helgaas
@ 2021-03-10 20:11   ` Bjorn Helgaas
  0 siblings, 0 replies; 4+ messages in thread
From: Bjorn Helgaas @ 2021-03-10 20:11 UTC (permalink / raw)
  To: tools; +Cc: Arnd Bergmann, Krzysztof Wilczyński

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".

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, back to index

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-03-10 19:44 b4 claims patches are missing Bjorn Helgaas
2021-03-10 19:50 ` Bjorn Helgaas
2021-03-10 20:11   ` Bjorn Helgaas
2021-03-10 19:57 ` Konstantin Ryabitsev

Linux maintainer tooling and workflows

Archives are clonable:
	git clone --mirror https://lore.kernel.org/tools/0 tools/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 tools tools/ https://lore.kernel.org/tools \
		tools@linux.kernel.org
	public-inbox-index tools

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.linux.tools


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git