Restore --head/-H to gclient

Restore --head/-H to gclient

R=dmazzoni@chromium.org, maruel@chromium.org
BUG=109184
Committed: https://chromium.googlesource.com/chromium/tools/depot_tools/+/2697cd19a7853609a4f06537306c9e5b8d34c65d

[agable, 2016-06-27 21:00:38]

I've uploaded this CL with two different ways of restoring the "sync the
top-level solutions too" functionality dmazzoni wants. PTAL and let me know
which you prefer.

https://codereview.chromium.org/2105493002/diff/1/gclient.py
File gclient.py (right):

https://codereview.chromium.org/2105493002/diff/1/gclient.py#newcode1478
gclient.py:1478: if not s.managed and not self._options.head:
This is way 1 of adding it back: check for --head, and if it was passed, leave
the "revision_overrides" for that solution blank (which will result in it being
`git fetched` and updated.)

https://codereview.chromium.org/2105493002/diff/1/gclient.py#newcode1489
gclient.py:1489: if rev != 'HEAD':
This is way 2 of adding it back: don't re-add the "-H" flag, but instead respect
"-r HEAD" (or verbosely, "--revision foo@HEAD") to mean "the HEAD of the remote,
not the HEAD of whatever I have locally".

[dmazzoni, 2016-06-27 21:08:15]

lgtm

Thanks. Either way is fine with me, I could use -r HEAD instead.

[agable, 2016-06-28 00:47:25]

Thanks. In chat with stip@, I've decided to bring back the --head/-H flag. It's
the less magical, more discoverable, more self-documenting choice.

[agable, 2016-06-28 00:47:45]

The CQ bit was checked by agable@chromium.org

[agable, 2016-06-28 00:47:45]

The patchset sent to the CQ was uploaded after l-g-t-m from
dmazzoni@chromium.org
Link to the patchset: https://codereview.chromium.org/2105493002/#ps20001
(title: "Choose the --head/-H choice")

[commit-bot: I haz the power, 2016-06-28 00:47:57]

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

[agable, 2016-06-28 00:48:01]

The CQ bit was unchecked by agable@chromium.org

[agable, 2016-06-28 00:48:17]

Description was changed from

==========
Try two different ways of restoring managed mode to gclient sync

R=dmazzoni@chromium.org, maruel@chromium.org
BUG=109184
==========

to

==========
Restore --head/-H to gclient

R=dmazzoni@chromium.org, maruel@chromium.org
BUG=109184
==========

[agable, 2016-06-28 00:48:19]

The CQ bit was checked by agable@chromium.org

[commit-bot: I haz the power, 2016-06-28 00:48:27]

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-06-28 00:54:47]

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

[commit-bot: I haz the power, 2016-06-28 00:54:48]

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

[agable, 2016-06-28 17:20:36]

The CQ bit was checked by agable@chromium.org

[agable, 2016-06-28 17:20:36]

The patchset sent to the CQ was uploaded after l-g-t-m from
dmazzoni@chromium.org
Link to the patchset: https://codereview.chromium.org/2105493002/#ps40001
(title: "Fix accidental deletion")

[commit-bot: I haz the power, 2016-06-28 17:20:48]

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-06-28 17:23:56]

Description was changed from

==========
Restore --head/-H to gclient

R=dmazzoni@chromium.org, maruel@chromium.org
BUG=109184
==========

to

==========
Restore --head/-H to gclient

R=dmazzoni@chromium.org, maruel@chromium.org
BUG=109184

Committed:
https://chromium.googlesource.com/chromium/tools/depot_tools/+/2697cd19a78536...
==========

[commit-bot: I haz the power, 2016-06-28 17:23:57]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/tools/depot_tools/+/2697cd19a78536...