Simon Glass | efeac06 | 2019-10-31 07:42:52 -0600 | [diff] [blame] | 1 | #!/usr/bin/env python3 |
Tom Rini | 10e4779 | 2018-05-06 17:58:06 -0400 | [diff] [blame] | 2 | # SPDX-License-Identifier: GPL-2.0+ |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 3 | # |
| 4 | # Copyright (c) 2011 The Chromium OS Authors. |
| 5 | # |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 6 | |
| 7 | """See README for more information""" |
| 8 | |
Simon Glass | eb101ac | 2020-07-05 21:41:53 -0600 | [diff] [blame] | 9 | from argparse import ArgumentParser |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 10 | import os |
| 11 | import re |
| 12 | import sys |
Simon Glass | dbac716 | 2020-07-05 21:41:59 -0600 | [diff] [blame] | 13 | import traceback |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 14 | import unittest |
| 15 | |
Simon Glass | bdaad40 | 2020-04-17 18:08:52 -0600 | [diff] [blame] | 16 | if __name__ == "__main__": |
Simon Glass | 4214316 | 2020-04-17 18:09:05 -0600 | [diff] [blame] | 17 | # Allow 'from patman import xxx to work' |
Simon Glass | bdaad40 | 2020-04-17 18:08:52 -0600 | [diff] [blame] | 18 | our_path = os.path.dirname(os.path.realpath(__file__)) |
| 19 | sys.path.append(os.path.join(our_path, '..')) |
| 20 | |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 21 | # Our modules |
Simon Glass | a997ea5 | 2020-04-17 18:09:04 -0600 | [diff] [blame] | 22 | from patman import command |
Simon Glass | 24725af | 2020-07-05 21:41:49 -0600 | [diff] [blame] | 23 | from patman import control |
Simon Glass | a997ea5 | 2020-04-17 18:09:04 -0600 | [diff] [blame] | 24 | from patman import gitutil |
Simon Glass | a997ea5 | 2020-04-17 18:09:04 -0600 | [diff] [blame] | 25 | from patman import project |
| 26 | from patman import settings |
| 27 | from patman import terminal |
Simon Glass | bddb438 | 2020-07-05 21:41:48 -0600 | [diff] [blame] | 28 | from patman import test_util |
Simon Glass | cf5f0b5 | 2020-06-14 10:54:04 -0600 | [diff] [blame] | 29 | from patman import test_checkpatch |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 30 | |
Simon Glass | 109e84e | 2020-07-05 21:41:55 -0600 | [diff] [blame] | 31 | def AddCommonArgs(parser): |
| 32 | parser.add_argument('-b', '--branch', type=str, |
| 33 | help="Branch to process (by default, the current branch)") |
| 34 | parser.add_argument('-c', '--count', dest='count', type=int, |
| 35 | default=-1, help='Automatically create patches from top n commits') |
| 36 | parser.add_argument('-e', '--end', type=int, default=0, |
| 37 | help='Commits to skip at end of patch list') |
Simon Glass | dbac716 | 2020-07-05 21:41:59 -0600 | [diff] [blame] | 38 | parser.add_argument('-D', '--debug', action='store_true', |
| 39 | help='Enabling debugging (provides a full traceback on error)') |
Simon Glass | 109e84e | 2020-07-05 21:41:55 -0600 | [diff] [blame] | 40 | parser.add_argument('-s', '--start', dest='start', type=int, |
| 41 | default=0, help='Commit to start creating patches from (0 = HEAD)') |
| 42 | |
Simon Glass | eb101ac | 2020-07-05 21:41:53 -0600 | [diff] [blame] | 43 | epilog = '''Create patches from commits in a branch, check them and email them |
| 44 | as specified by tags you place in the commits. Use -n to do a dry run first.''' |
Simon Glass | 9c501e1 | 2020-07-05 21:41:55 -0600 | [diff] [blame] | 45 | |
Simon Glass | eb101ac | 2020-07-05 21:41:53 -0600 | [diff] [blame] | 46 | parser = ArgumentParser(epilog=epilog) |
Simon Glass | ecb09da | 2020-07-05 21:41:54 -0600 | [diff] [blame] | 47 | subparsers = parser.add_subparsers(dest='cmd') |
| 48 | send = subparsers.add_parser('send') |
| 49 | send.add_argument('-H', '--full-help', action='store_true', dest='full_help', |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 50 | default=False, help='Display the README file') |
Simon Glass | ecb09da | 2020-07-05 21:41:54 -0600 | [diff] [blame] | 51 | send.add_argument('-i', '--ignore-errors', action='store_true', |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 52 | dest='ignore_errors', default=False, |
| 53 | help='Send patches email even if patch errors are found') |
Simon Glass | ecb09da | 2020-07-05 21:41:54 -0600 | [diff] [blame] | 54 | send.add_argument('-l', '--limit-cc', dest='limit', type=int, default=None, |
| 55 | help='Limit the cc list to LIMIT entries [default: %(default)s]') |
| 56 | send.add_argument('-m', '--no-maintainers', action='store_false', |
Simon Glass | 46b84d8 | 2014-09-14 20:23:17 -0600 | [diff] [blame] | 57 | dest='add_maintainers', default=True, |
| 58 | help="Don't cc the file maintainers automatically") |
Simon Glass | ecb09da | 2020-07-05 21:41:54 -0600 | [diff] [blame] | 59 | send.add_argument('-n', '--dry-run', action='store_true', dest='dry_run', |
Simon Glass | 33bdd9e | 2013-03-26 13:09:45 +0000 | [diff] [blame] | 60 | default=False, help="Do a dry run (create but don't email patches)") |
Simon Glass | ecb09da | 2020-07-05 21:41:54 -0600 | [diff] [blame] | 61 | send.add_argument('-p', '--project', default=project.DetectProject(), |
| 62 | help="Project name; affects default option values and " |
| 63 | "aliases [default: %(default)s]") |
| 64 | send.add_argument('-r', '--in-reply-to', type=str, action='store', |
Doug Anderson | 06f27ac | 2013-03-17 10:31:04 +0000 | [diff] [blame] | 65 | help="Message ID that this series is in reply to") |
Simon Glass | ecb09da | 2020-07-05 21:41:54 -0600 | [diff] [blame] | 66 | send.add_argument('-t', '--ignore-bad-tags', action='store_true', |
| 67 | default=False, help='Ignore bad tags / aliases') |
| 68 | send.add_argument('-v', '--verbose', action='store_true', dest='verbose', |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 69 | default=False, help='Verbose output of errors and warnings') |
Simon Glass | ecb09da | 2020-07-05 21:41:54 -0600 | [diff] [blame] | 70 | send.add_argument('-T', '--thread', action='store_true', dest='thread', |
| 71 | default=False, help='Create patches as a single thread') |
| 72 | send.add_argument('--cc-cmd', dest='cc_cmd', type=str, action='store', |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 73 | default=None, help='Output cc list for patch file (used by git)') |
Simon Glass | ecb09da | 2020-07-05 21:41:54 -0600 | [diff] [blame] | 74 | send.add_argument('--no-binary', action='store_true', dest='ignore_binary', |
| 75 | default=False, |
| 76 | help="Do not output contents of changes in binary files") |
| 77 | send.add_argument('--no-check', action='store_false', dest='check_patch', |
| 78 | default=True, |
| 79 | help="Don't check for patch compliance") |
| 80 | send.add_argument('--no-tags', action='store_false', dest='process_tags', |
| 81 | default=True, help="Don't process subject tags as aliases") |
| 82 | send.add_argument('--smtp-server', type=str, |
| 83 | help="Specify the SMTP server to 'git send-email'") |
Simon Glass | 109e84e | 2020-07-05 21:41:55 -0600 | [diff] [blame] | 84 | AddCommonArgs(send) |
Tom Rini | 5a9ecb2 | 2020-07-24 08:42:06 -0400 | [diff] [blame] | 85 | |
Simon Glass | ecb09da | 2020-07-05 21:41:54 -0600 | [diff] [blame] | 86 | send.add_argument('patchfiles', nargs='*') |
Simon Glass | 9c501e1 | 2020-07-05 21:41:55 -0600 | [diff] [blame] | 87 | |
Simon Glass | 109e84e | 2020-07-05 21:41:55 -0600 | [diff] [blame] | 88 | test_parser = subparsers.add_parser('test', help='Run tests') |
| 89 | AddCommonArgs(test_parser) |
| 90 | |
Doug Anderson | 31ffd7f | 2012-12-03 14:43:18 +0000 | [diff] [blame] | 91 | # Parse options twice: first to get the project and second to handle |
| 92 | # defaults properly (which depends on project). |
Simon Glass | eb101ac | 2020-07-05 21:41:53 -0600 | [diff] [blame] | 93 | argv = sys.argv[1:] |
Simon Glass | ecb09da | 2020-07-05 21:41:54 -0600 | [diff] [blame] | 94 | if len(argv) < 1 or argv[0].startswith('-'): |
| 95 | argv = ['send'] + argv |
Simon Glass | eb101ac | 2020-07-05 21:41:53 -0600 | [diff] [blame] | 96 | args = parser.parse_args(argv) |
Simon Glass | ecb09da | 2020-07-05 21:41:54 -0600 | [diff] [blame] | 97 | if hasattr(args, 'project'): |
| 98 | settings.Setup(gitutil, send, args.project, '') |
| 99 | args = parser.parse_args(argv) |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 100 | |
Simon Glass | 2b68b36 | 2015-07-30 13:47:41 -0600 | [diff] [blame] | 101 | if __name__ != "__main__": |
| 102 | pass |
| 103 | |
Simon Glass | dbac716 | 2020-07-05 21:41:59 -0600 | [diff] [blame] | 104 | if not args.debug: |
| 105 | sys.tracebacklimit = 0 |
| 106 | |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 107 | # Run our meagre tests |
Simon Glass | 109e84e | 2020-07-05 21:41:55 -0600 | [diff] [blame] | 108 | if args.cmd == 'test': |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 109 | import doctest |
Simon Glass | a997ea5 | 2020-04-17 18:09:04 -0600 | [diff] [blame] | 110 | from patman import func_test |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 111 | |
| 112 | sys.argv = [sys.argv[0]] |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 113 | result = unittest.TestResult() |
Simon Glass | cf5f0b5 | 2020-06-14 10:54:04 -0600 | [diff] [blame] | 114 | for module in (test_checkpatch.TestPatch, func_test.TestFunctional): |
Simon Glass | df1bc5c | 2017-05-29 15:31:31 -0600 | [diff] [blame] | 115 | suite = unittest.TestLoader().loadTestsFromTestCase(module) |
| 116 | suite.run(result) |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 117 | |
Simon Glass | 5f9325d | 2020-04-09 15:08:40 -0600 | [diff] [blame] | 118 | for module in ['gitutil', 'settings', 'terminal']: |
Doug Anderson | 06a9515 | 2012-12-03 14:43:19 +0000 | [diff] [blame] | 119 | suite = doctest.DocTestSuite(module) |
| 120 | suite.run(result) |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 121 | |
Simon Glass | bddb438 | 2020-07-05 21:41:48 -0600 | [diff] [blame] | 122 | sys.exit(test_util.ReportResult('patman', None, result)) |
Tom Rini | 5a9ecb2 | 2020-07-24 08:42:06 -0400 | [diff] [blame] | 123 | |
Simon Glass | 109e84e | 2020-07-05 21:41:55 -0600 | [diff] [blame] | 124 | # Process commits, produce patches files, check them, email them |
| 125 | elif args.cmd == 'send': |
| 126 | # Called from git with a patch filename as argument |
| 127 | # Printout a list of additional CC recipients for this patch |
| 128 | if args.cc_cmd: |
| 129 | fd = open(args.cc_cmd, 'r') |
| 130 | re_line = re.compile('(\S*) (.*)') |
| 131 | for line in fd.readlines(): |
| 132 | match = re_line.match(line) |
| 133 | if match and match.group(1) == args.patchfiles[0]: |
| 134 | for cc in match.group(2).split('\0'): |
| 135 | cc = cc.strip() |
| 136 | if cc: |
| 137 | print(cc) |
| 138 | fd.close() |
Tom Rini | 5a9ecb2 | 2020-07-24 08:42:06 -0400 | [diff] [blame] | 139 | |
Simon Glass | 109e84e | 2020-07-05 21:41:55 -0600 | [diff] [blame] | 140 | elif args.full_help: |
| 141 | pager = os.getenv('PAGER') |
| 142 | if not pager: |
| 143 | pager = 'more' |
| 144 | fname = os.path.join(os.path.dirname(os.path.realpath(sys.argv[0])), |
| 145 | 'README') |
| 146 | command.Run(pager, fname) |
Simon Glass | 2613288 | 2012-01-14 15:12:45 +0000 | [diff] [blame] | 147 | |
Simon Glass | 109e84e | 2020-07-05 21:41:55 -0600 | [diff] [blame] | 148 | else: |
| 149 | control.send(args) |