Linux maintainer tooling and workflows
 help / color / Atom feed
* b4 picking the wrong message
@ 2020-07-03 18:52 Mark Brown
  2020-07-06 22:11 ` Konstantin Ryabitsev
  0 siblings, 1 reply; 3+ messages in thread
From: Mark Brown @ 2020-07-03 18:52 UTC (permalink / raw)
  To: Konstantin Ryabitsev; +Cc: tools


[-- Attachment #1: Type: text/plain, Size: 588 bytes --]

Hi Konstantin,

When I ask b4 am to fetch either of these message IDs:

   20200626164623.87894-1-sebastian.reichel@collabora.com
   DB6PR1001MB10965E5BDA9B3EF570B607A4806A0@DB6PR1001MB1096.EURPRD10.PROD.OUTLOOK.COM

both of which are for v4 of a single patch that was previously part of a
series b4 fetches v3 of that series rather than the v4 I thought I was
asking for. I see that the patch (the first message ID) was sent in
reply to discussion of v3 of the series which is probably confusing
things, there's also the use of PATCHv4 rather than PATCH v4.

Thanks,
Mark

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 499 bytes --]

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

* Re: b4 picking the wrong message
  2020-07-03 18:52 b4 picking the wrong message Mark Brown
@ 2020-07-06 22:11 ` Konstantin Ryabitsev
  2020-07-07  9:38   ` Mark Brown
  0 siblings, 1 reply; 3+ messages in thread
From: Konstantin Ryabitsev @ 2020-07-06 22:11 UTC (permalink / raw)
  To: Mark Brown; +Cc: tools

On Fri, Jul 03, 2020 at 07:52:37PM +0100, Mark Brown wrote:
> Hi Konstantin,
> 
> When I ask b4 am to fetch either of these message IDs:
> 
>    20200626164623.87894-1-sebastian.reichel@collabora.com
>    DB6PR1001MB10965E5BDA9B3EF570B607A4806A0@DB6PR1001MB1096.EURPRD10.PROD.OUTLOOK.COM
> 
> both of which are for v4 of a single patch that was previously part of a
> series b4 fetches v3 of that series rather than the v4 I thought I was
> asking for. I see that the patch (the first message ID) was sent in
> reply to discussion of v3 of the series which is probably confusing
> things, there's also the use of PATCHv4 rather than PATCH v4.

That's the culprit -- we look for a standalone word "PATCH" inside the 
brackets, so if it's mangled as [PATCHv4], then we don't consider that 
message. I added logic to master/stable-0.5.y to deal with that case.

Thanks,
-K

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

* Re: b4 picking the wrong message
  2020-07-06 22:11 ` Konstantin Ryabitsev
@ 2020-07-07  9:38   ` Mark Brown
  0 siblings, 0 replies; 3+ messages in thread
From: Mark Brown @ 2020-07-07  9:38 UTC (permalink / raw)
  To: Konstantin Ryabitsev; +Cc: tools


[-- Attachment #1: Type: text/plain, Size: 320 bytes --]

On Mon, Jul 06, 2020 at 06:11:36PM -0400, Konstantin Ryabitsev wrote:

> That's the culprit -- we look for a standalone word "PATCH" inside the 
> brackets, so if it's mangled as [PATCHv4], then we don't consider that 
> message. I added logic to master/stable-0.5.y to deal with that case.

That worked, thanks!

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 499 bytes --]

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

end of thread, back to index

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-07-03 18:52 b4 picking the wrong message Mark Brown
2020-07-06 22:11 ` Konstantin Ryabitsev
2020-07-07  9:38   ` Mark Brown

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