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

fix(ModJava):错误将下载目录的 Java 当作特定路径 Java #5781

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

shimoranla
Copy link
Collaborator

现在会排除 PCL 下的 MyDownload 文件夹,防止用户误操作解压到此文件夹,进而因为特定路径优先选择的搜索逻辑导致游戏崩溃

Resolve #5780

@shimoranla shimoranla changed the title fix(ModJava):排除下载目录的 Java fix(ModJava):错误将下载目录的 Java 当作特定路径 Java Mar 4, 2025
@zkitefly zkitefly requested a review from LTCatt March 4, 2025 14:02
@zkitefly zkitefly added · Bug 等待确认 已经过社区确认,等待开发者确认 labels Mar 4, 2025
@SALTWOOD
Copy link
Contributor

SALTWOOD commented Mar 4, 2025

能修掉 #5780 的一半,但那个 issue 本身是因为逻辑错误导致的吧。
换句话说,把这个 Java 23 放到别的优先目录这个问题照样存在。

@LTCatt LTCatt added 处理中 开发者正在调查或处理该项 🟨 中 优先度:中 and removed 等待确认 已经过社区确认,等待开发者确认 labels Mar 4, 2025
@shimoranla
Copy link
Collaborator Author

能修掉 #5780 的一半,但那个 issue 本身是因为逻辑错误导致的吧。 换句话说,把这个 Java 23 放到别的优先目录这个问题照样存在。

龙猫在 #3547 的表态是

WAI,这个功能是用于给整合包作者以及高级用户来 强制指定 Java 的,所以会忽略兼容性检查。普通玩家也应当不会做出 “手动下一个 Java 放在优先目录” 这种事。

@biliyoyo520
Copy link

龙猫在 #3547 的表态是

普通玩家也应当不会做出 “手动下一个 Java 放在优先目录” 这种事。

主要还是PCL2作为下载器太方便了。
游戏开着摸鱼,顺便等其他东西下好,下好之后顺手就把项目jdk补上了。懒得去开idm、motrix什么的,毕竟mc占用内存太多了。
还是太好用了,少一步都不可能这样🐢

@allMagicNB
Copy link
Contributor

你怎么敢假定用户的下载路径一定是 MyDownload。(

@shimoranla
Copy link
Collaborator Author

你怎么敢假定用户的下载路径一定是 MyDownload。(

Constructive input is always welcome

如果你有更好的建议,请直接发出来,这比反问或者其他形式的话语更受欢迎

@biliyoyo520
Copy link

你怎么敢假定用户的下载路径一定是 MyDownload。(

如果你有更好的建议,请直接发出来,这比反问或者其他形式的话语更受欢迎

不引用PCL文件夹下面的深于二层文件夹里面的Java(?)
或者说不引用四层目录更深里面的java.exe(二层/zulu23.32.11-ca-jdk23.0.2-win_x64/bin/java.exe)

毕竟一般正常的整合包作者也不会把Java塞进这么深的地方,实在需要的话手动导入就行了吧

@shimoranla
Copy link
Collaborator Author

shimoranla commented Mar 10, 2025

不引用PCL文件夹下面的深于二层文件夹里面的Java(?) 或者说不引用四层目录更深里面的java.exe(二层/zulu23.32.11-ca-jdk23.0.2-win_x64/bin/java.exe)

毕竟一般正常的整合包作者也不会把Java塞进这么深的地方,实在需要的话手动导入就行了吧

PCL\MyDownload 这里已经两层了,再叠个 openjdk-xx\bin 这里已经四层了....

有些 Java 可能 openjdk-xx 和 bin 之间再叠个 jdk-xx,这里就五层了

还没考虑到可能嵌套文件夹分发,目录会更多层了

Edit:主要是自定义下载的源代码被扣掉了,不然也不会硬编码什么的

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
· Bug 处理中 开发者正在调查或处理该项 🟨 中 优先度:中
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PCL2"自动选择合适的Java"选项无法正常工作
6 participants