Add the ability to specify a target_os for gclient solutions

tests/gclient_test.py

Index: tests/gclient_test.py
===================================================================
--- tests/gclient_test.py	(revision 132861)
+++ tests/gclient_test.py	(working copy)
@@ -210,7 +210,7 @@
     # auto-fixed.
     d = gclient.Dependency(
         None, 'name', 'proto://host/path/@revision', None, None, None,
-        None, '', True)
+        None, '', None, True)
     self.assertEquals('proto://host/path@revision', d.url)
 
   def testStr(self):
@@ -220,24 +220,24 @@
     obj.add_dependencies_and_close(
         [
           gclient.Dependency(
-            obj, 'foo', 'url', None, None, None, None, 'DEPS', True),
+            obj, 'foo', 'url', None, None, None, None, 'DEPS', None, True),
           gclient.Dependency(
-            obj, 'bar', 'url', None, None, None, None, 'DEPS', True),
+            obj, 'bar', 'url', None, None, None, None, 'DEPS', None, True),
         ],
         [])
     obj.dependencies[0].add_dependencies_and_close(
         [
           gclient.Dependency(
             obj.dependencies[0], 'foo/dir1', 'url', None, None, None, None,
-            'DEPS', True),
+            'DEPS', None, True),
           gclient.Dependency(
             obj.dependencies[0], 'foo/dir2',
             gclient.GClientKeywords.FromImpl('bar'), None, None, None, None,
-            'DEPS', True),
+            'DEPS', None, True),
           gclient.Dependency(
             obj.dependencies[0], 'foo/dir3',
             gclient.GClientKeywords.FileImpl('url'), None, None, None, None,
-            'DEPS', True),
+            'DEPS', None, True),
         ],
         [])
     # Make sure __str__() works fine.
@@ -276,7 +276,44 @@
     work_queue.flush({}, None, [], options=options)
     self.assertEqual(client.GetHooks(options), [x['action'] for x in hooks])
 
+  def testTargetOS(self):
+    """Verifies that specifying a target_os pulls in all relevant dependencies.
 
+    The target_os variable allows specifying the name of an additional OS which
+    should be considered when selecting dependencies from a DEPS' deps_os. The
+    value will be appended to the enforced_os tuple.
+    """
+
+    write(
+        '.gclient',
+        """
+solutions = [
+  { "name": "foo",
+    "url": "svn://example.com/foo",
+    "target_os": "baz"
+  }
+]""")
+    write(
+        os.path.join('foo', 'DEPS'),
+        """
+deps = {
+  "foo/dir1": "/dir1"
+}
+deps_os = {
+  "unix": { "foo/dir2": "/dir2" },
+  "baz": { "foo/dir3": "/dir3" }
+}""")
+
+    parser = gclient.Parser()
+    options, args = parser.parse_args(['--jobs', '1'])
+    options.deps_os = "unix"
+
+    obj = gclient.GClient.LoadCurrentConfig(options)
+    obj.RunOnDeps('None', args)
+
+    self.assertEqual(4, len(self._get_processed()))
+    
+
 if __name__ == '__main__':
   sys.stdout = gclient_utils.MakeFileAutoFlush(sys.stdout)
   sys.stdout = gclient_utils.MakeFileAnnotated(sys.stdout, include_zero=True)