Add support for substring replacement in gypi_to_gn

build/gypi_to_gn.py

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 """Converts a given gypi file to a python scope and writes the result to stdout.
 
 It is assumed that the file contains a toplevel dictionary, and this script
 will return that dictionary as a GN "scope" (see example below). This script
 does not know anything about GYP and it will not expand variables or execute
 conditions (it will check for the presence of a "conditions" value in the
@@ skipping 28 matching lines @@
   component("mycomponent") {
     sources = gypi_values.sources
     defines = gypi_values.defines
   }
 
 Sometimes your .gypi file will include paths relative to a different
 directory than the current .gn file. In this case, you can rebase them to
 be relative to the current directory.
   sources = rebase_path(gypi_values.sources, ".",
                         "//path/gypi/input/values/are/relative/to")
+
+This script also has the ability to replace certain substrings in the input.
+Generally this is used to emulate GYP variable expansion. If you passed the
+argument "--replace=<(foo)=bar" then all instances of "<(foo)" in strings in
+the input will be replaced with "bar":
+
+  gypi_values = exec_script("//build/gypi_to_gn.py",
+                            [ rebase_path("your_file.gypi"),
+                              "--replace=<(foo)=bar"],
+                            "scope",
+                            [ "your_file.gypi" ])
+
 """
 
 import gn_helpers
 from optparse import OptionParser
 import sys
 
 def LoadPythonDictionary(path):
   file_string = open(path).read()
   try:
     file_data = eval(file_string, {'__builtins__': None}, None)
   except SyntaxError, e:
     e.filename = path
     raise
   except Exception, e:
     raise Exception("Unexpected error while reading %s: %s" % (path, str(e)))
 
   assert isinstance(file_data, dict), "%s does not eval to a dictionary" % path
   assert 'conditions' not in file_data, \
       "The file %s has conditions in it, these aren't supported." % path
 
   return file_data
 
 
+def ReplaceSubstrings(values, search_for, replace_with):
+  """Recursively replaces substrings in a value.
+
+  Replaces all substrings of the "search_for" with "repace_with" for all
+  strings occurring in "values". This is done by recursively iterating into
+  lists as well as the keys and values of dictionaries."""
+  if isinstance(values, str):
+    return values.replace(search_for, replace_with)
+
+  if isinstance(values, list):
+    result = []
+    for v in values:
+      result.append(ReplaceSubstrings(v, search_for, replace_with))
+    return result

[Dirk Pranke, 2014/04/10 01:18:20]

Nit: one line: "return [ReplaceSubstrings(v, search_for, replace_with) for v in
values]"

+
+  if isinstance(values, dict):
+    # For dictionaries, do the search for both the key and values.
+    result = {}
+    for v in values:
+      result[ReplaceSubstrings(v, search_for, replace_with)] = \
+          ReplaceSubstrings(values[v], search_for, replace_with)

[Dirk Pranke, 2014/04/10 01:18:20]

Nit: I'd probably rewrite this to:

result = {}
for key, value in values.items():
  new_key = ReplaceSubstrings(key, search_for, replace_with)
  new_value = ReplaceSubstrings(value, search_for, replace_with)
  result[new_key] = new_value

Perhaps a little more verbose, but clearer and a bit more idiomatic, I think.

+    return result
+
+  # Assume everything else is unchanged.
+  return values
+
 def main():
   parser = OptionParser()
+  parser.add_option("-r", "--replace", action="append",
+    help="Replaces substrings. If passed a=b, replaces all substrs a with b.")
   (options, args) = parser.parse_args()
 
   if len(args) != 1:
     raise Exception("Need one argument which is the .gypi file to read.")
 
   data = LoadPythonDictionary(args[0])
+  if options.replace:
+    # Do replacements for all specified patterns.
+    for replace in options.replace:
+      split = replace.split('=')
+      # Allow "foo=" to replace with nothing.
+      if len(split) == 1:
+        split.append('')
+      assert len(split) == 2, "Replacement must be of the form 'key=value'."
+      data = ReplaceSubstrings(data, split[0], split[1])
+
   print gn_helpers.ToGNString(data)
 
 if __name__ == '__main__':
   try:
     main()
   except Exception, e:
     print str(e)
     sys.exit(1)