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

Toolchain upgrade and Nostalgia mappings #101

Merged
merged 7 commits into from
Sep 17, 2024
Merged

Conversation

TechPizzaDev
Copy link
Contributor

babric-loom-extension is a Gradle plugin that plugs into the official fabric-loom and fabric-loader. This plugin replaces the specialized babric-loom and babric-loader which are poorly maintained as of today.

Remap codebase to Nostalgia mappings which is more complete and maintained. Nostalgia mappings are TinyV2 which is also required for the new toolchain, while the previous Babric Yarn (Barn) mappings were TinyV1.

This PR does not properly rename all members to the new terms:

  1. Package names have changed but our package/folder structure remained.
  2. @Mixin(Block.class) MixinBlock is now @Mixin(Tile.class) MixinBlock (correct target, bad name).
  3. All @Shadow members should be properly remapped (correctness not properly verified).
  4. Some interface methods have been remapped since they @Override-ed vanilla methods.
  5. Parameters or fields like World world remain as Level world.

This list is ordered in importance of fixing. N1 and N2 are next on the crafting table chopping block. N3 is difficult since I have no proper tools to verify and this PR already took too much manual and error-prone effort.
N4 and N5 are not important but are nice to have, and could be done all at once or piece by piece.

@TechPizzaDev TechPizzaDev merged commit 38b3636 into main Sep 17, 2024
2 checks passed
@TechPizzaDev TechPizzaDev deleted the nostalgia-toolchain branch September 17, 2024 21:02
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.

1 participant