tree 5c8d846a008a73f4d509da9cfea5548ffd34cfe2
parent fd2fd117ca109488aaff788b150b047c74da34c7
author Brian Norris <briannorris@chromium.org> 1645748203 -0800
committer Commit Bot <commit-bot@chromium.org> 1646093436 +0000

update_kernel: Fix lint, shellcheck issues

cros lint (part of PRESUBMIT) is now spitting out shellcheck errors.
Let's fix 'em.

Most are of the typical variety (lack of quoting; splitting 'local'
declarations from execution/assignment; ${variable} style). One odd one:

In update_kernel.sh line 80:
  ! remote_sh rootdev
  ^-- SC2251: This ! is not on a condition and skips errexit. Use `&& exit 1` instead, or make sure $? is checked.

Even if we were trying to handle errors, this is inverted; we expect
rootdev to succeed, not fail.

BUG=none
TEST=update_kernel.sh

Change-Id: I2e8217691325570197164863a04d1be8acf28f94
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosutils/+/3489915
Reviewed-by: Stephen Boyd <swboyd@chromium.org>
Tested-by: Brian Norris <briannorris@chromium.org>
Commit-Queue: Brian Norris <briannorris@chromium.org>