[pkgsrc/pkgsrc-2018Q4]: pkgsrc/www/nghttp2 Pullup ticket #5892 - requested by...

[bsiegert <bsiegert%pkgsrc.org@localhost>]

details:   https://anonhg.NetBSD.org/pkgsrc/rev/f750924ca38c
branches:  pkgsrc-2018Q4
changeset: 334048:f750924ca38c
user:      bsiegert <bsiegert%pkgsrc.org@localhost>
date:      Sat Jan 19 21:09:42 2019 +0000

description:
Pullup ticket #5892 - requested by gdt
www/nghttp2: build fix (don't require C++14 for the C library)

Revisions pulled up:
- www/nghttp2/Makefile          (patch)
- www/nghttp2/options.mk        (patch)

diffstat:

 www/nghttp2/Makefile   |  12 +++++++++---
 www/nghttp2/options.mk |  12 +++++++-----
 2 files changed, 16 insertions(+), 8 deletions(-)

diffs (61 lines):

diff -r be9d32c4c17e -r f750924ca38c www/nghttp2/Makefile
--- a/www/nghttp2/Makefile      Sat Jan 12 09:41:16 2019 +0000
+++ b/www/nghttp2/Makefile      Sat Jan 19 21:09:42 2019 +0000
@@ -1,7 +1,7 @@
-# $NetBSD: Makefile,v 1.46 2018/12/13 19:51:38 adam Exp $
+# $NetBSD: Makefile,v 1.46.2.1 2019/01/19 21:09:42 bsiegert Exp $
 
 DISTNAME=      nghttp2-1.35.1
-PKGREVISION=   1
+PKGREVISION=   2
 CATEGORIES=    www
 MASTER_SITES=  ${MASTER_SITE_GITHUB:=tatsuhiro-t/}
 EXTRACT_SUFX=  .tar.xz
@@ -12,7 +12,13 @@
 COMMENT=       Implementation of HTTP/2 in C
 LICENSE=       mit
 
-USE_LANGUAGES=         c c++14
+# Upstream documents that c++14 is required for C++ parts of the code;
+# the base package does not use C++.
+USE_LANGUAGES=         c
+# Upstream documents that gcc>=6 or clang >= 6 is required, but this
+# appears not to be true for the C parts.
+# https://github.com/nghttp2/nghttp2/issues/1293
+
 USE_LIBTOOL=           yes
 USE_TOOLS+=            gmake pkg-config
 GNU_CONFIGURE=         yes
diff -r be9d32c4c17e -r f750924ca38c www/nghttp2/options.mk
--- a/www/nghttp2/options.mk    Sat Jan 12 09:41:16 2019 +0000
+++ b/www/nghttp2/options.mk    Sat Jan 19 21:09:42 2019 +0000
@@ -1,4 +1,4 @@
-# $NetBSD: options.mk,v 1.5 2017/12/21 08:18:38 adam Exp $
+# $NetBSD: options.mk,v 1.5.10.1 2019/01/19 21:09:42 bsiegert Exp $
 
 PKG_OPTIONS_VAR=       PKG_OPTIONS.nghttp2
 PKG_SUPPORTED_OPTIONS= nghttp2-asio nghttp2-tools
@@ -12,8 +12,9 @@
 ### Build the ASIO C++ library
 ###
 .if !empty(PKG_OPTIONS:Mnghttp2-asio)
-# Requires emplace() for associative containers, introduced in gcc 4.8.0
-GCC_REQD+=             4.8
+# Upstream documents C++14 and gcc>=6 or clang>=6
+USE_LANGUAGES+=                c++14
+GCC_REQD+=             6
 CONFIGURE_ARGS+=       --enable-asio-lib
 CONFIGURE_ARGS+=       --with-boost=${BUILDLINK_PREFIX.boost-libs}
 CONFIGURE_ARGS+=       --with-boost-asio
@@ -30,8 +31,9 @@
 ### Build apps and tools
 ###
 .if !empty(PKG_OPTIONS:Mnghttp2-tools)
-# Requires emplace() for associative containers, introduced in gcc 4.8.0
-GCC_REQD+=             4.8
+# Upstream documents C++14 and gcc>=6 or clang>=6
+USE_LANGUAGES+=                c++14
+GCC_REQD+=             6
 CONFIGURE_ARGS+=       --enable-app
 CONFIGURE_ARGS+=       --enable-hpack-tools
 PLIST.tools=           yes