blob: 9605a36eff2e398ac7a80297838fd6a13969c8e1 [file] [log] [blame]
Simon Glass24b86ce2019-05-14 15:53:55 -06001#!/usr/bin/env python
Tom Rini10e47792018-05-06 17:58:06 -04002# SPDX-License-Identifier: GPL-2.0+
Simon Glass26132882012-01-14 15:12:45 +00003#
4# Copyright (c) 2011 The Chromium OS Authors.
5#
Simon Glass26132882012-01-14 15:12:45 +00006
7"""See README for more information"""
8
9from optparse import OptionParser
10import os
11import re
12import sys
13import unittest
14
15# Our modules
Chris Packham464a0e92015-07-22 21:21:46 +120016try:
17 from patman import checkpatch, command, gitutil, patchstream, \
18 project, settings, terminal, test
19except ImportError:
20 import checkpatch
21 import command
22 import gitutil
23 import patchstream
24 import project
25 import settings
26 import terminal
27 import test
Simon Glass26132882012-01-14 15:12:45 +000028
29
30parser = OptionParser()
31parser.add_option('-H', '--full-help', action='store_true', dest='full_help',
32 default=False, help='Display the README file')
33parser.add_option('-c', '--count', dest='count', type='int',
34 default=-1, help='Automatically create patches from top n commits')
35parser.add_option('-i', '--ignore-errors', action='store_true',
36 dest='ignore_errors', default=False,
37 help='Send patches email even if patch errors are found')
Simon Glass46b84d82014-09-14 20:23:17 -060038parser.add_option('-m', '--no-maintainers', action='store_false',
39 dest='add_maintainers', default=True,
40 help="Don't cc the file maintainers automatically")
Chris Packhamb84fb482018-06-07 20:45:06 +120041parser.add_option('-l', '--limit-cc', dest='limit', type='int',
42 default=None, help='Limit the cc list to LIMIT entries [default: %default]')
Simon Glass26132882012-01-14 15:12:45 +000043parser.add_option('-n', '--dry-run', action='store_true', dest='dry_run',
Simon Glass33bdd9e2013-03-26 13:09:45 +000044 default=False, help="Do a dry run (create but don't email patches)")
Vadim Bendeburyc549f082013-01-09 16:00:10 +000045parser.add_option('-p', '--project', default=project.DetectProject(),
46 help="Project name; affects default option values and "
47 "aliases [default: %default]")
Doug Anderson06f27ac2013-03-17 10:31:04 +000048parser.add_option('-r', '--in-reply-to', type='string', action='store',
49 help="Message ID that this series is in reply to")
Simon Glass26132882012-01-14 15:12:45 +000050parser.add_option('-s', '--start', dest='start', type='int',
51 default=0, help='Commit to start creating patches from (0 = HEAD)')
Simon Glass12ea5f42013-03-26 13:09:42 +000052parser.add_option('-t', '--ignore-bad-tags', action='store_true',
53 default=False, help='Ignore bad tags / aliases')
54parser.add_option('--test', action='store_true', dest='test',
Simon Glass26132882012-01-14 15:12:45 +000055 default=False, help='run tests')
56parser.add_option('-v', '--verbose', action='store_true', dest='verbose',
57 default=False, help='Verbose output of errors and warnings')
58parser.add_option('--cc-cmd', dest='cc_cmd', type='string', action='store',
59 default=None, help='Output cc list for patch file (used by git)')
Vadim Bendeburyc549f082013-01-09 16:00:10 +000060parser.add_option('--no-check', action='store_false', dest='check_patch',
61 default=True,
62 help="Don't check for patch compliance")
Simon Glass26132882012-01-14 15:12:45 +000063parser.add_option('--no-tags', action='store_false', dest='process_tags',
64 default=True, help="Don't process subject tags as aliaes")
Simon Glass8137e302018-06-19 09:56:07 -060065parser.add_option('--smtp-server', type='str',
66 help="Specify the SMTP server to 'git send-email'")
Mateusz Kulikowski80c2ebc2016-01-14 20:37:41 +010067parser.add_option('-T', '--thread', action='store_true', dest='thread',
68 default=False, help='Create patches as a single thread')
Simon Glass26132882012-01-14 15:12:45 +000069
Masahiro Yamadaa97f35f2014-08-21 14:28:03 +090070parser.usage += """
Simon Glass26132882012-01-14 15:12:45 +000071
72Create patches from commits in a branch, check them and email them as
Simon Glass33bdd9e2013-03-26 13:09:45 +000073specified by tags you place in the commits. Use -n to do a dry run first."""
Simon Glass26132882012-01-14 15:12:45 +000074
Doug Anderson3d3077c2012-12-03 14:43:17 +000075
Doug Anderson31ffd7f2012-12-03 14:43:18 +000076# Parse options twice: first to get the project and second to handle
77# defaults properly (which depends on project).
78(options, args) = parser.parse_args()
79settings.Setup(parser, options.project, '')
Simon Glass26132882012-01-14 15:12:45 +000080(options, args) = parser.parse_args()
81
Simon Glass2b68b362015-07-30 13:47:41 -060082if __name__ != "__main__":
83 pass
84
Simon Glass26132882012-01-14 15:12:45 +000085# Run our meagre tests
Simon Glass2b68b362015-07-30 13:47:41 -060086elif options.test:
Simon Glass26132882012-01-14 15:12:45 +000087 import doctest
Simon Glassdf1bc5c2017-05-29 15:31:31 -060088 import func_test
Simon Glass26132882012-01-14 15:12:45 +000089
90 sys.argv = [sys.argv[0]]
Simon Glass26132882012-01-14 15:12:45 +000091 result = unittest.TestResult()
Simon Glassdf1bc5c2017-05-29 15:31:31 -060092 for module in (test.TestPatch, func_test.TestFunctional):
93 suite = unittest.TestLoader().loadTestsFromTestCase(module)
94 suite.run(result)
Simon Glass26132882012-01-14 15:12:45 +000095
Doug Anderson06a95152012-12-03 14:43:19 +000096 for module in ['gitutil', 'settings']:
97 suite = doctest.DocTestSuite(module)
98 suite.run(result)
Simon Glass26132882012-01-14 15:12:45 +000099
100 # TODO: Surely we can just 'print' result?
Paul Burtonc3931342016-09-27 16:03:50 +0100101 print(result)
Simon Glass26132882012-01-14 15:12:45 +0000102 for test, err in result.errors:
Paul Burtonc3931342016-09-27 16:03:50 +0100103 print(err)
Simon Glass26132882012-01-14 15:12:45 +0000104 for test, err in result.failures:
Paul Burtonc3931342016-09-27 16:03:50 +0100105 print(err)
Simon Glass26132882012-01-14 15:12:45 +0000106
107# Called from git with a patch filename as argument
108# Printout a list of additional CC recipients for this patch
109elif options.cc_cmd:
110 fd = open(options.cc_cmd, 'r')
111 re_line = re.compile('(\S*) (.*)')
112 for line in fd.readlines():
113 match = re_line.match(line)
114 if match and match.group(1) == args[0]:
115 for cc in match.group(2).split(', '):
116 cc = cc.strip()
117 if cc:
Paul Burtonc3931342016-09-27 16:03:50 +0100118 print(cc)
Simon Glass26132882012-01-14 15:12:45 +0000119 fd.close()
120
121elif options.full_help:
122 pager = os.getenv('PAGER')
123 if not pager:
124 pager = 'more'
Simon Glassc29101f2016-03-06 19:45:34 -0700125 fname = os.path.join(os.path.dirname(os.path.realpath(sys.argv[0])),
126 'README')
Simon Glass26132882012-01-14 15:12:45 +0000127 command.Run(pager, fname)
128
129# Process commits, produce patches files, check them, email them
130else:
131 gitutil.Setup()
132
133 if options.count == -1:
134 # Work out how many patches to send if we can
135 options.count = gitutil.CountCommitsToBranch() - options.start
136
137 col = terminal.Color()
138 if not options.count:
139 str = 'No commits found to process - please use -c flag'
Masahiro Yamada880828d2014-08-16 00:59:26 +0900140 sys.exit(col.Color(col.RED, str))
Simon Glass26132882012-01-14 15:12:45 +0000141
142 # Read the metadata from the commits
143 if options.count:
144 series = patchstream.GetMetaData(options.start, options.count)
145 cover_fname, args = gitutil.CreatePatches(options.start, options.count,
146 series)
147
148 # Fix up the patch files to our liking, and insert the cover letter
Simon Glassfdf70022017-05-29 15:31:27 -0600149 patchstream.FixPatches(series, args)
150 if cover_fname and series.get('cover'):
Simon Glass26132882012-01-14 15:12:45 +0000151 patchstream.InsertCoverLetter(cover_fname, series, options.count)
152
153 # Do a few checks on the series
154 series.DoChecks()
155
156 # Check the patches, and run them through 'git am' just to be sure
Vadim Bendeburyc549f082013-01-09 16:00:10 +0000157 if options.check_patch:
158 ok = checkpatch.CheckPatches(options.verbose, args)
159 else:
160 ok = True
Simon Glass26132882012-01-14 15:12:45 +0000161
Simon Glass12ea5f42013-03-26 13:09:42 +0000162 cc_file = series.MakeCcFile(options.process_tags, cover_fname,
Simon Glass46b84d82014-09-14 20:23:17 -0600163 not options.ignore_bad_tags,
Chris Packhamb84fb482018-06-07 20:45:06 +1200164 options.add_maintainers, options.limit)
Doug Anderson507e8e82012-12-03 14:40:42 +0000165
Simon Glass26132882012-01-14 15:12:45 +0000166 # Email the patches out (giving the user time to check / cancel)
167 cmd = ''
Vadim Bendeburyfbf8ee02014-09-04 10:45:13 -0700168 its_a_go = ok or options.ignore_errors
169 if its_a_go:
Simon Glass26132882012-01-14 15:12:45 +0000170 cmd = gitutil.EmailPatches(series, cover_fname, args,
Simon Glass12ea5f42013-03-26 13:09:42 +0000171 options.dry_run, not options.ignore_bad_tags, cc_file,
Simon Glass8137e302018-06-19 09:56:07 -0600172 in_reply_to=options.in_reply_to, thread=options.thread,
173 smtp_server=options.smtp_server)
Vadim Bendeburyfbf8ee02014-09-04 10:45:13 -0700174 else:
Paul Burtonc3931342016-09-27 16:03:50 +0100175 print(col.Color(col.RED, "Not sending emails due to errors/warnings"))
Simon Glass26132882012-01-14 15:12:45 +0000176
177 # For a dry run, just show our actions as a sanity check
178 if options.dry_run:
179 series.ShowActions(args, cmd, options.process_tags)
Vadim Bendeburyfbf8ee02014-09-04 10:45:13 -0700180 if not its_a_go:
Paul Burtonc3931342016-09-27 16:03:50 +0100181 print(col.Color(col.RED, "Email would not be sent"))
Doug Anderson507e8e82012-12-03 14:40:42 +0000182
183 os.remove(cc_file)