[U-Boot] [PATCH 2/2] patman: Add all CC addresses to the cover letter

Simon Glass sjg at chromium.org
Tue Dec 4 00:00:18 CET 2012


On Fri, Nov 30, 2012 at 4:25 PM, Doug Anderson <dianders at chromium.org> wrote:
> If we're sending a cover letter make sure to CC everyone that we're
> CCing on each of the individual patches.
>
> Signed-off-by: Doug Anderson <dianders at chromium.org>

Looks good, but can you please add a note to the README under the
'Where Patches Are Sent' header which mentions where the cover letter
is sent?

> ---
>  tools/patman/patman.py |    2 +-
>  tools/patman/series.py |   12 +++++++++++-
>  2 files changed, 12 insertions(+), 2 deletions(-)
>
> diff --git a/tools/patman/patman.py b/tools/patman/patman.py
> index de8314a..4181d80 100755
> --- a/tools/patman/patman.py
> +++ b/tools/patman/patman.py
> @@ -140,7 +140,7 @@ else:
>              options.count + options.start):
>          ok = False
>
> -    cc_file = series.MakeCcFile(options.process_tags)
> +    cc_file = series.MakeCcFile(options.process_tags, cover_fname)
>
>      # Email the patches out (giving the user time to check / cancel)
>      cmd = ''
> diff --git a/tools/patman/series.py b/tools/patman/series.py
> index ad8288d..083af0f 100644
> --- a/tools/patman/series.py
> +++ b/tools/patman/series.py
> @@ -19,6 +19,7 @@
>  # MA 02111-1307 USA
>  #
>
> +import itertools
>  import os
>
>  import gitutil
> @@ -138,6 +139,9 @@ class Series(dict):
>          print 'Prefix:\t ', self.get('prefix')
>          if self.cover:
>              print 'Cover: %d lines' % len(self.cover)
> +            all_ccs = itertools.chain(*self._generated_cc.values())
> +            for email in set(all_ccs):
> +                    print '      Cc: ',email
>          if cmd:
>              print 'Git command: %s' % cmd
>
> @@ -201,27 +205,33 @@ class Series(dict):
>              str = 'Change log exists, but no version is set'
>              print col.Color(col.RED, str)
>
> -    def MakeCcFile(self, process_tags):
> +    def MakeCcFile(self, process_tags, cover_fname):
>          """Make a cc file for us to use for per-commit Cc automation
>
>          Also stores in self._generated_cc to make ShowActions() faster.
>
>          Args:
>              process_tags: Process tags as if they were aliases
> +            cover_fname: If non-None the name of the cover letter.
>          Return:
>              Filename of temp file created
>          """
>          # Look for commit tags (of the form 'xxx:' at the start of the subject)
>          fname = '/tmp/patman.%d' % os.getpid()
>          fd = open(fname, 'w')
> +        all_ccs = []
>          for commit in self.commits:
>              list = []
>              if process_tags:
>                  list += gitutil.BuildEmailList(commit.tags)
>              list += gitutil.BuildEmailList(commit.cc_list)
> +            all_ccs += list
>              print >>fd, commit.patch, ', '.join(list)
>              self._generated_cc[commit.patch] = list
>
> +        if cover_fname:
> +            print >>fd, cover_fname, ', '.join(set(all_ccs))
> +
>          fd.close()
>          return fname
>
> --
> 1.7.7.3
>

Regards,
Simon


More information about the U-Boot mailing list