Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Fix BIO_* functions method linking when compiled with libressl (OpenBSD).#4043

Merged
ethomson merged 1 commit intolibgit2:masterlibgit2/libgit2:masterfrom
fudanchii:fudanchii/openbsdfudanchii/libgit2:fudanchii/openbsdCopy head branch name to clipboard
Dec 30, 2016
Merged

Fix BIO_* functions method linking when compiled with libressl (OpenBSD).#4043
ethomson merged 1 commit intolibgit2:masterlibgit2/libgit2:masterfrom
fudanchii:fudanchii/openbsdfudanchii/libgit2:fudanchii/openbsdCopy head branch name to clipboard

Conversation

@fudanchii
Copy link
Contributor

Hello,

I tried to build libgit2 in OpenBSD (as dependencies of rust's cargo) and found that it can't find BIO_* functions defined in openssl_stream.h. Obviously because it was guarded with openssl version check. I found a reference (below) that we can use libressl as we check openssl for version < 1.1.0. This patch does just that.

Thanks.

ref:
https://github.com/gentoo/libressl/blob/672ac74ce7b7cb2e4799b2d66bc0b1b1efa3454e/media-video/ffmpeg/files/ffmpeg-3.2-libressl.patch

@jacquesg
Copy link
Contributor

I've observed the same behaviour in the perl bindings:

http://www.cpantesters.org/cpan/report/dd6ea9f8-c7bb-11e6-8b9c-8683a7a0ff84

@jacquesg
Copy link
Contributor

Can we get this merged?

@ethomson
Copy link
Member

I don't use libressl, but if @jacquesg says it's working for him then 👍 . Thanks @fudanchii ! 😀

@ethomson ethomson merged commit 42ad85e into libgit2:master Dec 30, 2016
@jacquesg
Copy link
Contributor

👍

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.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.