From 0973a55f82f259e903a5e158a6cd2e31ba9b03db Mon Sep 17 00:00:00 2001 From: Mirko Bonadei Date: Wed, 8 Jul 2020 13:50:13 +0000 Subject: [PATCH] Revert "Add possibility to expand strings in the autoroller." This reverts commit bfb6124a73af8c9fb806d9798c21aa28d36b125f. Reason for revert: Testing Autoroller notifications. I will reland this later. Original change's description: > 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} TBR=mbonadei@webrtc.org,jleconte@webrtc.org Change-Id: I69f75da132d6b838b96a559712ad4c0da929021c No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: None Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/178877 Reviewed-by: Mirko Bonadei Commit-Queue: Mirko Bonadei Cr-Commit-Position: refs/heads/master@{#31673} --- tools_webrtc/autoroller/roll_deps.py | 5 ----- 1 file changed, 5 deletions(-) diff --git a/tools_webrtc/autoroller/roll_deps.py b/tools_webrtc/autoroller/roll_deps.py index 4a505b5b7b..77dd7d8d73 100755 --- a/tools_webrtc/autoroller/roll_deps.py +++ b/tools_webrtc/autoroller/roll_deps.py @@ -97,10 +97,6 @@ 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] @@ -108,7 +104,6 @@ def VarLookup(local_scope): def ParseDepsDict(deps_content): local_scope = {} global_scope = { - 'Str': StrExpansion(), 'Var': VarLookup(local_scope), 'deps_os': {}, }