Add possibility to expand strings in the autoroller.
Starting from crrev.com/c/2265498, the Chromium DEPS file has a new function Str(), which seems to just expand to the string it wraps. This causes the following error: NameError: name 'Str' is not defined In //tools_webrtc/autoroller/roll_deps.py. This CL adds a way to expand the string. No-Try: True Bug: None Change-Id: I4cdb43410edeed72b393f200314c0ee7eea9cb2a Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/178865 Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org> Reviewed-by: Jeremy Leconte <jleconte@webrtc.org> Cr-Commit-Position: refs/heads/master@{#31661}
This commit is contained in:
committed by
Commit Bot
parent
52fd96fb73
commit
bfb6124a73
@ -97,6 +97,10 @@ class RollError(Exception):
|
||||
pass
|
||||
|
||||
|
||||
def StrExpansion():
|
||||
return lambda str_value: str_value
|
||||
|
||||
|
||||
def VarLookup(local_scope):
|
||||
return lambda var_name: local_scope['vars'][var_name]
|
||||
|
||||
@ -104,6 +108,7 @@ def VarLookup(local_scope):
|
||||
def ParseDepsDict(deps_content):
|
||||
local_scope = {}
|
||||
global_scope = {
|
||||
'Str': StrExpansion(),
|
||||
'Var': VarLookup(local_scope),
|
||||
'deps_os': {},
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user