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.

Sign up for GitHub

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

Jump to bottom

为什么将 zstd 的依赖导入从大写改成小写呢？ #10

Open
PanZ12580 opened this issue Sep 13, 2021 · 0 comments · May be fixed by #12
Open

为什么将 zstd 的依赖导入从大写改成小写呢？ #10

PanZ12580 opened this issue Sep 13, 2021 · 0 comments · May be fixed by #12

Comments

@PanZ12580
Copy link

PanZ12580 commented Sep 13, 2021 •
edited
Loading

go.mod 中 github.com/DataDog/zstd 这个依赖在 0.2.0 的时候还是大驼峰的 “DataDog”，到了 0.3.0 的时候就变成了全小写的 “github.com/datadog/zstd”，因为 github 源对大小写不敏感，所以在这个还是可以正常导入。不过看正确的路径应该是 github.com/DataDog/zstd，对方的 go.mod 文件中也指明了模块名。其他开源项目像 github.com/apache/pulsar-client-go 现在也是用的 “github.com/DataDog/zstd”，目前两个名字共存时会出现 “case-insensitive import collision” 的问题，希望可以尽快修复一下
dr-dime added a commit to dr-dime/tdmq-go-client that referenced this issue Jul 6, 2022
@dr-dime
s/datadog/DataDog/ to correctly reflect module name
305ba4b 
Fixes TencentCloud#10
@dr-dime dr-dime linked a pull request Jul 6, 2022 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Issue 10][go.mod] s/datadog/DataDog/g to correctly reflect module name dr-dime/tdmq-go-client
1 participant
@PanZ12580