-
Notifications
You must be signed in to change notification settings - Fork 32
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.
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
Upgrade sleep to 2.0 to fix yosemite compilation problem #16
base: master
Are you sure you want to change the base?
Conversation
@rultor try to merge |
@jimmys @yegor256 Oops, I failed. You can see the full log here (spent 12s)
|
@jimmys Thanks, let me find someone who can review this pull request |
@yegor256 review this pls (it's a rather short pull request) |
@rultor try to merge |
@jimmys @yegor256 Oops, I failed. You can see the full log here (spent 1min)
|
@jimmys there is something wrong with |
Looks like the rultor auto merge tool doesn't understand this project?
|
@jimmys that was a few months ago and it's fixed. The latest log is here: http://www.rultor.com/t/4637-148540699 |
I moved on from this software due to the installation issues, so I won't be able to help any more. |
This replaces the patch in yegor256#16, and works for me on El Capitan with nodejs 4.x+.
I've upgraded to sleep 2.0 to fix the problem I was having in #15
I also noted that other users have problems with the older version of sleep on Yosemite:
erikdubbelboer/node-sleep#26