Module "vm" missing. #6

Activity

1. parshap commented on Mar 20, 2018

   

   parshap

   on Mar 20, 2018

   Owner

   More actions

   vm-browserify uses window.document and dom elements in its implementation, so it won't work in RN. I don't know of a RN-compatible polyfill for vm, but it may be possible to create one. It'd be great if you can contribute one! Or even a "mock" one that stubs out empty functions if that's all you need to get your library working.
2. kegesch commented on Mar 20, 2018

   

   kegesch

   on Mar 20, 2018

   Author

   More actions

   Okay, thank you for the explanation. A mock would propably enough for my usage, but i got it working with adding vm-browserify (i think it is not using the the function at all but just checks if the library is there?) with configuring the rn-cli.config.js like:

   'use strict';
   const nodelibs = require('node-libs-react-native');
   nodelibs.vm  = require.resolve('vm-browserify');
   
   module.exports = {
       extraNodeModules: nodelibs,
   };
   

   👍React with 👍6Reacted by Parsha Pourkhomami, Tim Wu, Todd Morrison, Wenhao Ho, Chris Feist and Iwashi1t
3. ka-sh commented on Jul 11, 2018

   

   ka-sh

   on Jul 11, 2018

   More actions

   Am having the same issue. @parshap what would be an acceptable solution? a mock or actually using vm-browserify ? if so will create a pull request to cover the VM case
4. parshap commented on Jul 12, 2018

   

   parshap

   on Jul 12, 2018

   Owner

   More actions

   I think a mock is the best solution unless you actually need sandboxing for security. Adding a vm mock to mock/ would be awesome.
5. toddpi314 commented on Nov 29, 2018

   

   toddpi314

   on Nov 29, 2018

   Last edited by toddpi314

   More actions

   note above, you might want to use the resolver syntax:

   'use strict';
   const nodelibs = require('node-libs-react-native');
   nodelibs.vm  = require.resolve('vm-browserify');
   
   module.exports = {
     resolver: {
       extraNodeModules: nodelibs
     },
   };

   I'm not sure how @Y0nnyy's original snippet would work without that..

   👍React with 👍4Reacted by ChazZeromus, Paul Cowgill, leeeverest and Petra
6. devesrawat commented on Dec 24, 2019

   

   devesrawat

   on Dec 24, 2019

   More actions

   Can't find variablebtoaafter adding metro.config.js
7. pcowgill commented on Dec 30, 2019

   

   pcowgill

   on Dec 30, 2019

   Contributor

   More actions

   note above, you might want to use the resolver syntax:

   Yep, and in addition to using resolver syntax, these days the file would be named metro.config.js rather than rn-cli.config.js