search

mricon / b4

Tool to help with email-based patch workflows
GNU General Public License v2.0
59 stars 20 forks source link

b4 confusing separate patches in one series as different versions of one patch #31

Closed wens closed 2 months ago

wens commented 2 months ago

With b4 v0.12.4, while applying the patch series from message ID 0240516122102.16379-1-yunfei.dong@mediatek.com, b4 seems to be confusing the individual patches as different versions of one patch, and only applies the one patch it considers the latest one, while ignoring the other 23 patches:

$ b4 shazam -C  -l -s -t 20240516122102.16379-1-yunfei.dong@mediatek.com
Grabbing thread from lore.kernel.org/all/20240516122102.16379-1-yunfei.dong@mediatek.com/t.mbox.gz
Checking for newer revisions
Grabbing search results from lore.kernel.org
Analyzing 38 messages in the thread
Assuming new revision: v2 ([PATCH v6,19/24] media: mediatek: vcodec: disable wait interrupt for svp mode)
Assuming new revision: v3 ([PATCH v6,23/24] media: mediatek: vcodec: support av1 svp decoder for mt8188)
Assuming new revision: v4 ([PATCH v6,22/24] media: mediatek: vcodec: support hevc svp for mt8188)
Assuming new revision: v5 ([PATCH v6,24/24] media: mediatek: vcodec: support vp9 svp decoder for mt8188)
Assuming new revision: v6 ([PATCH v6,18/24] media: mediatek: vcodec: remove parse nal_info in kernel)
Assuming new revision: v7 ([PATCH v6,20/24] media: mediatek: vcodec: support tee decoder)
Assuming new revision: v8 ([PATCH v6,17/24] media: mediatek: vcodec: re-construct h264 driver to support svp mode)
Assuming new revision: v9 ([PATCH v6,21/24] media: mediatek: vcodec: move vdec init interface to setup callback)
Assuming new revision: v10 ([PATCH v6,16/24] media: mediatek: vcodec: support one plane capture buffer)
Will use the latest revision: v10
You can pick other revisions using the -vN flag
Checking attestation on all messages, may take a moment...
---
  ✓ [PATCH] media: mediatek: vcodec: support one plane capture buffer
    + Link: https://lore.kernel.org/r/20240516122102.16379-17-yunfei.dong@mediatek.com
    + Signed-off-by: Chen-Yu Tsai <wenst@chromium.org>
  ---
  ✓ Signed: DKIM/mediatek.com
---
Total patches: 1
---
Applying: media: mediatek: vcodec: support one plane capture buffer

Not sure if it's because of the comma in the subject prefix or something else.

mricon commented 2 months ago

Yes, it's because of the comma in the prefixes. Sorry, but I cannot fix this one -- please ask the author to submit a well-formed series.