remove VM-Dependency

[TimeForANinja]

Replace the VM-Dependency added in #1022 by switching to a Script like descriped in #1022 (comment)

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[fullerlee]

I'm keeping an eye on this one. I'm successfully using ytdl-core in reactnative by patching a few of the dependencies as shown here: https://github.com/fullerlee/react-native-ytdl

However, I've encountered the throttled downloads. Merging in the new sig.js results in a new dependency of vm, and I've not been able to sensibly patch that one.

If anyone has any ideas on how I might patch it, or replace the dynamic script creation, I'd be most grateful!

[TimeForANinja]

i mean it should be enough to replace new vm.Script(xxx) in sig.js with an eval(xxx) call

[fullerlee]

i mean it should be enough to replace new vm.Script(xxx) in sig.js with an eval(xxx) call

That's exactly what I've done to make it react-native compatible. Albeit by patching vm-browserify. It's a work-in-progress, but see https://github.com/fullerlee/react-native-ytdl/blob/master/lib/__REACT_NATIVE_YTDL_CUSTOM_MODULES__/vm/index.js