git cl set-commit: actually apply passed --rietveld / --gerrit options

git cl set-commit: actually apply passed --rietveld / --gerrit options

BUG=600945
Recipe-Tryjob-Bypass-Reason: expectation changes expected and wanted
Committed: https://chromium.googlesource.com/chromium/tools/depot_tools/+/7c3841abdfddcb33f25be98d20c9e9db70657f9f

[tandrii(chromium), 2016-08-10 15:03:59]

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

[tandrii(chromium), 2016-08-10 15:04:00]

LGTM, this is indeedn an oversight.

*But* how did you find this? AFAIU, the only way to hit this problem is if the
same branch is uploaded to both Gerrit and Rietveld, which shouldn't really be
the case.

[Paweł Hajdan Jr., 2016-08-10 15:16:58]

The CQ bit was checked by phajdan.jr@chromium.org

[commit-bot: I haz the power, 2016-08-10 15:17:07]

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-08-10 15:28:36]

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

[commit-bot: I haz the power, 2016-08-10 15:28:37]

Try jobs failed on following builders:
  build-internal-recipe-downstream-tryjob on master.internal.infra.try
(JOB_FAILED,
https://uberchromegw.corp.google.com/i/internal.infra.try/builders/build-inte...)

[Paweł Hajdan Jr., 2016-08-10 16:30:29]

Description was changed from

==========
git cl set-commit: actually apply passed --rietveld / --gerrit options

BUG=600945
==========

to

==========
git cl set-commit: actually apply passed --rietveld / --gerrit options

BUG=600945

Recipe-Tryjob-Bypass-Reason: expectation changes expected and wanted
==========

[Paweł Hajdan Jr., 2016-08-10 16:30:44]

The CQ bit was checked by phajdan.jr@chromium.org

[commit-bot: I haz the power, 2016-08-10 16:30:52]

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-08-10 16:35:25]

Description was changed from

==========
git cl set-commit: actually apply passed --rietveld / --gerrit options

BUG=600945

Recipe-Tryjob-Bypass-Reason: expectation changes expected and wanted
==========

to

==========
git cl set-commit: actually apply passed --rietveld / --gerrit options

BUG=600945

Recipe-Tryjob-Bypass-Reason: expectation changes expected and wanted
Committed:
https://chromium.googlesource.com/chromium/tools/depot_tools/+/7c3841abdfddcb...
==========

[commit-bot: I haz the power, 2016-08-10 16:35:26]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/tools/depot_tools/+/7c3841abdfddcb...