Reference Error: setImmediate is not defined

runtimejs / runtime

[not maintained] Lightweight JavaScript library operating system for the cloud

http://runtimejs.org

Apache License 2.0

1.93k stars 128 forks source link

Reference Error: setImmediate is not defined #63

Closed facekapow closed 9 years ago

facekapow commented 9 years ago

I think commit d26617b1cbe7ea3517e130eb8b5caf6740303382 broke TCP. I tried to use http.get (now that runtimeify supports runtime-net-node and http-node) and it gave the error in the title. I saw it in the commit but didn't see it get defined anywhere in the file. Pretty sure this should be fixed.

iefserge commented 9 years ago

@ArielAbreu hm, weird, it should be polyfilled here https://github.com/runtimejs/runtime/blob/master/js/core/polyfill.js#L18

facekapow commented 9 years ago

I must have missed that. I just copy pasted the updated tcp-socket.js to my node_modules/runtimejs. Hmm, should've git pull'ed and later copied the js folder. My fault! Sorry.