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

stricter portable_filenames check #2

Open
wollmers opened this issue May 8, 2014 · 4 comments
Open

stricter portable_filenames check #2

wollmers opened this issue May 8, 2014 · 4 comments

Comments

@wollmers
Copy link

wollmers commented May 8, 2014

Some filesystems do not support case-sensitive names (e.g. OSX, Win). This means, they treat Foo.pm the same as foo.pm.

@charsbar
Copy link
Contributor

charsbar commented May 8, 2014

Basically true. Do you think if we need to check stricter, in other words, if we actually need to check 02packages/06permissions to see if there're distributions with conflicting name(s), and which is released earlier? (theoretically, it should be "yes", but practically, it may be a bit too much burden considering the cost/performance). It would be awesome If you know some concrete examples to convince me to implement this :)

@karenetheridge
Copy link

if we actually need to check 02packages/06permissions to see if there're distributions with conflicting name(s), and which is released earlier?

Given that PAUSE no longer indexes same-but-for-case module names, I think the answer is yes.

@wollmers
Copy link
Author

wollmers commented May 9, 2014

I mean it generally for all files in the distribution, e.g.

t/Foo.t
t/foo.t

or

share/img/foo.jpg
share/img/foo.JPG
share/IMG/foo.jpg

Play around with https://github.com/wollmers/John-Doe (just created and released by Dist::Milla to CPAN).

Here is what I get after unpacking the tar.gz

Linux:
helmutw@dev:~/github/test_john_doe/John-Doe-0.00/t$ ls -1
basic.t
john_doe.t
John_Doe.t
JOHN_DOE.t
release-pod-syntax.t

OSX:

helmut@Helmuts-iMac:~/github/John-Doe-0.00/t$ ls -1
basic.t
john_doe.t
release-pod-syntax.t

The other package/PAUSE/permission problem is a subset and will be catched too.

@karenetheridge knows more about this, but here a some links to the "ElasticSearch renaming" problem:

http://www.dagolden.com/index.php/2341/perl-qa-hackathon-2014-report/

http://www.elasticsearch.org/blog/renaming-perl-client/

http://rjbs.manxome.org/rubric/entry/2040

andk/pause#85

@wollmers wollmers closed this as completed May 9, 2014
@wollmers wollmers reopened this May 9, 2014
@wollmers
Copy link
Author

wollmers commented May 9, 2014

It's on CPAN now

https://metacpan.org/release/John-Doe

and I can install it on Mac with cpanm, but shows the same content for different modules.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants