diff options
Diffstat (limited to 'tools/patman/patchstream.py')
| -rw-r--r-- | tools/patman/patchstream.py | 43 | 
1 files changed, 36 insertions, 7 deletions
| diff --git a/tools/patman/patchstream.py b/tools/patman/patchstream.py index c2045230a..684204c63 100644 --- a/tools/patman/patchstream.py +++ b/tools/patman/patchstream.py @@ -30,7 +30,10 @@ re_cover = re.compile('^Cover-letter:')  re_cover_cc = re.compile('^Cover-letter-cc: *(.*)')  # Patch series tag -re_series = re.compile('^Series-([a-z-]*): *(.*)') +re_series_tag = re.compile('^Series-([a-z-]*): *(.*)') + +# Commit series tag +re_commit_tag = re.compile('^Commit-([a-z-]*): *(.*)')  # Commit tags that we want to collect and keep  re_tag = re.compile('^(Tested-by|Acked-by|Reviewed-by|Cc): (.*)') @@ -90,6 +93,20 @@ class PatchStream:          if self.is_log:              self.series.AddTag(self.commit, line, name, value) +    def AddToCommit(self, line, name, value): +        """Add a new Commit-xxx tag. + +        When a Commit-xxx tag is detected, we come here to record it. + +        Args: +            line: Source line containing tag (useful for debug/error messages) +            name: Tag name (part after 'Commit-') +            value: Tag value (part after 'Commit-xxx: ') +        """ +        if name == 'notes': +            self.in_section = 'commit-' + name +            self.skip_blank = False +      def CloseCommit(self):          """Save the current commit into our commit list, and reset our state"""          if self.commit and self.is_log: @@ -138,7 +155,8 @@ class PatchStream:                  line = line[4:]          # Handle state transition and skipping blank lines -        series_match = re_series.match(line) +        series_tag_match = re_series_tag.match(line) +        commit_tag_match = re_commit_tag.match(line)          commit_match = re_commit.match(line) if self.is_log else None          cover_cc_match = re_cover_cc.match(line)          tag_match = None @@ -165,6 +183,9 @@ class PatchStream:                  elif self.in_section == 'notes':                      if self.is_log:                          self.series.notes += self.section +                elif self.in_section == 'commit-notes': +                    if self.is_log: +                        self.commit.notes += self.section                  else:                      self.warn.append("Unknown section '%s'" % self.in_section)                  self.in_section = None @@ -178,7 +199,7 @@ class PatchStream:              self.commit.subject = line          # Detect the tags we want to remove, and skip blank lines -        elif re_remove.match(line): +        elif re_remove.match(line) and not commit_tag_match:              self.skip_blank = True              # TEST= should be the last thing in the commit, so remove @@ -211,9 +232,9 @@ class PatchStream:              self.skip_blank = False          # Detect Series-xxx tags -        elif series_match: -            name = series_match.group(1) -            value = series_match.group(2) +        elif series_tag_match: +            name = series_tag_match.group(1) +            value = series_tag_match.group(2)              if name == 'changes':                  # value is the version number: e.g. 1, or 2                  try: @@ -226,6 +247,14 @@ class PatchStream:                  self.AddToSeries(line, name, value)                  self.skip_blank = True +        # Detect Commit-xxx tags +        elif commit_tag_match: +            name = commit_tag_match.group(1) +            value = commit_tag_match.group(2) +            if name == 'notes': +                self.AddToCommit(line, name, value) +                self.skip_blank = True +          # Detect the start of a new commit          elif commit_match:              self.CloseCommit() @@ -276,7 +305,7 @@ class PatchStream:                  out = []                  log = self.series.MakeChangeLog(self.commit)                  out += self.FormatTags(self.tags) -                out += [line] + log +                out += [line] + self.commit.notes + [''] + log              elif self.found_test:                  if not re_allowed_after_test.match(line):                      self.lines_after_test += 1 |