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

Issue 785324 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
OOO until Feb 4th
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug



Sign in to add a comment

errorprone update to 2.1.2 caused Android arm64 compile failures

Project Member Reported by charliea@chromium.org, Nov 15 2017

Issue description

I'm seeing compile failures on Android arm64 on the Chromium Perf waterfall that first occurred in this run: https://uberchromegw.corp.google.com/i/chromium.perf/builders/Android%20arm64%20Builder/builds/164548

The only new CL in that run was https://chromium.googlesource.com/chromium/src/+/9e5c2bb47fab12af0f8e05726de234bd2b6d4680, which rolled the version of errorprone to 2.1.2. The compilation errors also mention errorprone, which seems to confirm that this is indeed the cause.

CCing, wnwen@, the author of the CL

Also CCing the two current Android sheriffs, nyquist@ and clamy@

I'm going to go ahead and revert the CL.

 
Cc: wnwen@chromium.org
Woops, *actually* adding wnwen@, the CL author
Project Member

Comment 3 by bugdroid1@chromium.org, Nov 15 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/58ceae1a55b867cdae3db0a923605d44b34488c2

commit 58ceae1a55b867cdae3db0a923605d44b34488c2
Author: Charlie Andrews <charliea@chromium.org>
Date: Wed Nov 15 17:45:40 2017

Revert "Android: Update errorprone to 2.1.2"

This reverts commit 9e5c2bb47fab12af0f8e05726de234bd2b6d4680.

This CL caused Android arm64 compile failures.

NOTRY=true
TBR=nyquist@chromium.org, yuweih@chromium.org, kapishnikov@chromium.org, agrieve@chromium.org

