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

Chance fails on Windows with deep paths #80

Open
mitchless opened this issue Jan 25, 2012 · 2 comments
Open

Chance fails on Windows with deep paths #80

mitchless opened this issue Jan 25, 2012 · 2 comments

Comments

@mitchless
Copy link

Chance's temp file generation will fail when using deep paths on Windows. The NTFS path limit is 256 characters.

The affected file is vendor/chance/lib/chance/instance.rb, lines 448:455.

Is there a reason to preserve the full path, or can we hash that at least?

@mitchless
Copy link
Author

@ialexi Git lists you as having removed the md5; can I get your input on this? Can we hash the path and leave the file name?

@ialexi
Copy link
Contributor

ialexi commented Jan 25, 2012

Hm... I wonder if we could get a little more of the path in there. Perhaps we can use part of the MD5 + the relative path? I can't remember how the code works atm.

Could also switch to Sass importers, and then none of this would be an issue.

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

No branches or pull requests

2 participants