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

[建议]从源代码中移除“最新正式版.zip” #4043

Closed
2 tasks done
SALTWOOD opened this issue Jun 15, 2024 · 8 comments
Closed
2 tasks done

[建议]从源代码中移除“最新正式版.zip” #4043

SALTWOOD opened this issue Jun 15, 2024 · 8 comments
Labels
· 优化 拒绝 / 放弃 不会制作该功能,或无法修复该 Bug

Comments

@SALTWOOD
Copy link
Contributor

检查项

描述

大部分来这里的人都是为了交 issue 或者下源代码的吧
加一个“最新正式版.zip”有点多余(?
而且我没记错的话,git 版本管理系统似乎并不适合存储二进制文件
也不知道对不对,龙猫您自己定夺(

原因

似乎可以加快拉取仓库(?

@SALTWOOD SALTWOOD added · 优化 新提交 需要社区初步确认其有效的新提交 labels Jun 15, 2024
@JingHai-Lingyun JingHai-Lingyun added 等待确认 已经过社区确认,等待开发者确认 and removed 新提交 需要社区初步确认其有效的新提交 labels Jun 15, 2024
@allMagicNB
Copy link
Contributor

allMagicNB commented Jun 15, 2024

加快拉取仓库速度纯属无稽之谈,2 MB 能加快啥(

@SALTWOOD
Copy link
Contributor Author

SALTWOOD commented Jun 15, 2024

加快拉取仓库速度纯属无稽之谈,2 MB 能加快啥(

主要是没谁下罢,本着多余就可以砍的原则(
最主要是 git 不太适合二进制文件的存储

@Silverteal
Copy link
Contributor

Silverteal commented Jun 15, 2024

第一,Git不适合存储二进制这一观点不能绝对化。实际上只是提交对比差异的时候无法和纯文本一样逐行对比和快速合并变更。而且后果也只是不能发挥Git的优势,并不会使Git工作故障。
第二,目前除了这里之外没有一个可靠的地方存储历史正式版。
第三,这个文件在正式版更新的时候疑似会用到。

@wuliaodexiaoluo
Copy link
Contributor

wuliaodexiaoluo commented Jun 15, 2024

事实上,Release 的更新时请求的就是这个文件,龙猫扔这里面大概率是希望使用 GitHub Raw 来节约 COS 流量成本(国内 COS 提供商流量收的死贵)

至于 Git 不适合二进制 diff,其实并不影响的

而且 Git 拉取源代码只要可以连上 GitHub 速度就是只快不慢(当然,某些比较神奇的网络环境除外)

@3gf8jv4dv
Copy link
Collaborator

3gf8jv4dv commented Jun 15, 2024

事实上,Release 的更新时请求的就是这个文件,龙猫扔这里面大概率是希望使用 GitHub Raw 来节约 COS 流量成本(国内 COS 提供商流量收的死贵)

至于 Git 不适合二进制 diff,其实并不影响的

而且 Git 拉取源代码只要可以连上 GitHub 速度就是只快不慢(当然,某些比较神奇的网络环境除外)

One collaborator did not think so, or is it my misunderstanding? PCL-Community/PCL2-1930#59.

I will take a look at the code later.

Edit: I forgot that the code on GitHub does not include this feature (

@WorldHim
Copy link
Collaborator

不考虑其他因素下还是 release 更适合塞附件

@Silverteal
Copy link
Contributor

Silverteal commented Jun 15, 2024

不考虑其他因素下还是 release 更适合塞附件

说实话很有道理。无论是提供下载还是自动化更新都可以覆盖到。

不过这又回到那个问题了:

如果更改是中性的(没有明确地让功能更好,不过也没有更坏),则不采纳

我有一瞬间认为这句话应该成为所有新功能提案,pr 和 discussion 的最高指导方针

@LTCatt
Copy link
Member

LTCatt commented Jun 15, 2024

目前还没有移除它并且换到其他地方更新的必要……

@LTCatt LTCatt closed this as not planned Won't fix, can't repro, duplicate, stale Jun 15, 2024
@LTCatt LTCatt added 拒绝 / 放弃 不会制作该功能,或无法修复该 Bug and removed 等待确认 已经过社区确认,等待开发者确认 labels Jun 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
· 优化 拒绝 / 放弃 不会制作该功能,或无法修复该 Bug
Projects
None yet
Development

No branches or pull requests

8 participants