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

Retool to Essential Loom #1836

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

Conversation

Elephant1214
Copy link
Contributor

@Elephant1214 Elephant1214 commented Aug 11, 2024

Retool to Essential Loom, supporting up to Fabric 1.21.1 and Forge 1.21, and now requiring Java 21.

Things to note

  • A Java 21 JDK is REQUIRED, not optional
  • 1.12.2 runs MUST USE JAVA 8 or they won't work
  • Tasks on subprojects must be run through the main project using gradlew :subprojectName:task

Problems

  • Resources are not shaded into 1.12.2 and 1.16.5 projects when run in an IDE for some reason
  • No way to automatically update version numbers (yet)
  • Jar names are probably not to Don's liking, but this can be fixed easily
  • Access Transformer AND Access Widener required for modern Forge, but apparently Essential Loom can convert these itself on the fly (?)

@slava110
Copy link

Note that for 1.12.2 forge version will be downgraded from 14.23.5.2860 to 14.23.5.2840. Though, I don't think there's a lot of important changes for dev environment between these versions

@Elephant1214
Copy link
Contributor Author

Note that for 1.12.2 forge version will be downgraded from 14.23.5.2860 to 14.23.5.2840. Though, I don't think there's a lot of important changes for dev environment between these versions

No difference for MTS. All I've found is that Mixin has a few problems with mappings, but using UniMix fixes them.

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