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

cleanly added hpfeeds support #128

Open
wants to merge 18 commits into
base: master
Choose a base branch
from
Open

Conversation

gregcmartin
Copy link

This is compatible with hpfeeds out of the box and has no external dependencies like in the other PR

@desaster
Copy link
Owner

sorry for my huge delay with dealing with pull requests.

As it is now, the patch uses socket and blocking calls, which unfortunately cannot be used in a twisted application that relies on fully non-blocking operation.

http://twistedmatrix.com/documents/current/core/howto/clients.html

@desaster desaster mentioned this pull request Nov 21, 2014
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

6 participants