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

require('gaze') causes node to hang #98

Closed
floatdrop opened this issue Apr 17, 2014 · 3 comments
Closed

require('gaze') causes node to hang #98

floatdrop opened this issue Apr 17, 2014 · 3 comments
Labels
bug

Comments

@floatdrop
Copy link

Hi! @jh3y is stumbled in it in issue connected with gaze 0.6 - if you just require it, node will hang (due open timer or something stuck in event loop):

# This will never exit
node -e "require('gaze');"
@floatdrop floatdrop changed the title require('gaze') is hanging node require('gaze') causes node to hang Apr 17, 2014
@shama
Copy link
Owner

shama commented Apr 17, 2014

That is related to #40 where now gaze keeps the process alive (there is a setInterval but the native binding will keep it alive as well). If it's a problem, we could try adding an option to disable it.

floatdrop added a commit to floatdrop/gulp-watch that referenced this issue Apr 17, 2014
@floatdrop
Move gaze require into exported function
6453a43 
Due this issues:
 * shama/gaze#98
 * shama/gaze#40

Closes #25
@floatdrop
Copy link
Author

It is not huge problem, but it is unexpected to me - should requireing of gaze module do something, even thou constructor was not called in code below?

aslansky added a commit to aslansky/css-sprite that referenced this issue Apr 17, 2014
@aslansky
rollback gaze and vinyl-fs version because of shama/gaze#98
45ab4fd
@shama shama added the bug label Apr 17, 2014
@shama
Copy link
Owner

shama commented Apr 17, 2014

You're right, its a bug. It shouldn't keep the process alive until it is instantiated and should close the process when all instances are closed. Marking as a bug and will fix.

shama added a commit that referenced this issue Apr 19, 2014
@shama
Determine whether pathwatcher built without running it. Ref GH-98.
6ec2bff
@shama shama closed this as completed in faff6fb Apr 19, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@shama @floatdrop