scripts/slave/recipe_modules/auto_bisect/bisector.py - Issue 2225253003: Using the new convention for resolving the checkout directory.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: scripts/slave/recipe_modules/auto_bisect/bisector.py

Issue 2225253003: Using the new convention for resolving the checkout directory. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master

Patch Set: Created 4 years, 4 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 side-by-side diff with in-line comments

Download patch

Index: scripts/slave/recipe_modules/auto_bisect/bisector.py

diff --git a/scripts/slave/recipe_modules/auto_bisect/bisector.py b/scripts/slave/recipe_modules/auto_bisect/bisector.py

index 5cf20e35770a7b87b667cc6e24c1b003c2952bf9..4abe97cf9815e38d436aaf31a4e342c25c3971b0 100644

--- a/scripts/slave/recipe_modules/auto_bisect/bisector.py

+++ b/scripts/slave/recipe_modules/auto_bisect/bisector.py

@@ -348,7 +348,7 @@ class Bisector(object):

patched_contents = re.sub(deps_item_regexp, new_commit_hash,

original_contents)

- cwd = self.api.m.path['slave_build'].join(

+ cwd = (self.api.working_dir or self.api.m.path['slave_build']).join(

sullivan 2016/08/09 19:27:38 Please add a comment explaining why there are two

depot_config.DEPOT_DEPS_NAME[base_revision.depot_name]['src'])

deps_file = 'DEPS'

# This is to support backward compatibility on android-chrome because

« no previous file with comments | « no previous file | no next file » | no next file with comments »