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.

Sign up for GitHub

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

Jump to bottom

安装MCBBS整合包时不会添加整合包内JVM参数 #4395

Open
3 tasks done
tmdakm opened this issue Aug 2, 2024 · 14 comments · May be fixed by #4401
Open
3 tasks done

安装MCBBS整合包时不会添加整合包内JVM参数 #4395

tmdakm opened this issue Aug 2, 2024 · 14 comments · May be fixed by #4401
Labels
· 新功能 可合并 已有 Pull Request 处理了该 Issue

Comments

@tmdakm
Copy link

tmdakm commented Aug 2, 2024

检查项

  • 我已尝试使用其他启动器，其他启动器没有出现问题。 如果其他启动器也存在问题，证明是网络环境不佳（解决方法），并非 PCL 的问题，请 不要 提交反馈。下载 HMCL
  • 我知晓大多数此类问题都是网络环境不佳导致的，但我确实认为我的问题可能是 PCL 导致的，和网络环境无关。
  • 我已在 Issues 页面 和 常见&难检反馈及问题列表 中搜索，确认了这一 Bug 未被提交过。

描述

按照整合包规范制作了一个测试用MCBBS整合包
其中的mcbbs整合包packmeta文件和manifest文件都加上了
"launchInfo": { "minMemory": 0, "launchArgument": [], "javaArgument": [ "-XX:+ExplicitGCInvokesConcurrent" ] }
安装完成后，无法找到该jvm参数：
image

重现步骤

拖动带有jvm参数的整合包进入pcl2窗口并安装

日志与附件

test.zip
logs.zip
@tmdakm tmdakm added · Bug 新提交 需要社区初步确认其有效的新提交 labels Aug 2, 2024
@tmdakm
Copy link
Author

tmdakm commented Aug 2, 2024

经过测试 我添加另一个整合包是有JVM参数 但是别人添加那个就没有 而这个test.zip是都没有
另附setup.ini:
Count:9 LastVersionCode:51 Notice:11 LastMinecraftVersion:21w44a UiTheme:0 LaunchFolderSelect:$HuangyuRSCLIENT\.minecraft\ History:1.12.2 LaunchSplit:1 LaunchFolders:D:\Minecraft\xdms\.minecraft\ HomeUpdateRelease:True HomeForumPush:True LaunchVisibility:4 UiHiddenOtherHelp:False LaunchVersionSelect:555 LoginPageType:0 LaunchRamType:1 LaunchRamCustom:12 LaunchArgumentIndie:4 LaunchAdvanceGame:-XX:+DisableAttachMechanism UiCustomType:0

@tmdakm
Copy link
Author

tmdakm commented Aug 2, 2024 •
edited
Loading

经过测试 我添加另一个整合包是有JVM参数 但是别人添加那个就没有 而这个test.zip是都没有 另附setup.ini: Count:9 LastVersionCode:51 Notice:11 LastMinecraftVersion:21w44a UiTheme:0 LaunchFolderSelect:$HuangyuRSCLIENT\.minecraft\ History:1.12.2 LaunchSplit:1 LaunchFolders:D:\Minecraft\xdms\.minecraft\ HomeUpdateRelease:True HomeForumPush:True LaunchVisibility:4 UiHiddenOtherHelp:False LaunchVersionSelect:555 LoginPageType:0 LaunchRamType:1 LaunchRamCustom:12 LaunchArgumentIndie:4 LaunchAdvanceGame:-XX:+DisableAttachMechanism UiCustomType:0

确定你的代码写对了吗（ 也许是不符合格式？

我确定是按照格式写的 其他启动器可以正常导入
这个是setup.ini（

@Deep-Dark-Forest Deep-Dark-Forest added 等待确认 已经过社区确认，等待开发者确认 and removed 新提交 需要社区初步确认其有效的新提交 labels Aug 2, 2024
@Stardito
Copy link

Stardito commented Aug 2, 2024

Modules/Minecraft/ModModpack.vb 的确没有关于 JVM 的相关代码，连 launchInfo 都没有编写

@tmdakm
Copy link
Author

tmdakm commented Aug 2, 2024

经过测试 我添加另一个整合包是有JVM参数 但是别人添加那个就没有 而这个test.zip是都没有 另附setup.ini: Count:9 LastVersionCode:51 Notice:11 LastMinecraftVersion:21w44a UiTheme:0 LaunchFolderSelect:$HuangyuRSCLIENT\.minecraft\ History:1.12.2 LaunchSplit:1 LaunchFolders:D:\Minecraft\xdms\.minecraft\ HomeUpdateRelease:True HomeForumPush:True LaunchVisibility:4 UiHiddenOtherHelp:False LaunchVersionSelect:555 LoginPageType:0 LaunchRamType:1 LaunchRamCustom:12 LaunchArgumentIndie:4 LaunchAdvanceGame:-XX:+DisableAttachMechanism UiCustomType:0

另一个整合包导入有jvm参数是因为导出了那个版本的pcl配置文件

@tmdakm
Copy link
Author

tmdakm commented Aug 2, 2024

