Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Incorrect include_paths option format #24

Closed
Igorbek opened this Issue Jul 9, 2012 · 3 comments

Comments

Projects
None yet
2 participants
Contributor

Igorbek commented Jul 9, 2012

Now this parameter parsed via hardcoded specified separator character ':'.
This is not correct on some platforms, for ex. on Windows =)
I think, it would be better to specify custom separator or array of strings (better).

akhleung commented Jul 9, 2012

Sounds like a good idea. I'll make the change when I get a chance.

Contributor

Igorbek commented Jul 9, 2012

Thanks a lot!

Oy, sorry for taking forever on this. The separator will be ';' on Windows and ':' everywhere else. Using an arbitrary separator at runtime will take more effort than I can spare at the moment!

@akhleung akhleung closed this Nov 29, 2012

anlutro pushed a commit to alprs/libsass that referenced this issue Feb 2, 2015

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