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', 15d9917b0bSSimon 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: 72*070b781bSAlbert ARIBAUD if name=="notes": 73*070b781bSAlbert ARIBAUD self[name] = [value] 74*070b781bSAlbert ARIBAUD else: 750d24de9dSSimon Glass self[name] = value 760d24de9dSSimon Glass else: 770d24de9dSSimon Glass raise ValueError("In %s: line '%s': Unknown 'Series-%s': valid " 78ef0e9de8SSimon Glass "options are %s" % (commit.hash, line, name, 790d24de9dSSimon Glass ', '.join(valid_series))) 800d24de9dSSimon Glass 810d24de9dSSimon Glass def AddCommit(self, commit): 820d24de9dSSimon Glass """Add a commit into our list of commits 830d24de9dSSimon Glass 840d24de9dSSimon Glass We create a list of tags in the commit subject also. 850d24de9dSSimon Glass 860d24de9dSSimon Glass Args: 870d24de9dSSimon Glass commit: Commit object to add 880d24de9dSSimon Glass """ 890d24de9dSSimon Glass commit.CheckTags() 900d24de9dSSimon Glass self.commits.append(commit) 910d24de9dSSimon Glass 920d24de9dSSimon Glass def ShowActions(self, args, cmd, process_tags): 930d24de9dSSimon Glass """Show what actions we will/would perform 940d24de9dSSimon Glass 950d24de9dSSimon Glass Args: 960d24de9dSSimon Glass args: List of patch files we created 970d24de9dSSimon Glass cmd: The git command we would have run 980d24de9dSSimon Glass process_tags: Process tags as if they were aliases 990d24de9dSSimon Glass """ 1002181830fSPeter Tyser to_set = set(gitutil.BuildEmailList(self.to)); 1012181830fSPeter Tyser cc_set = set(gitutil.BuildEmailList(self.cc)); 1022181830fSPeter Tyser 1030d24de9dSSimon Glass col = terminal.Color() 1040d24de9dSSimon Glass print 'Dry run, so not doing much. But I would do this:' 1050d24de9dSSimon Glass print 1060d24de9dSSimon Glass print 'Send a total of %d patch%s with %scover letter.' % ( 1070d24de9dSSimon Glass len(args), '' if len(args) == 1 else 'es', 1080d24de9dSSimon Glass self.get('cover') and 'a ' or 'no ') 1090d24de9dSSimon Glass 1100d24de9dSSimon Glass # TODO: Colour the patches according to whether they passed checks 1110d24de9dSSimon Glass for upto in range(len(args)): 1120d24de9dSSimon Glass commit = self.commits[upto] 1130d24de9dSSimon Glass print col.Color(col.GREEN, ' %s' % args[upto]) 114d94566a1SDoug Anderson cc_list = list(self._generated_cc[commit.patch]) 1152181830fSPeter Tyser for email in set(cc_list) - to_set - cc_set: 1160d24de9dSSimon Glass if email == None: 1170d24de9dSSimon Glass email = col.Color(col.YELLOW, "<alias '%s' not found>" 1180d24de9dSSimon Glass % tag) 1190d24de9dSSimon Glass if email: 1200d24de9dSSimon Glass print ' Cc: ',email 1210d24de9dSSimon Glass print 1222181830fSPeter Tyser for item in to_set: 1230d24de9dSSimon Glass print 'To:\t ', item 1242181830fSPeter Tyser for item in cc_set - to_set: 1250d24de9dSSimon Glass print 'Cc:\t ', item 1260d24de9dSSimon Glass print 'Version: ', self.get('version') 1270d24de9dSSimon Glass print 'Prefix:\t ', self.get('prefix') 1280d24de9dSSimon Glass if self.cover: 1290d24de9dSSimon Glass print 'Cover: %d lines' % len(self.cover) 130fe2f8d9eSSimon Glass cover_cc = gitutil.BuildEmailList(self.get('cover_cc', '')) 131fe2f8d9eSSimon Glass all_ccs = itertools.chain(cover_cc, *self._generated_cc.values()) 1322181830fSPeter Tyser for email in set(all_ccs) - to_set - cc_set: 13331187255SDoug Anderson print ' Cc: ',email 1340d24de9dSSimon Glass if cmd: 1350d24de9dSSimon Glass print 'Git command: %s' % cmd 1360d24de9dSSimon Glass 1370d24de9dSSimon Glass def MakeChangeLog(self, commit): 1380d24de9dSSimon Glass """Create a list of changes for each version. 1390d24de9dSSimon Glass 1400d24de9dSSimon Glass Return: 1410d24de9dSSimon Glass The change log as a list of strings, one per line 1420d24de9dSSimon Glass 14327e97600SSimon Glass Changes in v4: 144244e6f97SOtavio Salvador - Jog the dial back closer to the widget 145244e6f97SOtavio Salvador 14627e97600SSimon Glass Changes in v3: None 14727e97600SSimon Glass Changes in v2: 1480d24de9dSSimon Glass - Fix the widget 1490d24de9dSSimon Glass - Jog the dial 1500d24de9dSSimon Glass 1510d24de9dSSimon Glass etc. 1520d24de9dSSimon Glass """ 1530d24de9dSSimon Glass final = [] 154645b271aSSimon Glass process_it = self.get('process_log', '').split(',') 155645b271aSSimon Glass process_it = [item.strip() for item in process_it] 1560d24de9dSSimon Glass need_blank = False 157244e6f97SOtavio Salvador for change in sorted(self.changes, reverse=True): 1580d24de9dSSimon Glass out = [] 1590d24de9dSSimon Glass for this_commit, text in self.changes[change]: 1600d24de9dSSimon Glass if commit and this_commit != commit: 1610d24de9dSSimon Glass continue 162645b271aSSimon Glass if 'uniq' not in process_it or text not in out: 1630d24de9dSSimon Glass out.append(text) 16427e97600SSimon Glass line = 'Changes in v%d:' % change 16527e97600SSimon Glass have_changes = len(out) > 0 166645b271aSSimon Glass if 'sort' in process_it: 167645b271aSSimon Glass out = sorted(out) 16827e97600SSimon Glass if have_changes: 16927e97600SSimon Glass out.insert(0, line) 17027e97600SSimon Glass else: 17127e97600SSimon Glass out = [line + ' None'] 1720d24de9dSSimon Glass if need_blank: 17327e97600SSimon Glass out.insert(0, '') 1740d24de9dSSimon Glass final += out 17527e97600SSimon Glass need_blank = have_changes 1760d24de9dSSimon Glass if self.changes: 1770d24de9dSSimon Glass final.append('') 1780d24de9dSSimon Glass return final 1790d24de9dSSimon Glass 1800d24de9dSSimon Glass def DoChecks(self): 1810d24de9dSSimon Glass """Check that each version has a change log 1820d24de9dSSimon Glass 1830d24de9dSSimon Glass Print an error if something is wrong. 1840d24de9dSSimon Glass """ 1850d24de9dSSimon Glass col = terminal.Color() 1860d24de9dSSimon Glass if self.get('version'): 1870d24de9dSSimon Glass changes_copy = dict(self.changes) 188d5f81d8aSOtavio Salvador for version in range(1, int(self.version) + 1): 1890d24de9dSSimon Glass if self.changes.get(version): 1900d24de9dSSimon Glass del changes_copy[version] 1910d24de9dSSimon Glass else: 192d5f81d8aSOtavio Salvador if version > 1: 1930d24de9dSSimon Glass str = 'Change log missing for v%d' % version 1940d24de9dSSimon Glass print col.Color(col.RED, str) 1950d24de9dSSimon Glass for version in changes_copy: 1960d24de9dSSimon Glass str = 'Change log for unknown version v%d' % version 1970d24de9dSSimon Glass print col.Color(col.RED, str) 1980d24de9dSSimon Glass elif self.changes: 1990d24de9dSSimon Glass str = 'Change log exists, but no version is set' 2000d24de9dSSimon Glass print col.Color(col.RED, str) 2010d24de9dSSimon Glass 202983a2749SSimon Glass def MakeCcFile(self, process_tags, cover_fname, raise_on_error, 203983a2749SSimon Glass add_maintainers): 2040d24de9dSSimon Glass """Make a cc file for us to use for per-commit Cc automation 2050d24de9dSSimon Glass 206d94566a1SDoug Anderson Also stores in self._generated_cc to make ShowActions() faster. 207d94566a1SDoug Anderson 2080d24de9dSSimon Glass Args: 2090d24de9dSSimon Glass process_tags: Process tags as if they were aliases 21031187255SDoug Anderson cover_fname: If non-None the name of the cover letter. 211a1318f7cSSimon Glass raise_on_error: True to raise an error when an alias fails to match, 212a1318f7cSSimon Glass False to just print a message. 213983a2749SSimon Glass add_maintainers: Call the get_maintainers to CC maintainers 2140d24de9dSSimon Glass Return: 2150d24de9dSSimon Glass Filename of temp file created 2160d24de9dSSimon Glass """ 2170d24de9dSSimon Glass # Look for commit tags (of the form 'xxx:' at the start of the subject) 2180d24de9dSSimon Glass fname = '/tmp/patman.%d' % os.getpid() 2190d24de9dSSimon Glass fd = open(fname, 'w') 22031187255SDoug Anderson all_ccs = [] 2210d24de9dSSimon Glass for commit in self.commits: 2220d24de9dSSimon Glass list = [] 2230d24de9dSSimon Glass if process_tags: 224a1318f7cSSimon Glass list += gitutil.BuildEmailList(commit.tags, 225a1318f7cSSimon Glass raise_on_error=raise_on_error) 226a1318f7cSSimon Glass list += gitutil.BuildEmailList(commit.cc_list, 227a1318f7cSSimon Glass raise_on_error=raise_on_error) 228983a2749SSimon Glass if add_maintainers: 22921a19d70SDoug Anderson list += get_maintainer.GetMaintainer(commit.patch) 23031187255SDoug Anderson all_ccs += list 2312181830fSPeter Tyser print >>fd, commit.patch, ', '.join(set(list)) 232d94566a1SDoug Anderson self._generated_cc[commit.patch] = list 2330d24de9dSSimon Glass 23431187255SDoug Anderson if cover_fname: 235fe2f8d9eSSimon Glass cover_cc = gitutil.BuildEmailList(self.get('cover_cc', '')) 236fe2f8d9eSSimon Glass print >>fd, cover_fname, ', '.join(set(cover_cc + all_ccs)) 23731187255SDoug Anderson 2380d24de9dSSimon Glass fd.close() 2390d24de9dSSimon Glass return fname 2400d24de9dSSimon Glass 2410d24de9dSSimon Glass def AddChange(self, version, commit, info): 2420d24de9dSSimon Glass """Add a new change line to a version. 2430d24de9dSSimon Glass 2440d24de9dSSimon Glass This will later appear in the change log. 2450d24de9dSSimon Glass 2460d24de9dSSimon Glass Args: 2470d24de9dSSimon Glass version: version number to add change list to 2480d24de9dSSimon Glass info: change line for this version 2490d24de9dSSimon Glass """ 2500d24de9dSSimon Glass if not self.changes.get(version): 2510d24de9dSSimon Glass self.changes[version] = [] 2520d24de9dSSimon Glass self.changes[version].append([commit, info]) 2530d24de9dSSimon Glass 2540d24de9dSSimon Glass def GetPatchPrefix(self): 2550d24de9dSSimon Glass """Get the patch version string 2560d24de9dSSimon Glass 2570d24de9dSSimon Glass Return: 2580d24de9dSSimon Glass Patch string, like 'RFC PATCH v5' or just 'PATCH' 2590d24de9dSSimon Glass """ 2603871cd85SWu, Josh git_prefix = gitutil.GetDefaultSubjectPrefix() 2613871cd85SWu, Josh if git_prefix: 2623871cd85SWu, Josh git_prefix = '%s][' % git_prefix 2633871cd85SWu, Josh else: 2643871cd85SWu, Josh git_prefix = '' 2653871cd85SWu, Josh 2660d24de9dSSimon Glass version = '' 2670d24de9dSSimon Glass if self.get('version'): 2680d24de9dSSimon Glass version = ' v%s' % self['version'] 2690d24de9dSSimon Glass 2700d24de9dSSimon Glass # Get patch name prefix 2710d24de9dSSimon Glass prefix = '' 2720d24de9dSSimon Glass if self.get('prefix'): 2730d24de9dSSimon Glass prefix = '%s ' % self['prefix'] 2743871cd85SWu, Josh return '%s%sPATCH%s' % (git_prefix, prefix, version) 275