blob: 5914ef5b9a2a53f51a16b377fb81c3978082ca74 [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())
145 args['source_line'] += \
146 '\n(also found at https://lkml.kernel.org/r/%s)' % \
147 message_id
148
149 if args['replace']:
150 subprocess.call(['git', 'reset', '--hard', 'HEAD~1'])
151
152 git_am = subprocess.Popen(['git', 'am', '-3'], stdin=subprocess.PIPE)
153 git_am.communicate(patch_contents)
154 return git_am.returncode
155
156def _match_linux(match, args):
157 """Match location: linux://HASH."""
158 commit = match.group(1)
159
160 if args['debug']:
161 print('_match_linux: commit=%s' % commit)
162
163 # Confirm a 'linux' remote is setup.
164 linux_remote = _find_linux_remote()
165 if not linux_remote:
Tzung-Bi Shih436fdba2019-09-04 19:05:00 +0800166 errprint('Error: need a valid upstream remote')
Tzung-Bi Shih886c9092019-09-02 12:46:16 +0800167 sys.exit(1)
168
169 linux_master = '%s/master' % linux_remote
170 ret = subprocess.call(['git', 'merge-base', '--is-ancestor',
171 commit, linux_master])
172 if ret:
Tzung-Bi Shih436fdba2019-09-04 19:05:00 +0800173 errprint('Error: Commit not in %s' % linux_master)
Tzung-Bi Shih886c9092019-09-02 12:46:16 +0800174 sys.exit(1)
175
176 if args['source_line'] is None:
177 git_pipe = subprocess.Popen(['git', 'rev-parse', commit],
178 stdout=subprocess.PIPE)
179 commit = git_pipe.communicate()[0].strip()
180
181 args['source_line'] = ('(cherry picked from commit %s)' %
182 (commit))
183 if args['tag'] is None:
184 args['tag'] = 'UPSTREAM: '
185
186 if args['replace']:
187 subprocess.call(['git', 'reset', '--hard', 'HEAD~1'])
188
189 return subprocess.call(['git', 'cherry-pick', commit])
190
191def _match_fromgit(match, args):
192 """Match location: git://remote/branch/HASH."""
193 remote = match.group(2)
194 branch = match.group(3)
195 commit = match.group(4)
196
197 if args['debug']:
198 print('_match_fromgit: remote=%s branch=%s commit=%s' %
199 (remote, branch, commit))
200
201 ret = subprocess.call(['git', 'merge-base', '--is-ancestor',
202 commit, '%s/%s' % (remote, branch)])
203 if ret:
Tzung-Bi Shih436fdba2019-09-04 19:05:00 +0800204 errprint('Error: Commit not in %s/%s' % (remote, branch))
Tzung-Bi Shih886c9092019-09-02 12:46:16 +0800205 sys.exit(1)
206
207 git_pipe = subprocess.Popen(['git', 'remote', 'get-url', remote],
208 stdout=subprocess.PIPE)
209 url = git_pipe.communicate()[0].strip()
210
211 if args['source_line'] is None:
212 git_pipe = subprocess.Popen(['git', 'rev-parse', commit],
213 stdout=subprocess.PIPE)
214 commit = git_pipe.communicate()[0].strip()
215
216 args['source_line'] = \
217 '(cherry picked from commit %s\n %s %s)' % \
218 (commit, url, branch)
219 if args['tag'] is None:
220 args['tag'] = 'FROMGIT: '
221
222 if args['replace']:
223 subprocess.call(['git', 'reset', '--hard', 'HEAD~1'])
224
225 return subprocess.call(['git', 'cherry-pick', commit])
226
227def _match_gitfetch(match, args):
228 """Match location: (git|https)://repoURL#branch/HASH."""
229 remote = match.group(1)
230 branch = match.group(3)
231 commit = match.group(4)
232
233 if args['debug']:
234 print('_match_gitfetch: remote=%s branch=%s commit=%s' %
235 (remote, branch, commit))
236
237 ret = subprocess.call(['git', 'fetch', remote, branch])
238 if ret:
Tzung-Bi Shih436fdba2019-09-04 19:05:00 +0800239 errprint('Error: Branch not in %s' % remote)
Tzung-Bi Shih886c9092019-09-02 12:46:16 +0800240 sys.exit(1)
241
242 url = remote
243
244 if args['source_line'] is None:
245 git_pipe = subprocess.Popen(['git', 'rev-parse', commit],
246 stdout=subprocess.PIPE)
247 commit = git_pipe.communicate()[0].strip()
248
249 args['source_line'] = \
250 '(cherry picked from commit %s\n %s %s)' % \
251 (commit, url, branch)
252 if args['tag'] is None:
253 args['tag'] = 'FROMGIT: '
254
255 return subprocess.call(['git', 'cherry-pick', commit])
256
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800257def main(args):
258 """This is the main entrypoint for fromupstream.
259
260 Args:
261 args: sys.argv[1:]
262
263 Returns:
264 An int return code.
265 """
266 parser = argparse.ArgumentParser()
267
268 parser.add_argument('--bug', '-b',
Guenter Roeckf47a50c2018-07-25 12:41:36 -0700269 type=str, help='BUG= line')
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800270 parser.add_argument('--test', '-t',
Guenter Roeckf47a50c2018-07-25 12:41:36 -0700271 type=str, help='TEST= line')
Stephen Boyd24b309b2018-11-06 22:11:00 -0800272 parser.add_argument('--crbug', action='append',
273 type=int, help='BUG=chromium: line')
274 parser.add_argument('--buganizer', action='append',
275 type=int, help='BUG=b: line')
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800276 parser.add_argument('--changeid', '-c',
277 help='Overrides the gerrit generated Change-Id line')
278
Tzung-Bi Shihf5d25a82019-09-02 11:40:09 +0800279 parser.add_argument('--replace', '-r',
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800280 action='store_true',
Tzung-Bi Shih196d31e2019-09-01 18:16:28 +0800281 help='Replaces the HEAD commit with this one, taking '
282 'its properties(BUG, TEST, Change-Id). Useful for '
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800283 'updating commits.')
284 parser.add_argument('--nosignoff',
285 dest='signoff', action='store_false')
Tzung-Bi Shih5100c742019-09-02 10:28:32 +0800286 parser.add_argument('--debug', '-d', action='store_true',
287 help='Prints more verbose logs.')
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800288
289 parser.add_argument('--tag',
290 help='Overrides the tag from the title')
291 parser.add_argument('--source', '-s',
292 dest='source_line', type=str,
Tzung-Bi Shih196d31e2019-09-01 18:16:28 +0800293 help='Overrides the source line, last line, ex: '
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800294 '(am from http://....)')
295 parser.add_argument('locations',
Douglas Andersonc77a8b82018-05-04 17:02:03 -0700296 nargs='+',
Tzung-Bi Shih196d31e2019-09-01 18:16:28 +0800297 help='Patchwork ID (pw://### or pw://PROJECT/###, '
298 'where PROJECT is defined in ~/.pwclientrc; if no '
299 'PROJECT is specified, the default is retrieved from '
300 '~/.pwclientrc), '
301 'linux commit like linux://HASH, or '
302 'git reference like git://remote/branch/HASH or '
303 'git://repoURL#branch/HASH or '
Stephen Boyd66ea1912018-10-30 11:26:54 -0700304 'https://repoURL#branch/HASH')
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800305
306 args = vars(parser.parse_args(args))
307
Stephen Boyd24b309b2018-11-06 22:11:00 -0800308 buglist = [args['bug']] if args['bug'] else []
309 if args['buganizer']:
310 buglist += ['b:{0}'.format(x) for x in args['buganizer']]
311 if args['crbug']:
312 buglist += ['chromium:{0}'.format(x) for x in args['crbug']]
Brian Norris667a0cb2018-12-07 09:28:46 -0800313 if buglist:
314 args['bug'] = ', '.join(buglist)
Stephen Boyd24b309b2018-11-06 22:11:00 -0800315
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800316 if args['replace']:
317 old_commit_message = subprocess.check_output(
318 ['git', 'show', '-s', '--format=%B', 'HEAD']
319 ).strip('\n')
Tzung-Bi Shih231fada2019-09-02 00:54:59 +0800320
321 # It is possible that multiple Change-Ids are in the commit message
322 # (due to cherry picking). We only want to pull out the first one.
323 changeid_match = re.search('^Change-Id: (.*)$',
324 old_commit_message, re.MULTILINE)
325 if changeid_match:
326 args['changeid'] = changeid_match.group(1)
327
Tzung-Bi Shih04345302019-09-02 12:04:01 +0800328 bugs = re.findall('BUG=(.*)$', old_commit_message, re.MULTILINE)
329 if args['bug'] is None and bugs:
330 args['bug'] = '\nBUG='.join(bugs)
331
332 tests = re.findall('TEST=(.*)$', old_commit_message, re.MULTILINE)
333 if args['test'] is None and tests:
334 args['test'] = '\nTEST='.join(tests)
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800335 # TODO: deal with multiline BUG/TEST better
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800336
Guenter Roeckf47a50c2018-07-25 12:41:36 -0700337 if args['bug'] is None or args['test'] is None:
Tzung-Bi Shih196d31e2019-09-01 18:16:28 +0800338 parser.error('BUG=/TEST= lines are required; --replace can help '
Stephen Boyde6fdf912018-11-09 10:30:57 -0800339 'automate, or set via --bug/--test')
Guenter Roeckf47a50c2018-07-25 12:41:36 -0700340
Tzung-Bi Shih5100c742019-09-02 10:28:32 +0800341 if args['debug']:
342 pprint.pprint(args)
343
Tzung-Bi Shih886c9092019-09-02 12:46:16 +0800344 re_matches = (
345 (re.compile(r'pw://(([^/]+)/)?(\d+)'), _match_patchwork),
346 (re.compile(r'linux://([0-9a-f]+)'), _match_linux),
347 (re.compile(r'(from)?git://([^/\#]+)/([^#]+)/([0-9a-f]+)$'),
348 _match_fromgit),
349 (re.compile(r'((git|https)://.+)#(.+)/([0-9a-f]+)$'), _match_gitfetch),
350 )
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800351
Tzung-Bi Shih886c9092019-09-02 12:46:16 +0800352 for location in args['locations']:
Tzung-Bi Shih5100c742019-09-02 10:28:32 +0800353 if args['debug']:
354 print('location=%s' % location)
355
Tzung-Bi Shih886c9092019-09-02 12:46:16 +0800356 for reg, handler in re_matches:
357 match = reg.match(location)
358 if match:
359 ret = handler(match, args)
360 break
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800361 else:
Tzung-Bi Shih436fdba2019-09-04 19:05:00 +0800362 errprint('Don\'t know what "%s" means.' % location)
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800363 sys.exit(1)
364
365 if ret != 0:
Guenter Roeckbdbb9cc2018-04-19 10:05:08 -0700366 conflicts = _get_conflicts()
Douglas Anderson2108e532018-04-30 09:50:42 -0700367 if args['tag'] == 'UPSTREAM: ':
368 args['tag'] = 'BACKPORT: '
369 else:
370 args['tag'] = 'BACKPORT: ' + args['tag']
Guenter Roeckbdbb9cc2018-04-19 10:05:08 -0700371 _pause_for_merge(conflicts)
372 else:
Douglas Andersonb6a10fe2019-08-12 13:53:30 -0700373 conflicts = ''
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800374
375 # extract commit message
376 commit_message = subprocess.check_output(
377 ['git', 'show', '-s', '--format=%B', 'HEAD']
378 ).strip('\n')
379
Guenter Roeck2e4f2512018-04-24 09:20:51 -0700380 # Remove stray Change-Id, most likely from merge resolution
381 commit_message = re.sub(r'Change-Id:.*\n?', '', commit_message)
382
Brian Norris7a41b982018-06-01 10:28:29 -0700383 # Note the source location before tagging anything else
384 commit_message += '\n' + args['source_line']
385
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800386 # add automatic Change ID, BUG, and TEST (and maybe signoff too) so
387 # next commands know where to work on
388 commit_message += '\n'
Guenter Roeckbdbb9cc2018-04-19 10:05:08 -0700389 commit_message += conflicts
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800390 commit_message += '\n' + 'BUG=' + args['bug']
Harry Cuttsae372f32019-02-12 18:01:14 -0800391 commit_message += '\n' + _wrap_commit_line('TEST', args['test'])
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800392 if args['signoff']:
393 extra = ['-s']
394 else:
395 extra = []
Stephen Boydaa4e7e02018-11-09 08:48:46 -0800396 subprocess.Popen(
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800397 ['git', 'commit'] + extra + ['--amend', '-F', '-'],
398 stdin=subprocess.PIPE
399 ).communicate(commit_message)
400
401 # re-extract commit message
402 commit_message = subprocess.check_output(
403 ['git', 'show', '-s', '--format=%B', 'HEAD']
404 ).strip('\n')
405
406 # replace changeid if needed
407 if args['changeid'] is not None:
408 commit_message = re.sub(r'(Change-Id: )(\w+)', r'\1%s' %
409 args['changeid'], commit_message)
410 args['changeid'] = None
411
412 # decorate it that it's from outside
413 commit_message = args['tag'] + commit_message
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800414
415 # commit everything
Stephen Boydaa4e7e02018-11-09 08:48:46 -0800416 subprocess.Popen(
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800417 ['git', 'commit', '--amend', '-F', '-'], stdin=subprocess.PIPE
418 ).communicate(commit_message)
419
Chirantan Ekbote4b08e712019-06-12 15:35:41 +0900420 return 0
Alexandru M Stanfb5b5ee2014-12-04 13:32:55 -0800421
422if __name__ == '__main__':
423 sys.exit(main(sys.argv[1:]))