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