经过测试 我添加另一个整合包是有JVM参数 但是别人添加那个就没有 而这个test.zip是都没有 另附setup.ini: Count:9 LastVersionCode:51 Notice:11 LastMinecraftVersion:21w44a UiTheme:0 LaunchFolderSelect:$HuangyuRSCLIENT\.minecraft\ History:1.12.2 LaunchSplit:1 LaunchFolders:D:\Minecraft\xdms\.minecraft\ HomeUpdateRelease:True HomeForumPush:True LaunchVisibility:4 UiHiddenOtherHelp:False LaunchVersionSelect:555 LoginPageType:0 LaunchRamType:1 LaunchRamCustom:12 LaunchArgumentIndie:4 LaunchAdvanceGame:-XX:+DisableAttachMechanism UiCustomType:0

另一个整合包导入有jvm参数是因为导出了那个版本的pcl配置文件

那就不奇怪了，可能你添加 JVM 参数的方法和 ta 的不一样，你的方法 PCL 也许不支持 修改 PCL 配置文件倒是可以办到

因为按照我找到的整合包manifest文件格式，里面是包含jvm参数的

@Stardito
Copy link

Stardito commented Aug 2, 2024 •
edited
Loading

我觉得这个应该算是新功能添加，是因为目前 PCL 代码中没有对 mcbbs.manifest 中的 JVM 参数做出相应处理。目前PCL对于导入游戏这一类的JVM参数处理好像就是PCL.ini了

@LTCatt
Copy link
Member

LTCatt commented Aug 2, 2024

@ZhaiSoul 有关于这个字段的文档么？

@LTCatt LTCatt added · 新功能 第三方 等待第三方处理，或是由第三方导致 and removed · Bug 等待确认 已经过社区确认，等待开发者确认 labels Aug 2, 2024
@tmdakm
Copy link
Author

tmdakm commented Aug 3, 2024

@ZhaiSoul 有关于这个字段的文档么？

"launchInfo": {
	"minMemory": 4096,
             //"支持的Java版本，该选项非强制，但需要警告玩家",
	"supportJava": [8, 9],
	"launchArgument": ["", ""],
	"javaArgument": ["", ""]
}

如果你说的是launchInfo的话（

@zkitefly
Copy link
Collaborator

zkitefly commented Aug 3, 2024

@ZhaiSoul 有关于这个字段的文档么？

"launchInfo": {
"minMemory": 4096,
//"支持的Java版本，该选项非强制，但需要警告玩家",
"supportJava": [8, 9],
"launchArgument": ["", ""],
"javaArgument": ["", ""]
}

如果你说的是launchInfo的话（

这个是来自哪里的？

@tmdakm
Copy link
Author

tmdakm commented Aug 3, 2024

@ZhaiSoul 有关于这个字段的文档么？

"launchInfo": {
"minMemory": 4096,
//"支持的Java版本，该选项非强制，但需要警告玩家",
"supportJava": [8, 9],
"launchArgument": ["", ""],
"javaArgument": ["", ""]
}
如果你说的是launchInfo的话（

这个是来自哪里的？

某个开发群

@tmdakm tmdakm linked a pull request Aug 3, 2024 that will close this issue
Open
@tmdakm
Copy link
Author

tmdakm commented Aug 3, 2024

写了个简单的读取，应该能用？（

@tmdakm tmdakm changed the title 安装整合包时不会添加整合包内JVM参数 安装MCBBS整合包时不会添加整合包内JVM参数 Aug 3, 2024
@LTCatt LTCatt added 处理中 开发者正在调查或处理该项 🟩 低 优先度：低 and removed 第三方 等待第三方处理，或是由第三方导致 labels Aug 3, 2024
@LTCatt LTCatt added 🟨 中 优先度：中 可合并 已有 Pull Request 处理了该 Issue and removed 🟩 低 优先度：低 处理中 开发者正在调查或处理该项 🟨 中 优先度：中 labels Aug 30, 2024
@LTCatt
Copy link
Member

LTCatt commented Oct 15, 2024

@ZhaiSoul
帖主的示例整合包里，这个字段是这样的：

  "launchInfo": {
    "minMemory": 0,
    "launchArgument": [],
    "javaArgument": [
      "-XX:+ExplicitGCInvokesConcurrent"
    ]
  }

也就是说 javaArgument 应该追加在启动器默认的参数之后？
但这样的话，如果整合包需要删除/覆盖启动器默认的参数，就没法做到了吧……
有没有什么字段是控制 ”是覆盖还是添加“ 这一行为的？

@zkitefly
Copy link
Collaborator

但这样的话，如果整合包需要删除/覆盖启动器默认的参数，就没法做到了吧……

整合包真的需要这样做吗

可以做到替换默认参数的某个值，但覆盖还是不用的吧

@LTCatt
Copy link
Member

LTCatt commented Oct 19, 2024

也行吧……（

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
· 新功能 可合并 已有 Pull Request 处理了该 Issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

读取MCBBS整合包文件中的Jvm和Launch参数 tmdakm/PCL2
6 participants
@LTCatt @zkitefly @Stardito @tmdakm @Deep-Dark-Forest and others