Issue 679831 link

Starred by 1 user

Issue metadata

Status:	Archived
Owner:	norvez@chromium.org
Closed:	Jan 2017
Components:	Infra>SDK
EstimatedDays:	----
NextAction:	----
OS:	Chrome
Pri:	3
Type:	Bug
VerifyIn-58 VerifyIn-59 VerifyIn-60 VerifyIn-61

After the board sysroot has been recreated, cros_workon'd packages point to non-workon version

Project Member Reported by norvez@chromium.org, Jan 10 2017

Issue description


The list of packages that are being worked on persists across creation of board sysroot (including cros_sdk --replace). However the state of portage is out of sync after that, and unexpectedly modified by any call to cros_workon:

$ cros_workon-${BOARD} start arc-mesa
$ ./setup_board --board=${BOARD} --force
$ equery-${BOARD} w arc-mesa
/mnt/host/source/src/third_party/chromiumos-overlay/media-libs/arc-mesa/arc-mesa-12.1.0_pre-r33.ebuild
$ cros_workon --board=${BOARD} list
media-libs/arc-mesa
$ equery-${BOARD} w arc-mesa
/mnt/host/source/src/third_party/chromiumos-overlay/media-libs/arc-mesa/arc-mesa-9999.ebuild

We should make the state consistent when recreating the board sysroot.

Project Member

Comment 1 by bugdroid1@chromium.org, Jan 13 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform/crosutils/+/2e722a283cdc2dc8d0e212c7df2c2631ee335bfb

commit 2e722a283cdc2dc8d0e212c7df2c2631ee335bfb
Author: Nicolas Norvez <norvez@chromium.org>
Date: Tue Jan 10 19:51:24 2017

setup_board: reload cros_workon'd packages

The list of packages worked on persists when the board's sysroot is
recreated (list stored in .config/cros_workon/) but the information is
not passed to portage's db. A dummy call to cros_workon regenerates the
necessary symlinks as per CL:267112 and fixes the discrepancy.

BUG= chromium:679831 
TEST=manual:
cros_workon start arc-mesa;
setup_board --force;
equery w arc-mesa # now points to -9999 ebuild instead of "stable"
TEST=./setup_board --board=cyan
TEST=./setup_board --board=veyron_minnie

Change-Id: I6dd251557f89198a8bdc9e0c7b57ed658b70c703
Reviewed-on: https://chromium-review.googlesource.com/426369
Commit-Ready: Nicolas Norvez <norvez@chromium.org>
Tested-by: Nicolas Norvez <norvez@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/2e722a283cdc2dc8d0e212c7df2c2631ee335bfb/setup_board