New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 607355 link

Starred by 1 user
Status: Verified
Owner:
steve...@chromium.org
Closed: May 2016
Cc:
█ davidjames@chromium.org
dpranke@chromium.org
hashimoto@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Simple Chrome: cros_chrome_sdk: Use environemnt from --chroot

Project Member Reported by steve...@chromium.org, Apr 27 2016 
cros_chrome_sdk.py takes a --chroot argument that allows Simple Chrome to use the sysroot from a local chromeos checkout.

It does not however currently support using the environment generated from the local chromeos checkout when 'emerge-$BOARD chromeos-chrome' or 'build_packages' has been run.

In order to test changes to the ebuild in Simple Chrome, it would be helpful to be able to extract the environment from a local checkout.
Project Member

Comment 1 by bugdroid1@chromium.org, May 2 2016 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/7392eb4e9c7ee382827fdb258b4c0d5ed28908fd

commit 7392eb4e9c7ee382827fdb258b4c0d5ed28908fd
Author: Steven Bennetts <stevenjb@chromium.org>
Date: Wed Apr 27 22:57:06 2016

cros_chrome_sdk: Use environemnt from --chroot

BUG= chromium:607355 
TEST=cros chrome-sdk --chroot=/path/to/chromeos  # Ensure GN_ARGS was generated from local environment

Change-Id: I95dec7fc7d1916979950b15d3840ae40dcb7be4c
Reviewed-on: https://chromium-review.googlesource.com/341090
Commit-Ready: Steven Bennetts <stevenjb@chromium.org>
Tested-by: Steven Bennetts <stevenjb@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/7392eb4e9c7ee382827fdb258b4c0d5ed28908fd/cli/cros/cros_chrome_sdk.py

Comment 2 by steve...@chromium.org, May 2 2016

Status: Fixed (was: Started)

Comment 3 by patricia@chromium.org, May 23 2016 

Bulk verified

Comment 4 by patricia@chromium.org, May 23 2016

Status: Verified (was: Fixed)
bulk verified
►

Sign in to add a comment