Fix cros_run_vm_update --proxy_port support

Fix cros_run_vm_update --proxy_port support

BUG=chromium-os:10318
TEST=Manual

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

[sosa, 2010-12-11 00:17:07]

http://codereview.chromium.org/5801001/diff/1/bin/cros_run_vm_update
File bin/cros_run_vm_update (right):

http://codereview.chromium.org/5801001/diff/1/bin/cros_run_vm_update#newcode44
bin/cros_run_vm_update:44: IMAGE_ARGS="--proxy_port="${FLAGS_proxy_port}""
Does the quotes in quotes work?  Don't need \" in there?

[dgarrett, 2010-12-11 00:25:38]

http://codereview.chromium.org/5801001/diff/1/bin/cros_run_vm_update
File bin/cros_run_vm_update (right):

http://codereview.chromium.org/5801001/diff/1/bin/cros_run_vm_update#newcode44
bin/cros_run_vm_update:44: IMAGE_ARGS="--proxy_port="${FLAGS_proxy_port}""
On 2010/12/11 00:17:07, sosa wrote:
> Does the quotes in quotes work?  Don't need \" in there?

To be honest, it's a copy mod of the option above. It does work, but is probably
suboptimal. I imagine it's just being passed along to a new shell that processes
it out.

[sosa, 2010-12-11 00:34:01]

Ah makes sense.  LGTM

On Fri, Dec 10, 2010 at 4:25 PM,  <dgarrett@chromium.org> wrote:
>
> http://codereview.chromium.org/5801001/diff/1/bin/cros_run_vm_update
> File bin/cros_run_vm_update (right):
>
> http://codereview.chromium.org/5801001/diff/1/bin/cros_run_vm_update#newcode44
> bin/cros_run_vm_update:44:
> IMAGE_ARGS="--proxy_port="${FLAGS_proxy_port}""
> On 2010/12/11 00:17:07, sosa wrote:
>>
>> Does the quotes in quotes work?  Don't need \" in there?
>
> To be honest, it's a copy mod of the option above. It does work, but is
> probably suboptimal. I imagine it's just being passed along to a new
> shell that processes it out.
>
> http://codereview.chromium.org/5801001/
>