diff options
| author | Konstantin Ryabitsev <konstantin@linuxfoundation.org> | 2020-06-08 14:03:27 -0400 |
|---|---|---|
| committer | Konstantin Ryabitsev <konstantin@linuxfoundation.org> | 2020-06-08 14:04:30 -0400 |
| commit | 8d14a7a089acad173917bb03d204b69c15450391 (patch) | |
| tree | f11de602728687f3c78e17cc30d3a87ebdf3b952 | |
| parent | c8c63b1a31c65cf22ac0bf969c8c66eee6151427 (diff) | |
| download | b4-8d14a7a089acad173917bb03d204b69c15450391.tar.gz | |
Output less confusing trailer mismatch messages
Show the actual trailer name we're complaining about.
Signed-off-by: Konstantin Ryabitsev <konstantin@linuxfoundation.org>
| -rw-r--r-- | b4/__init__.py | 4 | ||||
| -rw-r--r-- | b4/mbox.py | 6 |
2 files changed, 5 insertions, 5 deletions
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: @@ -127,9 +127,9 @@ def mbox_to_am(mboxfile, cmdargs): if len(lser.trailer_mismatches): logger.critical('---') logger.critical('NOTE: some trailers ignored due to from/email mismatches:') - for tvalue, fname, femail in lser.trailer_mismatches: - logger.critical(' ! Trailer: %s', tvalue) - logger.critical(' From: %s <%s>', fname, femail) + for tname, tvalue, fname, femail in lser.trailer_mismatches: + logger.critical(' ! Trailer: %s: %s', tname, tvalue) + logger.critical(' Msg From: %s <%s>', fname, femail) logger.critical('NOTE: Rerun with -S to apply them anyway') topdir = None |