Switch MB to write the GN args to args.gn.

build/gn_helpers.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.
 
 """Helper functions useful when writing scripts that integrate with GN.
 
 The main functions are ToGNString and FromGNString which convert between
 serialized GN veriables and Python variables.
 
 To use in a random python file in the build:
 
   import os
   import sys
 
   sys.path.append(os.path.join(os.path.dirname(__file__),
                                os.pardir, os.pardir, "build"))
   import gn_helpers
 
 Where the sequence of parameters to join is the relative path from your source
 file to the build directory."""
 
 class GNException(Exception):
   pass
 
 
 def ToGNString(value, allow_dicts = True):
   """Prints the given value to stdout.

[brettw, 2016/04/12 21:46:03]

While you're here can you fix this line? It returns a string that GN can read
instead of printing it.

[Dirk Pranke, 2016/04/12 21:47:52]

Sure.

 
   allow_dicts indicates if this function will allow converting dictionaries
   to GN scopes. This is only possible at the top level, you can't nest a
   GN scope in a list, so this should be set to False for recursive calls."""
   if isinstance(value, str):
     if value.find('\n') >= 0:
       raise GNException("Trying to print a string with a newline in it.")
     return '"' + \
         value.replace('\\', '\\\\').replace('"', '\\"').replace('$', '\\$') + \
         '"'
 
   if isinstance(value, unicode):
     return ToGNString(value.encode('utf-8'))
 
   if isinstance(value, bool):
     if value:
       return "true"
     return "false"
 
   if isinstance(value, list):
     return '[ %s ]' % ', '.join(ToGNString(v) for v in value)
 
   if isinstance(value, dict):
     if not allow_dicts:
       raise GNException("Attempting to recursively print a dictionary.")
     result = ""
-    for key in value:
+    for key in sorted(value):

[Dirk Pranke, 2016/04/12 18:06:49]

this makes things a little more deterministic and sensible.

       if not isinstance(key, str):
         raise GNException("Dictionary key is not a string.")
       result += "%s = %s\n" % (key, ToGNString(value[key], False))
     return result
 
   if isinstance(value, int):
     return str(value)
 
   raise GNException("Unsupported type when printing to GN.")
 
@@ skipping 277 matching lines @@
     location in the input. If it does, the text is consumed and the function
     returns true. Otherwise, returns false and the current position is
     unchanged."""
     end = self.cur + len(constant)
     if end > len(self.input):
       return False  # Not enough room.
     if self.input[self.cur:end] == constant:
       self.cur = end
       return True
     return False