Skip to content

Correctly register all sources#3611

Merged
oldergod merged 1 commit into
masterfrom
bquenaudon.2026-05-27.fixingstuff
May 27, 2026
Merged

Correctly register all sources#3611
oldergod merged 1 commit into
masterfrom
bquenaudon.2026-05-27.fixingstuff

Conversation

@oldergod
Copy link
Copy Markdown
Member

@oldergod oldergod commented May 27, 2026

fixes #3607

variant.sources.java?.addGeneratedSourceDirectory(wireTask) { it.outputDirectoriesList[index] }
}
is KotlinOutput -> {
variant.sources.kotlin?.addGeneratedSourceDirectory(wireTask) { it.outputDirectoriesList[index] }
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it was supposed to be fixed so........... not happy

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe I need to find the AOSP ticket and share

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

confusing with another piece of code, all good

@oldergod oldergod force-pushed the bquenaudon.2026-05-27.fixingstuff branch from 181a220 to c925f10 Compare May 27, 2026 11:13
@oldergod oldergod marked this pull request as ready for review May 27, 2026 11:54
@oldergod oldergod requested a review from JakeWharton May 27, 2026 11:54
@oldergod oldergod force-pushed the bquenaudon.2026-05-27.fixingstuff branch from c925f10 to 8253a4b Compare May 27, 2026 13:44
@oldergod oldergod enabled auto-merge May 27, 2026 13:46
@oldergod oldergod force-pushed the bquenaudon.2026-05-27.fixingstuff branch from 8253a4b to ceb3b19 Compare May 27, 2026 16:06
@oldergod oldergod merged commit 5823d31 into master May 27, 2026
17 checks passed
@oldergod oldergod deleted the bquenaudon.2026-05-27.fixingstuff branch May 27, 2026 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Wire 7.0.0-alpha (1/2/3): Android variant Kotlin source set empty — KSP and compileKotlin see no user sources

2 participants