EventEmitter memory leak

[ehmicky]

Steps to reproduce:

index.js:

import execa from 'execa'

(async () => {
  const childProcess = execa('echo', ['test'])
  await childProcess
  await childProcess
  await childProcess
  await childProcess
  await childProcess
})()

$ node index.js
(node:12764) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 close listeners added to [Socket]. Use emitter.setMaxListeners() to increase limit
(node:12764) MaxListenersExceededWarning: Possible EventEmitter memory leak detected. 11 close listeners added to [Socket]. Use emitter.setMaxListeners() to increase limit

Node v12.4.0
Ubuntu 19.10
Execa 2.0.0

This might not be an issue, but simply us setting more than 10 global listeners. But I would like to investigate this to make sure.

[ehmicky]

This seems to be a bug with pump, a dependency of get-stream that we use to retrieve streams with promises. Stream.pipeline() (available since Node 10.0.0) is a copy/paste of pump so I submitted a bug report directly to Node.

If this happened to a bug, I will submit a bug report to pump so they hopefully fix it (but the repository does not look actively maintained).

[ehmicky]

PR at #330.