Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cannot find module 'ws' from '/home/fabio/devel/require-emscripten/test' #1

Closed
fabiosantoscode opened this issue Mar 25, 2015 · 5 comments

Comments

@fabiosantoscode
Copy link
Owner

Emscripten's default output WILL try to require the excellent 'ws' module in some cases. However, browserify doesn't see the conditions outside it and tries to inline it all the time.

  • Proposed resolution: Figure out a way to fix this or patch upstream somehow.
  • Workaround: npm install ws and learn to live some unused code for now.

(I just rhymed)

@nfroidure
Copy link

Having the same issue while bundling emscripten modules with browserify. I used the --ignore-missing option to workaround https://github.com/substack/node-browserify#usage .

@fabiosantoscode
Copy link
Owner Author

Documented it in 32a55b3. Thank you so much for the pointer!

@nfroidure
Copy link

You're welcome, on my side, i borrowed you this snippet ;) https://github.com/nfroidure/ttf2woff2/blob/master/jssrc/post.js

@fabiosantoscode
Copy link
Owner Author

Soon it won't be necessary, I pushed it upstream into emscripten ;) emscripten-core/emscripten#3426

@nfroidure
Copy link

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants