Correct the order in which OutputDirectory and IntermediateDirectory are defined.

test/msvs/shared_output/gyptest-shared_output.py

+#!/usr/bin/env python
+
+# 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.
+
+"""
+Test checking that IntermediateDirectory can be defined in terms of
+OutputDirectory. We previously had emitted the definition of
+IntermediateDirectory before the definition of OutputDirectory.
+This is required so that $(IntDir) can be based on $(OutDir).
+"""
+
+import TestGyp
+import os
+
+test = TestGyp.TestGyp(workdir='workarea_shared_output', formats=['msvs'])

[scottmg, 2012/05/09 02:45:41]

if there's things actually relying on this style of expansion, it'll have to
work on ninja too. generally the windows tests look like this now:

if sys.platform == 'win32':
  ... TestGyp(..., formats['msvs', 'ninja'])

[bradn, 2012/05/10 01:48:52]

So this test is assuming that output goes in msvs style locations. My
understanding from you is that ninja on windows uses a ninja style output layout
instead of reproducing msvs's layout. So I believe we don't want to run this
test under ninja on windows. I've updated the comments.

+
+test.run_gyp('hello.gyp')
+test.set_configuration('Baz')
+
+test.build('there/there.gyp', test.ALL)
+test.must_exist(os.path.join(test.workdir, 'foo', 'there.exe'))
+test.must_exist(os.path.join(test.workdir, 'foo', 'bar', 'there.obj'))
+
+test.build('hello.gyp', test.ALL)
+test.must_exist(os.path.join(test.workdir, 'foo', 'hello.exe'))
+test.must_exist(os.path.join(test.workdir, 'foo', 'bar', 'hello.obj'))
+
+test.pass_test()