recipes.py - Issue 2343733003: Prefer overridden dependencies. - Code Review

Chromium Code Reviews

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

(350)

My Issues | Starred Open | Closed | All

Unified Diff: recipes.py

Issue 2343733003: Prefer overridden dependencies. (Closed)

Patch Set: Created 4 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:

View side-by-side diff with in-line comments

« recipe_engine/package.py ('K') | « recipe_engine/package.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: recipes.py

diff --git a/recipes.py b/recipes.py

index e839b59f78cd7011e2bab640b09aa978d2ee863a..744aa6f5d6e4215f253ffbf954d25c2ead5a46df 100755

--- a/recipes.py

+++ b/recipes.py

@@ -10,6 +10,7 @@ infra/config/recipes.cfg.

"""

import argparse

+import collections

import json

import logging

import os

@@ -198,7 +199,7 @@ class ProjectOverrideAction(argparse.Action):

v = getattr(namespace, self.dest, None)

if v is None:

- v = {}

+ v = collections.OrderedDict()

setattr(namespace, self.dest, v)

if v.get(project_id):

« recipe_engine/package.py ('K') | « recipe_engine/package.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine

This is Rietveld 408576698