-
Notifications
You must be signed in to change notification settings - Fork 17
feat(minecraft): 将实例下载迁移到 Core #75
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
base: main
Are you sure you want to change the base?
Conversation
8061eb3 to
38a9c9d
Compare
|
参考一下 #65?我个人希望实例下载(对了还是以实例称呼吧,改下标题?)是做成 HMCL patches 的形式,那样存储信息更具有扩展性,更完整。 题外话:这个 PCL.Core.sln 有必要传上来吗(?) |
具体的?
VSCode 不喜欢 slnx 应该( |
具体就是和 HMCL 完全一样的格式,你用 HMCL 下个实例看一下他的 JSON 就知道了 HMCL是 merge json 然后额外加入patches 字段的格式 |
8b5243a to
d7d7f8e
Compare
这个好说,因为写入 Json 和 ModLoader 安装都没做( 说到这个 @ruattd 考虑给 Downloader 支持多 Uri 下载? |
你自己用就好,不要传上来,这会让 msbuild 分不出默认项目的 |
0bf287d to
c745ddb
Compare
|
哦对想起来一个事情,用 patches 格式的话,读 libraries 等的各个参数应该都是读的 patches 列表里每项各自的 libraries 等(至少 HMCL 是这个逻辑) 但是因为有事先 merge 这个逻辑,所以正常你读外面的也一般不会有什么问题就是了 但是从整体架构来说的话,patchers 字段应该还是我们真正存信息的地方,外面只是保留兼容性,所以应该最好确实也还是读 patches 里面?你自己想一下? |
|
umm 这个有没有规定按什么顺序写入么
这个我之后再寻思下咋整好了 |
额,应该没有吧。反正都是平等的补丁,都一样吧,就像连原版 game 也在 patchers 里面) |
|
有顺序的,因为 Classpath 是有写入顺序的,不按照顺序来会爆炸 |
你这么说我想起来了,有 priority 这个参数啊,不用写入顺序 |
...那 patches 外面也得按照顺序写 |
那外面我就按 ModLoader -> Subassembly -> Game 这样的顺序写进去好了 Edit: 怎么 GitHub Ctrl+Enter 默认关 PR ( |
|
这个新配置系统.... 不会用,有无人知道怎么用的() |
PR ToDo
实现 Minecraft 的安装
实现附加组件安装
实现不依赖 ForgeInstallerBootstraper 的安装器
版本列表获取重构