Bug:  785324 
Cq-Include-Trybots: master.tryserver.chromium.android:android_cronet_tester;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: I235a28ff449f77fee3ab7f000773c4befdc032fa
Reviewed-on: https://chromium-review.googlesource.com/771995
Reviewed-by: Charlie Andrews <charliea@chromium.org>
Commit-Queue: Charlie Andrews <charliea@chromium.org>
Cr-Commit-Position: refs/heads/master@{#516738}
[modify] https://crrev.com/58ceae1a55b867cdae3db0a923605d44b34488c2/DEPS
[modify] https://crrev.com/58ceae1a55b867cdae3db0a923605d44b34488c2/base/android/javatests/src/org/chromium/base/ObserverListTest.java
[modify] https://crrev.com/58ceae1a55b867cdae3db0a923605d44b34488c2/build/android/gyp/javac.py
[modify] https://crrev.com/58ceae1a55b867cdae3db0a923605d44b34488c2/build/android/lint/suppressions.xml
[modify] https://crrev.com/58ceae1a55b867cdae3db0a923605d44b34488c2/chrome/android/BUILD.gn
[modify] https://crrev.com/58ceae1a55b867cdae3db0a923605d44b34488c2/chrome/android/java/ChromeVersionConstants.java.version
[modify] https://crrev.com/58ceae1a55b867cdae3db0a923605d44b34488c2/chrome/android/java/src/org/chromium/chrome/browser/AppHooks.java
[modify] https://crrev.com/58ceae1a55b867cdae3db0a923605d44b34488c2/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadNotificationService.java
[modify] https://crrev.com/58ceae1a55b867cdae3db0a923605d44b34488c2/chrome/android/java/src/org/chromium/chrome/browser/suggestions/SuggestionsRecyclerView.java
[modify] https://crrev.com/58ceae1a55b867cdae3db0a923605d44b34488c2/chrome/android/java/src/org/chromium/chrome/browser/widget/bottomsheet/BottomSheetNavigationItemView.java
[modify] https://crrev.com/58ceae1a55b867cdae3db0a923605d44b34488c2/chrome/android/third_party/widget_bottomsheet_base/java/src/org/chromium/chrome/browser/widget/bottomsheet/base/BottomNavigationItemView.java
[modify] https://crrev.com/58ceae1a55b867cdae3db0a923605d44b34488c2/chrome/android/third_party/widget_bottomsheet_base/java/src/org/chromium/chrome/browser/widget/bottomsheet/base/BottomNavigationMenu.java
[modify] https://crrev.com/58ceae1a55b867cdae3db0a923605d44b34488c2/chrome/android/third_party/widget_bottomsheet_base/java/src/org/chromium/chrome/browser/widget/bottomsheet/base/BottomNavigationMenuView.java
[modify] https://crrev.com/58ceae1a55b867cdae3db0a923605d44b34488c2/chrome/android/third_party/widget_bottomsheet_base/java/src/org/chromium/chrome/browser/widget/bottomsheet/base/BottomNavigationPresenter.java
[modify] https://crrev.com/58ceae1a55b867cdae3db0a923605d44b34488c2/chrome/android/third_party/widget_bottomsheet_base/java/src/org/chromium/chrome/browser/widget/bottomsheet/base/BottomNavigationView.java
[modify] https://crrev.com/58ceae1a55b867cdae3db0a923605d44b34488c2/components/cronet/android/java/src/org/chromium/net/impl/JavaUrlRequest.java
[modify] https://crrev.com/58ceae1a55b867cdae3db0a923605d44b34488c2/remoting/android/java/src/org/chromium/chromoting/SessionConnector.java
[modify] https://crrev.com/58ceae1a55b867cdae3db0a923605d44b34488c2/third_party/errorprone/BUILD.gn
[modify] https://crrev.com/58ceae1a55b867cdae3db0a923605d44b34488c2/third_party/errorprone/README.chromium

Labels: Performance-Sheriff-BotHealth

Comment 6 by wnwen@chromium.org, Nov 15 2017

Owner: wnwen@chromium.org
Status: Assigned (was: Untriaged)

Comment 7 by wnwen@chromium.org, Nov 15 2017

Status: Started (was: Assigned)
Project Member

Comment 8 by bugdroid1@chromium.org, Nov 15 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/6686c0001a1c308024788485783468485ddfce9d

commit 6686c0001a1c308024788485783468485ddfce9d
Author: Peter Wen <wnwen@chromium.org>
Date: Wed Nov 15 21:51:48 2017

Reland "Android: Update errorprone to 2.1.2"

Original CL: https://crrev.com/c/761318

Changes:
- Suppress additional warning in ChromeVersionConstants

BUG= 775031 , 785324 
TBR=nyquist@chromium.org,yuweih@chromium.org,kapishnikov@chromium.org,agrieve@chromium.org

Change-Id: I2c38e5fca25af5e6993477fcacf426df113fb2f0
Reviewed-on: https://chromium-review.googlesource.com/772000
Commit-Queue: Peter Wen <wnwen@chromium.org>
Reviewed-by: Peter Wen <wnwen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#516840}
[modify] https://crrev.com/6686c0001a1c308024788485783468485ddfce9d/DEPS
[modify] https://crrev.com/6686c0001a1c308024788485783468485ddfce9d/base/android/javatests/src/org/chromium/base/ObserverListTest.java
[modify] https://crrev.com/6686c0001a1c308024788485783468485ddfce9d/build/android/gyp/javac.py
[modify] https://crrev.com/6686c0001a1c308024788485783468485ddfce9d/build/android/lint/suppressions.xml
[modify] https://crrev.com/6686c0001a1c308024788485783468485ddfce9d/chrome/android/BUILD.gn
[modify] https://crrev.com/6686c0001a1c308024788485783468485ddfce9d/chrome/android/java/ChromeVersionConstants.java.version
[modify] https://crrev.com/6686c0001a1c308024788485783468485ddfce9d/chrome/android/java/src/org/chromium/chrome/browser/AppHooks.java
[modify] https://crrev.com/6686c0001a1c308024788485783468485ddfce9d/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadNotificationService.java
[modify] https://crrev.com/6686c0001a1c308024788485783468485ddfce9d/chrome/android/java/src/org/chromium/chrome/browser/suggestions/SuggestionsRecyclerView.java
[modify] https://crrev.com/6686c0001a1c308024788485783468485ddfce9d/chrome/android/java/src/org/chromium/chrome/browser/widget/bottomsheet/BottomSheetNavigationItemView.java
[modify] https://crrev.com/6686c0001a1c308024788485783468485ddfce9d/chrome/android/third_party/widget_bottomsheet_base/java/src/org/chromium/chrome/browser/widget/bottomsheet/base/BottomNavigationItemView.java
[modify] https://crrev.com/6686c0001a1c308024788485783468485ddfce9d/chrome/android/third_party/widget_bottomsheet_base/java/src/org/chromium/chrome/browser/widget/bottomsheet/base/BottomNavigationMenu.java
[modify] https://crrev.com/6686c0001a1c308024788485783468485ddfce9d/chrome/android/third_party/widget_bottomsheet_base/java/src/org/chromium/chrome/browser/widget/bottomsheet/base/BottomNavigationMenuView.java
[modify] https://crrev.com/6686c0001a1c308024788485783468485ddfce9d/chrome/android/third_party/widget_bottomsheet_base/java/src/org/chromium/chrome/browser/widget/bottomsheet/base/BottomNavigationPresenter.java
[modify] https://crrev.com/6686c0001a1c308024788485783468485ddfce9d/chrome/android/third_party/widget_bottomsheet_base/java/src/org/chromium/chrome/browser/widget/bottomsheet/base/BottomNavigationView.java
[modify] https://crrev.com/6686c0001a1c308024788485783468485ddfce9d/components/cronet/android/java/src/org/chromium/net/impl/JavaUrlRequest.java
[modify] https://crrev.com/6686c0001a1c308024788485783468485ddfce9d/remoting/android/java/src/org/chromium/chromoting/SessionConnector.java
[modify] https://crrev.com/6686c0001a1c308024788485783468485ddfce9d/third_party/errorprone/BUILD.gn
[modify] https://crrev.com/6686c0001a1c308024788485783468485ddfce9d/third_party/errorprone/README.chromium

Comment 9 by wnwen@chromium.org, Nov 15 2017

Status: Fixed (was: Started)

Sign in to add a comment