From 8d14a7a089acad173917bb03d204b69c15450391 Mon Sep 17 00:00:00 2001 From: Konstantin Ryabitsev Date: Mon, 8 Jun 2020 14:03:27 -0400 Subject: Output less confusing trailer mismatch messages Show the actual trailer name we're complaining about. Signed-off-by: Konstantin Ryabitsev --- b4/__init__.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'b4/__init__.py') diff --git a/b4/__init__.py b/b4/__init__.py index 79a0c2e..7edc799 100644 --- a/b4/__init__.py +++ b/b4/__init__.py @@ -284,7 +284,7 @@ class LoreMailbox: trailers, mismatches = fmsg.get_trailers(sloppy=sloppytrailers) for tname, tvalue in mismatches: - lser.trailer_mismatches.add((tvalue, fmsg.fromname, fmsg.fromemail)) + lser.trailer_mismatches.add((tname, tvalue, fmsg.fromname, fmsg.fromemail)) lvl = 1 while True: logger.debug('%sParent: %s', ' ' * lvl, pmsg.full_subject) @@ -881,7 +881,7 @@ class LoreMessage: # Do we have something that looks like a person-trailer? matches = re.findall(r'^\s*([\w-]+):[ \t]+(.*<\S+>)\s*$', self.body, re.MULTILINE) # These are commonly part of patch/commit metadata - badtrailers = ('from', 'author') + badtrailers = ('from', 'author', 'cc') if matches: for tname, tvalue in matches: if tname.lower() not in badtrailers: -- cgit v1.2.3