MinecraftForge / ForgeFlower

Forge's modifications to FernFlower. Fixing various bugs/inconsistencies. Main Repo: https://github.com/MinecraftForge/FernFlower
Apache License 2.0
80 stars 44 forks source link

ForgeFlower Legacy; FG-2.2 Base #139

Open TheCurle opened 1 year ago

TheCurle commented 1 year ago

This PR is the result of copying all the commits made on top of https://github.com/MinecraftForge/FernFlowerLegacy, converting those to patches, and applying them in the style of ForgeFlower.

All upstream patches are removed, and thus this should be considered a completely different product than the master branch; thus the branch this is PRed to uses a legacy- prefix.

This is Part 5 of the RetroGradle initiative.

Limitations

Being that this is forked from FernFlowerLegacy, it is limited to Java 9 features. Therefore, this version of ForgeFlower cannot decompile any J10+ features, such as records, try-with-resources, enhanced switch, or any other syntax sugar related code.

However, it does not need to; the Minecraft 1.11 (and below) code it is designed for does not use any of these.

TODO

zml2008 commented 1 year ago

I think for ease of management it makes sense to keep this in the same repo, but having a different artifact ID is imo the easiest way to distinguish the versions for consumers.

TheCurle commented 1 year ago

I have adjusted this so that it publishes under net.minecraftforge.forgeflowerlegacy with the versioning scheme of 0.{FG_VERSION}.{PATCH}.

The first build will thus be net.minecraftforge:forgeflowerlegacy:0.2.2.0.

With that, this is no longer a draft.

Codetoil commented 1 year ago

Good that RetroGradle is moving alongšŸ‘.