Make 'gclient update' and 'gclient status' quieter by default - they don't...

Make 'gclient update' and 'gclient status' quieter by default - they don't
print messages unless the underlying commands actually generate output.
This is a more conservative version of r28271.

  BUG=none
  TEST=none
  R=maruel


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=30296

[Dirk Pranke, 2009-10-27 22:25:11]

Hi Marc-Antoine,

This is a more conservative attempt to make things quieter by default. It should
be completely safe as it only changes log output levels. Let me know what you
think.

[M-A Ruel, 2009-10-27 23:31:13]

lgtm with one change.

http://codereview.chromium.org/344004/diff/1/3
File gclient_utils.py (right):

http://codereview.chromium.org/344004/diff/1/3#newcode217
Line 217: if not print_messages:
Move this inside the print_stdout. You don't want to print this line when
print_stdout is False.

[Dirk Pranke, 2009-10-27 23:44:30]

http://codereview.chromium.org/344004/diff/1/3
File gclient_utils.py (right):

http://codereview.chromium.org/344004/diff/1/3#newcode217
Line 217: if not print_messages:
On 2009/10/27 23:31:13, Marc-Antoine Ruel wrote:
> Move this inside the print_stdout. You don't want to print this line when
> print_stdout is False.

Are you sure? In the old version we would've printed the line regardless of the
value of print_stdout. What's the difference here?

[M-A Ruel, 2009-10-27 23:59:02]

On 2009/10/27 23:44:30, dpranke wrote:
> http://codereview.chromium.org/344004/diff/1/3
> File gclient_utils.py (right):
> 
> http://codereview.chromium.org/344004/diff/1/3#newcode217
> Line 217: if not print_messages:
> On 2009/10/27 23:31:13, Marc-Antoine Ruel wrote:
> > Move this inside the print_stdout. You don't want to print this line when
> > print_stdout is False.
> 
> Are you sure? In the old version we would've printed the line regardless of
the
> value of print_stdout. What's the difference here?

You are changing the behavior when print_message=False, print_stdout=False. This
is used to grep command outputs like svn status.