Allow to define vendor for the Daemon JVM criteria #28931
+223
−72
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Context
After #28731 PR was merged the initial vendor support was removed. This PR adds the support back by following the agreed public spec document but also the recent #28934 linked to the Daemon toolchain support.
Those are changes to add support for vendor on Daemon toolchain criteria:
JvmVendor
onDaemonContext
using systemProperties to obtain its value from running daemon processJvmVendorSpec
property onUpdateDaemonJvm
task by making the class serializableUpdateDaemonJvm
task so it can still update thedaemon-jvm
properties file## Demo
Daemon.toolchain.using.vendor.mov
Contributor Checklist
<subproject>/src/integTest
) to verify changes from a user perspective.<subproject>/src/test
) to verify logic../gradlew sanityCheck
../gradlew <changed-subproject>:quickTest
.