Skip to content

Loading…

Unbreak on Linux. #41

Merged
merged 1 commit into from

2 participants

@bernd

I'm getting the following exception under Linux.

/home/bernd/.rvm/gems/ruby-1.9.3-head/bundler/gems/tweetstream e414317ab775/lib/tweetstream/client.rb:307:in `kqueue=': kqueue is not supported on this platform (EventMachine::Unsupported)

EM.kqueue= can only be used on a system that actually supports kqueue. Calling EM.kqueue is just fine since it's a no-op on operating systems without kqueue support.

@stve
TweetStream member

I wasn't 100% sure if I had the syntax right. EM's documentation didn't make it clear but it makes sense that it would work like epoll. Thanks for the PR.

@stve stve merged commit 9d8cd7a into tweetstream:master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Sep 25, 2011
  1. @bernd

    Unbreak on Linux.

    bernd committed
Showing with 1 addition and 1 deletion.
  1. +1 −1 lib/tweetstream/client.rb
View
2 lib/tweetstream/client.rb
@@ -304,7 +304,7 @@ def start(path, query_parameters = {}, &block) #:nodoc:
}.merge(auth_params).merge(extra_stream_parameters)
EventMachine.epoll
- EventMachine.kqueue = EM.kqueue?
+ EventMachine.kqueue
EventMachine::run {
if @on_interval_proc.is_a?(Proc)
Something went wrong with that request. Please try again.