Lint fails for multiple APK build targets |
||||||
Issue description
[61811/62476] ACTION //chrome/android:chrome_test_vr_java__lint(//build/toolchain/android:android_clang_arm)
Unexpected failure during lint analysis of WebappActivityTestRule.java (this is a bug in lint or one of the libraries it depends on)
AbortCompilation:ProblemHandler.handle(ProblemHandler.java:159)<-ProblemHandler.handle(ProblemHandler.java:226)<-ProblemReporter.handle(ProblemReporter.java:2393)<-ProblemReporter.isClassPathCorre
ct(ProblemReporter.java:4761)
Unexpected failure during lint analysis of CustomTabActivityTestRule.java (this is a bug in lint or one of the libraries it depends on)
AbortCompilation:ProblemHandler.handle(ProblemHandler.java:159)<-ProblemHandler.handle(ProblemHandler.java:226)<-ProblemReporter.handle(ProblemReporter.java:2393)<-ProblemReporter.isClassPathCorre
ct(ProblemReporter.java:4761)
Unexpected failure during lint analysis of VrShellControllerInputTest.java (this is a bug in lint or one of the libraries it depends on)
AbortCompilation:ProblemHandler.handle(ProblemHandler.java:159)<-ProblemHandler.handle(ProblemHandler.java:226)<-ProblemReporter.handle(ProblemReporter.java:2393)<-ProblemReporter.isClassPathCorre
ct(ProblemReporter.java:4761)
Unexpected failure during lint analysis of WebVrTransitionTest.java (this is a bug in lint or one of the libraries it depends on)
AbortCompilation:ProblemHandler.handle(ProblemHandler.java:159)<-ProblemHandler.handle(ProblemHandler.java:226)<-ProblemReporter.handle(ProblemReporter.java:2393)<-ProblemReporter.isClassPathCorre
ct(ProblemReporter.java:4761)
Unexpected failure during lint analysis of WebVrInputTest.java (this is a bug in lint or one of the libraries it depends on)
AbortCompilation:ProblemHandler.handle(ProblemHandler.java:159)<-ProblemHandler.handle(ProblemHandler.java:226)<-ProblemReporter.handle(ProblemReporter.java:2393)<-ProblemReporter.isClassPathCorre
ct(ProblemReporter.java:4761)
Unexpected failure during lint analysis of VrShellCompositorViewHolderTest.java (this is a bug in lint or one of the libraries it depends on)
AbortCompilation:ProblemHandler.handle(ProblemHandler.java:159)<-ProblemHandler.handle(ProblemHandler.java:226)<-ProblemReporter.handle(ProblemReporter.java:2393)<-ProblemReporter.isClassPathCorre
ct(ProblemReporter.java:4761)
Unexpected failure during lint analysis of VrTestFramework.java (this is a bug in lint or one of the libraries it depends on)
AbortCompilation:ProblemHandler.handle(ProblemHandler.java:159)<-ProblemHandler.handle(ProblemHandler.java:226)<-ProblemReporter.handle(ProblemReporter.java:2393)<-ProblemReporter.isClassPathCorre
ct(ProblemReporter.java:4761)
Unexpected failure during lint analysis of VrShellNavigationTest.java (this is a bug in lint or one of the libraries it depends on)
AbortCompilation:ProblemHandler.handle(ProblemHandler.java:159)<-ProblemHandler.handle(ProblemHandler.java:226)<-ProblemReporter.handle(ProblemReporter.java:2393)<-ProblemReporter.isClassPathCorre
ct(ProblemReporter.java:4761)
,
Oct 30 2017
I've never seen this before. Brian, have you seen this?
,
Oct 30 2017
Nope, I don't remember ever seeing something like this for VR. I've seen similar errors pop up in the past, but they weren't VR related.
,
Oct 30 2017
This also happens to chrome_uiautomator_test_apk__apk__java, and probably others too
[1863/2296] ACTION //clank/test/geppetto/clanktests:chrome_uiautomator_test_apk__apk__java__lint(//build/toolchain/android:android_clang_arm)
Unexpected failure during lint analysis of PDFTest.java (this is a bug in lint or one of the libraries it depends on)
AbortCompilation:ProblemHandler.handle(ProblemHandler.java:159)<-ProblemHandler.handle(ProblemHandler.java:226)<-ProblemReporter.handle(ProblemReporter.java:2393)<-ProblemReporter.isClassPathCorre
ct(ProblemReporter.java:4761)
Unexpected failure during lint analysis of CrashReportTest.java (this is a bug in lint or one of the libraries it depends on)
AbortCompilation:ProblemHandler.handle(ProblemHandler.java:159)<-ProblemHandler.handle(ProblemHandler.java:226)<-ProblemReporter.handle(ProblemReporter.java:2393)<-ProblemReporter.isClassPathCorre
ct(ProblemReporter.java:4761)
Unexpected failure during lint analysis of YouTubeTest.java (this is a bug in lint or one of the libraries it depends on)
AbortCompilation:ProblemHandler.handle(ProblemHandler.java:159)<-ProblemHandler.handle(ProblemHandler.java:226)<-ProblemReporter.handle(ProblemReporter.java:2393)<-ProblemReporter.isClassPathCorre
ct(ProblemReporter.java:4761)
Unexpected failure during lint analysis of BookmarksTest.java (this is a bug in lint or one of the libraries it depends on)
AbortCompilation:ProblemHandler.handle(ProblemHandler.java:159)<-ProblemHandler.handle(ProblemHandler.java:226)<-ProblemReporter.handle(ProblemReporter.java:2393)<-ProblemReporter.isClassPathCorre
ct(ProblemReporter.java:4761)
Unexpected failure during lint analysis of OmniboxTest.java (this is a bug in lint or one of the libraries it depends on)
AbortCompilation:ProblemHandler.handle(ProblemHandler.java:159)<-ProblemHandler.handle(ProblemHandler.java:226)<-ProblemReporter.handle(ProblemReporter.java:2393)<-ProblemReporter.isClassPathCorre
ct(ProblemReporter.java:4761)
Unexpected failure during lint analysis of OptionsMenuTest.java (this is a bug in lint or one of the libraries it depends on)
AbortCompilation:ProblemHandler.handle(ProblemHandler.java:159)<-ProblemHandler.handle(ProblemHandler.java:226)<-ProblemReporter.handle(ProblemReporter.java:2393)<-ProblemReporter.isClassPathCorre
ct(ProblemReporter.java:4761)
Unexpected failure during lint analysis of OfflinePagesTest.java (this is a bug in lint or one of the libraries it depends on)
AbortCompilation:ProblemHandler.handle(ProblemHandler.java:159)<-ProblemHandler.handle(ProblemHandler.java:226)<-ProblemReporter.handle(ProblemReporter.java:2393)<-ProblemReporter.isClassPathCorre
ct(ProblemReporter.java:4761)
Unexpected failure during lint analysis of WebAppTest.java (this is a bug in lint or one of the libraries it depends on)
AbortCompilation:ProblemHandler.handle(ProblemHandler.java:159)<-ProblemHandler.handle(ProblemHandler.java:226)<-ProblemReporter.handle(ProblemReporter.java:2393)<-ProblemReporter.isClassPathCorre
ct(ProblemReporter.java:4761)
Unexpected failure during lint analysis of IncognitoTest.java (this is a bug in lint or one of the libraries it depends on)
AbortCompilation:ProblemHandler.handle(ProblemHandler.java:159)<-ProblemHandler.handle(ProblemHandler.java:226)<-ProblemReporter.handle(ProblemReporter.java:2393)<-ProblemReporter.isClassPathCorre
ct(ProblemReporter.java:4761)
Unexpected failure during lint analysis of SavePasswordTest.java (this is a bug in lint or one of the libraries it depends on)
AbortCompilation:ProblemHandler.handle(ProblemHandler.java:159)<-ProblemHandler.handle(ProblemHandler.java:226)<-ProblemReporter.handle(ProblemReporter.java:2393)<-ProblemReporter.isClassPathCorre
ct(ProblemReporter.java:4761)
... Plenty more I'm not including
,
Nov 2 2017
These errors generally occur when there's an sdk/lint version/platform xml version mismatch and our attempts to avoid these have been mostly unsuccessful. For this case, I _think_ what's happening is that we've updated the sdk to v26 (and platform xml is v26 as well) but lint is outdated and therefore fails on certain types of checks it doesn't know how to handle. I updated sdk tools via android studio locally and updated the lint path to reflect this and these errors go away, however, there are a whole bunch (hundreds) of new lint errors which will either need to be fixed/suppressed before we can update this. I think we should come up with a better plan for preventing this in the future before fixing this - it can take a few days of work just to suppress everything let alone fix all of the issues. Another thing that makes this problem harder is that upstream/downstream sdk versions may differ at any given time and downstream/upstream builds need to work with all the various gn arg configurations we have to handle that stuff. I believe historically the command line version of lint was out of date but I think that's since changed so we should just be able to update lint whenever the sdk is updated. John, do you have any thoughts as to what the best course of action is here? Does this fit with go/use-adjective-apis? It may be enough to just do a manual update one more time with many suppressions and then from here on out force lint to be updated with the sdk..
,
Jan 12 2018
,
Jan 13 2018
Guessing this bug can be archived. wnwen has been fixing lint recently, and I'd guess these failures don't happen any more (although not unlikely that they are now suppressed warnings)
,
Feb 8 2018
,
Jul 4
|
||||||
►
Sign in to add a comment |
||||||
Comment 1 by wnwen@chromium.org
, Oct 30 2017