blob: 6a968aad86ded514ab85f42f35c9bfaa279dfddd [file] [log] [blame]
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -08001#!/usr/bin/env python2
Mike Frysingerf80ca212018-07-13 15:02:52 -04002# -*- coding: utf-8 -*-
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -08003# Copyright 2017 The Chromium OS Authors. All rights reserved.
4# Use of this source code is governed by a BSD-style license that can be
5# found in the LICENSE file.
Mike Frysingerf80ca212018-07-13 15:02:52 -04006
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -08007"""This is a tool for picking patches from upstream and applying them."""
8
9from __future__ import print_function
10
Brian Norris9f8a2be2018-06-01 11:14:08 -070011import ConfigParser
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -080012import argparse
Tzung-Bi Shih436fdba2019-09-04 19:05:00 +080013import functools
Brian Norrisc3421042018-08-15 14:17:26 -070014import mailbox
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -080015import os
Tzung-Bi Shih5100c742019-09-02 10:28:32 +080016import pprint
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -080017import re
18import signal
19import subprocess
20import sys
Harry Cuttsae372f32019-02-12 18:01:14 -080021import textwrap
Brian Norris2d4e9762018-08-15 13:11:47 -070022import urllib
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -080023
Tzung-Bi Shih436fdba2019-09-04 19:05:00 +080024errprint = functools.partial(print, file=sys.stderr)
25
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -080026LINUX_URLS = (
27 'git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git',
28 'https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git',
29 'https://kernel.googlesource.com/pub/scm/linux/kernel/git/torvalds/linux.git',
30)
31
Harry Cuttsae372f32019-02-12 18:01:14 -080032COMMIT_MESSAGE_WIDTH = 75
33
Brian Norris9f8a2be2018-06-01 11:14:08 -070034_PWCLIENTRC = os.path.expanduser('~/.pwclientrc')
35
Guenter Roeckbdbb9cc2018-04-19 10:05:08 -070036def _get_conflicts():
37 """Report conflicting files."""
38 resolutions = ('DD', 'AU', 'UD', 'UA', 'DU', 'AA', 'UU')
39 conflicts = []
Douglas Anderson46287f92018-04-30 09:58:24 -070040 lines = subprocess.check_output(['git', 'status', '--porcelain',
Guenter Roeckbdbb9cc2018-04-19 10:05:08 -070041 '--untracked-files=no']).split('\n')
Douglas Anderson46287f92018-04-30 09:58:24 -070042 for line in lines:
43 if not line:
Guenter Roeckbdbb9cc2018-04-19 10:05:08 -070044 continue
Douglas Anderson46287f92018-04-30 09:58:24 -070045 resolution, name = line.split(None, 1)
Guenter Roeckbdbb9cc2018-04-19 10:05:08 -070046 if resolution in resolutions:
47 conflicts.append(' ' + name)
48 if not conflicts:
Douglas Andersonb6a10fe2019-08-12 13:53:30 -070049 return ''
Guenter Roeckbdbb9cc2018-04-19 10:05:08 -070050 return '\nConflicts:\n%s\n' % '\n'.join(conflicts)
51
Guenter Roeckd66daa72018-04-19 10:31:25 -070052def _find_linux_remote():
53 """Find a remote pointing to a Linux upstream repository."""
54 git_remote = subprocess.Popen(['git', 'remote'], stdout=subprocess.PIPE)
55 remotes = git_remote.communicate()[0].strip()
56 for remote in remotes.splitlines():
57 rurl = subprocess.Popen(['git', 'remote', 'get-url', remote],
58 stdout=subprocess.PIPE)
59 url = rurl.communicate()[0].strip()
60 if not rurl.returncode and url in LINUX_URLS:
61 return remote
62 return None
63
Guenter Roeckbdbb9cc2018-04-19 10:05:08 -070064def _pause_for_merge(conflicts):
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -080065 """Pause and go in the background till user resolves the conflicts."""
66
67 git_root = subprocess.check_output(['git', 'rev-parse',
68 '--show-toplevel']).strip('\n')
69
70 paths = (
71 os.path.join(git_root, '.git', 'rebase-apply'),
72 os.path.join(git_root, '.git', 'CHERRY_PICK_HEAD'),
73 )
74 for path in paths:
75 if os.path.exists(path):
Tzung-Bi Shih436fdba2019-09-04 19:05:00 +080076 errprint('Found "%s".' % path)
77 errprint(conflicts)
78 errprint('Please resolve the conflicts and restart the '
79 'shell job when done. Kill this job if you '
80 'aborted the conflict.')
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -080081 os.kill(os.getpid(), signal.SIGTSTP)
82 # TODO: figure out what the state is after the merging, and go based on
83 # that (should we abort? skip? continue?)
84 # Perhaps check last commit message to see if it's the one we were using.
85
Brian Norris9f8a2be2018-06-01 11:14:08 -070086def _get_pw_url(project):
87 """Retrieve the patchwork server URL from .pwclientrc.
88
Mike Frysingerf80ca212018-07-13 15:02:52 -040089 Args:
90 project: patchwork project name; if None, we retrieve the default
91 from pwclientrc
Brian Norris9f8a2be2018-06-01 11:14:08 -070092 """
93 config = ConfigParser.ConfigParser()
94 config.read([_PWCLIENTRC])
95
96 if project is None:
97 try:
98 project = config.get('options', 'default')
99 except (ConfigParser.NoSectionError, ConfigParser.NoOptionError):
Tzung-Bi Shih436fdba2019-09-04 19:05:00 +0800100 errprint('Error: no default patchwork project found in %s.'
101 % _PWCLIENTRC)
Brian Norris9f8a2be2018-06-01 11:14:08 -0700102 sys.exit(1)
103
104 if not config.has_option(project, 'url'):
Tzung-Bi Shih436fdba2019-09-04 19:05:00 +0800105 errprint("Error: patchwork URL not found for project '%s'" % project)
Brian Norris9f8a2be2018-06-01 11:14:08 -0700106 sys.exit(1)
107
108 url = config.get(project, 'url')
Brian Norris2d4e9762018-08-15 13:11:47 -0700109 # Strip trailing 'xmlrpc' and/or trailing slash.
110 return re.sub('/(xmlrpc/)?$', '', url)
Brian Norris9f8a2be2018-06-01 11:14:08 -0700111
Harry Cuttsae372f32019-02-12 18:01:14 -0800112def _wrap_commit_line(prefix, content):
113 line = prefix + '=' + content
114 indent = ' ' * (len(prefix) + 1)
115 return textwrap.fill(line, COMMIT_MESSAGE_WIDTH, subsequent_indent=indent)
116
Tzung-Bi Shih886c9092019-09-02 12:46:16 +0800117def _match_patchwork(match, args):
118 """Match location: pw://### or pw://PROJECT/###."""
119 pw_project = match.group(2)
120 patch_id = int(match.group(3))
121
122 if args['debug']:
123 print('_match_patchwork: pw_project=%s, patch_id=%d' %
124 (pw_project, patch_id))
125
126 if args['tag'] is None:
127 args['tag'] = 'FROMLIST: '
128
129 url = _get_pw_url(pw_project)
130 opener = urllib.urlopen('%s/patch/%d/mbox' % (url, patch_id))
131 if opener.getcode() != 200:
Tzung-Bi Shih436fdba2019-09-04 19:05:00 +0800132 errprint('Error: could not download patch - error code %d'
133 % opener.getcode())
Tzung-Bi Shih886c9092019-09-02 12:46:16 +0800134 sys.exit(1)
135 patch_contents = opener.read()
136
137 if not patch_contents:
Tzung-Bi Shih436fdba2019-09-04 19:05:00 +0800138 errprint('Error: No patch content found')
Tzung-Bi Shih886c9092019-09-02 12:46:16 +0800139 sys.exit(1)
140
141 if args['source_line'] is None:
142 args['source_line'] = '(am from %s/patch/%d/)' % (url, patch_id)
143 message_id = mailbox.Message(patch_contents)['Message-Id']
144 message_id = re.sub('^<|>$', '', message_id.strip())
Tzung-Bi Shiha8f310d2019-09-04 19:10:10 +0800145 args['source_line'] += (
146 '\n(also found at https://lkml.kernel.org/r/%s)' % message_id)
Tzung-Bi Shih886c9092019-09-02 12:46:16 +0800147
148 if args['replace']:
149 subprocess.call(['git', 'reset', '--hard', 'HEAD~1'])
150
151 git_am = subprocess.Popen(['git', 'am', '-3'], stdin=subprocess.PIPE)
152 git_am.communicate(patch_contents)
153 return git_am.returncode
154
155def _match_linux(match, args):
156 """Match location: linux://HASH."""
157 commit = match.group(1)
158
159 if args['debug']:
160 print('_match_linux: commit=%s' % commit)
161
162 # Confirm a 'linux' remote is setup.
163 linux_remote = _find_linux_remote()
164 if not linux_remote:
Tzung-Bi Shih436fdba2019-09-04 19:05:00 +0800165 errprint('Error: need a valid upstream remote')
Tzung-Bi Shih886c9092019-09-02 12:46:16 +0800166 sys.exit(1)
167
168 linux_master = '%s/master' % linux_remote
169 ret = subprocess.call(['git', 'merge-base', '--is-ancestor',
170 commit, linux_master])
171 if ret:
Tzung-Bi Shih436fdba2019-09-04 19:05:00 +0800172 errprint('Error: Commit not in %s' % linux_master)
Tzung-Bi Shih886c9092019-09-02 12:46:16 +0800173 sys.exit(1)
174
175 if args['source_line'] is None:
176 git_pipe = subprocess.Popen(['git', 'rev-parse', commit],
177 stdout=subprocess.PIPE)
178 commit = git_pipe.communicate()[0].strip()
179
180 args['source_line'] = ('(cherry picked from commit %s)' %
181 (commit))
182 if args['tag'] is None:
183 args['tag'] = 'UPSTREAM: '
184
185 if args['replace']:
186 subprocess.call(['git', 'reset', '--hard', 'HEAD~1'])
187
188 return subprocess.call(['git', 'cherry-pick', commit])
189
190def _match_fromgit(match, args):
191 """Match location: git://remote/branch/HASH."""
192 remote = match.group(2)
193 branch = match.group(3)
194 commit = match.group(4)
195
196 if args['debug']:
197 print('_match_fromgit: remote=%s branch=%s commit=%s' %
198 (remote, branch, commit))
199
200 ret = subprocess.call(['git', 'merge-base', '--is-ancestor',
201 commit, '%s/%s' % (remote, branch)])
202 if ret:
Tzung-Bi Shih436fdba2019-09-04 19:05:00 +0800203 errprint('Error: Commit not in %s/%s' % (remote, branch))
Tzung-Bi Shih886c9092019-09-02 12:46:16 +0800204 sys.exit(1)
205
206 git_pipe = subprocess.Popen(['git', 'remote', 'get-url', remote],
207 stdout=subprocess.PIPE)
208 url = git_pipe.communicate()[0].strip()
209
210 if args['source_line'] is None:
211 git_pipe = subprocess.Popen(['git', 'rev-parse', commit],
212 stdout=subprocess.PIPE)
213 commit = git_pipe.communicate()[0].strip()
214
Tzung-Bi Shiha8f310d2019-09-04 19:10:10 +0800215 args['source_line'] = (
216 '(cherry picked from commit %s\n %s %s)' % (commit, url, branch))
Tzung-Bi Shih886c9092019-09-02 12:46:16 +0800217 if args['tag'] is None:
218 args['tag'] = 'FROMGIT: '
219
220 if args['replace']:
221 subprocess.call(['git', 'reset', '--hard', 'HEAD~1'])
222
223 return subprocess.call(['git', 'cherry-pick', commit])
224
225def _match_gitfetch(match, args):
226 """Match location: (git|https)://repoURL#branch/HASH."""
227 remote = match.group(1)
228 branch = match.group(3)
229 commit = match.group(4)
230
231 if args['debug']:
232 print('_match_gitfetch: remote=%s branch=%s commit=%s' %
233 (remote, branch, commit))
234
235 ret = subprocess.call(['git', 'fetch', remote, branch])
236 if ret:
Tzung-Bi Shih436fdba2019-09-04 19:05:00 +0800237 errprint('Error: Branch not in %s' % remote)
Tzung-Bi Shih886c9092019-09-02 12:46:16 +0800238 sys.exit(1)
239
240 url = remote
241
242 if args['source_line'] is None:
243 git_pipe = subprocess.Popen(['git', 'rev-parse', commit],
244 stdout=subprocess.PIPE)
245 commit = git_pipe.communicate()[0].strip()
246
Tzung-Bi Shiha8f310d2019-09-04 19:10:10 +0800247 args['source_line'] = (
248 '(cherry picked from commit %s\n %s %s)' % (commit, url, branch))
Tzung-Bi Shih886c9092019-09-02 12:46:16 +0800249 if args['tag'] is None:
250 args['tag'] = 'FROMGIT: '
251
252 return subprocess.call(['git', 'cherry-pick', commit])
253
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800254def main(args):
255 """This is the main entrypoint for fromupstream.
256
257 Args:
258 args: sys.argv[1:]
259
260 Returns:
261 An int return code.
262 """
263 parser = argparse.ArgumentParser()
264
265 parser.add_argument('--bug', '-b',
Guenter Roeckf47a50c2018-07-25 12:41:36 -0700266 type=str, help='BUG= line')
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800267 parser.add_argument('--test', '-t',
Guenter Roeckf47a50c2018-07-25 12:41:36 -0700268 type=str, help='TEST= line')
Stephen Boyd24b309b2018-11-06 22:11:00 -0800269 parser.add_argument('--crbug', action='append',
270 type=int, help='BUG=chromium: line')
271 parser.add_argument('--buganizer', action='append',
272 type=int, help='BUG=b: line')
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800273 parser.add_argument('--changeid', '-c',
274 help='Overrides the gerrit generated Change-Id line')
275
Tzung-Bi Shihf5d25a82019-09-02 11:40:09 +0800276 parser.add_argument('--replace', '-r',
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800277 action='store_true',
Tzung-Bi Shih196d31e2019-09-01 18:16:28 +0800278 help='Replaces the HEAD commit with this one, taking '
279 'its properties(BUG, TEST, Change-Id). Useful for '
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800280 'updating commits.')
281 parser.add_argument('--nosignoff',
282 dest='signoff', action='store_false')
Tzung-Bi Shih5100c742019-09-02 10:28:32 +0800283 parser.add_argument('--debug', '-d', action='store_true',
284 help='Prints more verbose logs.')
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800285
286 parser.add_argument('--tag',
287 help='Overrides the tag from the title')
288 parser.add_argument('--source', '-s',
289 dest='source_line', type=str,
Tzung-Bi Shih196d31e2019-09-01 18:16:28 +0800290 help='Overrides the source line, last line, ex: '
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800291 '(am from http://....)')
292 parser.add_argument('locations',
Douglas Andersonc77a8b82018-05-04 17:02:03 -0700293 nargs='+',
Tzung-Bi Shih196d31e2019-09-01 18:16:28 +0800294 help='Patchwork ID (pw://### or pw://PROJECT/###, '
295 'where PROJECT is defined in ~/.pwclientrc; if no '
296 'PROJECT is specified, the default is retrieved from '
297 '~/.pwclientrc), '
298 'linux commit like linux://HASH, or '
299 'git reference like git://remote/branch/HASH or '
300 'git://repoURL#branch/HASH or '
Stephen Boyd66ea1912018-10-30 11:26:54 -0700301 'https://repoURL#branch/HASH')
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800302
303 args = vars(parser.parse_args(args))
304
Stephen Boyd24b309b2018-11-06 22:11:00 -0800305 buglist = [args['bug']] if args['bug'] else []
306 if args['buganizer']:
307 buglist += ['b:{0}'.format(x) for x in args['buganizer']]
308 if args['crbug']:
309 buglist += ['chromium:{0}'.format(x) for x in args['crbug']]
Brian Norris667a0cb2018-12-07 09:28:46 -0800310 if buglist:
311 args['bug'] = ', '.join(buglist)
Stephen Boyd24b309b2018-11-06 22:11:00 -0800312
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800313 if args['replace']:
314 old_commit_message = subprocess.check_output(
315 ['git', 'show', '-s', '--format=%B', 'HEAD']
316 ).strip('\n')
Tzung-Bi Shih231fada2019-09-02 00:54:59 +0800317
318 # It is possible that multiple Change-Ids are in the commit message
319 # (due to cherry picking). We only want to pull out the first one.
320 changeid_match = re.search('^Change-Id: (.*)$',
321 old_commit_message, re.MULTILINE)
322 if changeid_match:
323 args['changeid'] = changeid_match.group(1)
324
Tzung-Bi Shih04345302019-09-02 12:04:01 +0800325 bugs = re.findall('BUG=(.*)$', old_commit_message, re.MULTILINE)
326 if args['bug'] is None and bugs:
327 args['bug'] = '\nBUG='.join(bugs)
328
329 tests = re.findall('TEST=(.*)$', old_commit_message, re.MULTILINE)
330 if args['test'] is None and tests:
331 args['test'] = '\nTEST='.join(tests)
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800332 # TODO: deal with multiline BUG/TEST better
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800333
Guenter Roeckf47a50c2018-07-25 12:41:36 -0700334 if args['bug'] is None or args['test'] is None:
Tzung-Bi Shih196d31e2019-09-01 18:16:28 +0800335 parser.error('BUG=/TEST= lines are required; --replace can help '
Stephen Boyde6fdf912018-11-09 10:30:57 -0800336 'automate, or set via --bug/--test')
Guenter Roeckf47a50c2018-07-25 12:41:36 -0700337
Tzung-Bi Shih5100c742019-09-02 10:28:32 +0800338 if args['debug']:
339 pprint.pprint(args)
340
Tzung-Bi Shih886c9092019-09-02 12:46:16 +0800341 re_matches = (
342 (re.compile(r'pw://(([^/]+)/)?(\d+)'), _match_patchwork),
343 (re.compile(r'linux://([0-9a-f]+)'), _match_linux),
344 (re.compile(r'(from)?git://([^/\#]+)/([^#]+)/([0-9a-f]+)$'),
345 _match_fromgit),
346 (re.compile(r'((git|https)://.+)#(.+)/([0-9a-f]+)$'), _match_gitfetch),
347 )
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800348
Tzung-Bi Shih886c9092019-09-02 12:46:16 +0800349 for location in args['locations']:
Tzung-Bi Shih5100c742019-09-02 10:28:32 +0800350 if args['debug']:
351 print('location=%s' % location)
352
Tzung-Bi Shih886c9092019-09-02 12:46:16 +0800353 for reg, handler in re_matches:
354 match = reg.match(location)
355 if match:
356 ret = handler(match, args)
357 break
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800358 else:
Tzung-Bi Shih436fdba2019-09-04 19:05:00 +0800359 errprint('Don\'t know what "%s" means.' % location)
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800360 sys.exit(1)
361
362 if ret != 0:
Guenter Roeckbdbb9cc2018-04-19 10:05:08 -0700363 conflicts = _get_conflicts()
Douglas Anderson2108e532018-04-30 09:50:42 -0700364 if args['tag'] == 'UPSTREAM: ':
365 args['tag'] = 'BACKPORT: '
366 else:
367 args['tag'] = 'BACKPORT: ' + args['tag']
Guenter Roeckbdbb9cc2018-04-19 10:05:08 -0700368 _pause_for_merge(conflicts)
369 else:
Douglas Andersonb6a10fe2019-08-12 13:53:30 -0700370 conflicts = ''
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800371
372 # extract commit message
373 commit_message = subprocess.check_output(
374 ['git', 'show', '-s', '--format=%B', 'HEAD']
375 ).strip('\n')
376
Guenter Roeck2e4f2512018-04-24 09:20:51 -0700377 # Remove stray Change-Id, most likely from merge resolution
378 commit_message = re.sub(r'Change-Id:.*\n?', '', commit_message)
379
Brian Norris7a41b982018-06-01 10:28:29 -0700380 # Note the source location before tagging anything else
381 commit_message += '\n' + args['source_line']
382
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800383 # add automatic Change ID, BUG, and TEST (and maybe signoff too) so
384 # next commands know where to work on
385 commit_message += '\n'
Guenter Roeckbdbb9cc2018-04-19 10:05:08 -0700386 commit_message += conflicts
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800387 commit_message += '\n' + 'BUG=' + args['bug']
Harry Cuttsae372f32019-02-12 18:01:14 -0800388 commit_message += '\n' + _wrap_commit_line('TEST', args['test'])
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800389 if args['signoff']:
390 extra = ['-s']
391 else:
392 extra = []
Stephen Boydaa4e7e02018-11-09 08:48:46 -0800393 subprocess.Popen(
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800394 ['git', 'commit'] + extra + ['--amend', '-F', '-'],
395 stdin=subprocess.PIPE
396 ).communicate(commit_message)
397
398 # re-extract commit message
399 commit_message = subprocess.check_output(
400 ['git', 'show', '-s', '--format=%B', 'HEAD']
401 ).strip('\n')
402
403 # replace changeid if needed
404 if args['changeid'] is not None:
405 commit_message = re.sub(r'(Change-Id: )(\w+)', r'\1%s' %
406 args['changeid'], commit_message)
407 args['changeid'] = None
408
409 # decorate it that it's from outside
410 commit_message = args['tag'] + commit_message
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800411
412 # commit everything
Stephen Boydaa4e7e02018-11-09 08:48:46 -0800413 subprocess.Popen(
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800414 ['git', 'commit', '--amend', '-F', '-'], stdin=subprocess.PIPE
415 ).communicate(commit_message)
416
Chirantan Ekbote4b08e712019-06-12 15:35:41 +0900417 return 0
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800418
419if __name__ == '__main__':
420 sys.exit(main(sys.argv[1:]))