Eliminate all interactive terminal prompts from git.

Eliminate all interactive terminal prompts from git.

BUG=

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=265735

[szager1, 2014-04-22 14:33:31]

This will hopefully eliminate once and for all the 'gclient hangs in a
subprocess waiting for input' problem.

[ghost stip (do not use), 2014-04-22 18:12:14]

lgtm

[szager1, 2014-04-22 19:02:13]

The CQ bit was checked by szager@chromium.org

[commit-bot: I haz the power, 2014-04-22 19:02:26]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/szager@chromium.org/247493002/1

[commit-bot: I haz the power, 2014-04-22 19:04:06]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-04-22 19:04:07]

Presubmit check for 247493002-1 failed and returned exit status 1.

Running presubmit commit checks ...
Checking out rietveld...
Running save-description-on-failure.sh
Running push-basic.sh
Running upstream.sh
Running submit-from-new-dir.sh
Running abandon.sh
Running submodule-merge-test.sh
Running upload-local-tracking-branch.sh
Running hooks.sh
Running post-dcommit-hook-test.sh
Running upload-stale.sh
Running patch.sh
Running basic.sh

** Presubmit ERRORS **
tests/scm_unittest.py (23.10s) failed
..['ApplyEnvVars']
F...............
======================================================================
FAIL: testMembersChanged (__main__.GitWrapperTestCase)
----------------------------------------------------------------------
Traceback (most recent call last):
  File
"/b/infra_internal/commit_queue/workdir/tools/depot_tools/third_party/pymox/mox.py",
line 1618, in new_method
    func(self, *args, **kwargs)
  File "tests/scm_unittest.py", line 103, in testMembersChanged
    self.compareMembers(scm.GIT, members)
  File
"/b/infra_internal/commit_queue/workdir/tools/depot_tools/testing_support/super_mox.py",
line 78, in compareMembers
    self.assertEqual(actual_members, expected_members)
AssertionError: Lists differ: ['ApplyEnvVars', 'AssertVersio... !=
['AssertVersion', 'Capture', '...

First differing element 0:
ApplyEnvVars
AssertVersion

First list contains 1 additional elements.
First extra element 25:
current_version

- ['ApplyEnvVars',
-  'AssertVersion',
? ^

+ ['AssertVersion',
? ^

   'Capture',
   'CaptureStatus',
   'FetchUpstreamTuple',
   'GenerateDiff',
   'GetBlessedSha1ForSvnRev',
   'GetBranch',
   'GetBranchRef',
   'GetCheckoutRoot',
   'GetDifferentFiles',
   'GetEmail',
   'GetGitDir',
   'GetGitSvnHeadRev',
   'GetPatchName',
   'GetSVNBranch',
   'GetSha1ForSvnRev',
   'GetUpstreamBranch',
   'IsGitSvn',
   'IsInsideWorkTree',
   'IsValidRevision',
   'IsWorkTreeDirty',
   'MatchSvnGlob',
   'ParseGitSvnSha1',
   'ShortBranchName',
   'current_version']

----------------------------------------------------------------------
Ran 18 tests in 22.867s

FAILED (failures=1)


Presubmit checks took 97.2s to calculate.

[ghost stip (do not use), 2014-04-23 18:13:38]

you just have to add the export to that dang unittest

[szager1, 2014-04-23 20:56:45]

The CQ bit was checked by szager@chromium.org

[commit-bot: I haz the power, 2014-04-23 20:56:57]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/szager@chromium.org/247493002/20001

[commit-bot: I haz the power, 2014-04-23 20:59:24]

Change committed as 265735