Fix make and ninja backends to sensibly handle duplicate target names in different directories

test/same-target-name-different-directory/gyptest-all.gyp

Index: test/same-target-name-different-directory/gyptest-all.gyp
===================================================================
--- test/same-target-name-different-directory/gyptest-all.gyp	(revision 0)
+++ test/same-target-name-different-directory/gyptest-all.gyp	(revision 0)
@@ -0,0 +1,30 @@
+#!/usr/bin/env python

[Nico, 2012/06/07 19:19:29]

This needs to be called 'gyptest-all.py', else `python gyptest.py -a` won't run
it. That's why you have green trybots; if I run this test locally I get:


hummer:gyp thakis$ python gyptest.py
test/same-target-name-different-directory/gyptest-all.gyp 
PYTHONPATH=/Volumes/MacintoshHD2/src/chrome-git/src/tools/gyp/test/lib
TESTGYP_FORMAT=make
/usr/bin/python test/same-target-name-different-directory/gyptest-all.gyp
PASSED
TESTGYP_FORMAT=ninja
/usr/bin/python test/same-target-name-different-directory/gyptest-all.gyp
PASSED
TESTGYP_FORMAT=xcode
/usr/bin/python test/same-target-name-different-directory/gyptest-all.gyp
None returned 1
STDOUT =========================================================================
Build settings from command line:
    SYMROOT = $SRCROOT/build


STDERR =========================================================================
Unknown target: target
Usage: xcodebuild [-project <projectname>] [-activetarget] [-alltargets]
[-target <targetname>]... [-parallelizeTargets] [-activeconfiguration]
[-configuration <configurationname>] [-sdk <sdkfullpath>|<sdkname>] [-xcconfig
<filepath>] [<buildsetting>=<value>]... [<buildaction>]...
       xcodebuild [-version [-sdk [<sdkfullpath>|<sdkname>] [<item>] ]
       xcodebuild [-showsdks]
       xcodebuild [-find <binary> -sdk <sdkfullpath>|<sdkname>]
       xcodebuild [-list]

FAILED test at line 423 of
/Volumes/MacintoshHD2/src/chrome-git/src/Tools/gyp/test/lib/TestCommon.py
(_complete)
	from line 540 of
/Volumes/MacintoshHD2/src/chrome-git/src/Tools/gyp/test/lib/TestCommon.py (run)
	from line 244 of
/Volumes/MacintoshHD2/src/chrome-git/src/Tools/gyp/test/lib/TestGyp.py (run)
	from line 792 of
/Volumes/MacintoshHD2/src/chrome-git/src/Tools/gyp/test/lib/TestGyp.py (build)
	from line 20 of test/same-target-name-different-directory/gyptest-all.gyp

Failed the following test:
	test/same-target-name-different-directory/gyptest-all.gyp

[Steve Block, 2012/06/08 12:20:18]

Argh, not sure how the name got mangled. I don't have a mac machine here, so
didn't see that failure locally. Will fix.

+
+# 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 cases when multiple targets in different directories have the same name.
+"""
+
+import TestGyp
+
+test = TestGyp.TestGyp()
+
+test.run_gyp('subdirs.gyp', chdir='src')
+
+test.relocate('src', 'relocate/src')
+
+# Test that we build all targets.
+test.build('subdirs.gyp', 'target', chdir='relocate/src')
+test.must_exist('relocate/src/subdir1/action1.txt')
+test.must_exist('relocate/src/subdir2/action2.txt')
+
+# Test that we build all targets using the correct actions, even if they have
+# the same names.
+test.build('subdirs.gyp', 'target_same_action_name', chdir='relocate/src')
+test.must_exist('relocate/src/subdir1/action.txt')
+test.must_exist('relocate/src/subdir2/action.txt')
+
+test.pass_test()
Property changes on: test/same-target-name-different-directory/gyptest-all.gyp
___________________________________________________________________
Added: svn:eol-style
   + LF