search

pghalliday / node-BrowserStackTunnel

https://www.npmjs.com/package/browserstacktunnel-wrapper
MIT License
17 stars 24 forks source link

Node 6 graceful-fs warning #26

Closed dtothefp closed 6 years ago

dtothefp commented 8 years ago

looks like the version of unzip > fstream > graceful-fs is giving the dreaded Node 6 warning. 

fs: re-evaluating native module sources is not supported. If you are using the graceful-fs module, please update it to a more recent version.

I will put this issue on unzip as well but it looks like that package hasn't been updated in a while so I was hoping it could be handled here by subbing another more up to date module for unzip

screen shot 2016-06-01 at 11 45 14 pm
pghalliday commented 8 years ago

Will look into https://www.npmjs.com/package/yauzl

ChALkeR commented 8 years ago

Hi. browserstacktunnel-wrapper is currently #18 in my list of modules that would likely to be broken when https://github.com/nodejs/node/pull/6413 lands (which is anticipated to happen before v7.0).

Note that graceful-fs@1 and graceful-fs@4 are ok and would not be broken by this, but a require() on graceful-fs@2 or graceful-fs@3 would cause an error soon.

That is, runtime warning fs: re-evaluating native module sources is not supported. would be turned into a throw.

If you or your deps use graceful-fs@2 or graceful-fs@3 in runtime — you are affected.

Tracking: https://github.com/nodejs/node/issues/5213.