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

Issue 777086 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Oct 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

ec-utils.ebuild is missing dependency on chromeos-config

Project Member Reported by pberny@google.com, Oct 21 2017

Issue description

UserAgent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/62.0.3202.62 Safari/537.36
Platform: coral

Steps to reproduce the problem:
1. cros_workon ec-utils
2 apply CL:727373

What is the expected behavior?
no complaints

What went wrong?
/build/coral/tmp/portage/chromeos-base/ec-utils-9999/image//usr/share/chromeos-config/config.dtb missing. Do you have the right DEPEND='s

Did this work before? No 

Chrome version: 62.0.3202.62  Channel: canary
OS Version: 
Flash Version: 

but not exposed before, since chromeos-config-bsp was installing models before and still is on non-coral devices, but not with the change above.
 

Comment 1 by pbe...@chromium.org, Oct 21 2017

Cc: sjg@chromium.org jclinton@chromium.org
Components: OS>Firmware
Labels: -Pri-2 -Type-Bug-Regression Pri-1 Type-Bug
Owner: pbe...@chromium.org
Status: Assigned (was: Unconfirmed)

Comment 2 by pbe...@chromium.org, Oct 21 2017

I have the change, just need to upload the CL for review.
Project Member

Comment 3 by bugdroid1@chromium.org, Oct 21 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/737dd8034d712bd7f59e705dbedd457f39b83839

commit 737dd8034d712bd7f59e705dbedd457f39b83839
Author: Patrick Berny <pberny@chromium.org>
Date: Sat Oct 21 20:25:48 2017

ec-utils: add missing dependency to chromeos-config.

BUG= chromium:777086 
TEST=emerge-coral ec-utils
     Make sure the dependencies are pulled in correctly.

Change-Id: Idd3b0ec190527401a9b7e5b269844249995de726
Reviewed-on: https://chromium-review.googlesource.com/731889
Commit-Ready: Patrick Berny <pberny@chromium.org>
Tested-by: Patrick Berny <pberny@chromium.org>
Reviewed-by: Jason Clinton <jclinton@chromium.org>

[modify] https://crrev.com/737dd8034d712bd7f59e705dbedd457f39b83839/eclass/cros-ec-board.eclass
[modify] https://crrev.com/737dd8034d712bd7f59e705dbedd457f39b83839/chromeos-base/ec-utils/ec-utils-9999.ebuild

Comment 4 by pbe...@chromium.org, Oct 21 2017

Status: Fixed (was: Assigned)

Comment 5 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Comment 6 by dchan@chromium.org, Jan 23 2018

Status: Fixed (was: Archived)

Sign in to add a comment