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.

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

'required' signal n timer files,fixed namespace for calling global Signal class #9

Merged
merged 1 commit into from
Apr 30, 2015

Conversation

aninder
Copy link
Contributor

@aninder aninder commented Apr 29, 2015

i was playing with the watch_for_signal method , but was getting a 'signal:module cannot be initialised' , which later i realised was due the missing KQueue::Watcher::Signal constant in the constant lookup path , and due to which the new method on global Signal module was getting called
After requiring the files , and fixing the call Signal.list to ::Signal.list(list method which was getting called on Kqueue::Watcher::Signal) the wait_for_signal method is working alright.
i wonder if you have not required the files due to some reason?

@mat813
Copy link
Owner

mat813 commented Apr 29, 2015

Well, I only inherited the software so that I could keep it working after the original author went away, I don't really know why the files were not required, I've never used the signal bits.

mat813 added a commit that referenced this pull request Apr 30, 2015
'required' signal n timer files,fixed namespace for calling global Signal class
@mat813 mat813 merged commit 63b68e4 into mat813:master Apr 30, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants