Skip to content

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

Merged
LTCatt merged 4 commits intoMeloong-Git:mainfrom
LuoYun-Team:skip-download-folder
Apr 25, 2025
Merged

fix(ModJava):错误将下载目录的 Java 当作特定路径 Java#5781
LTCatt merged 4 commits intoMeloong-Git:mainfrom
LuoYun-Team:skip-download-folder

Conversation

@LinQingYuu
Copy link
Copy Markdown
Collaborator

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

Resolve #5780

@LinQingYuu LinQingYuu 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
Copy Markdown
Contributor

SALTWOOD commented Mar 4, 2025

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

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

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

龙猫在 #3547 的表态是

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

@biliyoyo520
Copy link
Copy Markdown

龙猫在 #3547 的表态是

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

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

@allMagicNB
Copy link
Copy Markdown
Contributor

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

@LinQingYuu
Copy link
Copy Markdown
Collaborator Author

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

Constructive input is always welcome

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

@biliyoyo520
Copy link
Copy Markdown

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

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

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

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

@LinQingYuu
Copy link
Copy Markdown
Collaborator Author

LinQingYuu 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:主要是自定义下载的源代码被扣掉了,不然也不会硬编码什么的

@LTCatt LTCatt merged commit 38b22c5 into Meloong-Git:main Apr 25, 2025
2 checks passed
@HexDragon-Bot HexDragon-Bot added 完成 已被处理,将在下次更新之后生效 and removed 处理中 开发者正在调查或处理该项 labels Apr 25, 2025
@LinQingYuu LinQingYuu deleted the skip-download-folder branch April 25, 2025 12:23
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"选项无法正常工作

7 participants