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

Issue 4154004: Fix custom_deps, hooks and custom_vars to be evaluated even if no DEPS file. (Closed)

Created:
10 years, 1 month ago by M-A Ruel
Modified:
9 years, 7 months ago
CC:
chromium-reviews, M-A Ruel, Rico
Visibility:
Public.

Description

Fix custom_deps, hooks and custom_vars to be evaluated even if no DEPS file. Previously, solution's properties would be ignored if there was no DEPS file in source control. Change the behavior to evaluate the properties anyway. TEST=new smoke test BUG=none Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=64232

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+43 lines, -16 lines) Patch
M gclient.py View 4 chunks +15 lines, -16 lines 0 comments Download
M tests/gclient_smoketest.py View 1 chunk +28 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
M-A Ruel
Adding Jochen because he's in the right time zone. Adding Rico FYI. Dependent on http://codereview.chromium.org/4215001
10 years, 1 month ago (2010-10-28 11:16:41 UTC) #1
jochen (gone - plz use gerrit)
10 years, 1 month ago (2010-10-28 11:25:56 UTC) #2
lgtm

Powered by Google App Engine
This is Rietveld 408576698