Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

process.EventEmitter is going to be removed in Node.js 7, use require('events') #9

Closed
ChALkeR opened this issue May 18, 2016 · 3 comments
Closed

process.EventEmitter is going to be removed in Node.js 7, use require('events') #9

ChALkeR opened this issue May 18, 2016 · 3 comments

Comments

@ChALkeR
Copy link

ChALkeR commented May 18, 2016

process.EventEmitter was deprecated in Node.js 6 and is going to be removed in Node.js 7, use require('events') as a drop-in replacement.

Refs: nodejs/node#5049, nodejs/node#6862.

Used on lib/server.js#L253 and in tests.
@jvilk
Copy link

jvilk commented Nov 7, 2016

+1. Please fix this. This library no longer functions in Node 7, which prevents other libraries -- like websockify -- from working.

@timkurvers timkurvers mentioned this issue Dec 18, 2016
Open
@mschaaf mschaaf mentioned this issue Mar 21, 2017
Open
@BrianLenzo
Copy link

This is now completely broken with Node 7.

3rd-Eden added a commit that referenced this issue Apr 12, 2017
@3rd-Eden
[fix] No longer use process.EventEmitter #9
02e9aa0
@3rd-Eden
Copy link
Owner

Published 0.0.6 with a fix, apologies for the inconvenience this caused.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@3rd-Eden @ChALkeR @jvilk @BrianLenzo