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

Issue 849211 link

Starred by 2 users

Issue metadata

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



Sign in to add a comment

ToT breakage: ./build_packages: x11-themes/cros-adapta

Project Member Reported by nya@chromium.org, Jun 4 2018

Issue description

tatl-paladin failing with the following error:

The BuildPackages stage failed: Packages failed in ./build_packages: x11-themes/cros-adapta

https://luci-milo.appspot.com/buildbot/chromiumos/tatl-paladin/3699
https://luci-milo.appspot.com/buildbot/chromiumos/tatl-paladin/3700
https://luci-milo.appspot.com/buildbot/chromiumos/tatl-paladin/3701

Looks related to crrev.com/c/1075828, which was chumped for no apparent reason.

 

Comment 1 by zsm@chromium.org, Jun 4 2018

The revert for crrev.com/c/1075828 can be found at crrev.com/c/1084631.

Comment 2 by zsm@google.com, Jun 4 2018

Owner: vpalatin@chromium.org
I'm reassigning this to vpalatin@ who created the revert CL. I guess this bug can be closed.
Status: Fixed (was: Untriaged)
tatl-paladin is green
Project Member

Comment 4 by bugdroid1@chromium.org, Jun 12 2018

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

commit 5e6716bef4f630d7fbbb78207cd0c990349b7416
Author: Josh Pratt <jopra@chromium.org>
Date: Tue Jun 12 07:36:26 2018

cros-adapta:Patch to only install the assets directory if it exists

Attempts to fix bug caused in
https://chromium-review.googlesource.com/1075828 .

This is a backwards compatible fix that allows the assets directory to
be moved into gtk-3.0.

BUG= 849211 

Change-Id: I60b570259c26775256f37e0fa4cacd4cc5358fc8
Reviewed-on: https://chromium-review.googlesource.com/1086817
Commit-Ready: Josh Pratt <jopra@chromium.org>
Tested-by: Josh Pratt <jopra@chromium.org>
Reviewed-by: Raymes Khoury <raymes@chromium.org>

[modify] https://crrev.com/5e6716bef4f630d7fbbb78207cd0c990349b7416/x11-themes/cros-adapta/cros-adapta-9999.ebuild

Sign in to add a comment