Tom Rini | 83d290c | 2018-05-06 17:58:06 -0400 | [diff] [blame] | 1 | # SPDX-License-Identifier: GPL-2.0+ |
Simon Glass | 0d24de9 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 2 | # Copyright (c) 2011 The Chromium OS Authors. |
| 3 | # |
Simon Glass | 0d24de9 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 4 | |
Simon Glass | 7207e2b | 2020-07-05 21:41:57 -0600 | [diff] [blame] | 5 | import collections |
Simon Glass | 0d24de9 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 6 | import re |
| 7 | |
| 8 | # Separates a tag: at the beginning of the subject from the rest of it |
Simon Glass | ed92227 | 2013-03-26 13:09:40 +0000 | [diff] [blame] | 9 | re_subject_tag = re.compile('([^:\s]*):\s*(.*)') |
Simon Glass | 0d24de9 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 10 | |
| 11 | class Commit: |
| 12 | """Holds information about a single commit/patch in the series. |
| 13 | |
| 14 | Args: |
| 15 | hash: Commit hash (as a string) |
| 16 | |
| 17 | Variables: |
| 18 | hash: Commit hash |
| 19 | subject: Subject line |
| 20 | tags: List of maintainer tag strings |
| 21 | changes: Dict containing a list of changes (single line strings). |
| 22 | The dict is indexed by change version (an integer) |
| 23 | cc_list: List of people to aliases/emails to cc on this commit |
Albert ARIBAUD | 5c8fdd9 | 2013-11-12 11:14:41 +0100 | [diff] [blame] | 24 | notes: List of lines in the commit (not series) notes |
Douglas Anderson | 833e419 | 2019-09-27 09:23:56 -0700 | [diff] [blame] | 25 | change_id: the Change-Id: tag that was stripped from this commit |
| 26 | and can be used to generate the Message-Id. |
Simon Glass | 7207e2b | 2020-07-05 21:41:57 -0600 | [diff] [blame] | 27 | rtags: Response tags (e.g. Reviewed-by) collected by the commit, dict: |
| 28 | key: rtag type (e.g. 'Reviewed-by') |
| 29 | value: Set of people who gave that rtag, each a name/email string |
Simon Glass | 313ef5f | 2020-10-29 21:46:24 -0600 | [diff] [blame] | 30 | warn: List of warnings for this commit, each a str |
Simon Glass | fc6c663 | 2022-03-02 19:12:24 -0700 | [diff] [blame] | 31 | patch (str): Filename of the patch file for this commit |
Simon Glass | 0d24de9 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 32 | """ |
| 33 | def __init__(self, hash): |
| 34 | self.hash = hash |
Simon Glass | 32cc6ae | 2022-02-11 13:23:18 -0700 | [diff] [blame] | 35 | self.subject = '' |
Simon Glass | 0d24de9 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 36 | self.tags = [] |
| 37 | self.changes = {} |
| 38 | self.cc_list = [] |
Simon Glass | 102061b | 2014-04-20 10:50:14 -0600 | [diff] [blame] | 39 | self.signoff_set = set() |
Albert ARIBAUD | 5c8fdd9 | 2013-11-12 11:14:41 +0100 | [diff] [blame] | 40 | self.notes = [] |
Douglas Anderson | 833e419 | 2019-09-27 09:23:56 -0700 | [diff] [blame] | 41 | self.change_id = None |
Simon Glass | 7207e2b | 2020-07-05 21:41:57 -0600 | [diff] [blame] | 42 | self.rtags = collections.defaultdict(set) |
Simon Glass | 313ef5f | 2020-10-29 21:46:24 -0600 | [diff] [blame] | 43 | self.warn = [] |
Simon Glass | fc6c663 | 2022-03-02 19:12:24 -0700 | [diff] [blame] | 44 | self.patch = '' |
Simon Glass | 0d24de9 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 45 | |
Simon Glass | a12ad7c | 2020-10-29 21:46:32 -0600 | [diff] [blame] | 46 | def __str__(self): |
| 47 | return self.subject |
| 48 | |
Simon Glass | a3eeadf | 2022-01-29 14:14:07 -0700 | [diff] [blame] | 49 | def add_change(self, version, info): |
Simon Glass | 0d24de9 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 50 | """Add a new change line to the change list for a version. |
| 51 | |
| 52 | Args: |
| 53 | version: Patch set version (integer: 1, 2, 3) |
| 54 | info: Description of change in this version |
| 55 | """ |
| 56 | if not self.changes.get(version): |
| 57 | self.changes[version] = [] |
| 58 | self.changes[version].append(info) |
| 59 | |
Simon Glass | a3eeadf | 2022-01-29 14:14:07 -0700 | [diff] [blame] | 60 | def check_tags(self): |
Simon Glass | 0d24de9 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 61 | """Create a list of subject tags in the commit |
| 62 | |
| 63 | Subject tags look like this: |
| 64 | |
Simon Glass | 0d99fe0 | 2013-03-26 13:09:41 +0000 | [diff] [blame] | 65 | propounder: fort: Change the widget to propound correctly |
Simon Glass | 0d24de9 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 66 | |
Simon Glass | 0d99fe0 | 2013-03-26 13:09:41 +0000 | [diff] [blame] | 67 | Here the tags are propounder and fort. Multiple tags are supported. |
| 68 | The list is updated in self.tag. |
Simon Glass | 0d24de9 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 69 | |
| 70 | Returns: |
| 71 | None if ok, else the name of a tag with no email alias |
| 72 | """ |
| 73 | str = self.subject |
| 74 | m = True |
| 75 | while m: |
| 76 | m = re_subject_tag.match(str) |
| 77 | if m: |
| 78 | tag = m.group(1) |
| 79 | self.tags.append(tag) |
| 80 | str = m.group(2) |
| 81 | return None |
| 82 | |
Simon Glass | a3eeadf | 2022-01-29 14:14:07 -0700 | [diff] [blame] | 83 | def add_cc(self, cc_list): |
Simon Glass | 0d24de9 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 84 | """Add a list of people to Cc when we send this patch. |
| 85 | |
| 86 | Args: |
| 87 | cc_list: List of aliases or email addresses |
| 88 | """ |
| 89 | self.cc_list += cc_list |
Simon Glass | 102061b | 2014-04-20 10:50:14 -0600 | [diff] [blame] | 90 | |
Simon Glass | a3eeadf | 2022-01-29 14:14:07 -0700 | [diff] [blame] | 91 | def check_duplicate_signoff(self, signoff): |
Simon Glass | 102061b | 2014-04-20 10:50:14 -0600 | [diff] [blame] | 92 | """Check a list of signoffs we have send for this patch |
| 93 | |
| 94 | Args: |
| 95 | signoff: Signoff line |
| 96 | Returns: |
| 97 | True if this signoff is new, False if we have already seen it. |
| 98 | """ |
| 99 | if signoff in self.signoff_set: |
| 100 | return False |
| 101 | self.signoff_set.add(signoff) |
| 102 | return True |
Simon Glass | 7207e2b | 2020-07-05 21:41:57 -0600 | [diff] [blame] | 103 | |
Simon Glass | a3eeadf | 2022-01-29 14:14:07 -0700 | [diff] [blame] | 104 | def add_rtag(self, rtag_type, who): |
Simon Glass | 7207e2b | 2020-07-05 21:41:57 -0600 | [diff] [blame] | 105 | """Add a response tag to a commit |
| 106 | |
| 107 | Args: |
| 108 | key: rtag type (e.g. 'Reviewed-by') |
| 109 | who: Person who gave that rtag, e.g. 'Fred Bloggs <fred@bloggs.org>' |
| 110 | """ |
| 111 | self.rtags[rtag_type].add(who) |