New issue
Advanced search Search tips

Issue 880490 link

Starred by 2 users

Issue metadata

Status: Duplicate
Merged: issue 880484
Owner:
Closed: Sep 4
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 0
Type: Bug-Regression



Sign in to add a comment

Build failure: failed generate_build_files failed in win and win64 continuous builder

Project Member Reported by ligim...@chromium.org, Sep 4

Issue description

Build failure: failed generate_build_files failed on win and win 64 Chrome continuous builder.

Link to the builder:
=====================
https://uberchromegw.corp.google.com/i/official.desktop.continuous/builders/win64%20trunk/builds/39799

Link to the log file:
=====================
https://logs.chromium.org/v/?s=chrome%2Fbb%2Fofficial.desktop.continuous%2Fwin64_trunk%2F39799%2F%2B%2Frecipes%2Fsteps%2Fgenerate_build_files%2F0%2Fstdout

Error log
=========
ERROR at //chrome/installer/setup/google_chrome_behaviors.cc:29:11: Can't include this header from here.
#include "third_party/crashpad/crashpad/util/misc/uuid.h"
          ^---------------------------------------------
The target:
  //chrome/installer/setup:lib
is including a file from the target:
  //third_party/crashpad/crashpad/util:util
It's usually best to depend directly on the destination target.
In some cases, the destination target is considered a subcomponent
of an intermediate target. In this case, the intermediate target
should depend publicly on the destination to forward the ability
to include headers.
Dependency chain (there may also be others):
  //chrome/installer/setup:lib -->
  //components/crash/content/app:app --[private]-->
  //third_party/crashpad/crashpad/util:util
GN gen failed: 1
step returned non-zero exit code: 1

Possible suspect
=================
https://chromium.googlesource.com/chromium/src/+/f82f3089ca14042a57ceac2fd0fb65022273addf

Assigning to Greg for further updates.

+chromium sheriff for awareness.
 
Cc: hua...@chromium.org
Mergedinto: 880484
Status: Duplicate (was: Assigned)
This is a duplicate, and should be fixed already.

Sign in to add a comment