Bug 232951 - www/nghttp2: Not broken with OpenSSL 1.1
Summary: www/nghttp2: Not broken with OpenSSL 1.1
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Po-Chuan Hsieh
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-11-04 10:33 UTC by Bernard Spil
Modified: 2018-11-06 21:17 UTC (History)
0 users

See Also:
bugzilla: maintainer-feedback? (sunpoet)


Attachments
svn diff for www/nghttp2 (481 bytes, patch)
2018-11-04 10:33 UTC, Bernard Spil
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Bernard Spil freebsd_committer freebsd_triage 2018-11-04 10:33:32 UTC
Created attachment 198929 [details]
svn diff for www/nghttp2

Hi,

Currently the www/nghttp2 port is marked broken with security/openssl-devel. The security/openssl-devel port was just deleted.

From the BROKEN_SSL_REASON I saw that this it was broken with very early versions of the openssl-devel port but that was fixed.
Additionally, looking at the nghttp2 code, I see that NPN support is optional. See e.g. https://github.com/nghttp2/nghttp2/blob/master/src/nghttp.cc#L1101
> #ifndef OPENSSL_NO_NEXTPROTONEG
>     SSL_get0_next_proto_negotiated(ssl, &next_proto, &next_proto_len);
> #endif // !OPENSSL_NO_NEXTPROTONEG

The relevant commit in nghttp is https://github.com/nghttp2/nghttp2/commit/400934e5a31da0bb756534deefbdc3046e603258
which I created myself I just found out when I dug a bit deeper...

You're probably not getting pkg-fallout messages for 12 or head for the port either, another indication that it builds OK with OpenSSL 1.1

Cheers, Bernard.
Comment 1 commit-hook freebsd_committer freebsd_triage 2018-11-06 21:13:18 UTC
A commit references this bug:

Author: sunpoet
Date: Tue Nov  6 21:12:32 UTC 2018
New revision: 484352
URL: https://svnweb.freebsd.org/changeset/ports/484352

Log:
  Remove outdated BROKEN_SSL

  PR:		232951
  Submitted by:	brnrd

Changes:
  head/www/nghttp2/Makefile
Comment 2 Po-Chuan Hsieh freebsd_committer freebsd_triage 2018-11-06 21:17:04 UTC
Committed. Thanks!