Fix usage of bash builtin "local" in update_depot_tools.

Fix usage of bash builtin "local" in update_depot_tools.

The bash command "local myvar=$(mycommand)" has the exit status of the
bash builtin "local" rather than the exit status of mycommand.  To get
the exit status of mycommand, "local myvar" should be run separately
on a previous line.


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

[newt (away), 2012-11-20 20:33:57]

PTAL

[nsylvain, 2012-11-20 20:35:38]

redirecting review to szager or cmp

[szager1, 2012-11-20 21:40:19]

https://codereview.chromium.org/11308129/diff/1/update_depot_tools
File update_depot_tools (right):

https://codereview.chromium.org/11308129/diff/1/update_depot_tools#newcode75
update_depot_tools:75: local REBASE_TXT, STATUS
No comma

[newt (away), 2012-11-20 21:56:44]

https://codereview.chromium.org/11308129/diff/1/update_depot_tools
File update_depot_tools (right):

https://codereview.chromium.org/11308129/diff/1/update_depot_tools#newcode75
update_depot_tools:75: local REBASE_TXT, STATUS
On 2012/11/20 21:40:19, szager1 wrote:
> No comma

Done.

[Isaac (away), 2012-11-25 04:51:54]

lgtm but needs owner approval

[M-A Ruel, 2012-11-26 16:03:41]

rubberstamp lgtm

[commit-bot: I haz the power, 2012-11-26 17:49:02]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/newt@chromium.org/11308129/2002

[commit-bot: I haz the power, 2012-11-26 17:51:16]

Change committed as 169446