From bfb6124a73af8c9fb806d9798c21aa28d36b125f Mon Sep 17 00:00:00 2001 From: Mirko Bonadei Date: Wed, 8 Jul 2020 10:27:58 +0200 Subject: [PATCH] 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 Reviewed-by: Jeremy Leconte Cr-Commit-Position: refs/heads/master@{#31661} --- tools_webrtc/autoroller/roll_deps.py | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/tools_webrtc/autoroller/roll_deps.py b/tools_webrtc/autoroller/roll_deps.py index 77dd7d8d73..4a505b5b7b 100755 --- a/tools_webrtc/autoroller/roll_deps.py +++ b/tools_webrtc/autoroller/roll_deps.py @@ -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': {}, }