www/nghttp2: Update to version 1.7.1

[brnrd]

Summary

Proposed commit log:

www/nghttp2: Update to version 1.7.1

  - Convert MASTER_SITES to USE_GITHUB
    - Remove USES=tar:xz
  - Remove unneeded RUN_DEPENDS
  - Add USES=autoreconf
  - Fix pkgconfig path
  - Remove defunct configure jemalloc fix
  - Fix LDFLAGS -ldl problem

Reviewed_by:	sunpoet (maintainer), koobs (mentor), feld (mentor)
Approved by:	(maintainer), (mentor)
Differential_Revision:	D5218
MFH:		2016Q1

Test Plan

• portlint -AC (clean)
• poudriere testport
• build mod_http2 1.2.4, run apache24 (OK, see comment)

[brnrd]

Apache error-log

[Sat Feb 06 13:07:05.454966 2016] [http2:info] [pid 17200] mod_http2 (v1.2.4, nghttp2 1.7.0), initializing...
[Sat Feb 06 13:07:05.581811 2016] [mpm_prefork:notice] [pid 17200] AH00163: Apache/2.4.18 (FreeBSD) LibreSSL/2.3.2 configured -- resuming normal operations

curl output

* SSL connection using TLSv1.2 / ECDHE-RSA-AES256-GCM-SHA384
* ALPN, server accepted to use h2

[koobs]

www/nghttp2/Makefile

21

The LIB_DEPEBDS above imply RUN time dependencies. I don't believe this is needed, nor am I sure its (:=) recommended any longer

68

USES=pathfix on Makefile.{in,am} instead

www/nghttp2/files/patch-configure.ac

2

Add patch header comment, breadcrumbs & references if applicable

[brnrd]

Use pathfix to fix .pc location, annotate patch

[brnrd]

www/nghttp2/Makefile

21

As this is not my port, that is for @sunpoet to decide.

68

Changed, works fine

[koobs]

Worst rationale ever, but OK. :]

[brnrd]

Remove RUN_DEPENDS, upstream libdl fix

[brnrd]

www/nghttp2/Makefile

20–22

Without RUN_DEPENDS, the package still reports correct dependencies.

deps: {
    libev: {
        origin: "devel/libev",
        version: "4.20,1"
    },
shlibs_required:
[
    "libev.so.4",
}

[koobs]

LGTM, well done @brnrd

[brnrd]

Update patches for 1.7.1

• Fixes CVE-2016-1544

[feld]

Approved, include the Security: in the commit log and Approved by: ports secteam (feld) in absence of the maintainer approval