Add cipd binary support to cit.py

Add cipd binary support to cit.py

BUG=

Committed: https://chromium.googlesource.com/chromium/tools/depot_tools/+/9a0de0b9f1249b128ac0fd1d3800f5a8c9a921ad

[Ryan Tseng, 2016-07-01 21:59:55]

The CQ bit was checked by hinoka@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-07-01 21:59:59]

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

[Ryan Tseng, 2016-07-01 22:00:32]

hinoka@google.com changed reviewers:
+ dnj@chromium.org, tandrii@chromium.org

[Ryan Tseng, 2016-07-01 22:00:33]

Since you're making cipd tools now, you might find this handy

+tandrii since he's been poking at cit.py a bit too

[commit-bot: I haz the power, 2016-07-01 22:03:58]

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

[commit-bot: I haz the power, 2016-07-01 22:03:59]

Dry run: This issue passed the CQ dry run.

[dnj (Google), 2016-07-01 22:42:36]

dnj@google.com changed reviewers:
+ dnj@google.com

[dnj (Google), 2016-07-01 22:42:36]

lgtm

[Ryan Tseng, 2016-07-01 22:44:17]

The CQ bit was checked by hinoka@google.com

[commit-bot: I haz the power, 2016-07-01 22:44:31]

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

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

[commit-bot: I haz the power, 2016-07-01 22:50:58]

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...)

[tandrii(chromium), 2016-07-14 12:59:26]

LGTM for cipd changes, but please keep stderr writing.

https://codereview.chromium.org/2117943002/diff/1/cit.py
File cit.py (left):

https://codereview.chromium.org/2117943002/diff/1/cit.py#oldcode62
cit.py:62: sys.stderr.flush()
you really don't like my stderr writing :(
the purpose of it was to avoid polluting stdout so that "cit log" output can be
piped without hackery of skipping first line.

The real fix would be to use logging with proper levels of verbosity.

[Ryan Tseng, 2016-07-14 19:38:08]

Stderr

[Ryan Tseng, 2016-07-14 19:38:13]

The CQ bit was checked by hinoka@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-07-14 19:38:15]

Dry run: 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-07-14 19:41:05]

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

[commit-bot: I haz the power, 2016-07-14 19:41:06]

Dry run: This issue passed the CQ dry run.

[Ryan Tseng, 2016-07-14 19:43:57]

Actually do it

[Ryan Tseng, 2016-07-14 19:43:58]

The CQ bit was checked by hinoka@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-07-14 19:44:05]

Dry run: 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-07-14 19:47:56]

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

[commit-bot: I haz the power, 2016-07-14 19:47:57]

Dry run: This issue passed the CQ dry run.

[hinoka, 2016-07-14 19:56:37]

The CQ bit was checked by hinoka@chromium.org

[hinoka, 2016-07-14 19:56:38]

The patchset sent to the CQ was uploaded after l-g-t-m from dnj@google.com,
tandrii@chromium.org
Link to the patchset: https://codereview.chromium.org/2117943002/#ps40001
(title: "Actually do it")

[commit-bot: I haz the power, 2016-07-14 19:56:47]

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-07-14 20:00:03]

Description was changed from

==========
Add cipd binary support to cit.py

BUG=
==========

to

==========
Add cipd binary support to cit.py

BUG=

Committed:
https://chromium.googlesource.com/chromium/tools/depot_tools/+/9a0de0b9f1249b...
==========

[commit-bot: I haz the power, 2016-07-14 20:00:04]

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

[tandrii(chromium), 2016-07-14 20:02:18]

On 2016/07/14 19:43:57, Ryan Tseng wrote:
> Actually do it

Thanks!

[Ryan Tseng, 2016-07-14 20:27:11]

https://codereview.chromium.org/2117943002/diff/1/cit.py
File cit.py (left):

https://codereview.chromium.org/2117943002/diff/1/cit.py#oldcode62
cit.py:62: sys.stderr.flush()
On 2016/07/14 12:59:26, tandrii(chromium) wrote:
> you really don't like my stderr writing :(
> the purpose of it was to avoid polluting stdout so that "cit log" output can
be
> piped without hackery of skipping first line.
> 
> The real fix would be to use logging with proper levels of verbosity.

Oops I think this was a mistake, or I rebased badly.