Correct the order in which OutputDirectory and IntermediateDirectory are defined.

pylib/gyp/generator/make.py

 # Copyright (c) 2012 Google Inc. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 # Notes:
 #
 # This is all roughly based on the Makefile system used by the Linux
 # kernel, but is a non-recursive make -- we put the entire dependency
 # graph in front of make and let it figure it out.
 #
@@ skipping 1778 matching lines @@
     # TODO(thakis): It would be nice to have some general mechanism instead.
     strip_save_file = self.xcode_settings.GetPerTargetSetting(
         'CHROMIUM_STRIP_SAVE_FILE', '')
     # Even if strip_save_file is empty, explicitly write it. Else a postbuild
     # might pick up an export from an earlier target.
     return self.GetXcodeEnv(
         additional_settings={'CHROMIUM_STRIP_SAVE_FILE': strip_save_file})
 
 
   def WriteXcodeEnv(self, target, env):
-    for k in gyp.xcode_emulation.TopologicallySortedEnvVarKeys(env):
+    for k in reversed(gyp.xcode_emulation.TopologicallySortedEnvVarKeys(env)):

[Nico, 2012/05/11 14:53:44]

When refactoring a function (gyp.xcode_emu.TopSortEnvVarKeys), try not to change
its interface. Then you don't need to change this file (and any of the other
calling sites you changed).

[bradn, 2012/05/11 18:57:23]

So I had been "correcting" what I had perceived as an inversion of "canonical"
definitions of the directionality of edges for variable dependencies.
Some informal polling of folks around me, that my sense of  is in fact
non-universal (its about a 50% 50% split :-).
Sorry for the confusion.
As such, keeping the current function signatures makes sense.
I've switched to that and added more clarifying comments.
I've ended up keeping the edge directions flipped from before (but hid
internally with a reverse) as its easier to find all variable references in a
one string.

       # For
       #  foo := a\ b
       # the escaped space does the right thing. For
       #  export foo := a\ b
       # it does not -- the backslash is written to the env as literal character.
       # So don't escape spaces in |env[k]|.
       self.WriteLn('%s: export %s := %s' % (QuoteSpaces(target), k, env[k]))
 
 
   def Objectify(self, path):
@@ skipping 318 matching lines @@
     root_makefile.write("  include " + include_file + "\n")
     root_makefile.write("endif\n")
   root_makefile.write('\n')
 
   if generator_flags.get('auto_regeneration', True):
     WriteAutoRegenerationRule(params, root_makefile, makefile_name, build_files)
 
   root_makefile.write(SHARED_FOOTER)
 
   root_makefile.close()