'X11/Xlib.h' file not found in ChromeOS amd64 Chromium Goma Canary |
|||||||
Issue descriptionStartintg failure from https://luci-milo.appspot.com/buildbot/chromium.fyi/ChromeOS%20amd64%20Chromium%20Goma%20Canary/6591 [1974/33015] CXX obj/remoting/host/linux/x11/x11_util.o FAILED: obj/remoting/host/linux/x11/x11_util.o /b/c/goma_client/gomacc x86_64-cros-linux-gnu-clang++ -B/b/c/builder/ChromeOS_amd64_Chromium_Goma_Canary/.cros_cache/chrome-sdk/tarballs/amd64-generic+10110.0.0+target_toolchain/usr/x86_64-pc-linux-gnu/x86_64-cros-linux-gnu/binutils-bin/2.27.0-gold -Wno-unknown-warning-option -MMD -MF obj/remoting/host/linux/x11/x11_util.o.d -DV8_DEPRECATION_WARNINGS -DUSE_UDEV -DUSE_AURA=1 -DUSE_NSS_CERTS=1 -DUSE_OZONE=1 -DFULL_SAFE_BROWSING -DSAFE_BROWSING_CSD -DSAFE_BROWSING_DB_LOCAL -DCHROMIUM_BUILD -DFIELDTRIAL_TESTING_ENABLED -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -DCR_CLANG_REVISION=\"317263-4\" -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -DOS_CHROMEOS -DNDEBUG -DNVALGRIND -DDYNAMIC_ANNOTATIONS_ENABLED=0 -DWEBRTC_NON_STATIC_TRACE_EVENT_HANDLERS=0 -DFEATURE_ENABLE_VOICEMAIL -DGTEST_RELATIVE_PATH -DWEBRTC_CHROMIUM_BUILD -DWEBRTC_POSIX -DWEBRTC_LINUX -DCHROMEOS -I../.. -Igen -I../../third_party/webrtc_overrides -I../../third_party/webrtc -fno-strict-aliasing -Wno-builtin-macro-redefined -D__DATE__= -D__TIME__= -D__TIMESTAMP__= -funwind-tables -fPIC -pipe -B../../third_party/binutils/Linux_x64/Release/bin -pthread -fcolor-diagnostics -no-canonical-prefixes -m64 -march=x86-64 -Wall -Wextra -Wno-missing-field-initializers -Wno-unused-parameter -Wno-c++11-narrowing -Wno-covered-switch-default -Wno-unneeded-internal-declaration -Wno-inconsistent-missing-override -Wno-undefined-var-template -Wno-nonportable-include-path -Wno-address-of-packed-member -Wno-unused-lambda-capture -Wno-user-defined-warnings -Wno-enum-compare-switch -Wno-tautological-unsigned-zero-compare -Wno-null-pointer-arithmetic -Wno-tautological-constant-compare -Wtautological-constant-out-of-range-compare -O2 -fno-ident -fdata-sections -ffunction-sections -fno-omit-frame-pointer -g2 -gsplit-dwarf -ggnu-pubnames -fvisibility=hidden -Wheader-hygiene -Wstring-conversion -Wtautological-overlap-compare -std=gnu++14 -fno-rtti --sysroot=../../../.cros_cache/chrome-sdk/tarballs/amd64-generic+10110.0.0+sysroot_chromeos-base_chromeos-chrome.tar.xz -fno-exceptions -fvisibility-inlines-hidden -pipe -pipe -pipe -march=x86-64 -msse3 -fno-split-dwarf-inlining -fdebug-info-for-profiling -D__google_stl_debug_vector=1 -Wno-unknown-warning-option -stdlib=libc++ -Wno-inline-asm -c ../../remoting/host/linux/x11_util.cc -o obj/remoting/host/linux/x11/x11_util.o In file included from ../../remoting/host/linux/x11_util.cc:5: ../../remoting/host/linux/x11_util.h:12:10: fatal error: 'X11/Xlib.h' file not found #include <X11/Xlib.h> ^~~~~~~~~~~~ 1 error generated.
,
Nov 15 2017
removed: [1] chrome-bot@slave9-c1:(Linux 14.04):~$ rm -rf /b/c/builder/ChromeOS_amd64_Chromium_Goma_Canary/.cros_cache/chrome-sdk
,
Nov 15 2017
I am not sure who I need to notify but I guess amd64-generic+10110.0.0+sysroot_chromeos-base_chromeos-chrome.tar.xz broken. It might not have Xlib.h. Luis and Caroline, do you know who I need to notify?
,
Nov 15 2017
I would start by checking with the ChromeOS build/infra team.
,
Nov 15 2017
we don't include X packages anymore because CrOS doesn't use X. so the sysroot itself omitting Xlib.h doesn't sound like a bug to me. looking at older recent sysroots suggests this isn't a new thing and Chrome has been building fine. it becomes a question of why the Chrome build is trying to include X components. maybe something on the Simple Chrome side (with the recent chrome-sdk changes) broke, or the Chrome side itself broke by incorrectly including X things, or Chrome started building components it shouldn't for a CrOS target. should compare the last passing run with the first failing run and see if anything on the GN args have changed. either way, fairly certain not a bug in the CrOS sysroot or toolchain, so i'll drop those peeps from the bug.
,
Nov 15 2017
Is this something I can try building on a Linux box? If so, please let me know what the GN configuration should be and I'll take a look.
,
Nov 15 2017
Looking at the first line of the build output, 6590 is building "chromiumos_preflight" and 6591 (where the failure starts) is building "All all". The build target is passed to "cros chrome-sdk", so while this may be due to a chromite change, it does not seem to be simple chrome per-se. Unfortunately our build output does not make it particularly clear what target is being build, but x11_util.cc is failing to build and appears to be part of remoting/host/linux.
,
Nov 15 2017
the Simple Chrome workflow can run on any Linux system w/out need for a CrOS SDK. but i'm not sure how to reproduce this particular bot config, and i don't think the CrOS bots are showing this issue (yet?).
,
Nov 15 2017
+dgarrett@chromium.org, +akeshet@chromium.org - Has anything changed that would cause this builder to build "All all" instead of "chromiumos_preflight" that you are aware of? Any idea where this command is coming from exactly?
,
Nov 15 2017
For referenc, this is the last succeeding build command: /b/rr/tmpSLUZNR/rw/scripts/slave/.recipe_deps/depot_tools/cros chrome-sdk '--board=amd64-generic' --nocolor '--log-level=debug' --use-external-config --gomadir /b/c/goma_client --nostart-goma -- /b/rr/tmpSLUZNR/rw/scripts/slave/.recipe_deps/depot_tools/ninja -w 'dupbuild=err' -C /b/c/builder/ChromeOS_amd64_Chromium_Goma_Canary/src/out_amd64-generic/Release -d stats -j 50 chromiumos_preflight And this one fails: /b/rr/tmp2s9ghw/rw/scripts/slave/.recipe_deps/depot_tools/cros chrome-sdk '--board=amd64-generic' --nocolor '--log-level=debug' --use-external-config --gomadir /b/c/goma_client --nostart-goma -- /b/rr/tmp2s9ghw/rw/scripts/slave/.recipe_deps/depot_tools/ninja -w 'dupbuild=err' -C /b/c/builder/ChromeOS_amd64_Chromium_Goma_Canary/src/out_amd64-generic/Release -d stats -j 50 All all
,
Nov 15 2017
The problem is not in chrome, or 'cros chrome-sdk', it's how we are invoking it. We have never tried to make building "all" work in Simple Chrome. Here is what chromiumos_preflight includes: https://cs.chromium.org/chromium/src/BUILD.gn?type=cs&q=chromiumos_preflight&sq=package:chromium&l=828 I think that this is where we are setting up the ninja command passed to simple chrome: https://cs.chromium.org/chromium/build/scripts/slave/recipe_modules/chromium/api.py?type=cs&q=_run_ninja&sq=package:chromium&l=424 I have no clue how to determine where that is getting called from though.
,
Nov 15 2017
the chromium.fyi waterfall isn't managed by the CrOS team. that's a Chrome waterfall.
,
Nov 15 2017
re: comment #12: I don't really understand how that is relevant? Aren't we all just trying to track down the problem? If there is someone more likely to understand the failure cause here, please cc them.
,
Nov 15 2017
i dropped CrOS infra from the bug earlier because, afaik, Aviv & Don don't monitor any Chromium waterfalls. if you have questions about those, you'll want to ping Chromium people.
,
Nov 16 2017
'All' and 'all' is come from https://cs.chromium.org/chromium/build/scripts/slave/recipe_modules/chromium/config.py?type=cs&sq=package:chromium&l=532 ?
,
Nov 16 2017
Looks configuration issue.
,
Nov 16 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/f4bfda2512e446f2236ff5c3d75706b0dbdad604 commit f4bfda2512e446f2236ff5c3d75706b0dbdad604 Author: Takuto Ikuta <tikuta@google.com> Date: Thu Nov 16 07:33:23 2017 Specify build target for 'ChromeOS amd64 Chromium Goma Canary' Without target specification, the builder tries to build 'All', 'all' target. But some targets are unsupported in simple chrome. Let me fix the issue by using the same target specified in chromeos-amd64-generic-rel. TBR=dpranke@chromium.org Bug: 785167 Change-Id: I66a5e99ec80a0df7524db199f206eb9cf22c3cd8 Reviewed-on: https://chromium-review.googlesource.com/773921 Commit-Queue: Takuto Ikuta <tikuta@google.com> Reviewed-by: Shinya Kawanaka <shinyak@chromium.org> Cr-Commit-Position: refs/heads/master@{#517015} [modify] https://crrev.com/f4bfda2512e446f2236ff5c3d75706b0dbdad604/testing/buildbot/chromium.fyi.json
,
Nov 16 2017
Thank you for your investigation! |
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by yyanagisawa@chromium.org
, Nov 15 2017