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

Introduction of case-sensitive test files (again) breaks Mac #11

Open
gkellogg opened this issue Mar 3, 2017 · 3 comments
Open

Introduction of case-sensitive test files (again) breaks Mac #11

gkellogg opened this issue Mar 3, 2017 · 3 comments
Assignees
Labels

Comments

@gkellogg
Copy link
Contributor

gkellogg commented Mar 3, 2017

The Mac filesystem is case-insensitive (although case-preserving for display). As has happened before, a couple of files differing only in case were added:

1literalPatterni_pass-lit-bC.val vs 1literalPatterni_pass-lit-BC.val

Is1_Ip1_LbC.ttl vs Is1_Ip1_LBC.ttl

We really need another paradigm here; the files are not duplicates, but contents differ in case. Perhaps 1literalPatterni_pass-lit-bC.val and 1literalPatterni_pass-lit-BC-uc.val. Better would be a Travis-CI test that looks for such files on checkin and fails.

Recovering on a Mac usually requires removing the repository, and re-creating it once it's been corrected, as doing nothing looks like a duplicate file, and removing it removes both files.

@gkellogg
Copy link
Contributor Author

gkellogg commented Mar 6, 2017

Seems to be fixed, it would be useful to have a guard against introducing the problem again in the future.

@gkellogg gkellogg closed this as completed Mar 6, 2017
@gkellogg
Copy link
Contributor Author

gkellogg commented Mar 6, 2017

Take it back, Is1_Ip1_LbC.ttl vs Is1_Ip1_LBC.ttl still is an issue.

@gkellogg gkellogg reopened this Mar 6, 2017
@gkellogg
Copy link
Contributor Author

gkellogg commented Mar 6, 2017

Remaining issues:

  • 1literalPatterni_pass-list-bC.val and BC.val
  • manifest.jsonld files need to be re-built (there's a makefile target for this)

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

No branches or pull requests

2 participants