Bug 197049

Summary: security/py-cryptography fix build failure with LibreSSL
Product: Ports & Packages Reporter: Bernard Spil <brnrd>
Component: Individual Port(s)Assignee: Kubilay Kocak <koobs>
Status: Closed FIXED    
Severity: Affects Some People Keywords: patch
Priority: --- Flags: koobs: maintainer-feedback+
Version: Latest   
Hardware: Any   
OS: Any   
Description Flags
svn diff against ports/head
Poudriere log of build
svn diff for security/py-cryptography
Poudriere build log of security/py-cryptography
svn diff for security/py-cryptography
Poudriere build log of security/py-cryptography none

Description Bernard Spil freebsd_committer 2015-01-24 13:58:57 UTC
Created attachment 152098 [details]
svn diff against ports/head

LibreSSL 2.1.3 has added ALPN support thereby breaking py-cryptography 0.7.2
This patch fixes that by making the definition of the ALPN features conditional on the existence of LIBRESSL_VERSION
Comment 1 Bugzilla Automation freebsd_committer 2015-01-24 13:58:57 UTC
Auto-assigned to maintainer koobs@FreeBSD.org
Comment 2 Kubilay Kocak freebsd_committer freebsd_triage 2015-01-27 04:50:33 UTC
Bernard, can you attach a successful poudriere build for this along with links to the upstream issue and/or pull request references please
Comment 3 Bernard Spil freebsd_committer 2015-01-27 19:36:52 UTC
Hi Kubilay, That was the dept. of misinformation... My apologies for your wasted time.
Attached patch is replacement of files/patch-src_cryptography_hazmat_bindings_openssl_ssl.py not an svndiff against port.

Upstream pull request https://github.com/pyca/cryptography/pull/1636
Comment 4 Bernard Spil freebsd_committer 2015-01-27 19:38:14 UTC
Created attachment 152271 [details]
Poudriere log of build
Comment 5 Kubilay Kocak freebsd_committer freebsd_triage 2015-02-11 09:56:17 UTC

The attached patch doesn't appear to match the upstream pull request changeset, which has not been merged yet. As such I'm hesitant to backport it (the upstream change).

Is there an upstream pull-request, issue or commit for the patch attached here? 

If so, please:

- Include the upstream issue, commit, or pr reference
- Update the attached patch with a patch against the port (not the cryptography sources)
Comment 6 Kubilay Kocak freebsd_committer freebsd_triage 2015-04-09 01:24:56 UTC
This issue is at risk of timing out on feedback
Comment 7 Bernard Spil freebsd_committer 2015-04-15 07:14:16 UTC
Created attachment 155602 [details]
svn diff for security/py-cryptography

Patch against current ports/head.

Port is missing ALPN check.
Comment 8 Bernard Spil freebsd_committer 2015-04-15 07:14:40 UTC
Created attachment 155603 [details]
Poudriere build log of security/py-cryptography
Comment 9 Bernard Spil freebsd_committer 2015-04-15 09:10:29 UTC
Current port contains most of the original fix but the ALPN support fix.

This last piece fixes the ALPN support by checking LIBRESSL_VERSION_NUMBER

Upstream in https://github.com/pyca/cryptography/pull/1849
Comment 10 Bernard Spil freebsd_committer 2015-04-17 15:41:57 UTC
Upstream imported this change
Comment 11 Bernard Spil freebsd_committer 2015-04-21 11:45:44 UTC
Created attachment 155798 [details]
svn diff for security/py-cryptography

The new version 0.8.2 adresses most issues building with LibreSSL. Most of the patches could therefor be removed, the only remaining patch has been merged upstream https://github.com/pyca/cryptography/pull/1849
Comment 12 Bernard Spil freebsd_committer 2015-04-21 11:46:12 UTC
Created attachment 155799 [details]
Poudriere build log of security/py-cryptography
Comment 13 commit-hook freebsd_committer 2015-04-24 13:37:18 UTC
A commit references this bug:

Author: koobs
Date: Fri Apr 24 13:36:30 UTC 2015
New revision: 384640
URL: https://svnweb.freebsd.org/changeset/ports/384640

  security/py-cryptography: Update to 0.8.2

  - 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]


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

  [1] https://github.com/pyca/cryptography/pull/1849

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

Comment 14 Kubilay Kocak freebsd_committer freebsd_triage 2015-04-24 13:38:17 UTC
Thanks Bernard!