Add 'git clean -f -d' to 'gclient revert' when in a git repo.

Add 'git clean -f -d' to 'gclient revert' when in a git repo.

The try slaves run 'git revert' to clean up from the previous run. This only runs 'git reset --hard', which does not remove any untracked files, so if a CL adds a new file it will still be present after 'gclient revert'. Adding git clean -f -d removes untracked files (-f) and directories (-d).

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

[M-A Ruel, 2012-09-25 20:10:31]

lgtm

[commit-bot: I haz the power, 2012-09-25 20:10:40]

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

[commit-bot: I haz the power, 2012-09-25 20:13:14]

Presubmit check for 10986032-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/gclient_smoketest.py failed!
Command tests/gclient_smoketest.py returned non-zero exit status 1 in
/b/commit-queue/workdir/tools/depot_tools
.................F......................
======================================================================
FAIL: testRevertAndStatus (__main__.GClientSmokeGIT)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "tests/gclient_smoketest.py", line 947, in testRevertAndStatus
    self.assertEquals(13, len(out))
AssertionError: 13 != 24

----------------------------------------------------------------------
Ran 40 tests in 38.778s

FAILED (failures=1)


Presubmit checks took 151.8s to calculate.

[lliabraa, 2012-09-26 12:47:18]

I've updated the test to reflect the addition of git clean. PTAL.

[M-A Ruel, 2012-09-27 14:03:08]

lgtm, thanks

[commit-bot: I haz the power, 2012-09-27 14:03:23]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/lliabraa@chromium.org/10986032/6001

[commit-bot: I haz the power, 2012-09-27 14:06:50]

Change committed as 159020