Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

IDEA-358562 Created projects via wizard defines Daemon JVM criteria by default #2883

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

vmadalin
Copy link
Contributor

@vmadalin vmadalin commented Dec 2, 2024

Context

The Daemon toolchain was introduced in Gradle 8.8 and the motivation behind it and other technical details can be found on the public spec document. The implementation follows the agreed details on spec document and UI/UX document.

Summary

When creating a new project via wizard, in case of using a Gradle version compatible with Daemon JVM criteria then specified JDK from the dropdown will be used to invoke updateDaemonJvm Gradle task with the resolved version and vendor from JDK. The vendor will only be specified if is known by foojay-plugin being able to generate the different toolchain download URLs for the auto-provisioning mechanism.

NOTE: For now this option will be disabled until Daemon JVM criteria supports the auto-provisioning but also until the feature is stable enough to be considered as a default for projects, keeping it as a opt-in option.

NOTE: This PR was build on top of #2882 to avoid conflicts.

Tasks

Tests

Demo

Screen.Recording.2024-11-29.at.15.05.35.mov

@vmadalin vmadalin force-pushed the feature/daemon-toolchain-wizard-default branch from 34ef2f6 to 51f3c63 Compare December 5, 2024 12:45
@vmadalin vmadalin force-pushed the feature/daemon-toolchain-wizard-default branch from 51f3c63 to f27aba3 Compare December 5, 2024 12:47
@vmadalin vmadalin marked this pull request as ready for review December 5, 2024 12:50
@HackerMadCat HackerMadCat self-assigned this Dec 18, 2024
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