New issue
Advanced search Search tips

Issue 725962 link

Starred by 1 user
Status: Fixed
Owner:
vapier@chromium.org
Closed: May 2017
Cc:
kerrnel@chromium.org
vapier@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

chromeos-base/imageloader probably missing a DEPEND

Project Member Reported by akes...@chromium.org, May 24 2017 
See https://luci-logdog.appspot.com/v/?s=chromeos%2Fbb%2Fchromeos%2Felm-paladin%2F2837%2F%2B%2Frecipes%2Fsteps%2FBuildPackages%2F0%2Fstdout

imageloader fails to build on first try due to missing dbus header file, succeeds on retry

Comment 1 by akes...@chromium.org, May 24 2017

Cc: vapier@chromium.org
Owner: kerrnel@chromium.org

Comment 2 by akes...@chromium.org, May 24 2017

Summary: chromeos-base/imageloader probably missing a DEPEND (was: chromeos-base/imageloader probably missing an RDEPEND)

Comment 3 by vapier@chromium.org, May 24 2017

Cc: kerrnel@chromium.org
Components: -Infra>Client>ChromeOS OS>Systems
Owner: vapier@chromium.org
https://chromium-review.googlesource.com/514382
Project Member

Comment 4 by bugdroid1@chromium.org, May 26 2017 

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

commit 058a82db97d6235d7afab863ce43d1ebb4c62bce
Author: Mike Frysinger <vapier@chromium.org>
Date: Fri May 26 00:16:09 2017

imageloader: depend on system_api

We use chromeos/dbus/service_constants.h.

BUG= chromium:725962 
TEST=precq passes

Change-Id: If2f7c76d6140e22d988bcdca93fba1640ea99ccf
Reviewed-on: https://chromium-review.googlesource.com/514382
Commit-Ready: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>
Reviewed-by: Aviv Keshet <akeshet@chromium.org>

[modify] https://crrev.com/058a82db97d6235d7afab863ce43d1ebb4c62bce/chromeos-base/imageloader/imageloader-9999.ebuild

Comment 5 by vapier@chromium.org, May 26 2017

Status: Fixed (was: Assigned)

Comment 6 by dchan@chromium.org, Aug 1 2017

Labels: VerifyIn-61

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

Status: Archived (was: Fixed)

Comment 8 by vapier@chromium.org, Jun 21 2018

Status: Fixed (was: Archived)
►

Sign in to add a comment