[yocto] [patchwork][PATCH v3] parsemail.py: Improve find_series_for_mail
Jose Lamego
jose.a.lamego at linux.intel.com
Fri Mar 31 10:24:57 PDT 2017
Patch Series that are sent as replies to other Series with
different patch quantity are not appropriately detected as
such, breaking the thread and causing missing revisions.
This change improves the find_series_for_mail function to
appropriately detect when a message is part of a thread
by performing an additional patch-query and detecting cover
letters that are replies to cover letters.
[YOCTO #10959]
Signed-off-by: Jose Lamego <jose.a.lamego at linux.intel.com>
---
Notes:
changes in v3: a couple of typos fixed
patchwork/bin/parsemail.py | 20 +++++++++++++++++---
1 file changed, 17 insertions(+), 3 deletions(-)
diff --git a/patchwork/bin/parsemail.py b/patchwork/bin/parsemail.py
index 61c18b2..3c3046d 100755
--- a/patchwork/bin/parsemail.py
+++ b/patchwork/bin/parsemail.py
@@ -399,7 +399,7 @@ def find_content(project, mail):
(x, n) = parse_series_marker(prefixes)
refs = build_references_list(mail)
is_root = refs == []
- is_cover_letter = is_root and x == 0
+ is_cover_letter = x == 0
patch_prefix = re.match('(\s*\[[^]]*\]\s*)*\[\s*PATCH',
mail.get('Subject'))
is_patch = patchbuf is not None and patch_prefix
@@ -441,7 +441,7 @@ def find_content(project, mail):
(ret.series, ret.revision, ret.patch_order, n) = \
find_series_for_mail(project, series_name, msgid, is_patch,
- ret.patch_order, n, refs)
+ ret.patch_order, n, refs, is_cover_letter)
ret.revision.n_patches = n or 1
date = mail_date(mail)
@@ -535,7 +535,7 @@ def find_patch_order(revisions, previous_patch, order, n_patches):
# - we need to create new revisions when the mail is actually a new version
# of a previous patch
def find_series_for_mail(project, name, msgid, is_patch, order, n_patches,
- refs):
+ refs, is_cover_letter):
if refs == []:
root_msgid = msgid
else:
@@ -560,6 +560,20 @@ def find_series_for_mail(project, name, msgid, is_patch, order, n_patches,
revision = revision.duplicate(exclude_patches=(order,))
# series has been updated, grab the new instance
series = revision.series
+ else:
+ try:
+ prev_patch = SeriesRevisionPatch.objects.get(
+ revision=revision, order=order).patch
+ if prev_patch and not prev_patch.msgid == msgid:
+ # this is a new patch in the thread
+ revision = revision.duplicate(exclude_patches=(order,))
+ series = revision.series
+ except SeriesRevisionPatch.DoesNotExist:
+ pass
+ elif is_cover_letter and not refs == []:
+ revision = revision.duplicate(
+ exclude_patches=range(1, revision.n_patches+1),)
+ series = revision.series
except IndexError:
if not name:
name = SERIES_DEFAULT_NAME
--
2.7.4
More information about the yocto
mailing list