Fix args to RunCommand

Fix equals in RunCommand's in cbuildbot.

Change-Id: If8cdde42a470c3d07cfd41552cf95dea7567688b

BUG=
TEST=Ran cbuildbot.

Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=aa46703

[davidjames, 2010-10-20 23:34:38]

Some nits.

http://codereview.chromium.org/3967002/diff/2001/3001
File bin/cbuildbot.py (right):

http://codereview.chromium.org/3967002/diff/2001/3001#newcode358
bin/cbuildbot.py:358: '--push_options', '--bypass-hooks','-f', 'push'],
Is this change right? I thought that -f was a flag to push_options here. Looking
at cros_mark_as_stable, it doesn't accept the -f flag.

http://codereview.chromium.org/3967002/diff/2001/3003
File lib/cros_build_lib.py (right):

http://codereview.chromium.org/3967002/diff/2001/3003#newcode63
lib/cros_build_lib.py:63: Warning('You are using spaces in arg %s' % arg)
Is there any way to suppress this warning if you really want to have spaces in
the command?

[sosa, 2010-10-20 23:37:18]

http://codereview.chromium.org/3967002/diff/2001/3001
File bin/cbuildbot.py (right):

http://codereview.chromium.org/3967002/diff/2001/3001#newcode358
bin/cbuildbot.py:358: '--push_options', '--bypass-hooks','-f', 'push'],
You're right ... that was correct as it was before (one arg to push_options

On 2010/10/20 23:34:38, davidjames wrote:
> Is this change right? I thought that -f was a flag to push_options here.
Looking
> at cros_mark_as_stable, it doesn't accept the -f flag.

http://codereview.chromium.org/3967002/diff/2001/3003
File lib/cros_build_lib.py (right):

http://codereview.chromium.org/3967002/diff/2001/3003#newcode63
lib/cros_build_lib.py:63: Warning('You are using spaces in arg %s' % arg)
Do you want another RunCommand option or just to take the check out in general?

On 2010/10/20 23:34:38, davidjames wrote:
> Is there any way to suppress this warning if you really want to have spaces in
> the command?

[davidjames, 2010-10-20 23:43:13]

http://codereview.chromium.org/3967002/diff/2001/3003
File lib/cros_build_lib.py (right):

http://codereview.chromium.org/3967002/diff/2001/3003#newcode63
lib/cros_build_lib.py:63: Warning('You are using spaces in arg %s' % arg)
On 2010/10/20 23:37:18, sosa wrote:
> Do you want another RunCommand option or just to take the check out in
general?

I'd suggest to take out this warning, and instead just improve the printing so
that any arguments are quoted when we print out the command. That way, in the
logs we can easily see that the command is incorrect.

Cheers,

David

[sosa, 2010-10-21 00:33:19]

Cleaned up PTAL

[davidjames, 2010-10-21 01:20:19]

LGTM