10d24de9dSSimon Glass# Copyright (c) 2011 The Chromium OS Authors. 20d24de9dSSimon Glass# 31a459660SWolfgang Denk# SPDX-License-Identifier: GPL-2.0+ 40d24de9dSSimon Glass# 50d24de9dSSimon Glass 631187255SDoug Andersonimport itertools 70d24de9dSSimon Glassimport os 80d24de9dSSimon Glass 921a19d70SDoug Andersonimport get_maintainer 100d24de9dSSimon Glassimport gitutil 110d24de9dSSimon Glassimport terminal 120d24de9dSSimon Glass 130d24de9dSSimon Glass# Series-xxx tags that we understand 14fe2f8d9eSSimon Glassvalid_series = ['to', 'cc', 'version', 'changes', 'prefix', 'notes', 'name', 15645b271aSSimon Glass 'cover-cc', 'process_log'] 160d24de9dSSimon Glass 170d24de9dSSimon Glassclass Series(dict): 180d24de9dSSimon Glass """Holds information about a patch series, including all tags. 190d24de9dSSimon Glass 200d24de9dSSimon Glass Vars: 210d24de9dSSimon Glass cc: List of aliases/emails to Cc all patches to 220d24de9dSSimon Glass commits: List of Commit objects, one for each patch 230d24de9dSSimon Glass cover: List of lines in the cover letter 240d24de9dSSimon Glass notes: List of lines in the notes 250d24de9dSSimon Glass changes: (dict) List of changes for each version, The key is 260d24de9dSSimon Glass the integer version number 27f0b739f1SSimon Glass allow_overwrite: Allow tags to overwrite an existing tag 280d24de9dSSimon Glass """ 290d24de9dSSimon Glass def __init__(self): 300d24de9dSSimon Glass self.cc = [] 310d24de9dSSimon Glass self.to = [] 32fe2f8d9eSSimon Glass self.cover_cc = [] 330d24de9dSSimon Glass self.commits = [] 340d24de9dSSimon Glass self.cover = None 350d24de9dSSimon Glass self.notes = [] 360d24de9dSSimon Glass self.changes = {} 37f0b739f1SSimon Glass self.allow_overwrite = False 380d24de9dSSimon Glass 39d94566a1SDoug Anderson # Written in MakeCcFile() 40d94566a1SDoug Anderson # key: name of patch file 41d94566a1SDoug Anderson # value: list of email addresses 42d94566a1SDoug Anderson self._generated_cc = {} 43d94566a1SDoug Anderson 440d24de9dSSimon Glass # These make us more like a dictionary 450d24de9dSSimon Glass def __setattr__(self, name, value): 460d24de9dSSimon Glass self[name] = value 470d24de9dSSimon Glass 480d24de9dSSimon Glass def __getattr__(self, name): 490d24de9dSSimon Glass return self[name] 500d24de9dSSimon Glass 510d24de9dSSimon Glass def AddTag(self, commit, line, name, value): 520d24de9dSSimon Glass """Add a new Series-xxx tag along with its value. 530d24de9dSSimon Glass 540d24de9dSSimon Glass Args: 550d24de9dSSimon Glass line: Source line containing tag (useful for debug/error messages) 560d24de9dSSimon Glass name: Tag name (part after 'Series-') 570d24de9dSSimon Glass value: Tag value (part after 'Series-xxx: ') 580d24de9dSSimon Glass """ 590d24de9dSSimon Glass # If we already have it, then add to our list 60fe2f8d9eSSimon Glass name = name.replace('-', '_') 61f0b739f1SSimon Glass if name in self and not self.allow_overwrite: 620d24de9dSSimon Glass values = value.split(',') 630d24de9dSSimon Glass values = [str.strip() for str in values] 640d24de9dSSimon Glass if type(self[name]) != type([]): 650d24de9dSSimon Glass raise ValueError("In %s: line '%s': Cannot add another value " 660d24de9dSSimon Glass "'%s' to series '%s'" % 670d24de9dSSimon Glass (commit.hash, line, values, self[name])) 680d24de9dSSimon Glass self[name] += values 690d24de9dSSimon Glass 700d24de9dSSimon Glass # Otherwise just set the value 710d24de9dSSimon Glass elif name in valid_series: 720d24de9dSSimon Glass self[name] = value 730d24de9dSSimon Glass else: 740d24de9dSSimon Glass raise ValueError("In %s: line '%s': Unknown 'Series-%s': valid " 75ef0e9de8SSimon Glass "options are %s" % (commit.hash, line, name, 760d24de9dSSimon Glass ', '.join(valid_series))) 770d24de9dSSimon Glass 780d24de9dSSimon Glass def AddCommit(self, commit): 790d24de9dSSimon Glass """Add a commit into our list of commits 800d24de9dSSimon Glass 810d24de9dSSimon Glass We create a list of tags in the commit subject also. 820d24de9dSSimon Glass 830d24de9dSSimon Glass Args: 840d24de9dSSimon Glass commit: Commit object to add 850d24de9dSSimon Glass """ 860d24de9dSSimon Glass commit.CheckTags() 870d24de9dSSimon Glass self.commits.append(commit) 880d24de9dSSimon Glass 890d24de9dSSimon Glass def ShowActions(self, args, cmd, process_tags): 900d24de9dSSimon Glass """Show what actions we will/would perform 910d24de9dSSimon Glass 920d24de9dSSimon Glass Args: 930d24de9dSSimon Glass args: List of patch files we created 940d24de9dSSimon Glass cmd: The git command we would have run 950d24de9dSSimon Glass process_tags: Process tags as if they were aliases 960d24de9dSSimon Glass """ 972181830fSPeter Tyser to_set = set(gitutil.BuildEmailList(self.to)); 982181830fSPeter Tyser cc_set = set(gitutil.BuildEmailList(self.cc)); 992181830fSPeter Tyser 1000d24de9dSSimon Glass col = terminal.Color() 1010d24de9dSSimon Glass print 'Dry run, so not doing much. But I would do this:' 1020d24de9dSSimon Glass print 1030d24de9dSSimon Glass print 'Send a total of %d patch%s with %scover letter.' % ( 1040d24de9dSSimon Glass len(args), '' if len(args) == 1 else 'es', 1050d24de9dSSimon Glass self.get('cover') and 'a ' or 'no ') 1060d24de9dSSimon Glass 1070d24de9dSSimon Glass # TODO: Colour the patches according to whether they passed checks 1080d24de9dSSimon Glass for upto in range(len(args)): 1090d24de9dSSimon Glass commit = self.commits[upto] 1100d24de9dSSimon Glass print col.Color(col.GREEN, ' %s' % args[upto]) 111d94566a1SDoug Anderson cc_list = list(self._generated_cc[commit.patch]) 1122181830fSPeter Tyser for email in set(cc_list) - to_set - cc_set: 1130d24de9dSSimon Glass if email == None: 1140d24de9dSSimon Glass email = col.Color(col.YELLOW, "<alias '%s' not found>" 1150d24de9dSSimon Glass % tag) 1160d24de9dSSimon Glass if email: 1170d24de9dSSimon Glass print ' Cc: ',email 1180d24de9dSSimon Glass print 1192181830fSPeter Tyser for item in to_set: 1200d24de9dSSimon Glass print 'To:\t ', item 1212181830fSPeter Tyser for item in cc_set - to_set: 1220d24de9dSSimon Glass print 'Cc:\t ', item 1230d24de9dSSimon Glass print 'Version: ', self.get('version') 1240d24de9dSSimon Glass print 'Prefix:\t ', self.get('prefix') 1250d24de9dSSimon Glass if self.cover: 1260d24de9dSSimon Glass print 'Cover: %d lines' % len(self.cover) 127fe2f8d9eSSimon Glass cover_cc = gitutil.BuildEmailList(self.get('cover_cc', '')) 128fe2f8d9eSSimon Glass all_ccs = itertools.chain(cover_cc, *self._generated_cc.values()) 1292181830fSPeter Tyser for email in set(all_ccs) - to_set - cc_set: 13031187255SDoug Anderson print ' Cc: ',email 1310d24de9dSSimon Glass if cmd: 1320d24de9dSSimon Glass print 'Git command: %s' % cmd 1330d24de9dSSimon Glass 1340d24de9dSSimon Glass def MakeChangeLog(self, commit): 1350d24de9dSSimon Glass """Create a list of changes for each version. 1360d24de9dSSimon Glass 1370d24de9dSSimon Glass Return: 1380d24de9dSSimon Glass The change log as a list of strings, one per line 1390d24de9dSSimon Glass 14027e97600SSimon Glass Changes in v4: 141244e6f97SOtavio Salvador - Jog the dial back closer to the widget 142244e6f97SOtavio Salvador 14327e97600SSimon Glass Changes in v3: None 14427e97600SSimon Glass Changes in v2: 1450d24de9dSSimon Glass - Fix the widget 1460d24de9dSSimon Glass - Jog the dial 1470d24de9dSSimon Glass 1480d24de9dSSimon Glass etc. 1490d24de9dSSimon Glass """ 1500d24de9dSSimon Glass final = [] 151645b271aSSimon Glass process_it = self.get('process_log', '').split(',') 152645b271aSSimon Glass process_it = [item.strip() for item in process_it] 1530d24de9dSSimon Glass need_blank = False 154244e6f97SOtavio Salvador for change in sorted(self.changes, reverse=True): 1550d24de9dSSimon Glass out = [] 1560d24de9dSSimon Glass for this_commit, text in self.changes[change]: 1570d24de9dSSimon Glass if commit and this_commit != commit: 1580d24de9dSSimon Glass continue 159645b271aSSimon Glass if 'uniq' not in process_it or text not in out: 1600d24de9dSSimon Glass out.append(text) 16127e97600SSimon Glass line = 'Changes in v%d:' % change 16227e97600SSimon Glass have_changes = len(out) > 0 163645b271aSSimon Glass if 'sort' in process_it: 164645b271aSSimon Glass out = sorted(out) 16527e97600SSimon Glass if have_changes: 16627e97600SSimon Glass out.insert(0, line) 16727e97600SSimon Glass else: 16827e97600SSimon Glass out = [line + ' None'] 1690d24de9dSSimon Glass if need_blank: 17027e97600SSimon Glass out.insert(0, '') 1710d24de9dSSimon Glass final += out 17227e97600SSimon Glass need_blank = have_changes 1730d24de9dSSimon Glass if self.changes: 1740d24de9dSSimon Glass final.append('') 1750d24de9dSSimon Glass return final 1760d24de9dSSimon Glass 1770d24de9dSSimon Glass def DoChecks(self): 1780d24de9dSSimon Glass """Check that each version has a change log 1790d24de9dSSimon Glass 1800d24de9dSSimon Glass Print an error if something is wrong. 1810d24de9dSSimon Glass """ 1820d24de9dSSimon Glass col = terminal.Color() 1830d24de9dSSimon Glass if self.get('version'): 1840d24de9dSSimon Glass changes_copy = dict(self.changes) 185d5f81d8aSOtavio Salvador for version in range(1, int(self.version) + 1): 1860d24de9dSSimon Glass if self.changes.get(version): 1870d24de9dSSimon Glass del changes_copy[version] 1880d24de9dSSimon Glass else: 189d5f81d8aSOtavio Salvador if version > 1: 1900d24de9dSSimon Glass str = 'Change log missing for v%d' % version 1910d24de9dSSimon Glass print col.Color(col.RED, str) 1920d24de9dSSimon Glass for version in changes_copy: 1930d24de9dSSimon Glass str = 'Change log for unknown version v%d' % version 1940d24de9dSSimon Glass print col.Color(col.RED, str) 1950d24de9dSSimon Glass elif self.changes: 1960d24de9dSSimon Glass str = 'Change log exists, but no version is set' 1970d24de9dSSimon Glass print col.Color(col.RED, str) 1980d24de9dSSimon Glass 199983a2749SSimon Glass def MakeCcFile(self, process_tags, cover_fname, raise_on_error, 200983a2749SSimon Glass add_maintainers): 2010d24de9dSSimon Glass """Make a cc file for us to use for per-commit Cc automation 2020d24de9dSSimon Glass 203d94566a1SDoug Anderson Also stores in self._generated_cc to make ShowActions() faster. 204d94566a1SDoug Anderson 2050d24de9dSSimon Glass Args: 2060d24de9dSSimon Glass process_tags: Process tags as if they were aliases 20731187255SDoug Anderson cover_fname: If non-None the name of the cover letter. 208a1318f7cSSimon Glass raise_on_error: True to raise an error when an alias fails to match, 209a1318f7cSSimon Glass False to just print a message. 210983a2749SSimon Glass add_maintainers: Call the get_maintainers to CC maintainers 2110d24de9dSSimon Glass Return: 2120d24de9dSSimon Glass Filename of temp file created 2130d24de9dSSimon Glass """ 2140d24de9dSSimon Glass # Look for commit tags (of the form 'xxx:' at the start of the subject) 2150d24de9dSSimon Glass fname = '/tmp/patman.%d' % os.getpid() 2160d24de9dSSimon Glass fd = open(fname, 'w') 21731187255SDoug Anderson all_ccs = [] 2180d24de9dSSimon Glass for commit in self.commits: 2190d24de9dSSimon Glass list = [] 2200d24de9dSSimon Glass if process_tags: 221a1318f7cSSimon Glass list += gitutil.BuildEmailList(commit.tags, 222a1318f7cSSimon Glass raise_on_error=raise_on_error) 223a1318f7cSSimon Glass list += gitutil.BuildEmailList(commit.cc_list, 224a1318f7cSSimon Glass raise_on_error=raise_on_error) 225983a2749SSimon Glass if add_maintainers: 22621a19d70SDoug Anderson list += get_maintainer.GetMaintainer(commit.patch) 22731187255SDoug Anderson all_ccs += list 2282181830fSPeter Tyser print >>fd, commit.patch, ', '.join(set(list)) 229d94566a1SDoug Anderson self._generated_cc[commit.patch] = list 2300d24de9dSSimon Glass 23131187255SDoug Anderson if cover_fname: 232fe2f8d9eSSimon Glass cover_cc = gitutil.BuildEmailList(self.get('cover_cc', '')) 233fe2f8d9eSSimon Glass print >>fd, cover_fname, ', '.join(set(cover_cc + all_ccs)) 23431187255SDoug Anderson 2350d24de9dSSimon Glass fd.close() 2360d24de9dSSimon Glass return fname 2370d24de9dSSimon Glass 2380d24de9dSSimon Glass def AddChange(self, version, commit, info): 2390d24de9dSSimon Glass """Add a new change line to a version. 2400d24de9dSSimon Glass 2410d24de9dSSimon Glass This will later appear in the change log. 2420d24de9dSSimon Glass 2430d24de9dSSimon Glass Args: 2440d24de9dSSimon Glass version: version number to add change list to 2450d24de9dSSimon Glass info: change line for this version 2460d24de9dSSimon Glass """ 2470d24de9dSSimon Glass if not self.changes.get(version): 2480d24de9dSSimon Glass self.changes[version] = [] 2490d24de9dSSimon Glass self.changes[version].append([commit, info]) 2500d24de9dSSimon Glass 2510d24de9dSSimon Glass def GetPatchPrefix(self): 2520d24de9dSSimon Glass """Get the patch version string 2530d24de9dSSimon Glass 2540d24de9dSSimon Glass Return: 2550d24de9dSSimon Glass Patch string, like 'RFC PATCH v5' or just 'PATCH' 2560d24de9dSSimon Glass """ 257*3871cd85SWu, Josh git_prefix = gitutil.GetDefaultSubjectPrefix() 258*3871cd85SWu, Josh if git_prefix: 259*3871cd85SWu, Josh git_prefix = '%s][' % git_prefix 260*3871cd85SWu, Josh else: 261*3871cd85SWu, Josh git_prefix = '' 262*3871cd85SWu, Josh 2630d24de9dSSimon Glass version = '' 2640d24de9dSSimon Glass if self.get('version'): 2650d24de9dSSimon Glass version = ' v%s' % self['version'] 2660d24de9dSSimon Glass 2670d24de9dSSimon Glass # Get patch name prefix 2680d24de9dSSimon Glass prefix = '' 2690d24de9dSSimon Glass if self.get('prefix'): 2700d24de9dSSimon Glass prefix = '%s ' % self['prefix'] 271*3871cd85SWu, Josh return '%s%sPATCH%s' % (git_prefix, prefix, version) 272