cros_build_lib.run: clean up error_code_ok->check migration
BUG=chromium:1006587
TEST=`./run_tests` passes
Change-Id: Ie416c5f7b16314a34a3a53f503469b3cc93fb9e2
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/1965957
Commit-Queue: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>
Reviewed-by: Chris McDonald <cjmcdonald@chromium.org>
diff --git a/scripts/cros_sdk.py b/scripts/cros_sdk.py
index efcb457..6970dfb 100644
--- a/scripts/cros_sdk.py
+++ b/scripts/cros_sdk.py
@@ -247,7 +247,7 @@
# ThinLTO opens lots of files at the same time.
resource.setrlimit(resource.RLIMIT_NOFILE, (32768, 32768))
ret = cros_build_lib.run(
- cmd, print_cmd=False, error_code_ok=True, mute_output=False)
+ cmd, print_cmd=False, check=False, mute_output=False)
# If we were in interactive mode, ignore the exit code; it'll be whatever
# they last ran w/in the chroot and won't matter to us one way or another.
# Note this does allow chroot entrance to fail and be ignored during
@@ -399,7 +399,7 @@
chroot_lv_path = '%s/%s' % (chroot_vg, chroot_lv)
cmd = ['lvchange', '-kn', chroot_lv_path]
cros_build_lib.run(
- cmd, print_cmd=False, capture_output=True, error_code_ok=True)
+ cmd, print_cmd=False, capture_output=True, check=False)
# Activate the LV in case the lvchange above was needed. Activating an LV
# that is already active shouldn't do anything, so this is safe to run even if