-
Notifications
You must be signed in to change notification settings - Fork 679
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
修复自动安装在 1.20.4 的严重问题 #3023
修复自动安装在 1.20.4 的严重问题 #3023
Conversation
做的差不多了,需要社区帮助测试 |
Forge 做了更改,需要处理兼容性,将更改代码逻辑 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested using a build based on burningtnt@560a9e9.
When Forge 49.0.49
and 49.0.50
are installed respectively and tested with OptiFine HD_U_I8_pre4
, both can run normally. When building using the main branch (without this PR), the former cannot start the game, while the latter works fine.
In other launcher-installed instances, the Modloaders / OptiFine interface will prompt as expected: * Installed by external process, which cannot be configured. I tested Forge and Fabric installed by PCL2 and both prompted as expected.
Good. Thanks for your detailed testing. |
By the way, may you try installing forge + optifine with the main branch, and launch the game with this PR. I've made some codes to automatically fix the optifine installed by legacy version. |
If I understand correctly: I tried installing Forge But the game did not launch successfully. I checked the library file and found that the |
可能是因为删除 mods.toml 是在安装 optifine 的时候 |
I've made some changes to check OptiFine while checking whether the game is conplete. Maybe there's a bug here. |
Has been fixed yesterday but I forgot to send a messsage. There should NOT be any bugs. (?) |
I saw the commit you pushed earlier, but I was dealing with other things at the time. I will test it later. |
Tested using Builds based on this PR now correctly fix OptiFine library files under Forge & OptiFine instances for 1.20.4 installed through a launcher that does not contain this PR. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall it seems there is no problem.
Close #2975 Close #3021