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.

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

选择 Java 时参考版本 Json 文件的 javaVersion 字段 #5659

Open
3 tasks done
shimoranla opened this issue Feb 14, 2025 · 5 comments
Open
3 tasks done

选择 Java 时参考版本 Json 文件的 javaVersion 字段 #5659

shimoranla opened this issue Feb 14, 2025 · 5 comments
Labels
· 新功能 处理中 开发者正在调查或处理该项 🟥 高 优先度:高

Comments

@shimoranla
Copy link

检查项

  • 我已在 Issues 页面常见&难检反馈及问题列表 中搜索,确认了这一提案未被提交过。
  • 我已查看 功能投票页面,确认了这一提案未在投票列表中。
  • 我知晓还没做的新功能真的太多了,忙不过来,所以新功能提案几乎不会被处理,也不建议再提交新功能提案 qwq……

描述

Mojang 给的 Json 文件有这样一个字段(以 25w06a 为例)

"javaVersion": { "component": "java-runtime-delta", "majorVersion": 21 }

指定了最低版本,并且要下什么版本的 Java 都给你写好了

原因

这样至少 Mojang 换新版本 Java 时启动不大可能爆炸(

至少在源代码没有搜到这个字段,并且依稀记得 PCL 的处理方法似乎都是 hardcode 然后发更新?(直接节约一笔流量费.jpg)

@shimoranla shimoranla added · 新功能 新提交 需要社区初步确认其有效的新提交 labels Feb 14, 2025
@allMagicNB
Copy link
Contributor

allMagicNB commented Feb 14, 2025

整合包作者:☝🤓我不写你不就炸了,我 1.16.5 带个 Modern UI 你这么读你不也炸了。

@shimoranla shimoranla changed the title 自动补全 Java 时读取版本 Json 文件的 javaVersion 字段 选择 Java 时参考版本 Json 文件的 javaVersion 字段 Feb 14, 2025
@shimoranla
Copy link
Author

shimoranla commented Feb 14, 2025

整合包作者:☝🤓我不写你不就炸了,我 1.16.5 带个 Modern UI 你这么读你不也炸了。

?天上飞个锅可还行?

首先,我没说这个可以取代现有的选择逻辑,这实际上只是多了个参考来源,读不到完全可以继续使用硬编码的选择范围,相当于 Fallback
其次,我的表述是 指定了最低版本 而不是 指定了最高版本,这个结果可以作为最低兼容版本使用,这就可以规避 Modern UI 需要 Java 11-15 的问题
最后 PCL 启动 1.16.5 自动补全的也是 Java 8,理论上也会炸(实际不知道)

综上,我觉得这个 Issue 并没有大问题,至少解决掉了未来版本的启动问题

题外话

对于你在最近 Issues/Discussions 加的这两个神奇表情,还有这个评论的语气,我建议你重新读一遍这个

@allMagicNB
Copy link
Contributor

allMagicNB commented Feb 14, 2025

1.7.2 Forge 怎么说?

@allMagicNB
Copy link
Contributor

我并没有否认此 Issue,只是说明需要考虑部分特殊情况。所以硬编码的处理方式还是需要保留。

@wuliaodexiaoluo
Copy link
Contributor

wuliaodexiaoluo commented Feb 14, 2025

我并没有否认此 Issue,只是说明需要考虑部分特殊情况。所以硬编码的处理方式还是需要保留。

首先,我没说这个可以取代现有的选择逻辑,这实际上只是多了个参考来源,读不到完全可以继续使用硬编码的选择范围,相当于 Fallback

Edit:人家也没说把这东西移除掉,只是改善自动选择 Java 逻辑,又不是完全重写,也没说旧逻辑和新逻辑不能共存(((

我并没有否认此 Issue

Edit2:BTW:人家说的是语气问题,全文没有一个字提到否认不否认的

@3gf8jv4dv 3gf8jv4dv added 等待确认 已经过社区确认,等待开发者确认 and removed 新提交 需要社区初步确认其有效的新提交 labels Feb 14, 2025
@LTCatt LTCatt added 处理中 开发者正在调查或处理该项 🟥 高 优先度:高 and removed 等待确认 已经过社区确认,等待开发者确认 labels Feb 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
· 新功能 处理中 开发者正在调查或处理该项 🟥 高 优先度:高
Projects
None yet
Development

No branches or pull requests

5 participants