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

OpenSSL 1.1.x compilation fixes for "legacy" branch #615

Closed
wants to merge 2 commits into from

Conversation

kedazo
Copy link

@kedazo kedazo commented Apr 13, 2018

Hello there,

Yes I am aware about that this branch shouldn't be used anylonger, however i made some small mods which makes this compilable with OpenSSL 1.1.x.
These are very little mods, maybe someone is interested.

Br,
David

@acmorrow
Copy link
Contributor

Merging this would just encourage people to continue to use the legacy driver as they find their OpenSSL versions upgraded, so I don't think we should merge it. Still, I appreciate your mentioning it on the associated ticket, as that should get people here.

On the other hand, I'd rather not leave PRs open forever, and I'm not merging it because, as you note, the driver is EOL. So, I think it is best to close it, but I think you have left enough breadcrumbs that doing so won't remove its value. Let me know if you strongly disagree.

@acmorrow
Copy link
Contributor

Closing, driver is EOL.

@acmorrow acmorrow closed this Apr 24, 2018
@Climax777
Copy link

Climax777 commented Feb 22, 2019

@kedazo Thanks for this. Resurrected an old project and this came in handy

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants