search

jdan / cleaver

30-second slideshows for hackers
http://jdan.github.io/cleaver
MIT License
4.03k stars 305 forks source link

Error when Debug not installed #86

Closed astalker closed 10 years ago

astalker commented 10 years ago

Hi,

Debug is currently a dev dependency but it is required to actually run Cleaver even when debug is not specified in the cmd line option. It might be best to set it as a required dependency so that it is always installed.

Cheers

jdan commented 10 years ago

From https://www.npmjs.org/doc/json.html

These things will be installed when doing npm link or npm install from the root of a package, and can be managed like any other npm configuration param. See npm-config(7) for more on the topic.

But you're right, that's not actually the case. I fixed it and pushed out a quick patch. Thanks for catching this!

jdan commented 10 years ago

Ah, I misread. So if you clone the package and run npm install then dev dependencies will, in fact, be installed. Normal installations won't yield the same result.