Scott Zawalski | 6bc41ac | 2010-09-08 12:47:28 -0700 | [diff] [blame] | 1 | # Copyright (c) 2010 The Chromium OS Authors. All rights reserved. |
| 2 | # Use of this source code is governed by a BSD-style license that can be |
| 3 | # found in the LICENSE file. |
| 4 | |
| 5 | """Common python commands used by various build scripts.""" |
| 6 | |
Chris Sosa | 471532a | 2011-02-01 15:10:06 -0800 | [diff] [blame] | 7 | import inspect |
Scott Zawalski | 98ac6b2 | 2010-09-08 15:59:23 -0700 | [diff] [blame] | 8 | import os |
Tan Gao | 2990a4d | 2010-09-22 09:34:27 -0700 | [diff] [blame] | 9 | import re |
Doug Anderson | 6781f94 | 2011-01-14 16:21:39 -0800 | [diff] [blame] | 10 | import signal |
Scott Zawalski | 6bc41ac | 2010-09-08 12:47:28 -0700 | [diff] [blame] | 11 | import subprocess |
| 12 | import sys |
David James | 645bf9e | 2011-05-26 23:50:49 -0700 | [diff] [blame^] | 13 | import time |
Simon Glass | 53ed230 | 2011-02-08 18:42:16 -0800 | [diff] [blame] | 14 | from terminal import Color |
| 15 | |
Scott Zawalski | 6bc41ac | 2010-09-08 12:47:28 -0700 | [diff] [blame] | 16 | |
| 17 | _STDOUT_IS_TTY = hasattr(sys.stdout, 'isatty') and sys.stdout.isatty() |
| 18 | |
David James | 645bf9e | 2011-05-26 23:50:49 -0700 | [diff] [blame^] | 19 | class GitPushFailed(Exception): |
| 20 | """Raised when a git push failed after retry.""" |
| 21 | pass |
Tan Gao | 2f31088 | 2010-09-10 14:50:47 -0700 | [diff] [blame] | 22 | |
| 23 | class CommandResult(object): |
| 24 | """An object to store various attributes of a child process.""" |
| 25 | |
| 26 | def __init__(self): |
| 27 | self.cmd = None |
| 28 | self.error = None |
| 29 | self.output = None |
| 30 | self.returncode = None |
| 31 | |
| 32 | |
| 33 | class RunCommandError(Exception): |
| 34 | """Error caught in RunCommand() method.""" |
Don Garrett | b85946a | 2011-03-10 18:11:08 -0800 | [diff] [blame] | 35 | def __init__(self, msg, cmd): |
| 36 | self.cmd = cmd |
| 37 | Exception.__init__(self, msg) |
Tan Gao | 2f31088 | 2010-09-10 14:50:47 -0700 | [diff] [blame] | 38 | |
Don Garrett | b85946a | 2011-03-10 18:11:08 -0800 | [diff] [blame] | 39 | def __eq__(self, other): |
| 40 | return (type(self) == type(other) and |
| 41 | str(self) == str(other) and |
| 42 | self.cmd == other.cmd) |
| 43 | |
| 44 | def __ne__(self, other): |
| 45 | return not self.__eq__(other) |
Tan Gao | 2f31088 | 2010-09-10 14:50:47 -0700 | [diff] [blame] | 46 | |
Scott Zawalski | 6bc41ac | 2010-09-08 12:47:28 -0700 | [diff] [blame] | 47 | def RunCommand(cmd, print_cmd=True, error_ok=False, error_message=None, |
| 48 | exit_code=False, redirect_stdout=False, redirect_stderr=False, |
Doug Anderson | 6781f94 | 2011-01-14 16:21:39 -0800 | [diff] [blame] | 49 | cwd=None, input=None, enter_chroot=False, shell=False, |
Chris Sosa | 66c8c25 | 2011-02-17 11:44:09 -0800 | [diff] [blame] | 50 | env=None, ignore_sigint=False, combine_stdout_stderr=False): |
David James | 6db8f52 | 2010-09-09 10:49:11 -0700 | [diff] [blame] | 51 | """Runs a command. |
Scott Zawalski | 6bc41ac | 2010-09-08 12:47:28 -0700 | [diff] [blame] | 52 | |
Tan Gao | 2990a4d | 2010-09-22 09:34:27 -0700 | [diff] [blame] | 53 | Args: |
| 54 | cmd: cmd to run. Should be input to subprocess.Popen. |
| 55 | print_cmd: prints the command before running it. |
| 56 | error_ok: does not raise an exception on error. |
| 57 | error_message: prints out this message when an error occurrs. |
| 58 | exit_code: returns the return code of the shell command. |
| 59 | redirect_stdout: returns the stdout. |
| 60 | redirect_stderr: holds stderr output until input is communicated. |
| 61 | cwd: the working directory to run this cmd. |
| 62 | input: input to pipe into this command through stdin. |
| 63 | enter_chroot: this command should be run from within the chroot. If set, |
Scott Zawalski | 6bc41ac | 2010-09-08 12:47:28 -0700 | [diff] [blame] | 64 | cwd must point to the scripts directory. |
Tan Gao | 2990a4d | 2010-09-22 09:34:27 -0700 | [diff] [blame] | 65 | shell: If shell is True, the specified command will be executed through |
| 66 | the shell. |
Doug Anderson | 6781f94 | 2011-01-14 16:21:39 -0800 | [diff] [blame] | 67 | env: If non-None, this is the environment for the new process. |
| 68 | ignore_sigint: If True, we'll ignore signal.SIGINT before calling the |
| 69 | child. This is the desired behavior if we know our child will handle |
| 70 | Ctrl-C. If we don't do this, I think we and the child will both get |
| 71 | Ctrl-C at the same time, which means we'll forcefully kill the child. |
Chris Sosa | 66c8c25 | 2011-02-17 11:44:09 -0800 | [diff] [blame] | 72 | combine_stdout_stderr: Combines stdout and stdin streams into stdout. |
Tan Gao | 2990a4d | 2010-09-22 09:34:27 -0700 | [diff] [blame] | 73 | |
| 74 | Returns: |
| 75 | A CommandResult object. |
| 76 | |
Scott Zawalski | 6bc41ac | 2010-09-08 12:47:28 -0700 | [diff] [blame] | 77 | Raises: |
| 78 | Exception: Raises generic exception on error with optional error_message. |
| 79 | """ |
| 80 | # Set default for variables. |
| 81 | stdout = None |
| 82 | stderr = None |
| 83 | stdin = None |
Tan Gao | 2f31088 | 2010-09-10 14:50:47 -0700 | [diff] [blame] | 84 | cmd_result = CommandResult() |
Scott Zawalski | 6bc41ac | 2010-09-08 12:47:28 -0700 | [diff] [blame] | 85 | |
| 86 | # Modify defaults based on parameters. |
Tan Gao | 2990a4d | 2010-09-22 09:34:27 -0700 | [diff] [blame] | 87 | if redirect_stdout: stdout = subprocess.PIPE |
| 88 | if redirect_stderr: stderr = subprocess.PIPE |
Chris Sosa | 66c8c25 | 2011-02-17 11:44:09 -0800 | [diff] [blame] | 89 | if combine_stdout_stderr: stderr = subprocess.STDOUT |
Tan Gao | 2990a4d | 2010-09-22 09:34:27 -0700 | [diff] [blame] | 90 | # TODO(sosa): gpylint complains about redefining built-in 'input'. |
| 91 | # Can we rename this variable? |
| 92 | if input: stdin = subprocess.PIPE |
David James | 6db8f52 | 2010-09-09 10:49:11 -0700 | [diff] [blame] | 93 | if isinstance(cmd, basestring): |
| 94 | if enter_chroot: cmd = './enter_chroot.sh -- ' + cmd |
| 95 | cmd_str = cmd |
| 96 | else: |
| 97 | if enter_chroot: cmd = ['./enter_chroot.sh', '--'] + cmd |
| 98 | cmd_str = ' '.join(cmd) |
Scott Zawalski | 6bc41ac | 2010-09-08 12:47:28 -0700 | [diff] [blame] | 99 | |
| 100 | # Print out the command before running. |
| 101 | if print_cmd: |
David James | 6db8f52 | 2010-09-09 10:49:11 -0700 | [diff] [blame] | 102 | Info('RunCommand: %s' % cmd_str) |
Doug Anderson | a8d22de | 2011-01-13 16:22:58 -0800 | [diff] [blame] | 103 | cmd_result.cmd = cmd |
Scott Zawalski | 6bc41ac | 2010-09-08 12:47:28 -0700 | [diff] [blame] | 104 | |
| 105 | try: |
David James | 9102a89 | 2010-12-02 10:21:49 -0800 | [diff] [blame] | 106 | proc = subprocess.Popen(cmd, cwd=cwd, stdin=stdin, stdout=stdout, |
Doug Anderson | 6781f94 | 2011-01-14 16:21:39 -0800 | [diff] [blame] | 107 | stderr=stderr, shell=shell, env=env) |
| 108 | if ignore_sigint: |
| 109 | old_sigint = signal.signal(signal.SIGINT, signal.SIG_IGN) |
| 110 | try: |
| 111 | (cmd_result.output, cmd_result.error) = proc.communicate(input) |
| 112 | finally: |
| 113 | if ignore_sigint: |
| 114 | signal.signal(signal.SIGINT, old_sigint) |
| 115 | |
Scott Zawalski | 6bc41ac | 2010-09-08 12:47:28 -0700 | [diff] [blame] | 116 | if exit_code: |
Tan Gao | 2f31088 | 2010-09-10 14:50:47 -0700 | [diff] [blame] | 117 | cmd_result.returncode = proc.returncode |
Scott Zawalski | 6bc41ac | 2010-09-08 12:47:28 -0700 | [diff] [blame] | 118 | |
| 119 | if not error_ok and proc.returncode: |
Tan Gao | 2f31088 | 2010-09-10 14:50:47 -0700 | [diff] [blame] | 120 | msg = ('Command "%s" failed.\n' % cmd_str + |
| 121 | (error_message or cmd_result.error or cmd_result.output or '')) |
Don Garrett | b85946a | 2011-03-10 18:11:08 -0800 | [diff] [blame] | 122 | raise RunCommandError(msg, cmd) |
Tan Gao | 2990a4d | 2010-09-22 09:34:27 -0700 | [diff] [blame] | 123 | # TODO(sosa): is it possible not to use the catch-all Exception here? |
| 124 | except Exception, e: |
Scott Zawalski | 6bc41ac | 2010-09-08 12:47:28 -0700 | [diff] [blame] | 125 | if not error_ok: |
| 126 | raise |
| 127 | else: |
| 128 | Warning(str(e)) |
| 129 | |
Tan Gao | 2f31088 | 2010-09-10 14:50:47 -0700 | [diff] [blame] | 130 | return cmd_result |
Scott Zawalski | 6bc41ac | 2010-09-08 12:47:28 -0700 | [diff] [blame] | 131 | |
| 132 | |
Simon Glass | 5329b93 | 2011-03-14 16:49:04 -0700 | [diff] [blame] | 133 | #TODO(sjg): Remove this in favor of operation.Die |
Scott Zawalski | 6bc41ac | 2010-09-08 12:47:28 -0700 | [diff] [blame] | 134 | def Die(message): |
| 135 | """Emits a red error message and halts execution. |
| 136 | |
Tan Gao | 2990a4d | 2010-09-22 09:34:27 -0700 | [diff] [blame] | 137 | Args: |
Scott Zawalski | 6bc41ac | 2010-09-08 12:47:28 -0700 | [diff] [blame] | 138 | message: The message to be emitted before exiting. |
| 139 | """ |
| 140 | print >> sys.stderr, ( |
| 141 | Color(_STDOUT_IS_TTY).Color(Color.RED, '\nERROR: ' + message)) |
| 142 | sys.exit(1) |
| 143 | |
| 144 | |
Simon Glass | 5329b93 | 2011-03-14 16:49:04 -0700 | [diff] [blame] | 145 | #TODO(sjg): Remove this in favor of operation.Warning |
Tan Gao | 2990a4d | 2010-09-22 09:34:27 -0700 | [diff] [blame] | 146 | # pylint: disable-msg=W0622 |
Scott Zawalski | 6bc41ac | 2010-09-08 12:47:28 -0700 | [diff] [blame] | 147 | def Warning(message): |
| 148 | """Emits a yellow warning message and continues execution. |
| 149 | |
Tan Gao | 2990a4d | 2010-09-22 09:34:27 -0700 | [diff] [blame] | 150 | Args: |
Scott Zawalski | 6bc41ac | 2010-09-08 12:47:28 -0700 | [diff] [blame] | 151 | message: The message to be emitted. |
| 152 | """ |
| 153 | print >> sys.stderr, ( |
| 154 | Color(_STDOUT_IS_TTY).Color(Color.YELLOW, '\nWARNING: ' + message)) |
| 155 | |
| 156 | |
Simon Glass | 5329b93 | 2011-03-14 16:49:04 -0700 | [diff] [blame] | 157 | # This command is deprecated in favor of operation.Info() |
| 158 | # It is left here for the moment so people are aware what happened. |
| 159 | # The reason is that this is not aware of the terminal output restrictions such |
| 160 | # as verbose, quiet and subprocess output. You should not be calling this. |
Simon Glass | 6b069ef | 2011-03-14 17:24:10 -0700 | [diff] [blame] | 161 | def Info(message): |
| 162 | """Emits a blue informational message and continues execution. |
| 163 | |
| 164 | Args: |
| 165 | message: The message to be emitted. |
| 166 | """ |
| 167 | print >> sys.stderr, ( |
| 168 | Color(_STDOUT_IS_TTY).Color(Color.BLUE, '\nINFO: ' + message)) |
Scott Zawalski | 98ac6b2 | 2010-09-08 15:59:23 -0700 | [diff] [blame] | 169 | |
| 170 | |
| 171 | def ListFiles(base_dir): |
| 172 | """Recurively list files in a directory. |
| 173 | |
Tan Gao | 2990a4d | 2010-09-22 09:34:27 -0700 | [diff] [blame] | 174 | Args: |
Scott Zawalski | 98ac6b2 | 2010-09-08 15:59:23 -0700 | [diff] [blame] | 175 | base_dir: directory to start recursively listing in. |
| 176 | |
| 177 | Returns: |
| 178 | A list of files relative to the base_dir path or |
| 179 | An empty list of there are no files in the directories. |
| 180 | """ |
| 181 | directories = [base_dir] |
| 182 | files_list = [] |
| 183 | while directories: |
| 184 | directory = directories.pop() |
| 185 | for name in os.listdir(directory): |
| 186 | fullpath = os.path.join(directory, name) |
| 187 | if os.path.isfile(fullpath): |
| 188 | files_list.append(fullpath) |
| 189 | elif os.path.isdir(fullpath): |
| 190 | directories.append(fullpath) |
| 191 | |
| 192 | return files_list |
Tan Gao | 2990a4d | 2010-09-22 09:34:27 -0700 | [diff] [blame] | 193 | |
| 194 | |
| 195 | def IsInsideChroot(): |
| 196 | """Returns True if we are inside chroot.""" |
| 197 | return os.path.exists('/etc/debian_chroot') |
| 198 | |
| 199 | |
| 200 | def GetSrcRoot(): |
| 201 | """Get absolute path to src/scripts/ directory. |
| 202 | |
| 203 | Assuming test script will always be run from descendent of src/scripts. |
| 204 | |
| 205 | Returns: |
| 206 | A string, absolute path to src/scripts directory. None if not found. |
| 207 | """ |
| 208 | src_root = None |
| 209 | match_str = '/src/scripts/' |
| 210 | test_script_path = os.path.abspath('.') |
| 211 | |
| 212 | path_list = re.split(match_str, test_script_path) |
| 213 | if path_list: |
| 214 | src_root = os.path.join(path_list[0], match_str.strip('/')) |
| 215 | Info ('src_root = %r' % src_root) |
| 216 | else: |
| 217 | Info ('No %r found in %r' % (match_str, test_script_path)) |
| 218 | |
| 219 | return src_root |
| 220 | |
| 221 | |
| 222 | def GetChromeosVersion(str_obj): |
| 223 | """Helper method to parse output for CHROMEOS_VERSION_STRING. |
| 224 | |
| 225 | Args: |
| 226 | str_obj: a string, which may contain Chrome OS version info. |
| 227 | |
| 228 | Returns: |
| 229 | A string, value of CHROMEOS_VERSION_STRING environment variable set by |
| 230 | chromeos_version.sh. Or None if not found. |
| 231 | """ |
| 232 | if str_obj is not None: |
| 233 | match = re.search('CHROMEOS_VERSION_STRING=([0-9_.]+)', str_obj) |
| 234 | if match and match.group(1): |
| 235 | Info ('CHROMEOS_VERSION_STRING = %s' % match.group(1)) |
| 236 | return match.group(1) |
| 237 | |
| 238 | Info ('CHROMEOS_VERSION_STRING NOT found') |
| 239 | return None |
| 240 | |
| 241 | |
| 242 | def GetOutputImageDir(board, cros_version): |
| 243 | """Construct absolute path to output image directory. |
| 244 | |
| 245 | Args: |
| 246 | board: a string. |
| 247 | cros_version: a string, Chrome OS version. |
| 248 | |
| 249 | Returns: |
| 250 | a string: absolute path to output directory. |
| 251 | """ |
| 252 | src_root = GetSrcRoot() |
| 253 | rel_path = 'build/images/%s' % board |
| 254 | # ASSUME: --build_attempt always sets to 1 |
| 255 | version_str = '-'.join([cros_version, 'a1']) |
| 256 | output_dir = os.path.join(os.path.dirname(src_root), rel_path, version_str) |
| 257 | Info ('output_dir = %s' % output_dir) |
| 258 | return output_dir |
Chris Sosa | 471532a | 2011-02-01 15:10:06 -0800 | [diff] [blame] | 259 | |
| 260 | |
| 261 | def FindRepoDir(path=None): |
| 262 | """Returns the nearest higher-level repo dir from the specified path. |
| 263 | |
| 264 | Args: |
| 265 | path: The path to use. Defaults to cwd. |
| 266 | """ |
| 267 | if path is None: |
| 268 | path = os.getcwd() |
| 269 | path = os.path.abspath(path) |
| 270 | while path != '/': |
| 271 | repo_dir = os.path.join(path, '.repo') |
| 272 | if os.path.isdir(repo_dir): |
| 273 | return repo_dir |
| 274 | path = os.path.dirname(path) |
| 275 | return None |
| 276 | |
| 277 | |
| 278 | def ReinterpretPathForChroot(path): |
| 279 | """Returns reinterpreted path from outside the chroot for use inside. |
| 280 | |
| 281 | Args: |
| 282 | path: The path to reinterpret. Must be in src tree. |
| 283 | """ |
| 284 | root_path = os.path.join(FindRepoDir(path), '..') |
| 285 | |
| 286 | path_abs_path = os.path.abspath(path) |
| 287 | root_abs_path = os.path.abspath(root_path) |
| 288 | |
| 289 | # Strip the repository root from the path and strip first /. |
| 290 | relative_path = path_abs_path.replace(root_abs_path, '')[1:] |
| 291 | |
| 292 | if relative_path == path_abs_path: |
| 293 | raise Exception('Error: path is outside your src tree, cannot reinterpret.') |
| 294 | |
| 295 | new_path = os.path.join('/home', os.getenv('USER'), 'trunk', relative_path) |
| 296 | return new_path |
| 297 | |
| 298 | |
David James | 645bf9e | 2011-05-26 23:50:49 -0700 | [diff] [blame^] | 299 | def GetPushBranch(branch, cwd): |
| 300 | """Gets the appropriate push branch for the specified branch / directory. |
| 301 | |
| 302 | If branch has a valid tracking branch, we should push to that branch. If |
| 303 | the tracking branch is a revision, we can't push to that, so we should look |
| 304 | at the default branch from the manifest. |
| 305 | |
| 306 | Args: |
| 307 | branch: Branch to examine for tracking branch. |
| 308 | cwd: Directory to look in. |
| 309 | """ |
| 310 | info = {} |
| 311 | for key in ('remote', 'merge'): |
| 312 | cmd = ['git', 'config', 'branch.%s.%s' % (branch, key)] |
| 313 | info[key] = RunCommand(cmd, redirect_stdout=True, cwd=cwd).output.strip() |
| 314 | if not info['merge'].startswith('refs/heads/'): |
| 315 | output = RunCommand(['repo', 'manifest', '-o', '-'], redirect_stdout=True, |
| 316 | cwd=cwd).output |
| 317 | m = re.search(r'<default[^>]*revision="(refs/heads/[^"]*)"', output) |
| 318 | assert m |
| 319 | info['merge'] = m.group(1) |
| 320 | assert info['merge'].startswith('refs/heads/') |
| 321 | return info['remote'], info['merge'].replace('refs/heads/', '') |
| 322 | |
| 323 | |
| 324 | def GitPushWithRetry(branch, cwd, dryrun=False, retries=5): |
| 325 | """General method to push local git changes. |
| 326 | |
| 327 | Args: |
| 328 | branch: Local branch to push. Branch should have already been created |
| 329 | with a local change committed ready to push to the remote branch. Must |
| 330 | also already be checked out to that branch. |
| 331 | cwd: Directory to push in. |
| 332 | dryrun: Git push --dry-run if set to True. |
| 333 | retries: The number of times to retry before giving up, default: 5 |
| 334 | |
| 335 | Raises: |
| 336 | GitPushFailed if push was unsuccessful after retries |
| 337 | """ |
| 338 | remote, push_branch = GetPushBranch(branch, cwd) |
| 339 | for retry in range(1, retries + 1): |
| 340 | try: |
| 341 | RunCommand(['git', 'remote', 'update'], cwd=cwd) |
| 342 | RunCommand(['git', 'rebase', '%s/%s' % (remote, push_branch)], cwd=cwd) |
| 343 | push_command = ['git', 'push', remote, '%s:%s' % (branch, push_branch)] |
| 344 | if dryrun: |
| 345 | push_command.append('--dry-run') |
| 346 | |
| 347 | RunCommand(push_command, cwd=cwd) |
| 348 | break |
| 349 | except RunCommandError: |
| 350 | if retry < retries: |
| 351 | print 'Error pushing changes trying again (%s/%s)' % (retry, retries) |
| 352 | time.sleep(5 * retry) |
| 353 | else: |
| 354 | raise GitPushFailed('Failed to push change after %s retries' % retries) |
| 355 | |
| 356 | |
Chris Sosa | 471532a | 2011-02-01 15:10:06 -0800 | [diff] [blame] | 357 | def GetCallerName(): |
| 358 | """Returns the name of the calling module with __main__.""" |
| 359 | top_frame = inspect.stack()[-1][0] |
| 360 | return os.path.basename(top_frame.f_code.co_filename) |
| 361 | |
| 362 | |
| 363 | class RunCommandException(Exception): |
| 364 | """Raised when there is an error in OldRunCommand.""" |
Don Garrett | b85946a | 2011-03-10 18:11:08 -0800 | [diff] [blame] | 365 | def __init__(self, msg, cmd): |
| 366 | self.cmd = cmd |
| 367 | Exception.__init__(self, msg) |
| 368 | |
| 369 | def __eq__(self, other): |
| 370 | return (type(self) == type(other) and |
| 371 | str(self) == str(other) and |
| 372 | self.cmd == other.cmd) |
| 373 | |
| 374 | def __ne__(self, other): |
| 375 | return not self.__eq__(other) |
Chris Sosa | 471532a | 2011-02-01 15:10:06 -0800 | [diff] [blame] | 376 | |
| 377 | |
| 378 | def OldRunCommand(cmd, print_cmd=True, error_ok=False, error_message=None, |
| 379 | exit_code=False, redirect_stdout=False, redirect_stderr=False, |
| 380 | cwd=None, input=None, enter_chroot=False, num_retries=0): |
| 381 | """Legacy run shell command. |
| 382 | |
| 383 | Arguments: |
| 384 | cmd: cmd to run. Should be input to subprocess.POpen. If a string, |
| 385 | converted to an array using split(). |
| 386 | print_cmd: prints the command before running it. |
| 387 | error_ok: does not raise an exception on error. |
| 388 | error_message: prints out this message when an error occurrs. |
| 389 | exit_code: returns the return code of the shell command. |
| 390 | redirect_stdout: returns the stdout. |
| 391 | redirect_stderr: holds stderr output until input is communicated. |
| 392 | cwd: the working directory to run this cmd. |
| 393 | input: input to pipe into this command through stdin. |
| 394 | enter_chroot: this command should be run from within the chroot. If set, |
| 395 | cwd must point to the scripts directory. |
| 396 | num_retries: the number of retries to perform before dying |
| 397 | |
| 398 | Returns: |
| 399 | If exit_code is True, returns the return code of the shell command. |
| 400 | Else returns the output of the shell command. |
| 401 | |
| 402 | Raises: |
| 403 | Exception: Raises RunCommandException on error with optional error_message. |
| 404 | """ |
| 405 | # Set default for variables. |
| 406 | stdout = None |
| 407 | stderr = None |
| 408 | stdin = None |
| 409 | output = '' |
| 410 | |
| 411 | # Modify defaults based on parameters. |
| 412 | if redirect_stdout: stdout = subprocess.PIPE |
| 413 | if redirect_stderr: stderr = subprocess.PIPE |
| 414 | if input: stdin = subprocess.PIPE |
| 415 | if enter_chroot: cmd = ['./enter_chroot.sh', '--'] + cmd |
| 416 | |
| 417 | # Print out the command before running. |
| 418 | if print_cmd: |
| 419 | Info('PROGRAM(%s) -> RunCommand: %r in dir %s' % |
| 420 | (GetCallerName(), cmd, cwd)) |
| 421 | |
| 422 | for retry_count in range(num_retries + 1): |
| 423 | try: |
| 424 | proc = subprocess.Popen(cmd, cwd=cwd, stdin=stdin, |
| 425 | stdout=stdout, stderr=stderr) |
| 426 | (output, error) = proc.communicate(input) |
| 427 | if exit_code and retry_count == num_retries: |
| 428 | return proc.returncode |
| 429 | |
| 430 | if proc.returncode == 0: |
| 431 | break |
| 432 | |
| 433 | raise RunCommandException('Command "%r" failed.\n' % (cmd) + |
Don Garrett | b85946a | 2011-03-10 18:11:08 -0800 | [diff] [blame] | 434 | (error_message or error or output or ''), |
| 435 | cmd) |
Chris Sosa | 471532a | 2011-02-01 15:10:06 -0800 | [diff] [blame] | 436 | except RunCommandException as e: |
| 437 | if not error_ok and retry_count == num_retries: |
| 438 | raise e |
| 439 | else: |
| 440 | Warning(str(e)) |
| 441 | if print_cmd: |
| 442 | Info('PROGRAM(%s) -> RunCommand: retrying %r in dir %s' % |
| 443 | (GetCallerName(), cmd, cwd)) |
| 444 | |
| 445 | return output |