commit 27a67cc595a9e81a75c83f02a927bd5e0b70951a
author Chris McDonald <cjmcdonald@chromium.org> Wed May 06 16:47:44 2020 -0600
committer Commit Bot <commit-bot@chromium.org> Fri May 08 00:17:31 2020 +0000
tree 139d88320790c2562b7b2678cd009b30d7867ec8
parent 4ef8d11f251e446b3cc57066fb3f2066c2cc6ab2

build_packages: Add missing array expansion

This command is supposed to run `grep` and the array of arguments but
was mistakenly expanding only to the first token.

PAIR=saklein@chromium.org
BUG=None
TEST=`build_packages` no longer shows a grep error

Change-Id: Iaaa1dea7f0a619c13ee9646f40b02112b166ced0
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosutils/+/2186290
Tested-by: Chris McDonald <cjmcdonald@chromium.org>
Reviewed-by: Alex Klein <saklein@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Commit-Queue: Chris McDonald <cjmcdonald@chromium.org>

build_packages

diff --git a/build_packages b/build_packages
index 388847b..0bfb6df 100755
--- a/build_packages
+++ b/build_packages
@@ -366,7 +366,7 @@
       for pkg in "${CHROME_PACKAGES[@]}"; do
         grep_cmd+=( -e "${pkg}" )
       done
-      RMOD_PKGS=( $(printf '%s\n' "${RMOD_PKGS[@]}" | "${grep_cmd}" || :) )
+      RMOD_PKGS=( $(printf '%s\n' "${RMOD_PKGS[@]}" | "${grep_cmd[@]}" || :) )
       info "final reverse dependencies that will be rebuilt: ${RMOD_PKGS[*]}"
       FORCE_LOCAL_BUILD_PKGS+=( "${RMOD_PKGS[@]}" )
     fi