Skip to content
This repository has been archived by the owner on Jun 9, 2019. It is now read-only.

npm module requires 'lame' module, but it's not listed as a depndency #18

Open
thejsj opened this issue May 26, 2014 · 3 comments · May be fixed by #62
Open

npm module requires 'lame' module, but it's not listed as a depndency #18

thejsj opened this issue May 26, 2014 · 3 comments · May be fixed by #62

Comments

@thejsj
Copy link

thejsj commented May 26, 2014

When you try to require('timbre') it throws the following error.

Warning: Cannot find module 'lame' from '/Users/jorgesilva/Sites/2014/clickOnJorge/node_modules/timbre' Use --force to continue.

This seems to be because this package is not listed as a dependency in the package.json.

Should be as simple as:

npm install lame --save
@mohayonao
Copy link
Owner

'lame' support is optional.
if lame is not installed, timbre.js is expected to ignore it.

https://github.com/mohayonao/timbre.js/blob/master/timbre.node.js#L64

var lame = (function() {
    try { return require("lame"); } catch (e) {}
})();

umm.. please tell me the detail of the error message.

@fionnbharra
Copy link

I have the same problem.

Error: Cannot find module 'lame' from '/node_modules/timbre'

@data-doge
Copy link

data-doge commented Jul 10, 2016

if yall are getting this error when using timbre.js with browserify, you can use browserify's --ignore-missing flag to skip the unresolved requires for lame, ogg, and vorbis

http://stackoverflow.com/questions/26127804/browserify-building-with-dependencies-that-have-require-statements-in-try-catch

@data-doge data-doge linked a pull request Jul 10, 2016 that will close this issue
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants