New issue
Advanced search Search tips

Issue 721462 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jul 2017
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Minor fixes and changes 2017Q2

Project Member Reported by sjg@chromium.org, May 11 2017

Issue description

Tracking bug for misc things too minor to track on other bugs.

 

Comment 1 by sjg@chromium.org, May 11 2017

Summary: Minor fixes and changes 2017Q2 (was: Minor fixed and changes 2017Q2)
Project Member

Comment 2 by bugdroid1@chromium.org, May 13 2017

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

commit 7e8a97f89440c0e97008b3ea040286854c178c01
Author: Simon Glass <sjg@chromium.org>
Date: Sat May 13 03:58:07 2017

coreboot-utils: Add a dependency on pciutils

I noticed when building coreboot-utils on reef that it fails with:

x86_64-cros-linux-gnu-clang ... -c -o amd.o amd.c
In file included from ali.c:17:
./superiotool.h:40:10: fatal error: 'pci/pci.h' file not found
         ^~~~~~~~~~~

It looks like we should have a dependency on pciutils which provides this
file. Add it.

BUG= chromium:721462 
TEST=emerge-pyro -q coreboot-utils (without pciutils installed)
See that we can now emerge without any errors
Signed-off-by: Simon Glass <sjg@chromium.org>

Change-Id: I7cadc409f9a125306624bd9f30342f6762fbd5ee
Reviewed-on: https://chromium-review.googlesource.com/503467
Commit-Ready: Simon Glass <sjg@chromium.org>
Tested-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/7e8a97f89440c0e97008b3ea040286854c178c01/sys-apps/coreboot-utils/coreboot-utils-9999.ebuild

Comment 3 by sjg@google.com, Jul 6 2017

Status: Fixed (was: Untriaged)

Sign in to add a comment