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

Issue 831624 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Sep 6
Cc:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug



Sign in to add a comment

eve-arcnext-chrome-pfq is failing in the PFQ due to libassistant compilation failure

Project Member Reported by newcomer@chromium.org, Apr 11 2018

Issue description

After having a red PFQ for a while due to issue 830865 which is now fixed, eve-arcnext-chrome-pfq is broken by this bug:

LINK:
https://logs.chromium.org/v/?s=chromeos%2Fbb%2Fchromeos%2Feve-arcnext-chrome-pfq%2F100%2F%2B%2Frecipes%2Fsteps%2FBuildPackages%2F0%2Fstdout


LOGS:
chromeos-chrome-67.0.3394.0_rc-r1: FAILED: obj/libassistant/internal/net/net/udp_socket_posix.o 
chromeos-chrome-67.0.3394.0_rc-r1: /home/chrome-bot/goma/gomacc x86_64-cros-linux-gnu-clang++ -B/usr/x86_64-pc-linux-gnu/x86_64-cros-linux-gnu/binutils-bin/2.27.0-gold -MMD -MF obj/libassistant/internal/net/net/udp_socket_posix.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 -DOFFICIAL_BUILD -DGOOGLE_CHROME_BUILD -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -DNO_UNWIND_TABLES -DCR_CLANG_REVISION=\"328716-2\" -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -D_FORTIFY_SOURCE=2 -DOS_CHROMEOS -DCR_SYSROOT_HASH=85ac8d5e0f6cff99fc323fd3d29cb73e2aa970e2 -DNDEBUG -DNVALGRIND -DDYNAMIC_ANNOTATIONS_ENABLED=0 -I../../../../../../../home/chrome-bot/chrome_root/src -Igen -I../../../../../../../home/chrome-bot/chrome_root/src/third_party/boringssl/src/include -fno-strict-aliasing -fmerge-all-constants -fno-unwind-tables -fno-asynchronous-unwind-tables -fPIC -pipe -pthread -fcolor-diagnostics -no-canonical-prefixes -m64 -march=x86-64 -Wall -Wextra -Wimplicit-fallthrough -Wthread-safety -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-null-pointer-arithmetic -Wno-ignored-pragma-optimize -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-exceptions -fno-rtti --sysroot=../../../../../../../build/eve-arcnext -fvisibility-inlines-hidden -pipe -pipe -pipe -march=corei7 -fno-split-dwarf-inlining -fdebug-info-for-profiling -D__google_stl_debug_vector=1 -Wno-unknown-warning-option -stdlib=libc++  -c ../../../../../../../home/chrome-bot/chrome_root/src/libassistant/internal/net/socket/udp_socket_posix.cc -o obj/libassistant/internal/net/net/udp_socket_posix.o
chromeos-chrome-67.0.3394.0_rc-r1: In file included from ../../../../../../../home/chrome-bot/chrome_root/src/libassistant/internal/net/socket/udp_socket_posix.cc:7:
chromeos-chrome-67.0.3394.0_rc-r1: ../../../../../../../home/chrome-bot/chrome_root/src/libassistant/internal/net/socket/udp_socket_posix.h:164:54: error: expected class name
chromeos-chrome-67.0.3394.0_rc-r1:   class ReadWatcher : public base::MessageLoopForIO::Watcher {
chromeos-chrome-67.0.3394.0_rc-r1:                                                      ^
chromeos-chrome-67.0.3394.0_rc-r1: ../../../../../../../home/chrome-bot/chrome_root/src/libassistant/internal/net/socket/udp_socket_posix.h:170:47: error: only virtual member functions can be marked 'override'
chromeos-chrome-67.0.3394.0_rc-r1:     void OnFileCanReadWithoutBlocking(int fd) override;
chromeos-chrome-67.0.3394.0_rc-r1:                                               ^~~~~~~~
chromeos-chrome-67.0.3394.0_rc-r1: ../../../../../../../home/chrome-bot/chrome_root/src/libassistant/internal/net/socket/udp_socket_posix.h:172:48: error: only virtual member functions can be marked 'override'
chromeos-chrome-67.0.3394.0_rc-r1:     void OnFileCanWriteWithoutBlocking(int fd) override {}
chromeos-chrome-67.0.3394.0_rc-r1:                                                ^~~~~~~~~
chromeos-chrome-67.0.3394.0_rc-r1: ../../../../../../../home/chrome-bot/chrome_root/src/libassistant/internal/net/socket/udp_socket_posix.h:180:55: error: expected class name
chromeos-chrome-67.0.3394.0_rc-r1:   class WriteWatcher : public base::MessageLoopForIO::Watcher {
chromeos-chrome-67.0.3394.0_rc-r1:                                                       ^
chromeos-chrome-67.0.3394.0_rc-r1: ../../../../../../../home/chrome-bot/chrome_root/src/libassistant/internal/net/socket/udp_socket_posix.h:186:47: error: only virtual member functions can be marked 'override'
chromeos-chrome-67.0.3394.0_rc-r1:     void OnFileCanReadWithoutBlocking(int fd) override {}
chromeos-chrome-67.0.3394.0_rc-r1:                                               ^~~~~~~~~
chromeos-chrome-67.0.3394.0_rc-r1: ../../../../../../../home/chrome-bot/chrome_root/src/libassistant/internal/net/socket/udp_socket_posix.h:188:48: error: only virtual member functions can be marked 'override'
chromeos-chrome-67.0.3394.0_rc-r1:     void OnFileCanWriteWithoutBlocking(int fd) override;
chromeos-chrome-67.0.3394.0_rc-r1:                                                ^~~~~~~~
chromeos-chrome-67.0.3394.0_rc-r1: ../../../../../../../home/chrome-bot/chrome_root/src/libassistant/internal/net/socket/udp_socket_posix.h:240:27: error: no type named 'FileDescriptorWatcher' in 'base::MessageLoopForIO'
chromeos-chrome-67.0.3394.0_rc-r1:   base::MessageLoopForIO::FileDescriptorWatcher read_socket_watcher_;
chromeos-chrome-67.0.3394.0_rc-r1:   ~~~~~~~~~~~~~~~~~~~~~~~~^
chromeos-chrome-67.0.3394.0_rc-r1: ../../../../../../../home/chrome-bot/chrome_root/src/libassistant/internal/net/socket/udp_socket_posix.h:241:27: error: no type named 'FileDescriptorWatcher' in 'base::MessageLoopForIO'
chromeos-chrome-67.0.3394.0_rc-r1:   base::MessageLoopForIO::FileDescriptorWatcher write_socket_watcher_;
chromeos-chrome-67.0.3394.0_rc-r1:   ~~~~~~~~~~~~~~~~~~~~~~~~^
chromeos-chrome-67.0.3394.0_rc-r1: ../../../../../../../home/chrome-bot/chrome_root/src/libassistant/internal/net/socket/udp_socket_posix.cc:166:26: error: no member named 'WATCH_READ' in 'base::MessageLoopForIO'; did you mean 'base::MessagePumpLibevent::WATCH_READ'?
chromeos-chrome-67.0.3394.0_rc-r1:           socket_, true, base::MessageLoopForIO::WATCH_READ,
chromeos-chrome-67.0.3394.0_rc-r1:                          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
chromeos-chrome-67.0.3394.0_rc-r1:                          base::MessagePumpLibevent::WATCH_READ
chromeos-chrome-67.0.3394.0_rc-r1: ../../../../../../../home/chrome-bot/chrome_root/src/base/message_loop/watchable_io_message_pump_posix.h:56:5: note: 'base::MessagePumpLibevent::WATCH_READ' declared here
chromeos-chrome-67.0.3394.0_rc-r1:     WATCH_READ = 1 << 0,
chromeos-chrome-67.0.3394.0_rc-r1:     ^
chromeos-chrome-67.0.3394.0_rc-r1: ../../../../../../../home/chrome-bot/chrome_root/src/libassistant/internal/net/socket/udp_socket_posix.cc:208:26: error: no member named 'WATCH_WRITE' in 'base::MessageLoopForIO'; did you mean 'base::MessagePumpLibevent::WATCH_WRITE'?
chromeos-chrome-67.0.3394.0_rc-r1:           socket_, true, base::MessageLoopForIO::WATCH_WRITE,
chromeos-chrome-67.0.3394.0_rc-r1:                          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
chromeos-chrome-67.0.3394.0_rc-r1:                          base::MessagePumpLibevent::WATCH_WRITE
chromeos-chrome-67.0.3394.0_rc-r1: ../../../../../../../home/chrome-bot/chrome_root/src/base/message_loop/watchable_io_message_pump_posix.h:57:5: note: 'base::MessagePumpLibevent::WATCH_WRITE' declared here
chromeos-chrome-67.0.3394.0_rc-r1:     WATCH_WRITE = 1 << 1,
chromeos-chrome-67.0.3394.0_rc-r1:     ^
chromeos-chrome-67.0.3394.0_rc-r1: In file included from ../../../../../../../home/chrome-bot/chrome_root/src/libassistant/internal/net/socket/udp_socket_posix.cc:7:
chromeos-chrome-67.0.3394.0_rc-r1: ../../../../../../../home/chrome-bot/chrome_root/src/libassistant/internal/net/socket/udp_socket_posix.h:240:49: warning: private field 'read_socket_watcher_' is not used [-Wunused-private-field]
chromeos-chrome-67.0.3394.0_rc-r1:   base::MessageLoopForIO::FileDescriptorWatcher read_socket_watcher_;
chromeos-chrome-67.0.3394.0_rc-r1:                                                 ^
chromeos-chrome-67.0.3394.0_rc-r1: ../../../../../../../home/chrome-bot/chrome_root/src/libassistant/internal/net/socket/udp_socket_posix.h:241:49: warning: private field 'write_socket_watcher_' is not used [-Wunused-private-field]
chromeos-chrome-67.0.3394.0_rc-r1:   base::MessageLoopForIO::FileDescriptorWatcher write_socket_watcher_;
chromeos-chrome-67.0.3394.0_rc-r1:                                                 ^
chromeos-chrome-67.0.3394.0_rc-r1: 2 warnings and 10 errors generated.


 
Status: Started (was: Unconfirmed)
https://libassistant-internal-review.git.corp.google.com/c/libassistant/internal/+/39242
https://libassistant-internal-review.git.corp.google.com/c/libassistant/internal/+/39241

These two CLs should fix it.  But we need to coordinate with the Cast team to fix it in their repo too.

Comment 2 by pbe...@chromium.org, Apr 12 2018

Cc: pbe...@chromium.org

Comment 3 by ihf@chromium.org, Apr 12 2018

Cc: ihf@chromium.org
Labels: -Pri-3 M-67 Pri-1
Update, waiting for ChromeCast team to merge their side and also including this fix. Looks like merging very soon. https://eureka-internal-review.git.corp.google.com/c/chromium/src/+/161818

Comment 5 by pbe...@chromium.org, Apr 16 2018

Labels: Hotlist-CrOS-Sheriffing
Status: Fixed (was: Started)

Sign in to add a comment