Build warning: dagger.lint.DaggerIssueRegistry does not specify a vendor

[bubenheimer]

Building with Dagger 2.35.1, Gradle 7.0, AGP 7.0.0-alpha14 I see warnings:

Task :app:lintAnalyzeDebug

dagger.lint.DaggerIssueRegistry in /Users/me/.gradle/caches/transforms-3/d0ee303d2fb2dbd304a5fff6d69d1127/transformed/jetified-dagger-lint-aar-2.35.1/jars/lint.jar does not specify a vendor; see IssueRegistry#vendor

This likely existed in prior versions, I just had not looked into it.

Would be nice if these could be eliminated, I see a bunch of these for different modules. Thanks.

[danysantiago]

Thanks - I think we might need to update our lint-api dependency to declare a vendor property in our IssueRegistry but should be easy to do.

[ZacSweers]

Anything preventing #2639 from being merged now that AGP 7 stable is out?

[danysantiago]

There is, the test which uses LintDetectorTest started to fail because there seems to be a requirement of absolute path dependencies:

java.lang.IllegalArgumentException: Relative Path found: ../maven/v1/https/repo1.maven.org/maven2/org/jetbrains/kotlin/kotlin-stdlib-jdk7/1.5.20/kotlin-stdlib-jdk7-1.5.20.jar. All paths should be absolute.
	at com.android.tools.lint.LintCliClient.initializeProjects(LintCliClient.kt:1346)
	at com.android.tools.lint.client.api.LintClient.performInitializeProjects$lint_api(LintClient.kt:1012)
	at com.android.tools.lint.client.api.LintDriver.initializeProjectRoots(LintDriver.kt:569)
	at com.android.tools.lint.client.api.LintDriver.doAnalyze(LintDriver.kt:484)
	at com.android.tools.lint.client.api.LintDriver.doAnalyze$default(LintDriver.kt:473)
	at com.android.tools.lint.client.api.LintDriver.analyze(LintDriver.kt:417)
	at com.android.tools.lint.checks.infrastructure.TestLintClient.analyze(TestLintClient.java:671)
	at com.android.tools.lint.checks.infrastructure.TestLintClient.analyze(TestLintClient.java:602)
	at com.android.tools.lint.checks.infrastructure.TestLintClient.checkLint(TestLintClient.java:349)
	at com.android.tools.lint.checks.infrastructure.TestLintRunner.checkLint(TestLintRunner.kt:229)
	at com.android.tools.lint.checks.infrastructure.TestLintRunner.run(TestLintRunner.kt:172)
	at com.android.tools.lint.checks.infrastructure.TestLintTask.run(TestLintTask.java:933)
	at dagger.lint.DaggerKotlinIssueDetectorTest.simpleSmokeTestForQualifiersAndProviders(DaggerKotlinIssueDetectorTest.kt:186)

I'll check with Lint team (a.k.a Tor) to see what we can do...

[danysantiago]

Alright, the LintDetectorTest is fixed, but now there is a concern regarding this new API usage. Specifically users in AGP below 7.0 will have errors because the Vendor class will be loaded and fail to do so because their AGP + Lint version does not contain it, so we might have to hold off on actually adding this until 7.0 is more widely adopted. =/

[G00fY2]

APG 7 stable is now available for 106 days. Any plans to address this issue soon? I can imagine that people who keep dagger up-to-date also take care of switching to the latest AGP.

[Berki2021]

Any new info on this?? WHAT TO DO?

[liutikas]

Which version of Dagger is expected to have the fix?

[danysantiago]

Version 2.41 and onwards should contain the fix for this issue.