search

slushjs / slush

The streaming scaffolding system - Gulp as a replacement for Yeoman
http://slushjs.github.io/generators
MIT License
1.24k stars 58 forks source link

Issues with OSX and case sensitive filesystem #17

Closed chadrien closed 9 years ago

chadrien commented 10 years ago

So I have a case sensitive filesystem, running OSX, using NVM and I can't use slush.

$ npm install -g slush slush-angular
$ slush angular
[slush] No generator by name: "angular" was found!
[slush] Try installing it with `npm install -g slush-angular` first.

I actually found the issue: bin/slush.js:L189 :: return path.toLowerCase() resulting in /users/chadrien/.nvm/v0.10.29/lib/node_modules instead of /Users/chadrien/.nvm/v0.10.29/lib/node_modules (notice the uppercase U)

So is the toLowerCase() intended? Initial commit is here: f396bd6af7b58087b57917c3cbce42df3a5e3df9 (ping @alexeyraspopov)

If not I'd be happy to fork/add test case/pull request :smiley:

joakimbeng commented 10 years ago

No, the toLowerCase() is not intended and shouldn't be necessary.

Go ahead! I'd be happy to accept your PR! :)