docs: Removing references to SVN

docs: Removing references to SVN

BUG=643697
TEST=None

Committed: https://chromium.googlesource.com/chromium/tools/depot_tools/+/186449d3e74a84a1fbd5394fab24da997629c2f4

[benry, 2016-09-19 20:34:33]

benhenry@chromium.org changed reviewers:
+ agable@chromium.org, benhenry@chromium.org, sergiyb@chromium.org

[benry, 2016-09-19 20:34:33]

PTAL

[Sergiy Byelozyorov, 2016-09-19 21:55:51]

lgtm

https://codereview.chromium.org/2350293002/diff/1/README.gclient.md
File README.gclient.md (right):

https://codereview.chromium.org/2350293002/diff/1/README.gclient.md#newcode14
README.gclient.md:14: "url"         :
"https://scmserver/component/trunk/src.git",
Why not "gitserver"?

https://codereview.chromium.org/2350293002/diff/1/README.gclient.md#newcode17
README.gclient.md:17: #"component": "https://scmserver/component/trunk/",
ditto

[benry, 2016-09-19 22:20:12]

The CQ bit was checked by benhenry@chromium.org

[commit-bot: I haz the power, 2016-09-19 22:20:15]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-19 22:26:25]

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

[commit-bot: I haz the power, 2016-09-19 22:26:26]

Try jobs failed on following builders:
  Depot Tools Presubmit on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/315d5d7a340c5f10)

[benry, 2016-09-19 22:39:34]

benhenry@chromium.org changed reviewers:
+ hinoka@chromium.org, iannucci@chromium.org, stip@chromium.org,
tandrii@chromium.org

[benry, 2016-09-19 22:39:35]

Adding more folks from the OWNERS file.

[iannucci, 2016-09-19 22:46:36]

lgtm

[tandrii(chromium), 2016-09-19 22:46:38]

LGTM % strong nits

https://codereview.chromium.org/2350293002/diff/1/README.gclient.md
File README.gclient.md (right):

https://codereview.chromium.org/2350293002/diff/1/README.gclient.md#newcode14
README.gclient.md:14: "url"         :
"https://scmserver/component/trunk/src.git",
On 2016/09/19 21:55:50, Sergiy Byelozyorov wrote:
> Why not "gitserver"?

Don't use trunk and scmserver. Use
"https://chromium.googlesource.com/chromium/src.git"

https://codereview.chromium.org/2350293002/diff/1/README.gclient.md#newcode17
README.gclient.md:17: #"component": "https://scmserver/component/trunk/",
On 2016/09/19 21:55:50, Sergiy Byelozyorov wrote:
> ditto

"https://github.com/luci/luci-go" :)

[iannucci, 2016-09-19 22:47:12]

On 2016/09/19 22:46:36, iannucci wrote:
> lgtm

Though there's still a bunch of svn tools and crap in depot_tools.

[Sergiy Byelozyorov, 2016-09-19 23:01:16]

On 2016/09/19 22:47:12, iannucci wrote:
> On 2016/09/19 22:46:36, iannucci wrote:
> > lgtm
> 
> Though there's still a bunch of svn tools and crap in depot_tools.

They must die!

[benry, 2016-09-19 23:03:26]

The CQ bit was checked by benhenry@chromium.org

[benry, 2016-09-19 23:03:26]

The patchset sent to the CQ was uploaded after l-g-t-m from
iannucci@chromium.org, tandrii@chromium.org, sergiyb@chromium.org
Link to the patchset: https://codereview.chromium.org/2350293002/#ps20001
(title: "addressing nits")

[commit-bot: I haz the power, 2016-09-19 23:03:35]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[benry, 2016-09-19 23:03:42]

On 2016/09/19 23:01:16, Sergiy Byelozyorov wrote:
> On 2016/09/19 22:47:12, iannucci wrote:
> > On 2016/09/19 22:46:36, iannucci wrote:
> > > lgtm
> > 
> > Though there's still a bunch of svn tools and crap in depot_tools.
> 
> They must die!

Carrying LGTM from patch1 as I've fixed nits.

[commit-bot: I haz the power, 2016-09-19 23:06:50]

Description was changed from

==========
docs: Removing references to SVN

BUG=643697
TEST=None
==========

to

==========
docs: Removing references to SVN

BUG=643697
TEST=None

Committed:
https://chromium.googlesource.com/chromium/tools/depot_tools/+/186449d3e74a84...
==========

[commit-bot: I haz the power, 2016-09-19 23:06:51]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/tools/depot_tools/+/186449d3e74a84...