'dep_os' paths override 'deps' paths.

tests/gclient_test.py

 #!/usr/bin/env python
 # Copyright (c) 2012 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.
 
 """Unit tests for gclient.py.
 
 See gclient_smoketest.py for integration tests.
 """
 
@@ skipping 407 matching lines @@
     self.assertEquals(
         [
           'svn://example.com/bar',
           'svn://example.com/bar/unix',
           'svn://example.com/foo',
           'svn://example.com/foo/baz',
           'svn://example.com/foo/unix',
           ],
         sorted(self._get_processed()))
 
+  def testDepsOsOverrideDepsInDepsFile(self):
+    """Verifies that a 'deps_os' path can override a 'deps' path.
+    """
+
+    write(
+        '.gclient',
+        'solutions = [\n'
+        '  { "name": "foo",\n'
+        '    "url": "svn://example.com/foo",\n'
+        '  },]\n')
+    write(
+        os.path.join('foo', 'DEPS'),
+        'target_os = ["baz"]\n'
+        'deps = {\n'
+        '  "foo/src": "/src",\n' # This path is to be overridded by similar path
+                                 # in deps_os['unix'].
+        '}\n'
+        'deps_os = {\n'
+        '  "unix": { "foo/unix": "/unix",'
+        '            "foo/src": "/src_unix"},\n'
+        '  "baz": { "foo/baz": "/baz", },\n'
+        '  "jaz": { "foo/jaz": "/jaz", },\n'
+        '}')
+
+    parser = gclient.Parser()
+    options, _ = parser.parse_args(['--jobs', '1'])

[M-A Ruel, 2012/11/05 19:48:05]

Why not use: '--deps', 'unix' ?

[Siva Chandra, 2012/11/05 22:13:13]

Can you clarify? Are you suggesting I use '--deps' option instead of the line
below? If yes, the reason I think is because we are not running any of those
commands which add '--deps' option. We are running with RunOnDeps with 'None' on
line 457 below.

[M-A Ruel, 2012/11/06 15:13:18]

You are right, I was confused.

+    options.deps_os = 'unix'
+
+    obj = gclient.GClient.LoadCurrentConfig(options)
+    obj.RunOnDeps('None', [])
+    self.assertEqual(['unix'], sorted(obj.enforced_os))
+    self.assertEquals(
+        [
+          'svn://example.com/foo',
+          'svn://example.com/foo/baz',
+          'svn://example.com/foo/src_unix',
+          'svn://example.com/foo/unix',
+          ],
+        sorted(self._get_processed()))
+
   def testRecursionOverride(self):
     """Verifies gclient respects the recursion var syntax.
 
     We check several things here:
     - recursion = 3 sets recursion on the foo dep to exactly 3
       (we pull /fizz, but not /fuzz)
     - pulling foo/bar at recursion level 1 (in .gclient) is overriden by
       a later pull of foo/bar at recursion level 2 (in the dep tree)
     """
     write(
@@ skipping 42 matching lines @@
   sys.stdout = gclient_utils.MakeFileAutoFlush(sys.stdout)
   sys.stdout = gclient_utils.MakeFileAnnotated(sys.stdout, include_zero=True)
   sys.stderr = gclient_utils.MakeFileAutoFlush(sys.stderr)
   sys.stderr = gclient_utils.MakeFileAnnotated(sys.stderr, include_zero=True)
   logging.basicConfig(
       level=[logging.ERROR, logging.WARNING, logging.INFO, logging.DEBUG][
         min(sys.argv.count('-v'), 3)],
       format='%(relativeCreated)4d %(levelname)5s %(module)13s('
               '%(lineno)d) %(message)s')
   unittest.main()