Switch FFmpeg from svn/git-svn translation to using git directly.

Switch FFmpeg from svn/git-svn translation to using git directly.

Includes the deps2git roll to remove special cased ffmpeg setup.

BUG=329002
TEST=gclient sync works
R=mmoss@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=271050

[szager1, 2014-03-07 22:47:03]

https://codereview.chromium.org/191553002/diff/1/DEPS
File DEPS (right):

https://codereview.chromium.org/191553002/diff/1/DEPS#newcode271
DEPS:271: "/trunk/tools/deps2git@XXXXXX-CHANGE-THIS",
Strictly speaking, this doesn't have to happen atomically.  deps2git is only run
by an infrastructure cron job, which always floats deps2git to tip-of-tree.

This change is safe to land before the deps2git change.

[DaleCurtis, 2014-05-13 23:52:30]

PTAL.  borenet's gclient changes have landed, so once skia has switched over
without issue I'll land this.

https://codereview.chromium.org/191553002/diff/1/DEPS
File DEPS (right):

https://codereview.chromium.org/191553002/diff/1/DEPS#newcode271
DEPS:271: "/trunk/tools/deps2git@XXXXXX-CHANGE-THIS",
On 2014/03/07 22:47:03, szager1 wrote:
> Strictly speaking, this doesn't have to happen atomically.  deps2git is only
run
> by an infrastructure cron job, which always floats deps2git to tip-of-tree.
> 
> This change is safe to land before the deps2git change.

Done.

[DaleCurtis, 2014-05-15 19:18:32]

+mmoss since it looks like he may be rolling deps2git soon.

[Michael Moss, 2014-05-15 19:41:54]

On 2014/05/15 19:18:32, DaleCurtis wrote:
> +mmoss since it looks like he may be rolling deps2git soon.

Should that roll be incorporated into this? If so, you can set it to r270777.

[DaleCurtis, 2014-05-15 19:51:41]

Done.

[Michael Moss, 2014-05-15 20:07:32]

lgtm

[DaleCurtis, 2014-05-15 20:11:56]

Hmm, looks like borenet's change got reverted since svn left a skia/ dir around.
 FFmpeg shouldn't have the same issues since there's no ffmpeg dir in svn repo,
it's pulled from deps.

I'll go ahead and CQ this and see what happens on the bots.

+borenet fyi

[DaleCurtis, 2014-05-15 20:12:13]

The CQ bit was checked by dalecurtis@chromium.org

[commit-bot: I haz the power, 2014-05-15 20:13:40]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/dalecurtis@chromium.org/191553002/40001

[borenet, 2014-05-15 20:14:57]

Yeah, this shouldn't have all of the problems of the Skia change...

https://codereview.chromium.org/191553002/diff/40001/DEPS
File DEPS (right):

https://codereview.chromium.org/191553002/diff/40001/DEPS#newcode216
DEPS:216:
"/chromium/third_party/ffmpeg.git@d3a060bd1d6a4f43e9bc7d07f193b0513df4f244",
You might consider leaving the "ffmpeg_revision" variable around in case you
want to use the AutoRoll bot for DEPS rolls in the future
(http://www.chromium.org/blink/blinkrollbot).

[DaleCurtis, 2014-05-15 20:27:33]

FFmpeg isn't something we want auto-rolled at this time, so I'm okay with
removing it.

Looks like presubmit is failing because it thinks I've checked in a change to
.DEPS.git

[DaleCurtis, 2014-05-15 20:46:47]

Hmm, CQ bots don't seem to be using the new DEPS file, they're instead using the
SVN checkout process; presumably from some cached DEPS file or something??

[commit-bot: I haz the power, 2014-05-16 00:19:37]

FYI, CQ is re-trying this CL (attempt #1).
Please consider checking whether the failures are real,
and report flakes to chrome-troopers@google.com.
The failing builders are:
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)

[commit-bot: I haz the power, 2014-05-16 00:22:17]

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

[commit-bot: I haz the power, 2014-05-16 00:22:17]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/chromium_presubmit/bu...)

[DaleCurtis, 2014-05-16 17:20:06]

Going to land this manually since presubmit doesn't like the .DEPS.git changes
that occur.

[DaleCurtis, 2014-05-16 17:51:04]

Committed patchset #3 manually as r271050 (presubmit successful).

[DaleCurtis, 2014-05-16 18:02:46]

Looking good so far,  "Conflicting directory found in ...\third_party\ffmpeg.
Removing."