Remove invocation of cros_workon from setup_board
This fixes a hole where upgrading from older chroots could:
- run chroot update scripts as part of the chroot update process
- chroot update scripts call setup_board
- setup_board invokes cros_workon
- cros_workon is a symlink in /usr/bin that exists because we haven't
finishd upgrading the chroot (a successful upgrade will re-emerge
chromeos-base/cros-devutils, removing the symlink)
- cros_workon the symlink points to src/scripts/cros_workon, which
got upgraded by merely syncing sources
- the upgraded src/scripts/cros_workon always exits with an error code
because its functionality has been moved
- we therefore cannot upgrade the chroot out of this mess.
Break this chain by just removing the callsite, which seems at first
glance to be mostly superfluous.
BUG=brillo:1021
TEST=None
Change-Id: I6836cde88cc08ed9862bacf97c355a3ca1b4d7cd
Reviewed-on: https://chromium-review.googlesource.com/270076
Tested-by: Christopher Wiley <wiley@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Commit-Queue: Christopher Wiley <wiley@chromium.org>
diff --git a/setup_board b/setup_board
index 294f806..66cc00d 100755
--- a/setup_board
+++ b/setup_board
@@ -318,13 +318,3 @@
# cros_workon do not support board_root.
exit 0
fi
-
-# NOTE: Printing the working-on ebuilds does not only serve the informative
-# purpose. It also causes the ${BOARD_ROOT}/etc/portage/package.* files to be
-# regenerated.
-WORKING_ON=$(cros_workon --board=${BOARD_VARIANT} list)
-if [ -n "${WORKING_ON}" ]; then
- echo
- echo "Currently working on the following ebuilds for this board:"
- echo "${WORKING_ON}"
-fi