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

LibreSSL added ALPN support in 2.1.3 #1849

Merged
merged 1 commit into from
Apr 17, 2015
Merged

LibreSSL added ALPN support in 2.1.3 #1849

merged 1 commit into from
Apr 17, 2015

Conversation

Sp1l
Copy link
Contributor

@Sp1l Sp1l commented Apr 15, 2015

This fixes build issues on FreeBSD

This fixes build issues on FreeBSD
@alex
Copy link
Member

alex commented Apr 15, 2015

ok to test

@reaperhulk
Copy link
Member

@Sp1l could you give a bit more detail on this? Why does this fix a problem? LibreSSL declares both OPENSSL_VERSION_NUMBER and LIBRESSL_VERSION_NUMBER as 0x20000000L, so what's the scenario where OPENSSL_VERSION_NUMBER is < 0x10002001L but LIBRESSL_VERSION_NUMBER is defined?

@Sp1l
Copy link
Contributor Author

Sp1l commented Apr 16, 2015

@reaperhulk The maintainer of the port decided to modify the OPENSSL_VERSION_NUMBER to 0x1000107fL in the port to make more ports compile without errors https://svnweb.freebsd.org/ports/head/security/libressl/files/patch-include-openssl-opensslv.h?revision=381701&view=markup
This obviously comes at the cost of having to maintain checks for features that were added after 1.0.1g.

@reaperhulk
Copy link
Member

I'm extremely conflicted here. Having a distribution make a change like this is a very, very bad thing to do and I don't want to encourage it (even though this is almost certainly what LibreSSL should have done). I guess we'll merge this, but since we don't directly test it this will continue to be problematic for the FreeBSD port.

reaperhulk added a commit that referenced this pull request Apr 17, 2015
LibreSSL added ALPN support in 2.1.3
@reaperhulk reaperhulk merged commit 97a49a3 into pyca:master Apr 17, 2015
uqs pushed a commit to freebsd/freebsd-ports that referenced this pull request Apr 24, 2015
- Update to 0.8.2
- Update COMMENT (upstream)
- Pet portlint (TEST_DEPENDS must appear earlier)
- Remove patches that have been upstreamed
- Backport ALPN patch merged upstream, not yet released [1]

Changes:

  * https://github.com/pyca/cryptography/blob/0.8.2/CHANGELOG.rst

[1] pyca/cryptography#1849

PR:		197049
Submitted by:	Bernard Spil <spil.oss gmail com>


git-svn-id: svn+ssh://svn.freebsd.org/ports/head@384640 35697150-7ecd-e111-bb59-0022644237b5
koobs added a commit to freebsd/freebsd-ports that referenced this pull request Apr 24, 2015
- Update to 0.8.2
- Update COMMENT (upstream)
- Pet portlint (TEST_DEPENDS must appear earlier)
- Remove patches that have been upstreamed
- Backport ALPN patch merged upstream, not yet released [1]

Changes:

  * https://github.com/pyca/cryptography/blob/0.8.2/CHANGELOG.rst

[1] pyca/cryptography#1849

PR:		197049
Submitted by:	Bernard Spil <spil.oss gmail com>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants