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

Macro to handle legacy OpenSSL SSLeay_add_all_algorithms() #5294

Closed
wants to merge 1 commit into from
Closed

Macro to handle legacy OpenSSL SSLeay_add_all_algorithms() #5294

wants to merge 1 commit into from

Conversation

Germano0
Copy link

@Germano0 Germano0 commented Nov 2, 2016

Hi, I am one of the Fedora Owncloud-Client maintainers.
Fedora Rawhide (26) ships OpenSSL 1.1.0 that no longer permits to call the old SSLeay_add_all_algorithms() that has been replaced with OpenSSL_add_all_algorithms() since OpenSSL version 0.9.5. So we started getting build failures
I solved by adding a macro that calls

Best regards
Germano Massullo

@CLAassistant
Copy link

CLAassistant commented Nov 2, 2016

CLA assistant check
All committers have signed the CLA.

@mention-bot
Copy link

@Germano0, thanks for your PR! By analyzing the history of the files in this pull request, we identified @nocteau and @danimo to be potential reviewers.

@Germano0 Germano0 added this to the 2.3.0 milestone Nov 4, 2016
@guruz
Copy link
Contributor

guruz commented Nov 4, 2016

Thanks for the PR.
After merging https://github.com/owncloud/client/pull/5289/files we would not need your PR anymore, sorry :-)

I'll keep it here still since we'd need to have it if we ever decide to do a 2.2.5 then we could cherry-pick your PR there.

@ogoffart
Copy link
Contributor

@guruz: maybe the patch that gets rid of p12topem.cpp could be merged now already?

@guruz guruz modified the milestones: 2.3.0, 2.2.5-maybe Nov 22, 2016
@guruz
Copy link
Contributor

guruz commented Mar 15, 2017

2.3.0 is released, thanks anyway :)

@guruz guruz closed this Mar 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants