search

malditogeek / vmux

Secure P2P text, audio and video chats in your browser.
https://vmux.co
BSD 4-Clause "Original" or "Old" License
605 stars 67 forks source link

README assumes gulp installed #28

Closed rjmoggach closed 9 years ago

rjmoggach commented 10 years ago

maybe add the following somewhere so it's totally obvious:

sudo npm install -g gulp
malditogeek commented 10 years ago

Hey @mogga, gulp is listed in the dev dependencies (https://github.com/malditogeek/vmux/blob/master/package.json#L14). Once you cloned vmux if you run npm install it should install gulp and all its plugins. That didn't work for you?

AntouanK commented 9 years ago

@malditogeek That's not going to work if you want to run gulp in the terminal ( like you suggest in the README ). You must install it globally, and usually with sudo. package.json cannot do that.

malditogeek commented 9 years ago

Yeah, you're right. I have the current directory node_modules in my PATH, that's why it works. Try with:

export PATH="node_modules/.bin:$PATH"

I prefer not to install modules globally to avoid conflicts with other projects.

On Mon, Dec 1, 2014 at 5:16 PM, Antonis Karamitros <notifications@github.com

wrote:

@malditogeek https://github.com/malditogeek That's not going to work if you want to run gulp in the terminal ( like you suggest in the README ). You must install it globally, and usually with sudo. package.json cannot do that.

— Reply to this email directly or view it on GitHub https://github.com/malditogeek/vmux/issues/28#issuecomment-65099304.

AntouanK commented 9 years ago

That might be working, but you cannot expect everyone to do that just to use a project. And gulp instructions recommend anyway to install it globally. It's designed to work that way.