safely-roll-deps: print every command

safely-roll-deps: print every command

BUG=363147
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=268239

[eseidel, 2014-05-01 18:26:27]

If we're going to do this, timestamps/runtimes might be nice.

[eseidel, 2014-05-01 18:26:37]

lgtm, but I don't think I have power here.

[borenet, 2014-05-02 15:47:19]

On 2014/05/01 18:26:37, eseidel wrote:
> lgtm, but I don't think I have power here.

Ping @maruel

[M-A Ruel, 2014-05-02 16:34:41]

lgtm but this is quite drastic. In general a wrapper function is less
aggressive.

https://codereview.chromium.org/260903009/diff/1/tools/safely-roll-deps.py
File tools/safely-roll-deps.py (right):

https://codereview.chromium.org/260903009/diff/1/tools/safely-roll-deps.py#ne...
tools/safely-roll-deps.py:52: print cmd
you'll want to sys.stdout.flush()

[borenet, 2014-05-02 19:51:15]

Uploaded patch set 2.  I guess it's a little aggressive, but it seemed nicer
than writing a wrapper function for every subprocess2 function we call...

https://codereview.chromium.org/260903009/diff/1/tools/safely-roll-deps.py
File tools/safely-roll-deps.py (right):

https://codereview.chromium.org/260903009/diff/1/tools/safely-roll-deps.py#ne...
tools/safely-roll-deps.py:52: print cmd
On 2014/05/02 16:34:41, M-A Ruel wrote:
> you'll want to sys.stdout.flush()

Done.

[borenet, 2014-05-02 19:51:47]

The CQ bit was checked by borenet@google.com

[commit-bot: I haz the power, 2014-05-02 19:52:36]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/borenet@google.com/260903009/20001

[commit-bot: I haz the power, 2014-05-02 20:10:44]

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

[commit-bot: I haz the power, 2014-05-02 20:10:44]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel on tryserver.chromium

[borenet, 2014-05-02 20:11:29]

The CQ bit was checked by borenet@google.com

[commit-bot: I haz the power, 2014-05-02 20:11:43]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/borenet@google.com/260903009/20001

[commit-bot: I haz the power, 2014-05-02 20:26:29]

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

[commit-bot: I haz the power, 2014-05-02 20:26:30]

Try jobs failed on following builders:
  linux_chromium_rel on tryserver.chromium

[eseidel, 2014-05-05 15:22:01]

The CQ bit was checked by eseidel@chromium.org

[commit-bot: I haz the power, 2014-05-05 15:22:09]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/borenet@google.com/260903009/20001

[commit-bot: I haz the power, 2014-05-05 19:07:13]

Change committed as 268239