Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 1 | #!/usr/bin/python |
| 2 | |
Mike Frysinger | 110750a | 2012-03-26 14:19:20 -0400 | [diff] [blame] | 3 | # Copyright (c) 2012 The Chromium OS Authors. All rights reserved. |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 4 | # Use of this source code is governed by a BSD-style license that can be |
| 5 | # found in the LICENSE file. |
| 6 | |
| 7 | """This module uprevs a given package's ebuild to the next revision.""" |
| 8 | |
Chris Sosa | 62ad852 | 2011-03-08 17:46:17 -0800 | [diff] [blame] | 9 | import optparse |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 10 | import os |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 11 | import sys |
| 12 | |
Mike Frysinger | 6cb624a | 2012-05-24 18:17:38 -0400 | [diff] [blame] | 13 | from chromite.buildbot import constants |
David James | 6600946 | 2012-03-25 10:08:38 -0700 | [diff] [blame] | 14 | from chromite.buildbot import portage_utilities |
Chris Sosa | c13bba5 | 2011-05-24 15:14:09 -0700 | [diff] [blame] | 15 | from chromite.lib import cros_build_lib |
David James | 97d9587 | 2012-11-16 15:09:56 -0800 | [diff] [blame] | 16 | from chromite.lib import git |
Mike Frysinger | fddaeb5 | 2012-11-20 11:17:31 -0500 | [diff] [blame] | 17 | from chromite.lib import osutils |
David James | 6450a0a | 2012-12-04 07:59:53 -0800 | [diff] [blame] | 18 | from chromite.lib import parallel |
Chris Sosa | c13bba5 | 2011-05-24 15:14:09 -0700 | [diff] [blame] | 19 | |
David James | 15ed130 | 2013-04-25 09:21:19 -0700 | [diff] [blame] | 20 | |
David James | 29e86d5 | 2013-04-19 09:41:29 -0700 | [diff] [blame] | 21 | # Commit message for uprevving Portage packages. |
| 22 | _GIT_COMMIT_MESSAGE = 'Marking 9999 ebuild for %s as stable.' |
| 23 | |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 24 | # Dictionary of valid commands with usage information. |
| 25 | COMMAND_DICTIONARY = { |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 26 | 'commit': |
| 27 | 'Marks given ebuilds as stable locally', |
| 28 | 'push': |
| 29 | 'Pushes previous marking of ebuilds to remote repo', |
| 30 | } |
| 31 | |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 32 | |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 33 | # ======================= Global Helper Functions ======================== |
| 34 | |
| 35 | |
David James | cc09c9b | 2012-01-26 22:10:13 -0800 | [diff] [blame] | 36 | def CleanStalePackages(boards, package_atoms): |
Chris Sosa | bf15387 | 2011-04-28 14:21:09 -0700 | [diff] [blame] | 37 | """Cleans up stale package info from a previous build. |
| 38 | Args: |
David James | cc09c9b | 2012-01-26 22:10:13 -0800 | [diff] [blame] | 39 | boards: Boards to clean the packages from. |
Mike Frysinger | de5ab0e | 2013-03-21 20:48:36 -0400 | [diff] [blame] | 40 | package_atoms: A list of package atoms to unmerge. |
Chris Sosa | bf15387 | 2011-04-28 14:21:09 -0700 | [diff] [blame] | 41 | """ |
David James | 15ed130 | 2013-04-25 09:21:19 -0700 | [diff] [blame] | 42 | if package_atoms: |
| 43 | cros_build_lib.Info('Cleaning up stale packages %s.' % package_atoms) |
| 44 | |
Mike Frysinger | b7ab9b8 | 2012-04-04 16:22:43 -0400 | [diff] [blame] | 45 | # First unmerge all the packages for a board, then eclean it. |
| 46 | # We need these two steps to run in order (unmerge/eclean), |
| 47 | # but we can let all the boards run in parallel. |
| 48 | def _CleanStalePackages(board): |
| 49 | if board: |
| 50 | suffix = '-' + board |
| 51 | runcmd = cros_build_lib.RunCommand |
| 52 | else: |
| 53 | suffix = '' |
| 54 | runcmd = cros_build_lib.SudoRunCommand |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 55 | |
David James | 59a0a2b | 2013-03-22 14:04:44 -0700 | [diff] [blame] | 56 | emerge, eclean = 'emerge' + suffix, 'eclean' + suffix |
| 57 | if not osutils.FindMissingBinaries([emerge, eclean]): |
Mike Frysinger | de5ab0e | 2013-03-21 20:48:36 -0400 | [diff] [blame] | 58 | # If nothing was found to be unmerged, emerge will exit(1). |
David James | 59a0a2b | 2013-03-22 14:04:44 -0700 | [diff] [blame] | 59 | result = runcmd([emerge, '-q', '--unmerge'] + package_atoms, |
Mike Frysinger | de5ab0e | 2013-03-21 20:48:36 -0400 | [diff] [blame] | 60 | extra_env={'CLEAN_DELAY': '0'}, error_code_ok=True) |
| 61 | if not result.returncode in (0, 1): |
| 62 | raise cros_build_lib.RunCommandError('unexpected error', result) |
David James | 59a0a2b | 2013-03-22 14:04:44 -0700 | [diff] [blame] | 63 | runcmd([eclean, '-d', 'packages'], |
| 64 | redirect_stdout=True, redirect_stderr=True) |
Mike Frysinger | b7ab9b8 | 2012-04-04 16:22:43 -0400 | [diff] [blame] | 65 | |
| 66 | tasks = [] |
David James | cc09c9b | 2012-01-26 22:10:13 -0800 | [diff] [blame] | 67 | for board in boards: |
Mike Frysinger | b7ab9b8 | 2012-04-04 16:22:43 -0400 | [diff] [blame] | 68 | tasks.append([board]) |
| 69 | tasks.append([None]) |
| 70 | |
David James | 6450a0a | 2012-12-04 07:59:53 -0800 | [diff] [blame] | 71 | parallel.RunTasksInProcessPool(_CleanStalePackages, tasks) |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 72 | |
| 73 | |
Mike Frysinger | 2ebe373 | 2012-05-08 17:04:12 -0400 | [diff] [blame] | 74 | # TODO(build): This code needs to be gutted and rebased to cros_build_lib. |
| 75 | def _DoWeHaveLocalCommits(stable_branch, tracking_branch, cwd): |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 76 | """Returns true if there are local commits.""" |
David James | 97d9587 | 2012-11-16 15:09:56 -0800 | [diff] [blame] | 77 | current_branch = git.GetCurrentBranch(cwd) |
Brian Harring | 609dc4e | 2012-05-07 02:17:44 -0700 | [diff] [blame] | 78 | |
| 79 | if current_branch != stable_branch: |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 80 | return False |
David James | 97d9587 | 2012-11-16 15:09:56 -0800 | [diff] [blame] | 81 | output = git.RunGit( |
Brian Harring | 609dc4e | 2012-05-07 02:17:44 -0700 | [diff] [blame] | 82 | cwd, ['rev-parse', 'HEAD', tracking_branch]).output.split() |
Brian Harring | 5b86b5e | 2012-05-25 18:27:40 -0700 | [diff] [blame] | 83 | return output[0] != output[1] |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 84 | |
| 85 | |
David James | 1b36358 | 2012-12-17 11:53:11 -0800 | [diff] [blame] | 86 | def _CheckSaneArguments(command, options): |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 87 | """Checks to make sure the flags are sane. Dies if arguments are not sane.""" |
| 88 | if not command in COMMAND_DICTIONARY.keys(): |
| 89 | _PrintUsageAndDie('%s is not a valid command' % command) |
Chris Sosa | 62ad852 | 2011-03-08 17:46:17 -0800 | [diff] [blame] | 90 | if not options.packages and command == 'commit' and not options.all: |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 91 | _PrintUsageAndDie('Please specify at least one package') |
Mike Frysinger | 8fd67dc | 2012-12-03 23:51:18 -0500 | [diff] [blame] | 92 | if options.boards: |
| 93 | cros_build_lib.AssertInsideChroot() |
Chris Sosa | 62ad852 | 2011-03-08 17:46:17 -0800 | [diff] [blame] | 94 | if not os.path.isdir(options.srcroot): |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 95 | _PrintUsageAndDie('srcroot is not a valid path') |
Chris Sosa | 62ad852 | 2011-03-08 17:46:17 -0800 | [diff] [blame] | 96 | options.srcroot = os.path.abspath(options.srcroot) |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 97 | |
| 98 | |
| 99 | def _PrintUsageAndDie(error_message=''): |
| 100 | """Prints optional error_message the usage and returns an error exit code.""" |
| 101 | command_usage = 'Commands: \n' |
| 102 | # Add keys and usage information from dictionary. |
| 103 | commands = sorted(COMMAND_DICTIONARY.keys()) |
| 104 | for command in commands: |
| 105 | command_usage += ' %s: %s\n' % (command, COMMAND_DICTIONARY[command]) |
| 106 | commands_str = '|'.join(commands) |
Chris Sosa | c13bba5 | 2011-05-24 15:14:09 -0700 | [diff] [blame] | 107 | cros_build_lib.Warning('Usage: %s FLAGS [%s]\n\n%s' % ( |
| 108 | sys.argv[0], commands_str, command_usage)) |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 109 | if error_message: |
Chris Sosa | c13bba5 | 2011-05-24 15:14:09 -0700 | [diff] [blame] | 110 | cros_build_lib.Die(error_message) |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 111 | else: |
| 112 | sys.exit(1) |
| 113 | |
| 114 | |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 115 | # ======================= End Global Helper Functions ======================== |
| 116 | |
| 117 | |
Mike Frysinger | 2ebe373 | 2012-05-08 17:04:12 -0400 | [diff] [blame] | 118 | def PushChange(stable_branch, tracking_branch, dryrun, cwd): |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 119 | """Pushes commits in the stable_branch to the remote git repository. |
| 120 | |
David James | ee2da62 | 2012-02-23 09:32:16 -0800 | [diff] [blame] | 121 | Pushes local commits from calls to CommitChange to the remote git |
David James | 6600946 | 2012-03-25 10:08:38 -0700 | [diff] [blame] | 122 | repository specified by current working directory. If changes are |
| 123 | found to commit, they will be merged to the merge branch and pushed. |
| 124 | In that case, the local repository will be left on the merge branch. |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 125 | |
| 126 | Args: |
| 127 | stable_branch: The local branch with commits we want to push. |
| 128 | tracking_branch: The tracking branch of the local branch. |
Chris Sosa | 62ad852 | 2011-03-08 17:46:17 -0800 | [diff] [blame] | 129 | dryrun: Use git push --dryrun to emulate a push. |
Mike Frysinger | 2ebe373 | 2012-05-08 17:04:12 -0400 | [diff] [blame] | 130 | cwd: The directory to run commands in. |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 131 | Raises: |
| 132 | OSError: Error occurred while pushing. |
| 133 | """ |
Mike Frysinger | 2ebe373 | 2012-05-08 17:04:12 -0400 | [diff] [blame] | 134 | if not _DoWeHaveLocalCommits(stable_branch, tracking_branch, cwd): |
Brian Harring | 609dc4e | 2012-05-07 02:17:44 -0700 | [diff] [blame] | 135 | cros_build_lib.Info('No work found to push in %s. Exiting', cwd) |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 136 | return |
| 137 | |
David James | 6600946 | 2012-03-25 10:08:38 -0700 | [diff] [blame] | 138 | # For the commit queue, our local branch may contain commits that were |
| 139 | # just tested and pushed during the CommitQueueCompletion stage. Sync |
| 140 | # and rebase our local branch on top of the remote commits. |
David James | 97d9587 | 2012-11-16 15:09:56 -0800 | [diff] [blame] | 141 | remote, push_branch = git.GetTrackingBranch(cwd, for_push=True) |
| 142 | git.SyncPushBranch(cwd, remote, push_branch) |
David James | 6600946 | 2012-03-25 10:08:38 -0700 | [diff] [blame] | 143 | |
| 144 | # Check whether any local changes remain after the sync. |
Brian Harring | 609dc4e | 2012-05-07 02:17:44 -0700 | [diff] [blame] | 145 | if not _DoWeHaveLocalCommits(stable_branch, push_branch, cwd): |
| 146 | cros_build_lib.Info('All changes already pushed for %s. Exiting', cwd) |
David James | 6600946 | 2012-03-25 10:08:38 -0700 | [diff] [blame] | 147 | return |
| 148 | |
David James | 67d7325 | 2013-09-19 17:33:12 -0700 | [diff] [blame] | 149 | description = git.RunGit(cwd, |
| 150 | ['log', '--format=format:%s%n%n%b', '%s..%s' % ( |
| 151 | push_branch, stable_branch)]).output |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 152 | description = 'Marking set of ebuilds as stable\n\n%s' % description |
Brian Harring | 609dc4e | 2012-05-07 02:17:44 -0700 | [diff] [blame] | 153 | cros_build_lib.Info('For %s, using description %s', cwd, description) |
David James | 97d9587 | 2012-11-16 15:09:56 -0800 | [diff] [blame] | 154 | git.CreatePushBranch(constants.MERGE_BRANCH, cwd) |
| 155 | git.RunGit(cwd, ['merge', '--squash', stable_branch]) |
| 156 | git.RunGit(cwd, ['commit', '-m', description]) |
| 157 | git.RunGit(cwd, ['config', 'push.default', 'tracking']) |
| 158 | git.PushWithRetry(constants.MERGE_BRANCH, cwd, dryrun=dryrun) |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 159 | |
| 160 | |
| 161 | class GitBranch(object): |
| 162 | """Wrapper class for a git branch.""" |
| 163 | |
Mike Frysinger | 2ebe373 | 2012-05-08 17:04:12 -0400 | [diff] [blame] | 164 | def __init__(self, branch_name, tracking_branch, cwd): |
David James | c7c4ff5 | 2013-09-18 17:57:13 -0700 | [diff] [blame] | 165 | """Sets up variables but does not create the branch. |
| 166 | |
| 167 | Arguments: |
| 168 | branch_name: The name of the branch. |
| 169 | tracking_branch: The associated tracking branch. |
| 170 | cwd: The git repository to work in. |
| 171 | """ |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 172 | self.branch_name = branch_name |
| 173 | self.tracking_branch = tracking_branch |
Mike Frysinger | 2ebe373 | 2012-05-08 17:04:12 -0400 | [diff] [blame] | 174 | self.cwd = cwd |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 175 | |
| 176 | def CreateBranch(self): |
Mike Frysinger | 2ebe373 | 2012-05-08 17:04:12 -0400 | [diff] [blame] | 177 | self.Checkout() |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 178 | |
Mike Frysinger | 2ebe373 | 2012-05-08 17:04:12 -0400 | [diff] [blame] | 179 | def Checkout(self, branch=None): |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 180 | """Function used to check out to another GitBranch.""" |
Mike Frysinger | 2ebe373 | 2012-05-08 17:04:12 -0400 | [diff] [blame] | 181 | if not branch: |
| 182 | branch = self.branch_name |
| 183 | if branch == self.tracking_branch or self.Exists(branch): |
| 184 | git_cmd = ['git', 'checkout', '-f', branch] |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 185 | else: |
Mike Frysinger | 2ebe373 | 2012-05-08 17:04:12 -0400 | [diff] [blame] | 186 | git_cmd = ['repo', 'start', branch, '.'] |
| 187 | cros_build_lib.RunCommandCaptureOutput(git_cmd, print_cmd=False, |
| 188 | cwd=self.cwd) |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 189 | |
Mike Frysinger | 2ebe373 | 2012-05-08 17:04:12 -0400 | [diff] [blame] | 190 | def Exists(self, branch=None): |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 191 | """Returns True if the branch exists.""" |
Mike Frysinger | 2ebe373 | 2012-05-08 17:04:12 -0400 | [diff] [blame] | 192 | if not branch: |
| 193 | branch = self.branch_name |
David James | 67d7325 | 2013-09-19 17:33:12 -0700 | [diff] [blame] | 194 | branches = git.RunGit(self.cwd, ['branch']).output |
Mike Frysinger | 2ebe373 | 2012-05-08 17:04:12 -0400 | [diff] [blame] | 195 | return branch in branches.split() |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 196 | |
| 197 | |
David James | 1b36358 | 2012-12-17 11:53:11 -0800 | [diff] [blame] | 198 | def main(_argv): |
Chris Sosa | 62ad852 | 2011-03-08 17:46:17 -0800 | [diff] [blame] | 199 | parser = optparse.OptionParser('cros_mark_as_stable OPTIONS packages') |
| 200 | parser.add_option('--all', action='store_true', |
| 201 | help='Mark all packages as stable.') |
Mike Frysinger | 587bd56 | 2012-11-19 16:41:39 -0500 | [diff] [blame] | 202 | parser.add_option('-b', '--boards', default='', |
David James | cc09c9b | 2012-01-26 22:10:13 -0800 | [diff] [blame] | 203 | help='Colon-separated list of boards') |
Chris Sosa | 62ad852 | 2011-03-08 17:46:17 -0800 | [diff] [blame] | 204 | parser.add_option('--drop_file', |
| 205 | help='File to list packages that were revved.') |
| 206 | parser.add_option('--dryrun', action='store_true', |
| 207 | help='Passes dry-run to git push if pushing a change.') |
| 208 | parser.add_option('-o', '--overlays', |
| 209 | help='Colon-separated list of overlays to modify.') |
| 210 | parser.add_option('-p', '--packages', |
| 211 | help='Colon separated list of packages to rev.') |
| 212 | parser.add_option('-r', '--srcroot', |
Mike Frysinger | fddaeb5 | 2012-11-20 11:17:31 -0500 | [diff] [blame] | 213 | default=os.path.join(constants.SOURCE_ROOT, 'src'), |
Chris Sosa | 62ad852 | 2011-03-08 17:46:17 -0800 | [diff] [blame] | 214 | help='Path to root src directory.') |
Chris Sosa | 62ad852 | 2011-03-08 17:46:17 -0800 | [diff] [blame] | 215 | parser.add_option('--verbose', action='store_true', |
| 216 | help='Prints out debug info.') |
| 217 | (options, args) = parser.parse_args() |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 218 | |
J. Richard Barnette | 2fa9fd6 | 2011-12-02 17:10:10 -0800 | [diff] [blame] | 219 | portage_utilities.EBuild.VERBOSE = options.verbose |
Chris Sosa | 62ad852 | 2011-03-08 17:46:17 -0800 | [diff] [blame] | 220 | |
| 221 | if len(args) != 1: |
Ryan Cui | 05a31ba | 2011-05-31 17:47:37 -0700 | [diff] [blame] | 222 | _PrintUsageAndDie('Must specify a valid command [commit, push]') |
Chris Sosa | 62ad852 | 2011-03-08 17:46:17 -0800 | [diff] [blame] | 223 | |
| 224 | command = args[0] |
| 225 | package_list = None |
| 226 | if options.packages: |
| 227 | package_list = options.packages.split(':') |
| 228 | |
David James | 1b36358 | 2012-12-17 11:53:11 -0800 | [diff] [blame] | 229 | _CheckSaneArguments(command, options) |
Chris Sosa | 62ad852 | 2011-03-08 17:46:17 -0800 | [diff] [blame] | 230 | if options.overlays: |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 231 | overlays = {} |
Chris Sosa | 62ad852 | 2011-03-08 17:46:17 -0800 | [diff] [blame] | 232 | for path in options.overlays.split(':'): |
Ryan Cui | 05a31ba | 2011-05-31 17:47:37 -0700 | [diff] [blame] | 233 | if not os.path.isdir(path): |
Chris Sosa | c13bba5 | 2011-05-24 15:14:09 -0700 | [diff] [blame] | 234 | cros_build_lib.Die('Cannot find overlay: %s' % path) |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 235 | overlays[path] = [] |
| 236 | else: |
Chris Sosa | c13bba5 | 2011-05-24 15:14:09 -0700 | [diff] [blame] | 237 | cros_build_lib.Warning('Missing --overlays argument') |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 238 | overlays = { |
Chris Sosa | 62ad852 | 2011-03-08 17:46:17 -0800 | [diff] [blame] | 239 | '%s/private-overlays/chromeos-overlay' % options.srcroot: [], |
| 240 | '%s/third_party/chromiumos-overlay' % options.srcroot: [] |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 241 | } |
| 242 | |
David James | 97d9587 | 2012-11-16 15:09:56 -0800 | [diff] [blame] | 243 | manifest = git.ManifestCheckout.Cached(options.srcroot) |
Ryan Cui | 4656a3c | 2011-05-24 12:30:30 -0700 | [diff] [blame] | 244 | |
David James | 84e953c | 2013-04-23 18:44:06 -0700 | [diff] [blame] | 245 | if command == 'commit': |
| 246 | portage_utilities.BuildEBuildDictionary(overlays, options.all, package_list) |
| 247 | |
David James | 15ed130 | 2013-04-25 09:21:19 -0700 | [diff] [blame] | 248 | # Contains the array of packages we actually revved. |
| 249 | revved_packages = [] |
| 250 | new_package_atoms = [] |
David James | a8457b5 | 2011-05-28 00:03:20 -0700 | [diff] [blame] | 251 | |
David James | 15ed130 | 2013-04-25 09:21:19 -0700 | [diff] [blame] | 252 | # Slight optimization hack: process the chromiumos overlay before any other |
| 253 | # cros-workon overlay first so we can do background cache generation in it. |
| 254 | # A perfect solution would walk all the overlays, figure out any dependencies |
| 255 | # between them (with layout.conf), and then process them in dependency order. |
| 256 | # However, this operation isn't slow enough to warrant that level of |
| 257 | # complexity, so we'll just special case the main overlay. |
| 258 | # |
| 259 | # Similarly, generate the cache in the portage-stable tree asap. We know |
| 260 | # we won't have any cros-workon packages in there, so generating the cache |
| 261 | # is the only thing it'll be doing. The chromiumos overlay instead might |
| 262 | # have revbumping to do before it can generate the cache. |
| 263 | keys = overlays.keys() |
| 264 | for overlay in ('/third_party/chromiumos-overlay', |
| 265 | '/third_party/portage-stable'): |
| 266 | for k in keys: |
| 267 | if k.endswith(overlay): |
| 268 | keys.remove(k) |
| 269 | keys.insert(0, k) |
| 270 | break |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 271 | |
David James | 15ed130 | 2013-04-25 09:21:19 -0700 | [diff] [blame] | 272 | with parallel.BackgroundTaskRunner(portage_utilities.RegenCache) as queue: |
| 273 | for overlay in keys: |
| 274 | ebuilds = overlays[overlay] |
| 275 | if not os.path.isdir(overlay): |
| 276 | cros_build_lib.Warning("Skipping %s" % overlay) |
| 277 | continue |
Chris Sosa | dad0d32 | 2011-01-31 16:37:33 -0800 | [diff] [blame] | 278 | |
David James | 15ed130 | 2013-04-25 09:21:19 -0700 | [diff] [blame] | 279 | # Note we intentionally work from the non push tracking branch; |
| 280 | # everything built thus far has been against it (meaning, http mirrors), |
| 281 | # thus we should honor that. During the actual push, the code switches |
| 282 | # to the correct urls, and does an appropriate rebasing. |
| 283 | tracking_branch = git.GetTrackingBranchViaManifest( |
| 284 | overlay, manifest=manifest)[1] |
Brian Harring | 609dc4e | 2012-05-07 02:17:44 -0700 | [diff] [blame] | 285 | |
David James | 15ed130 | 2013-04-25 09:21:19 -0700 | [diff] [blame] | 286 | if command == 'push': |
| 287 | PushChange(constants.STABLE_EBUILD_BRANCH, tracking_branch, |
| 288 | options.dryrun, cwd=overlay) |
| 289 | elif command == 'commit': |
| 290 | existing_branch = git.GetCurrentBranch(overlay) |
| 291 | work_branch = GitBranch(constants.STABLE_EBUILD_BRANCH, tracking_branch, |
| 292 | cwd=overlay) |
| 293 | work_branch.CreateBranch() |
| 294 | if not work_branch.Exists(): |
| 295 | cros_build_lib.Die('Unable to create stabilizing branch in %s' % |
| 296 | overlay) |
| 297 | |
| 298 | # In the case of uprevving overlays that have patches applied to them, |
| 299 | # include the patched changes in the stabilizing branch. |
| 300 | if existing_branch: |
David James | 67d7325 | 2013-09-19 17:33:12 -0700 | [diff] [blame] | 301 | git.RunGit(overlay, ['rebase', existing_branch]) |
David James | 15ed130 | 2013-04-25 09:21:19 -0700 | [diff] [blame] | 302 | |
| 303 | messages = [] |
| 304 | for ebuild in ebuilds: |
| 305 | if options.verbose: |
| 306 | cros_build_lib.Info('Working on %s', ebuild.package) |
| 307 | try: |
| 308 | new_package = ebuild.RevWorkOnEBuild(options.srcroot, manifest) |
| 309 | if new_package: |
| 310 | revved_packages.append(ebuild.package) |
| 311 | new_package_atoms.append('=%s' % new_package) |
| 312 | messages.append(_GIT_COMMIT_MESSAGE % ebuild.package) |
| 313 | except (OSError, IOError): |
| 314 | cros_build_lib.Warning('Cannot rev %s\n' % ebuild.package + |
| 315 | 'Note you will have to go into %s ' |
| 316 | 'and reset the git repo yourself.' % overlay) |
| 317 | raise |
| 318 | |
| 319 | if messages: |
| 320 | portage_utilities.EBuild.CommitChange('\n\n'.join(messages), overlay) |
| 321 | |
| 322 | if cros_build_lib.IsInsideChroot(): |
| 323 | # Regenerate caches if need be. We do this all the time to |
| 324 | # catch when users make changes without updating cache files. |
| 325 | queue.put([overlay]) |
| 326 | |
| 327 | if command == 'commit': |
| 328 | if cros_build_lib.IsInsideChroot(): |
| 329 | CleanStalePackages(options.boards.split(':'), new_package_atoms) |
| 330 | if options.drop_file: |
| 331 | osutils.WriteFile(options.drop_file, ' '.join(revved_packages)) |