gclient: Prune dangling remote tracking refs when fetching with --force

gclient: Prune dangling remote tracking refs when fetching with --force

BUG=644722
Committed: https://chromium.googlesource.com/chromium/tools/depot_tools/+/2a3e9913bec46ddaf1c82f3328202f7a6092df2f

[bpastene, 2016-09-07 21:18:50]

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

[commit-bot: I haz the power, 2016-09-07 21:18:54]

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

[bpastene, 2016-09-07 21:40:05]

bpastene@chromium.org changed reviewers:
+ iannucci@chromium.org

[bpastene, 2016-09-07 21:40:05]

talp!

[iannucci, 2016-09-07 21:46:13]

lgtm

https://chromiumcodereview.appspot.com/2320573005/diff/20001/gclient_scm.py
File gclient_scm.py (right):

https://chromiumcodereview.appspot.com/2320573005/diff/20001/gclient_scm.py#n...
gclient_scm.py:1239: self._Fetch(options, prune=True)
self._Fetch(options, prune=options.force)

[bpastene, 2016-09-07 21:51:29]

https://codereview.chromium.org/2320573005/diff/20001/gclient_scm.py
File gclient_scm.py (right):

https://codereview.chromium.org/2320573005/diff/20001/gclient_scm.py#newcode1239
gclient_scm.py:1239: self._Fetch(options, prune=True)
On 2016/09/07 21:46:13, iannucci wrote:
> self._Fetch(options, prune=options.force)

Done.

[bpastene, 2016-09-07 21:51:35]

The CQ bit was checked by bpastene@chromium.org

[bpastene, 2016-09-07 21:51:36]

The patchset sent to the CQ was uploaded after l-g-t-m from
iannucci@chromium.org
Link to the patchset: https://codereview.chromium.org/2320573005/#ps40001
(title: "inline")

[commit-bot: I haz the power, 2016-09-07 21:51:44]

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-07 22:05:36]

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

[commit-bot: I haz the power, 2016-09-07 22:05:37]

Try jobs failed on following builders:
  Recipe Roll Downstream Tester on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/311f74533abc6b10)

[jbudorick, 2016-09-07 22:09:58]

The CQ bit was checked by jbudorick@chromium.org

[commit-bot: I haz the power, 2016-09-07 22:10:07]

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

[jbudorick, 2016-09-07 22:12:14]

On 2016/09/07 22:09:58, jbudorick wrote:
> The CQ bit was checked by mailto:jbudorick@chromium.org

I think this should work now that https://codereview.chromium.org/2322663002/
rolled into chromium, but this would appear to be a structural flaw: nothing can
land here between a build/ side recipe change that causes a nontrivial
expectations change in chromium and the resulting recipe roll in chromium.

[commit-bot: I haz the power, 2016-09-07 22:23:20]

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

[commit-bot: I haz the power, 2016-09-07 22:23:23]

Try jobs failed on following builders:
  Recipe Roll Downstream Tester on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/311f852406ce4410)

[jbudorick, 2016-09-07 22:27:16]

On 2016/09/07 22:23:23, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   Recipe Roll Downstream Tester on luci.infra.try (JOB_FAILED,
> https://luci-milo.appspot.com/swarming/task/311f852406ce4410)

Now borked on https://codereview.chromium.org/2318413002 rolling into chromium.

[bpastene, 2016-09-07 22:44:02]

The CQ bit was checked by bpastene@chromium.org

[commit-bot: I haz the power, 2016-09-07 22:44:05]

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-07 22:57:14]

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

[commit-bot: I haz the power, 2016-09-07 22:57:15]

Try jobs failed on following builders:
  Recipe Roll Downstream Tester on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/311faa0b84ffd110)

[bpastene, 2016-09-07 23:19:02]

The CQ bit was checked by bpastene@chromium.org

[commit-bot: I haz the power, 2016-09-07 23:19:13]

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-07 23:22:27]

Description was changed from

==========
gclient: Prune dangling remote tracking refs when fetching with --force

BUG=644722
==========

to

==========
gclient: Prune dangling remote tracking refs when fetching with --force

BUG=644722

Committed:
https://chromium.googlesource.com/chromium/tools/depot_tools/+/2a3e9913bec46d...
==========

[commit-bot: I haz the power, 2016-09-07 23:22:28]

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