diff options
author | Konstantin Ryabitsev <konstantin@linuxfoundation.org> | 2020-04-10 11:19:42 -0400 |
---|---|---|
committer | Konstantin Ryabitsev <konstantin@linuxfoundation.org> | 2020-04-10 11:19:42 -0400 |
commit | 39181293c0cb1780e56e3eaee9ac2cc8abd8d43f (patch) | |
tree | 479b4bdd27f20ca8389ce16086df7cd0bf69abe0 /b4/mbox.py | |
parent | 7210e188cc087f2f9a2aba23420d781d6d1a8691 (diff) | |
download | b4-39181293c0cb1780e56e3eaee9ac2cc8abd8d43f.tar.gz |
Don't use git patch-id for patch tracking
Looks like some of the patches sent from quilt (at least from Andrew
Morton) are not properly parsed by git patch-id, so use our own patch
hashing routines, even if this means we're likely going to miss some of
the patches that were edited by maintainers.
Signed-off-by: Konstantin Ryabitsev <konstantin@linuxfoundation.org>
Diffstat (limited to 'b4/mbox.py')
-rw-r--r-- | b4/mbox.py | 11 |
1 files changed, 4 insertions, 7 deletions
@@ -155,16 +155,13 @@ def thanks_record_am(lser): if entry == filename: return - # Get patch-id of each patch in the series - gitargs = ['patch-id', '--stable'] patches = list() for pmsg in lser.patches[1:]: - ecode, out = b4.git_run_command(None, gitargs, stdin=pmsg.body.encode('utf-8')) - if ecode > 0 or not len(out.strip()): - logger.debug('Could not get patch-id of %s', pmsg.full_subject) + pmsg.load_hashes() + if pmsg.attestation is None: + logger.debug('Unable to get hashes for all patches, not tracking for thanks') return - chunks = out.split() - patches.append((pmsg.subject, chunks[0])) + patches.append((pmsg.subject, pmsg.attestation.p)) lmsg = lser.patches[0] if lmsg is None: |