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

建议增加修改版本名前检查是否和NTFS 8.3 短文件名冲突 #4505

Closed
2 tasks done
AptS-1547 opened this issue Aug 15, 2024 · 5 comments · Fixed by #4506
Closed
2 tasks done

建议增加修改版本名前检查是否和NTFS 8.3 短文件名冲突 #4505

AptS-1547 opened this issue Aug 15, 2024 · 5 comments · Fixed by #4506
Labels
· 优化 完成 已被处理,将在下次更新之后生效

Comments

@AptS-1547
Copy link

AptS-1547 commented Aug 15, 2024

检查项

描述

#4504
历史遗留问题 NTFS 8.3 短文件名 可能会导致在修改版本名的时候出现报错
建议增加新的检查项目防止出现这类问题

原因

有可能让玩家不知所措
可能会让我这样不懂的人开个bug issue(

@AptS-1547 AptS-1547 added · 优化 新提交 需要社区初步确认其有效的新提交 labels Aug 15, 2024
@SALTWOOD
Copy link
Contributor

SALTWOOD commented Aug 15, 2024

引自维基百科

长文件名只会保留基本名称的首6位半角字符,以一个~号连接着,再以一个数字作结尾以作识别

当玩家尝试使用类似 abcdef~1 这样的格式命名(原 issue 为两个中文字符、波浪号、数字的组合)版本时,会因为 Windows 的 NTFS 8.3 文件名导致错误。
因此,本人认为,有必要增加一个检测,防止出现此类意外情况。

@SALTWOOD
Copy link
Contributor

SALTWOOD commented Aug 15, 2024

顺带在此询问一下:是直接选择屏蔽波浪号还是选择检测 NTFS 8.3 文件名?

@AptS-1547
Copy link
Author

看各位大大哪个开发容易吧……幸苦了
我个人认为屏蔽波浪号相比于检测 NTFS 8.3 文件名 会不太好一点,但开发会更加麻烦(低头)
幸苦各位大大了

@JingHai-Lingyun JingHai-Lingyun added 等待确认 已经过社区确认,等待开发者确认 and removed 新提交 需要社区初步确认其有效的新提交 labels Aug 15, 2024
SALTWOOD added a commit to SALTWOOD/PCL2 that referenced this issue Aug 15, 2024
@LTCatt LTCatt added 处理中 开发者正在调查或处理该项 🟩 低 优先度:低 and removed 等待确认 已经过社区确认,等待开发者确认 labels Aug 15, 2024
@genshincoder
Copy link

个人建议:直接屏蔽~加数字的组合

@SALTWOOD
Copy link
Contributor

个人建议:直接屏蔽~加数字的组合

目前版本已经能够很好的防止此类情况的发生,没必要再改成这样

@LTCatt LTCatt added 可合并 已有 Pull Request 处理了该 Issue and removed 处理中 开发者正在调查或处理该项 🟩 低 优先度:低 labels Aug 30, 2024
@LTCatt LTCatt closed this as completed Oct 12, 2024
@HexDragon-Bot HexDragon-Bot added 完成 已被处理,将在下次更新之后生效 and removed 可合并 已有 Pull Request 处理了该 Issue labels Oct 12, 2024
@LTCatt LTCatt added 可合并 已有 Pull Request 处理了该 Issue and removed 完成 已被处理,将在下次更新之后生效 labels Oct 12, 2024
@HexDragon-Bot HexDragon-Bot reopened this Oct 12, 2024
HexDragon-Bot added a commit that referenced this issue Oct 12, 2024
通过检测 NTFS 8.3 文件名修复 #4505
@HexDragon-Bot HexDragon-Bot reopened this Oct 12, 2024
@LTCatt LTCatt closed this as completed Oct 12, 2024
@HexDragon-Bot HexDragon-Bot added 完成 已被处理,将在下次更新之后生效 and removed 可合并 已有 Pull Request 处理了该 Issue labels Oct 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
· 优化 完成 已被处理,将在下次更新之后生效
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants