trentm/node-bunyan

Remove/replace dependency on MV

nielm opened this issue · 3 comments

npm package MV has not been updated in 9 years, and depends on a deprecated package inflight which leaks memory:

Bunyan should find a replacement for mv.

> npm install --save
npm warn deprecated inflight@1.0.6: This module is not supported, and leaks memory. Do not use it. Check out lru-cache if you want a good and tested way to coalesce async requests by a key value, which is much more comprehensive and powerful.
> npm ls inflight
my-package
└─┬ bunyan@1.8.15
  └─┬ mv@2.1.1
    └─┬ rimraf@2.4.5
      └─┬ glob@6.0.4
        └── inflight@1.0.6

Workaround: add the following to package.json:

  "overrides": {
    "mv": "npm:dry-uninstall"
  },

which will prevent mv from being installed by using the dummy package dry-uninstall instead.

Mv is an optional module used for "rotating-file" stream type

Interesting workaround! Thanks for sharing.

I also found the same problem: renovatebot/renovate#29265, any chance you're coming from Renovate too?

Related/duplicate #667