Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(14)

Side by Side Diff: scripts/slave/recipe_modules/auto_bisect/bisector.py

Issue 1917243002: Revert "build: roll infra_paths changes" (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 # Copyright 2015 The Chromium Authors. All rights reserved. 1 # Copyright 2015 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 import json 5 import json
6 import re 6 import re
7 import time 7 import time
8 import urllib 8 import urllib
9 9
10 from . import config_validation 10 from . import config_validation
(...skipping 327 matching lines...) Expand 10 before | Expand all | Expand 10 after
338 r'(?<=["\']%s["\']: ["\'])([a-fA-F0-9]+)(?=["\'])' % deps_var, 338 r'(?<=["\']%s["\']: ["\'])([a-fA-F0-9]+)(?=["\'])' % deps_var,
339 re.MULTILINE) 339 re.MULTILINE)
340 if not re.search(deps_item_regexp, original_contents): # pragma: no cover 340 if not re.search(deps_item_regexp, original_contents): # pragma: no cover
341 reason = 'DEPS file does not contain entry for ' + deps_var 341 reason = 'DEPS file does not contain entry for ' + deps_var
342 self.api.m.halt(reason) 342 self.api.m.halt(reason)
343 raise self.api.m.step.StepFailure(reason) 343 raise self.api.m.step.StepFailure(reason)
344 344
345 patched_contents = re.sub(deps_item_regexp, new_commit_hash, 345 patched_contents = re.sub(deps_item_regexp, new_commit_hash,
346 original_contents) 346 original_contents)
347 347
348 cwd = self.api.m.infra_paths['slave_build'].join( 348 cwd = self.api.m.path['slave_build'].join(
349 depot_config.DEPOT_DEPS_NAME[base_revision.depot_name]['src']) 349 depot_config.DEPOT_DEPS_NAME[base_revision.depot_name]['src'])
350 deps_file = 'DEPS' 350 deps_file = 'DEPS'
351 # This is to support backward compatibility on android-chrome because 351 # This is to support backward compatibility on android-chrome because
352 # .DEPS.git got migrated to DEPS on April 5, 2016 352 # .DEPS.git got migrated to DEPS on April 5, 2016
353 if (base_revision.depot_name == 'android-chrome' and 353 if (base_revision.depot_name == 'android-chrome' and
354 self.api.m.path.exists(self.api.m.path.join(cwd, '.DEPS.git'))): 354 self.api.m.path.exists(self.api.m.path.join(cwd, '.DEPS.git'))):
355 deps_file = '.DEPS.git' # pragma: no cover 355 deps_file = '.DEPS.git' # pragma: no cover
356 356
357 interned_deps_hash = self._git_intern_file( 357 interned_deps_hash = self._git_intern_file(
358 patched_contents, cwd, new_commit_hash) 358 patched_contents, cwd, new_commit_hash)
(...skipping 581 matching lines...) Expand 10 before | Expand all | Expand 10 after
940 }) 940 })
941 return revision_rows 941 return revision_rows
942 942
943 def _get_build_url(self): 943 def _get_build_url(self):
944 properties = self.api.m.properties 944 properties = self.api.m.properties
945 bot_url = properties.get('buildbotURL', 945 bot_url = properties.get('buildbotURL',
946 'http://build.chromium.org/p/chromium/') 946 'http://build.chromium.org/p/chromium/')
947 builder_name = urllib.quote(properties.get('buildername', '')) 947 builder_name = urllib.quote(properties.get('buildername', ''))
948 builder_number = str(properties.get('buildnumber', '')) 948 builder_number = str(properties.get('buildnumber', ''))
949 return '%sbuilders/%s/builds/%s' % (bot_url, builder_name, builder_number) 949 return '%sbuilders/%s/builds/%s' % (bot_url, builder_name, builder_number)
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698