Support Gradle caching for some parts of FG - Githubissues

MinecraftForge / ForgeGradle

Minecraft mod development framework used by Forge and FML for the gradle build system

GNU Lesser General Public License v2.1

508 stars 437 forks source link

Support Gradle caching for some parts of FG #923

Closed PaintNinja closed 1 year ago

PaintNinja commented 1 year ago

This PR adds support for Gradle caching in some parts of FG, mainly focused towards userdev AT applying.

To test this PR:

Publish this to local maven and use it in the MDK
Enable caching in the MDK's gradle.properties with org.gradle.caching=true
Add an AT the MDK and build the mod
Change the AT and build it again
Change the AT back to exactly what it was in step 3 and build again

To compare performance with and without this PR:

For the PR: use the steps above
Without the PR: use a fresh copy of the MDK for step 1, skip step 2 and then do steps 3, 4 and 5 above.

Known limitations:

The AccessTransformerFunction creates a temporary AT file which prevents caching in some cases
Due to the relative path sensitivity of inputs including an input for the user-supplied AT, it can't fully cache across different mod projects using the same mappings, AT and Forge version, however the main recompilation stage should still use the cache

PaintNinja commented 1 year ago

Baseline (without this PR, caching not enabled): https://scans.gradle.com/s/oakl6gsps6iti (1m 7s) With this PR (when getting a cache hit): https://scans.gradle.com/s/p2jeystmnwjlq (33s)