Disable set -e for extract build step

Disable set -e for extract build step

This python script is returning warnings via error codes and
we don't want to halt our tests from these warnings.

BUG=132707
TEST=


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

[newt (away), 2012-06-22 00:40:19]

LGTM with one nit

https://chromiumcodereview.appspot.com/10629011/diff/1/build/android/buildbot...
File build/android/buildbot_functions.sh (right):

https://chromiumcodereview.appspot.com/10629011/diff/1/build/android/buildbot...
build/android/buildbot_functions.sh:279: if (( $extract_exitcode > 1 )); then
Are you intentionally ignoring an error code of 1?  If so, it's probably worth
mentioning why in the comment above.

[commit-bot: I haz the power, 2012-06-22 02:15:15]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a lowly provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[Yaron, 2012-06-22 02:31:50]

https://chromiumcodereview.appspot.com/10629011/diff/5001/build/android/build...
File build/android/buildbot_functions.sh (right):

https://chromiumcodereview.appspot.com/10629011/diff/5001/build/android/build...
build/android/buildbot_functions.sh:275: set +x
Do you mean "set +e" ? "x" refers to echo. You should also turn it back on
before returning to avoid you having side-effects

[Isaac (away), 2012-06-22 02:56:16]

yep, should have been set +e.  Don't need to reenable -- am using a subshell so
the setting doesn't affect the caller.

[Yaron, 2012-06-22 03:03:45]

lgtm

[commit-bot: I haz the power, 2012-06-22 03:03:55]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ilevy@chromium.org/10629011/6001

[commit-bot: I haz the power, 2012-06-22 03:56:36]

Try job failure for 10629011-6001 (retry) on mac_rel for step "browser_tests".
It's a second try, previously, step "browser_tests" failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=mac_rel&nu...

[commit-bot: I haz the power, 2012-06-22 05:09:32]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/ilevy@chromium.org/10629011/6001

[commit-bot: I haz the power, 2012-06-22 06:36:09]

Change committed as 143550