blob: 28a9a260879a16910b9065885aa021f1ad5ebd73 [file] [log] [blame]
Simon Glassefeac062019-10-31 07:42:52 -06001#!/usr/bin/env python3
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
Tom Rini5a9ecb22020-07-24 08:42:06 -04009from optparse import OptionParser
Simon Glass26132882012-01-14 15:12:45 +000010import os
11import re
12import sys
13import unittest
14
Simon Glassbdaad402020-04-17 18:08:52 -060015if __name__ == "__main__":
Simon Glass42143162020-04-17 18:09:05 -060016 # Allow 'from patman import xxx to work'
Simon Glassbdaad402020-04-17 18:08:52 -060017 our_path = os.path.dirname(os.path.realpath(__file__))
18 sys.path.append(os.path.join(our_path, '..'))
19
Simon Glass26132882012-01-14 15:12:45 +000020# Our modules
Tom Rini5a9ecb22020-07-24 08:42:06 -040021from patman import checkpatch
Simon Glassa997ea52020-04-17 18:09:04 -060022from patman import command
23from patman import gitutil
Tom Rini5a9ecb22020-07-24 08:42:06 -040024from patman import patchstream
Simon Glassa997ea52020-04-17 18:09:04 -060025from patman import project
26from patman import settings
27from patman import terminal
Simon Glasscf5f0b52020-06-14 10:54:04 -060028from patman import test_checkpatch
Simon Glass26132882012-01-14 15:12:45 +000029
Simon Glass9c501e12020-07-05 21:41:55 -060030
Tom Rini5a9ecb22020-07-24 08:42:06 -040031parser = OptionParser()
32parser.add_option('-H', '--full-help', action='store_true', dest='full_help',
Simon Glass26132882012-01-14 15:12:45 +000033 default=False, help='Display the README file')
Tom Rini5a9ecb22020-07-24 08:42:06 -040034parser.add_option('-c', '--count', dest='count', type='int',
35 default=-1, help='Automatically create patches from top n commits')
36parser.add_option('-i', '--ignore-errors', action='store_true',
Simon Glass26132882012-01-14 15:12:45 +000037 dest='ignore_errors', default=False,
38 help='Send patches email even if patch errors are found')
Tom Rini5a9ecb22020-07-24 08:42:06 -040039parser.add_option('-l', '--limit-cc', dest='limit', type='int',
40 default=None, help='Limit the cc list to LIMIT entries [default: %default]')
41parser.add_option('-m', '--no-maintainers', action='store_false',
Simon Glass46b84d82014-09-14 20:23:17 -060042 dest='add_maintainers', default=True,
43 help="Don't cc the file maintainers automatically")
Tom Rini5a9ecb22020-07-24 08:42:06 -040044parser.add_option('-n', '--dry-run', action='store_true', dest='dry_run',
Simon Glass33bdd9e2013-03-26 13:09:45 +000045 default=False, help="Do a dry run (create but don't email patches)")
Tom Rini5a9ecb22020-07-24 08:42:06 -040046parser.add_option('-p', '--project', default=project.DetectProject(),
Simon Glassc16f3242020-07-05 21:41:54 -060047 help="Project name; affects default option values and "
Tom Rini5a9ecb22020-07-24 08:42:06 -040048 "aliases [default: %default]")
49parser.add_option('-r', '--in-reply-to', type='string', action='store',
Doug Anderson06f27ac2013-03-17 10:31:04 +000050 help="Message ID that this series is in reply to")
Tom Rini5a9ecb22020-07-24 08:42:06 -040051parser.add_option('-s', '--start', dest='start', type='int',
52 default=0, help='Commit to start creating patches from (0 = HEAD)')
53parser.add_option('-t', '--ignore-bad-tags', action='store_true',
Simon Glassc16f3242020-07-05 21:41:54 -060054 default=False, help='Ignore bad tags / aliases')
Tom Rini5a9ecb22020-07-24 08:42:06 -040055parser.add_option('-v', '--verbose', action='store_true', dest='verbose',
Simon Glass26132882012-01-14 15:12:45 +000056 default=False, help='Verbose output of errors and warnings')
Tom Rini5a9ecb22020-07-24 08:42:06 -040057parser.add_option('-T', '--thread', action='store_true', dest='thread',
Simon Glassc16f3242020-07-05 21:41:54 -060058 default=False, help='Create patches as a single thread')
Tom Rini5a9ecb22020-07-24 08:42:06 -040059parser.add_option('--cc-cmd', dest='cc_cmd', type='string', action='store',
Simon Glass26132882012-01-14 15:12:45 +000060 default=None, help='Output cc list for patch file (used by git)')
Tom Rini5a9ecb22020-07-24 08:42:06 -040061parser.add_option('--no-binary', action='store_true', dest='ignore_binary',
Simon Glassc16f3242020-07-05 21:41:54 -060062 default=False,
63 help="Do not output contents of changes in binary files")
Tom Rini5a9ecb22020-07-24 08:42:06 -040064parser.add_option('--no-check', action='store_false', dest='check_patch',
Simon Glassc16f3242020-07-05 21:41:54 -060065 default=True,
66 help="Don't check for patch compliance")
Tom Rini5a9ecb22020-07-24 08:42:06 -040067parser.add_option('--no-tags', action='store_false', dest='process_tags',
Simon Glassc16f3242020-07-05 21:41:54 -060068 default=True, help="Don't process subject tags as aliases")
Tom Rini5a9ecb22020-07-24 08:42:06 -040069parser.add_option('--smtp-server', type='str',
Simon Glassc16f3242020-07-05 21:41:54 -060070 help="Specify the SMTP server to 'git send-email'")
Tom Rini5a9ecb22020-07-24 08:42:06 -040071parser.add_option('--test', action='store_true', dest='test',
72 default=False, help='run tests')
73
74parser.usage += """
Simon Glass26132882012-01-14 15:12:45 +000075
Tom Rini5a9ecb22020-07-24 08:42:06 -040076Create patches from commits in a branch, check them and email them as
77specified by tags you place in the commits. Use -n to do a dry run first."""
Doug Anderson3d3077c2012-12-03 14:43:17 +000078
Simon Glass9c501e12020-07-05 21:41:55 -060079
Doug Anderson31ffd7f2012-12-03 14:43:18 +000080# Parse options twice: first to get the project and second to handle
81# defaults properly (which depends on project).
Tom Rini5a9ecb22020-07-24 08:42:06 -040082(options, args) = parser.parse_args()
83settings.Setup(gitutil, parser, options.project, '')
84(options, args) = parser.parse_args()
Simon Glass26132882012-01-14 15:12:45 +000085
Simon Glass2b68b362015-07-30 13:47:41 -060086if __name__ != "__main__":
87 pass
88
Simon Glass26132882012-01-14 15:12:45 +000089# Run our meagre tests
Tom Rini5a9ecb22020-07-24 08:42:06 -040090elif options.test:
Simon Glass26132882012-01-14 15:12:45 +000091 import doctest
Simon Glassa997ea52020-04-17 18:09:04 -060092 from patman import func_test
Simon Glass26132882012-01-14 15:12:45 +000093
94 sys.argv = [sys.argv[0]]
Simon Glass26132882012-01-14 15:12:45 +000095 result = unittest.TestResult()
Simon Glasscf5f0b52020-06-14 10:54:04 -060096 for module in (test_checkpatch.TestPatch, func_test.TestFunctional):
Simon Glassdf1bc5c2017-05-29 15:31:31 -060097 suite = unittest.TestLoader().loadTestsFromTestCase(module)
98 suite.run(result)
Simon Glass26132882012-01-14 15:12:45 +000099
Simon Glass5f9325d2020-04-09 15:08:40 -0600100 for module in ['gitutil', 'settings', 'terminal']:
Doug Anderson06a95152012-12-03 14:43:19 +0000101 suite = doctest.DocTestSuite(module)
102 suite.run(result)
Simon Glass26132882012-01-14 15:12:45 +0000103
Tom Rini5a9ecb22020-07-24 08:42:06 -0400104 # TODO: Surely we can just 'print' result?
105 print(result)
106 for test, err in result.errors:
107 print(err)
108 for test, err in result.failures:
109 print(err)
110
111# Called from git with a patch filename as argument
112# Printout a list of additional CC recipients for this patch
113elif options.cc_cmd:
114 fd = open(options.cc_cmd, 'r')
115 re_line = re.compile('(\S*) (.*)')
116 for line in fd.readlines():
117 match = re_line.match(line)
118 if match and match.group(1) == args[0]:
119 for cc in match.group(2).split('\0'):
120 cc = cc.strip()
121 if cc:
122 print(cc)
123 fd.close()
124
125elif options.full_help:
126 pager = os.getenv('PAGER')
127 if not pager:
128 pager = 'more'
129 fname = os.path.join(os.path.dirname(os.path.realpath(sys.argv[0])),
130 'README')
131 command.Run(pager, fname)
Simon Glass26132882012-01-14 15:12:45 +0000132
Simon Glass9c501e12020-07-05 21:41:55 -0600133# Process commits, produce patches files, check them, email them
Tom Rini5a9ecb22020-07-24 08:42:06 -0400134else:
135 gitutil.Setup()
136
137 if options.count == -1:
138 # Work out how many patches to send if we can
139 options.count = gitutil.CountCommitsToBranch() - options.start
140
141 col = terminal.Color()
142 if not options.count:
143 str = 'No commits found to process - please use -c flag'
144 sys.exit(col.Color(col.RED, str))
Simon Glass26132882012-01-14 15:12:45 +0000145
Tom Rini5a9ecb22020-07-24 08:42:06 -0400146 # Read the metadata from the commits
147 if options.count:
148 series = patchstream.GetMetaData(options.start, options.count)
149 cover_fname, args = gitutil.CreatePatches(options.start, options.count,
150 options.ignore_binary, series)
Simon Glass26132882012-01-14 15:12:45 +0000151
Tom Rini5a9ecb22020-07-24 08:42:06 -0400152 # Fix up the patch files to our liking, and insert the cover letter
153 patchstream.FixPatches(series, args)
154 if cover_fname and series.get('cover'):
155 patchstream.InsertCoverLetter(cover_fname, series, options.count)
156
157 # Do a few checks on the series
158 series.DoChecks()
159
160 # Check the patches, and run them through 'git am' just to be sure
161 if options.check_patch:
162 ok = checkpatch.CheckPatches(options.verbose, args)
Simon Glass9c501e12020-07-05 21:41:55 -0600163 else:
Tom Rini5a9ecb22020-07-24 08:42:06 -0400164 ok = True
165
166 cc_file = series.MakeCcFile(options.process_tags, cover_fname,
167 not options.ignore_bad_tags,
168 options.add_maintainers, options.limit)
169
170 # Email the patches out (giving the user time to check / cancel)
171 cmd = ''
172 its_a_go = ok or options.ignore_errors
173 if its_a_go:
174 cmd = gitutil.EmailPatches(series, cover_fname, args,
175 options.dry_run, not options.ignore_bad_tags, cc_file,
176 in_reply_to=options.in_reply_to, thread=options.thread,
177 smtp_server=options.smtp_server)
178 else:
179 print(col.Color(col.RED, "Not sending emails due to errors/warnings"))
180
181 # For a dry run, just show our actions as a sanity check
182 if options.dry_run:
183 series.ShowActions(args, cmd, options.process_tags)
184 if not its_a_go:
185 print(col.Color(col.RED, "Email would not be sent"))
186
187 os.remove(cc_file)