infra/config/recipes.cfg - Issue 1345143004: CrOS: Use local Chromite checkout for configs.

Side by Side Diff

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

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

Side by Side Diff: infra/config/recipes.cfg

Issue 1345143004: CrOS: Use local Chromite checkout for configs. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build

Patch Set: Created 5 years, 3 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:

OLD	NEW
1 api_version: 1	1 api_version: 1

2 project_id: "build"	2 project_id: "build"

3 recipes_path: "scripts/slave"	3 recipes_path: "scripts/slave"

4 deps {	4 deps {

5 project_id: "recipe_engine"	5 project_id: "recipe_engine"

6 url: "https://chromium.googlesource.com/external/github.com/luci/recipes-py.gi t"	6 url: "https://chromium.googlesource.com/external/github.com/luci/recipes-py.gi t"

7 branch: "master"	7 branch: "master"

8 revision: "ffb4869a6eee2515ebdd50aee8b90b871b99a36c"	8 revision: "d4e143cac58ee880b72b788ba37f3e06cd4e1668"
	iannucci 2015/09/29 19:17:30 this should probably be done in a separate CL? Or this should probably be done in a separate CL? Or at least have a comment (in rietveld) saying why the roll is necessary. dnj 2015/09/29 19:28:40 I think this roll is actually obsolete, but if it Show quoted text On 2015/09/29 19:17:30, iannucci wrote: > this should probably be done in a separate CL? Or at least have a comment (in > rietveld) saying why the roll is necessary. I think this roll is actually obsolete, but if it is necessary will do.
9 }	9 }

OLD	NEW