xref: /rk3399_rockchip-uboot/tools/patman/series.py (revision 070b781b2beb5298cd904fd514ba50e8530b0e2c)
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