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

Finish deprecation of functions in __init__ #16

Closed
wants to merge 1 commit into from
Closed

Finish deprecation of functions in __init__ #16

wants to merge 1 commit into from

Conversation

rm-you
Copy link

@rm-you rm-you commented Jan 12, 2016

Importing from twisted in init means twisted and pyopenssl are
required dependencies for core as well. Finish removing these methods to
remove the depencency on twisted and pyopenssl.

Removing deprecated functions:

  • certificateOptionsFromFiles
  • certificateOptionsFromPEMs

@hynek
Copy link
Owner

hynek commented Jan 13, 2016

Deprecation interval is 1 year. But yeah, I completely fucked this up. You could fix it by moving the imports into the compatibility functions I think?

Importing from twisted in __init__ means twisted and pyopenssl are
required dependencies for core as well. Fix the imports for these to
remove the depencency on twisted and pyopenssl.
@rm-you
Copy link
Author

rm-you commented Feb 3, 2016

Does this look better?

@hynek
Copy link
Owner

hynek commented Feb 4, 2016

No, I’m sorry but that’s not what I had in mind. Before I keep you busy with my confusing pipe dreams, I had a shot myself. Can you confirm that d097f44 fixed your problem?

@rm-you
Copy link
Author

rm-you commented Feb 5, 2016

Yeah, it appears to have fixed it. No worries, just wanted it to work, and now it does! Thanks! :)

@rm-you rm-you closed this Feb 5, 2016
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