Open ghost opened 7 years ago
Same issue v2.4.0 gifify with giflossy. Ubuntu 16.04 based, node v6.9.4, npm 3.10.10
I think the error is due to: https://github.com/vvo/gifify/blob/master/index.js#L49
[ffmpeg, convert, gifsicle].forEach(function handleErrors(child) {
child.on('error', gifsicle.emit.bind(gifsicle, 'error'));
//...
});
The gifsicle
error event, would emit another gifsicle
error.
Any of you can try to fix this? I am not yet using node v8.
It's not v8 specific. Whenever the gifsicle object emits an error, you get an infinite loop of emitting more errors. You need to handle gifsicle's error events differently and it will fix this issue. What the error was that gifsicle emitted is a different issue.
Taking PRs to solve this.
I'll pass since I haven't looked into it enough to see how you want error handling to work. I just butchered it to find my underlying problem was #95, but it could be a different problem for @UNST4BL3
OS: Debian 8 x64 Node: v8.1.0 NPM: v5.0.3
Receiving this error:
With this code: