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

v0.0.9 and v0.0.10 contains a case-insensitive file name collision #2

Closed
dmitshur opened this issue Oct 25, 2019 · 1 comment
Closed
Assignees
Labels
bug Something isn't working

Comments

@dmitshur
Copy link

dmitshur commented Oct 25, 2019

Describe the bug
It's not possible to use v0.0.9 in module mode because it contains "tests/fileutils/FindMe.txt" and "tests/fileutils/findme.txt" files that cause a case-insensitive file name collision. The go command rejects it as a result.

To Reproduce
Steps to reproduce the behavior:

$ cd $(mktemp -d)
$ export GOPATH=$(pwd)
$ export GO111MODULE=on
$ export GOPROXY=direct
$ export GOSUMDB=off
$ go get -d github.com/alessiosavi/GoGPUtils@v0.0.9
go: finding github.com/alessiosavi/GoGPUtils v0.0.9
go: downloading github.com/alessiosavi/GoGPUtils v0.0.9
go: extracting github.com/alessiosavi/GoGPUtils v0.0.9
-> unzip /tmp/NPdspiwu/pkg/mod/cache/download/github.com/alessiosavi/!go!g!p!utils/@v/v0.0.9.zip: case-insensitive file name collision: "tests/fileutils/FindMe.txt" and "tests/fileutils/findme.txt"
go get github.com/alessiosavi/GoGPUtils@v0.0.9: unzip /tmp/NPdspiwu/pkg/mod/cache/download/github.com/alessiosavi/!go!g!p!utils/@v/v0.0.9.zip: case-insensitive file name collision: "tests/fileutils/FindMe.txt" and "tests/fileutils/findme.txt"

Expected behavior
It should work without an error.

Additional context
There is an open issue in the Go issue tracker about documenting the restrictions on contents that can be a part of modules. See golang/go#33778.

/cc @jayconrod FYI.

@alessiosavi
Copy link
Owner

alessiosavi commented Oct 26, 2019

Fixed in d1c4229.

Thank you so much @dmitshur for the effort during the analysis (: 👍

@alessiosavi alessiosavi changed the title v0.0.9 contains a case-insensitive file name collision v0.0.9 and v0.0.10 contains a case-insensitive file name collision Oct 26, 2019
@alessiosavi alessiosavi added the bug Something isn't working label Oct 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants