search

yeoman / doctor

Detect potential issues with users system that could prevent Yeoman from working correctly
http://yeoman.io
BSD 2-Clause "Simplified" License
61 stars 17 forks source link

Typo #15

Closed ondblclick closed 9 years ago

ondblclick commented 9 years ago

Some typos.

hemanth commented 9 years ago

Thanks. NONE_PATH sounds cool though :wink:

ondblclick commented 9 years ago

Found another issue actually... Please take a look:

[Info]
  NONE_PATH = /usr/lib/nodejs:/usr/lib/node_modules:/usr/share/javascript:/usr/lib/nodejs:/usr/lib/node_modules:/usr/share/javascript
  npm root  = /home/myusername/local/lib/node_modules

[Fix] Append the npm root value to your NODE_PATH variable
Add this line to your .bashrc
    export NODE_PATH=$NODE_PATH:/usr/lib/nodejs:/usr/lib/node_modules:/usr/share/javascript:/usr/lib/nodejs:/usr/lib/node_modules:/usr/share/javascript
  Or run this command
    echo "export NODE_PATH=$NODE_PATH:/usr/lib/nodejs:/usr/lib/node_modules:/usr/share/javascript:/usr/lib/nodejs:/usr/lib/node_modules:/usr/share/javascript" >> ~/.bashrc && source ~/.bashrc

As far as I understand doctor tries to suggest a solution - but the paths he suggests to add to NODE_PATH are not quite right - I've managed to fix my own problem by adding the npm root to NODE_PATH manually. I'm pretty sure paths and things on my machine is a big big mess, but doctor can have an issues as well.

hemanth commented 9 years ago

//cc @SBoudrias ^

SBoudrias commented 9 years ago

Fixed here: https://github.com/yeoman/doctor/commit/504b91a25f2af239a5c86ac66d29dfd479a6d7af

Next time please open a new issue so we can keep tracks of our bugs.

hemanth commented 9 years ago

:+1: