Default gclient's GIT_MODE to True

recipe_modules/gclient/config.py

 # Copyright 2013 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.
 
 import types
 
 from recipe_engine.config import config_item_context, ConfigGroup, BadConf
 from recipe_engine.config import ConfigList, Dict, Single, Static, Set, List
 
 from . import api as gclient_api
 
 
-def BaseConfig(USE_MIRROR=True, GIT_MODE=False, CACHE_DIR=None,
+def BaseConfig(USE_MIRROR=True, GIT_MODE=True, CACHE_DIR=None,
                PATCH_PROJECT=None, BUILDSPEC_VERSION=None,
                **_kwargs):
   deps = '.DEPS.git' if GIT_MODE else 'DEPS'
   cache_dir = str(CACHE_DIR) if CACHE_DIR else None
   return ConfigGroup(
     solutions = ConfigList(
       lambda: ConfigGroup(
         name = Single(basestring),
         url = Single(basestring),
         deps_file = Single(basestring, empty_val=deps, required=False,
@@ skipping 65 matching lines @@
     GIT_MODE = Static(bool(GIT_MODE)),
     USE_MIRROR = Static(bool(USE_MIRROR)),
     # TODO(tandrii): remove PATCH_PROJECT field.
     # DON'T USE THIS. WILL BE REMOVED.
     PATCH_PROJECT = Static(str(PATCH_PROJECT), hidden=True),
     BUILDSPEC_VERSION= Static(BUILDSPEC_VERSION, hidden=True),
   )
 
 config_ctx = config_item_context(BaseConfig)
 
-def ChromiumSvnSubURL(c, *pieces):
+def ChromiumSvnSubURL(c, *pieces):  # pragma: no cover
   BASES = ('https://src.chromium.org',
            'svn://svn-mirror.golo.chromium.org')
   return '/'.join((BASES[c.USE_MIRROR],) + pieces)
 
 def ChromiumGitURL(_c, *pieces):
   return '/'.join(('https://chromium.googlesource.com',) + pieces)
 
 def ChromiumSrcURL(c):
   # TODO(phajdan.jr): Move to proper repo and add coverage.
-  if c.GIT_MODE:  # pragma: no cover
+  if c.GIT_MODE:
     return ChromiumGitURL(c, 'chromium', 'src.git')
-  else:
+  else:  # pragma: no cover
     return ChromiumSvnSubURL(c, 'chrome', 'trunk', 'src')
 
 def BlinkURL(c):
   # TODO(phajdan.jr): Move to proper repo and add coverage.
-  if c.GIT_MODE:  # pragma: no cover
+  if c.GIT_MODE:
     return ChromiumGitURL(c, 'chromium', 'blink.git')
-  else:
+  else:  # pragma: no cover
     return ChromiumSvnSubURL(c, 'blink', 'trunk')
 
-def ChromeSvnSubURL(c, *pieces):
+def ChromeSvnSubURL(c, *pieces):  # pragma: no cover
   BASES = ('svn://svn.chromium.org',
            'svn://svn-mirror.golo.chromium.org')
   return '/'.join((BASES[c.USE_MIRROR],) + pieces)
 
 # TODO(phajdan.jr): Move to proper repo and add coverage.
 def ChromeInternalGitURL(_c, *pieces):  # pragma: no cover
   return '/'.join(('https://chrome-internal.googlesource.com',) + pieces)
 
 def ChromeInternalSrcURL(c):
   # TODO(phajdan.jr): Move to proper repo and add coverage.
-  if c.GIT_MODE:  # pragma: no cover
+  if c.GIT_MODE:
     return ChromeInternalGitURL(c, 'chrome', 'src-internal.git')
-  else:
+  else:  # pragma: no cover
     return ChromeSvnSubURL(c, 'chrome-internal', 'trunk', 'src-internal')
 
 def mirror_only(c, obj, default=None):
   return obj if c.USE_MIRROR else (default or obj.__class__())
 
 @config_ctx()
 def chromium_bare(c):
   s = c.solutions.add()
   s.name = 'src'
   s.url = ChromiumSrcURL(c)
@@ skipping 198 matching lines @@
 
 @config_ctx()
 def gyp(c):
   s = c.solutions.add()
   s.name = 'gyp'
   s.url = ChromiumGitURL(c, 'external', 'gyp.git')
   m = c.got_revision_mapping
   m['gyp'] = 'got_revision'
 
 @config_ctx(config_vars={'GIT_MODE': True})
-def build(c):  # pragma: no cover
-  if not c.GIT_MODE:
+def build(c):
+  if not c.GIT_MODE:  # pragma: no cover
     raise BadConf('build only supports git')
   s = c.solutions.add()
   s.name = 'build'
   s.url = ChromiumGitURL(c, 'chromium', 'tools', 'build.git')
   m = c.got_revision_mapping
   m['build'] = 'got_revision'
 
 @config_ctx(config_vars={'GIT_MODE': True})
 def depot_tools(c):  # pragma: no cover
   if not c.GIT_MODE:
     raise BadConf('depot_tools only supports git')
   s = c.solutions.add()
   s.name = 'depot_tools'
   s.url = ChromiumGitURL(c, 'chromium', 'tools', 'depot_tools.git')
   m = c.got_revision_mapping
   m['depot_tools'] = 'got_revision'
 
 @config_ctx(config_vars={'GIT_MODE': True})
 def skia(c):  # pragma: no cover
   if not c.GIT_MODE:
     raise BadConf('skia only supports git')
   s = c.solutions.add()
   s.name = 'skia'
   s.url = 'https://skia.googlesource.com/skia.git'
   m = c.got_revision_mapping
   m['skia'] = 'got_revision'
 
 @config_ctx(config_vars={'GIT_MODE': True})
-def chrome_golo(c): # pragma: no cover
+def chrome_golo(c):  # pragma: no cover
   if not c.GIT_MODE:
     raise BadConf('chrome_golo only supports git')
   s = c.solutions.add()
   s.name = 'chrome_golo'
   s.url = 'https://chrome-internal.googlesource.com/chrome-golo/chrome-golo.git'
   c.got_revision_mapping['chrome_golo'] = 'got_revision'
 
 @config_ctx(config_vars={'GIT_MODE': True})
 def build_internal(c):
   if not c.GIT_MODE:  # pragma: no cover
@@ skipping 277 matching lines @@
   """Add Valgrind binaries to the gclient solution."""
   c.solutions[0].custom_deps['src/third_party/valgrind'] = \
     ChromiumGitURL(c, 'chromium', 'deps', 'valgrind', 'binaries')
 
 @config_ctx(includes=['chromium'])
 def chromedriver(c):
   """Add Selenium Java tests to the gclient solution."""
   c.solutions[0].custom_deps[
       'src/chrome/test/chromedriver/third_party/java_tests'] = (
           ChromiumGitURL(c, 'chromium', 'deps', 'webdriver'))