V3s_Allwinner
/
uboot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases 2 Wiki Activity

patman: Add a -m option to avoid copying the maintainers 

The get_maintainers script is a useful default, but sometimes is copies
too many people, or takes a long time to run.

Add an option to disable it and update the README.

Signed-off-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
Simon Glass 2014-09-14 20:23:17 -06:00
parent 7798e2285f
commit 983a2749e2
3 changed files with 12 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
tools/patman/README
View File

@ -27,8 +27,8 @@ Series-to: fred.blogs@napier.co.nz
in one of your commits, the series will be sent there. in one of your commits, the series will be sent there.
In Linux this will also call get_maintainer.pl on each of your In Linux and U-Boot this will also call get_maintainer.pl on each of your
patches automatically. patches automatically (unless you use -m to disable this).
How to use this tool How to use this tool

6
tools/patman/patman.py
View File

@ -32,6 +32,9 @@ parser.add_option('-c', '--count', dest='count', type='int',
parser.add_option('-i', '--ignore-errors', action='store_true', parser.add_option('-i', '--ignore-errors', action='store_true',
dest='ignore_errors', default=False, dest='ignore_errors', default=False,
help='Send patches email even if patch errors are found') help='Send patches email even if patch errors are found')
parser.add_option('-m', '--no-maintainers', action='store_false',
dest='add_maintainers', default=True,
help="Don't cc the file maintainers automatically")
parser.add_option('-n', '--dry-run', action='store_true', dest='dry_run', parser.add_option('-n', '--dry-run', action='store_true', dest='dry_run',
default=False, help="Do a dry run (create but don't email patches)") default=False, help="Do a dry run (create but don't email patches)")
parser.add_option('-p', '--project', default=project.DetectProject(), parser.add_option('-p', '--project', default=project.DetectProject(),
@ -142,7 +145,8 @@ else:
ok = True ok = True
cc_file = series.MakeCcFile(options.process_tags, cover_fname, cc_file = series.MakeCcFile(options.process_tags, cover_fname,
not options.ignore_bad_tags) not options.ignore_bad_tags,
options.add_maintainers)
# Email the patches out (giving the user time to check / cancel) # Email the patches out (giving the user time to check / cancel)
cmd = '' cmd = ''

5
tools/patman/series.py
View File

@ -201,7 +201,8 @@ class Series(dict):
str = 'Change log exists, but no version is set' str = 'Change log exists, but no version is set'
print col.Color(col.RED, str) print col.Color(col.RED, str)
def MakeCcFile(self, process_tags, cover_fname, raise_on_error): def MakeCcFile(self, process_tags, cover_fname, raise_on_error,
add_maintainers):
"""Make a cc file for us to use for per-commit Cc automation """Make a cc file for us to use for per-commit Cc automation
Also stores in self._generated_cc to make ShowActions() faster. Also stores in self._generated_cc to make ShowActions() faster.
@ -211,6 +212,7 @@ class Series(dict):
cover_fname: If non-None the name of the cover letter. cover_fname: If non-None the name of the cover letter.
raise_on_error: True to raise an error when an alias fails to match, raise_on_error: True to raise an error when an alias fails to match,
False to just print a message. False to just print a message.
add_maintainers: Call the get_maintainers to CC maintainers
Return: Return:
Filename of temp file created Filename of temp file created
""" """
@ -225,6 +227,7 @@ class Series(dict):
raise_on_error=raise_on_error) raise_on_error=raise_on_error)
list += gitutil.BuildEmailList(commit.cc_list, list += gitutil.BuildEmailList(commit.cc_list,
raise_on_error=raise_on_error) raise_on_error=raise_on_error)
if add_maintainers:
list += get_maintainer.GetMaintainer(commit.patch) list += get_maintainer.GetMaintainer(commit.patch)
all_ccs += list all_ccs += list
print >>fd, commit.patch, ', '.join(list) print >>fd, commit.patch, ', '.join(list)