Skip to content

Fix DataSourceChannel#9812

Open
vvb2060 wants to merge 7 commits into
topjohnwu:masterfrom
vvb2060:fix
Open

Fix DataSourceChannel#9812
vvb2060 wants to merge 7 commits into
topjohnwu:masterfrom
vvb2060:fix

Conversation

@vvb2060
Copy link
Copy Markdown
Collaborator

@vvb2060 vvb2060 commented Apr 19, 2026

No description provided.

Copy link
Copy Markdown
Owner

@topjohnwu topjohnwu left a comment

Choose a reason for hiding this comment

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

Can you remove this commit (54f4de1) from the PR? I would prefer to always use Android Studio's internal JDK (which is still JDK 21 at the moment), and would prefer NOT to use gradle daemon JVM resolvers.

@vvb2060
Copy link
Copy Markdown
Collaborator Author

vvb2060 commented Apr 19, 2026

I also want to using the IDE's built-in JDK, but JEP 513 requires JDK25.

@topjohnwu
Copy link
Copy Markdown
Owner

Can refactor the code to avoid using JEP 513?

@vvb2060 vvb2060 marked this pull request as ready for review April 27, 2026 15:26
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.

2 participants