ABANDONED. Remove obsolete git cl dcommit.

ABANDONED. Remove obsolete git cl dcommit.

R=mmoss@chromium.org,benhenry@chromium.org
BUG=600451

[tandrii(chromium), 2016-06-16 20:20:37]

The CQ bit was checked by tandrii@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-06-16 20:20:43]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2076683002/20001

[commit-bot: I haz the power, 2016-06-16 20:27:15]

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

[commit-bot: I haz the power, 2016-06-16 20:27:16]

Dry run: Try jobs failed on following builders:
  Depot Tools Presubmit on tryserver.infra (JOB_FAILED,
https://build.chromium.org/p/tryserver.infra/builders/Depot%20Tools%20Presubm...)

[tandrii(chromium), 2016-08-15 12:58:17]

tandrii@chromium.org changed reviewers:
+ agable@chromium.org

[tandrii(chromium), 2016-08-15 12:58:18]

PTAL && WDYT?

[Michael Moss, 2016-08-15 15:26:32]

https://codereview.chromium.org/2076683002/diff/20001/git_cl.py
File git_cl.py (right):

https://codereview.chromium.org/2076683002/diff/20001/git_cl.py#newcode4134
git_cl.py:4134: print('Are you sure you didn\'t mean \'git cl dcommit\'?')
Change this too, maybe remove the get_footer_svn_id() check, and just check if
they have git-svn configured, and if so, suggest they undo it?

[agable, 2016-08-15 16:26:05]

I was planning to remove all SVN functionality from depot_tools in one go, and
to send a PSA at the same time. But if you want to do this bit yourself, go for
it. I defer to mmoss for the review itself since he's already started.

[tandrii(chromium), 2016-08-15 17:44:55]

On 2016/08/15 16:26:05, agable wrote:
> I was planning to remove all SVN functionality from depot_tools in one go, and
> to send a PSA at the same time. But if you want to do this bit yourself, go
for
> it. I defer to mmoss for the review itself since he's already started.

oh, if so, i'll let you, because git cl dcommit is tested elsewhere in
depot_tools, i believe some kind of smoke tests that actually run it end to end.
SO, this CL is ABANDONED.

[tandrii(chromium), 2016-08-15 17:45:12]

Description was changed from

==========
Remove obsolete git cl dcommit.

R=mmoss@chromium.org,benhenry@chromium.org
BUG=600451
==========

to

==========
ABANDONED. Remove obsolete git cl dcommit.

R=mmoss@chromium.org,benhenry@chromium.org
BUG=600451
==========