Skip to content

Conversation

tildejustin
Copy link
Member

sources:
tildejustin/optifabric@fd961db...legacy-fabric
tildejustin/optifabric@cd6dffa...ornithe

sorry for all the commits, the only thing for MCSR is fixing the 1.1 OptiFine releases (patching out the Signature attribute from xt.class), and that's in the ornithe branch only

@tildejustin tildejustin added the mod update An update to an existing mod label Jul 22, 2025
@contariaa
Copy link
Member

I looked through the legacy fabric diff, these were the actual changes that werent just upgradling or cleanup I found and all approve of, let me know if I've missed anything of importance:

Ignore Hash Failure
tildejustin/optifabric@fd961db...legacy-fabric#diff-841103ed55e978eb3a8f7fcdb2a604ee06cc090ac75ae957c4405fd75dbf02d7R1-R14

Cache targetClassName
tildejustin/optifabric@fd961db...legacy-fabric#diff-e0e197d4a0560176c39eebdafd9a9f0705f38bc4624460b69d5d38c4774286e0R22-R24

Add "optifabric.keepExtractedJar" System Property
tildejustin/optifabric@fd961db...legacy-fabric#diff-da5bb78a5f5a8071fd9e26c20bdbeff23b0f9e6d8e124403eea464b2000b7397R109-R112

Better Minecraft Jar Detection In Dev
tildejustin/optifabric@fd961db...legacy-fabric#diff-da5bb78a5f5a8071fd9e26c20bdbeff23b0f9e6d8e124403eea464b2000b7397R23-R24
tildejustin/optifabric@fd961db...legacy-fabric#diff-da5bb78a5f5a8071fd9e26c20bdbeff23b0f9e6d8e124403eea464b2000b7397R222-R246

Manual Remap Conflict Fixes in Dev
tildejustin/optifabric@013081f

Most of this seems to be for dev or debugging, what exactly does ignoring the hash failure do? Is it just suppressing some log messages?

@contariaa
Copy link
Member

Ornithe seems to be the same stuff plus the fix you mentioned which seems fine assuming it does in fact work

@tildejustin
Copy link
Member Author

tildejustin commented Jul 26, 2025

ignoring the hash failure is for when running the Patcher to extract the _MOD jar from the binary class installer jar using a in-dev loom obfuscated mc jar that has Environment annotations etc in it and is not a bit-for-bit match. however those mixins never get applied since the configuration gets added afterwards and it's all in prelaunch anyway so I would have to bump the mixin phase or smth in the first place (that mixin is already in 1.8-1.12.2 & 1.13.2 subprojects, but installers also exist for modern 1.7.10 releases so I copied it over with hopes to get it working someday. (hoping this loader change helps FabricMC/fabric-loader#1064)

@tildejustin
Copy link
Member Author

targetClassName is just to clean up the diff from the ornithe branch, I'm not entirely sure what it does but I think it suppresses all errors now? (The old implementation was a little flawed iirc, the first one always got through.)

Copy link
Member

@contariaa contariaa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alright, makes enough sense to me

@tildejustin tildejustin merged commit e362042 into Minecraft-Java-Edition-Speedrunning:main Jul 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mod update An update to an existing mod
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants