Re: git: 6ab07aa32fc9 - main - www/nghttp2: Change back to make

From: Daniel Engberg <diizzy_at_FreeBSD.org>
Date: Sun, 12 Feb 2023 21:11:51 UTC

Thanks,

Can we also do the same for the library?

https://cgit.freebsd.org/ports/commit/www/libnghttp2?id=3bbd8494cddb03fe841ad150a4e8da7737e9d36b

Best regards,

Daniel

On 2023-02-12 20:15, Po-Chuan Hsieh wrote:

> It is using cmake now.
> 
> On Fri, Feb 10, 2023 at 4:52 AM Daniel Engberg <diizzy@freebsd.org> 
> wrote:
> 
> Hi,
> 
> Sounds fair, thanks for looking into it.
> 
> Best regards,
> 
> Daniel
> 
> On 2023-02-07 20:37, Po-Chuan Hsieh wrote:
> 
> Hello,
> 
> I want to emphasize again that I'd like to follow the official build 
> process in most cases to avoid unexpected problems.
> Since it's supported by upstream, I'm OK to convert this port to cmake.
> But I need some time to read the cmake files and do it by myself 
> (better for future maintenance) with your version as a reference.
> 
> Regards,
> sunpoet
> 
> On Mon, Feb 6, 2023 at 3:13 AM Daniel Engberg <diizzy@freebsd.org> 
> wrote:
> 
> CMake is supported by upstream (see also submitted PRs and issue 
> reports) and is included in CI
> 
> https://github.com/nghttp2/nghttp2/blob/master/.github/workflows/build.yml
> 
> Many upstream projects lags when it comes to documentation 
> unfortunately
> 
> Best regards,
> 
> Daniel
> 
> On 2023-02-05 15:17, Po-Chuan Hsieh wrote:
> 
> Hello,
> 
> I'm following the official build process [1].
> 
> [1] https://github.com/nghttp2/nghttp2/blob/master/README.rst
> 
> Regards,
> sunpoet
> 
> On Wed, Feb 1, 2023 at 2:28 AM Tobias C. Berner <tcberner@gmail.com> 
> wrote: Moin moin
> 
> What's the reasoning behind this change?
> 
> mfg Tobias
> 
> On Mon, 30 Jan 2023 at 16:23, Daniel Engberg <diizzy@freebsd.org> 
> wrote:
>> 
>> On 2023-01-30 14:06, Po-Chuan Hsieh wrote:
>>> The branch main has been updated by sunpoet:
>>> 
>>> URL:
>>> https://cgit.FreeBSD.org/ports/commit/?id=6ab07aa32fc9b73564a66f81feef636225bdd941
>>> 
>>> commit 6ab07aa32fc9b73564a66f81feef636225bdd941
>>> Author:     Po-Chuan Hsieh <sunpoet@FreeBSD.org>
>>> AuthorDate: 2023-01-30 12:28:54 +0000
>>> Commit:     Po-Chuan Hsieh <sunpoet@FreeBSD.org>
>>> CommitDate: 2023-01-30 12:59:12 +0000
>>> 
>>>     www/nghttp2: Change back to make
>>> 
>>>     - Bump PORTREVISION for package change
>>> ---
>>>  www/nghttp2/Makefile                               | 46 +++++++-----
>>>  www/nghttp2/files/patch-CMakeLists.txt             | 11 ---
>>>  www/nghttp2/files/patch-Makefile.in                | 11 +++
>>>  www/nghttp2/files/patch-examples-Makefile.in       | 66
>>> +++++++++++++++++
>>>  www/nghttp2/files/patch-script-fetch-ocsp-response |  7 ++
>>>  www/nghttp2/files/patch-src-Makefile.in            | 83
>>> ++++++++++++++++++++++
>>>  www/nghttp2/files/patch-tests-Makefile.in          | 31 ++++++++
>>>  www/nghttp2/pkg-plist                              |  4 ++
>>>  8 files changed, 231 insertions(+), 28 deletions(-)
>>> 
>>> diff --git a/www/nghttp2/Makefile b/www/nghttp2/Makefile
>>> index 8ef93d6c83f7..e6a38583cd3d 100644
>>> --- a/www/nghttp2/Makefile
>>> +++ b/www/nghttp2/Makefile
>>> @@ -1,7 +1,7 @@
>>>  PORTNAME=    nghttp2
>>> -DISTVERSION= 1.51.0
>>> +PORTVERSION= 1.51.0
>>>  CATEGORIES=  www net
>>> -MASTER_SITES=        
>>> https://github.com/${PORTNAME}/${PORTNAME}/releases/download/v${DISTVERSION}/ 
>>> [1]
>>> +MASTER_SITES=        
>>> https://github.com/nghttp2/nghttp2/releases/download/v${PORTVERSION}/ 
>>> [2]
>>> 
>>>  MAINTAINER=  sunpoet@FreeBSD.org
>>>  COMMENT=     HTTP/2.0 C Library
>>> @@ -10,32 +10,44 @@ WWW=              https://nghttp2.org/
>>>  LICENSE=     MIT
>>>  LICENSE_FILE=        ${WRKSRC}/COPYING
>>> 
>>> -BUILD_DEPENDS=       libnghttp2>=${DISTVERSION}:www/libnghttp2
>>> +BUILD_DEPENDS=       libnghttp2>=${PORTVERSION}:www/libnghttp2
>>>  LIB_DEPENDS= libnghttp2.so:www/libnghttp2
>>> 
>>> -USES=                cmake compiler:c++14-lang cpe localbase:ldflags 
>>> pathfix \
>>> -             pkgconfig python:env shebangfix ssl tar:xz
>>> -USE_RC_SUBR= nghttpx
>>> -SHEBANG_FILES=       script/fetch-ocsp-response
>>> +USES=                compiler:c++14-lang cpe gmake libtool localbase 
>>> pathfix
>>> pkgconfig ssl tar:xz
>>> 
>>> -CMAKE_OFF=   ENABLE_PYTHON_BINDINGS ENABLE_HTTP3
>>> -CMAKE_ARGS=  -DCMAKE_INSTALL_MANDIR:PATH=man
>>> +CONFIGURE_ARGS=      --disable-python-bindings --with-jemalloc
>>> +CONFIGURE_ENV=       OPENSSL_CFLAGS="-I${OPENSSLINC}" \
>>> +             OPENSSL_LIBS="-L${OPENSSLLIB} -lcrypto -lssl"
>>> +GNU_CONFIGURE=       yes
>>> +INSTALL_TARGET=      install-strip
>>> +USE_CXXSTD=  c++14
>>> +USE_RC_SUBR= nghttpx
>>> 
>>>  OPTIONS_DEFINE=      APP DOCS EXAMPLES HPACK
>>>  OPTIONS_DEFAULT=APP HPACK
>>>  OPTIONS_SUB= yes
>>> -
>>>  APP_DESC=    Build h2load, nghttp, nghttpd and nghttpx
>>> 
>>> +APP_BUILD_DEPENDS=   c-ares>=1.7.5:dns/c-ares \
>>> +                     libev>=4.11:devel/libev
>>> +APP_CONFIGURE_ENABLE=        app
>>> +APP_CONFIGURE_OFF=   --without-libxml2
>>>  APP_LIB_DEPENDS=     libcares.so:dns/c-ares \
>>>                       libev.so:devel/libev
>>> -APP_USES=            gnome
>>>  APP_USE=             GNOME=libxml2
>>> -APP_CMAKE_BOOL=              ENABLE_APP WITH_LIBXML2
>>> -HPACK_LIB_DEPENDS=   libjansson.so:devel/jansson
>>> -HPACK_CMAKE_BOOL=    ENABLE_HPACK_TOOLS
>>> +APP_USES=            gnome
>>> +EXAMPLES_CONFIGURE_ENABLE=   examples
>>> +EXAMPLES_BUILD_DEPENDS=      libevent>=2.0.8:devel/libevent
>>>  EXAMPLES_LIB_DEPENDS=        libevent_openssl.so:devel/libevent
>>> -EXAMPLES_CMAKE_BOOL= ENABLE_EXAMPLES
>>> +HPACK_BUILD_DEPENDS= jansson>=2.5:devel/jansson
>>> +HPACK_CONFIGURE_ENABLE=      hpack-tools
>>> +HPACK_LIB_DEPENDS=   libjansson.so:devel/jansson
>>> +
>>> +# lib/libnghttp2.* are not available until configure phase
>>> +post-configure:
>>> +     @${MKDIR} ${WRKSRC}/lib/.libs/
>>> +     @${CP} ${LOCALBASE}/lib/libnghttp2.so* ${WRKSRC}/lib/.libs/
>>> +     @cd ${WRKSRC}/lib/.libs/ && ${AR} x 
>>> ${LOCALBASE}/lib/libnghttp2.a
>>> 
>>>  post-install:
>>>       ${MKDIR} ${STAGEDIR}${PREFIX}/etc/nghttpx/
>>> @@ -43,7 +55,7 @@ post-install:
>>> 
>>>  post-install-EXAMPLES-on:
>>>       ${MKDIR} ${STAGEDIR}${EXAMPLESDIR}/
>>> -     cd ${WRKSRC}/examples/ && ${INSTALL_DATA} client.c deflate.c
>>> libevent-client.c libevent-server.c ${STAGEDIR}${EXAMPLESDIR}/
>>> -     cd ${INSTALL_WRKSRC}/examples/ && ${INSTALL_PROGRAM} client 
>>> deflate
>>> libevent-client libevent-server ${STAGEDIR}${EXAMPLESDIR}/
>>> +     ${INSTALL_DATA} ${WRKSRC}/examples/*.c* 
>>> ${STAGEDIR}${EXAMPLESDIR}/
>>> +     cd ${WRKSRC}/examples/ && ${INSTALL_PROGRAM} client deflate
>>> libevent-client libevent-server ${STAGEDIR}${EXAMPLESDIR}/
>>> 
>>>  .include <bsd.port.mk [3]>
>>> diff --git a/www/nghttp2/files/patch-CMakeLists.txt
>>> b/www/nghttp2/files/patch-CMakeLists.txt
>>> deleted file mode 100644
>>> index e8cf364abae5..000000000000
>>> --- a/www/nghttp2/files/patch-CMakeLists.txt
>>> +++ /dev/null
>>> @@ -1,11 +0,0 @@
>>> ---- CMakeLists.txt.orig      2022-12-24 08:52:48 UTC
>>> -+++ CMakeLists.txt
>>> -@@ -505,7 +505,7 @@ set(PKGLIBDIR
>>> "${CMAKE_INSTALL_FULL_LIBDIR}/${CMAKE_PR
>>> -
>>> - install(FILES README.rst DESTINATION "${CMAKE_INSTALL_DOCDIR}")
>>> -
>>> --add_subdirectory(lib)
>>> -+#add_subdirectory(lib)
>>> - #add_subdirectory(lib/includes)
>>> - add_subdirectory(third-party)
>>> - add_subdirectory(src)
>>> diff --git a/www/nghttp2/files/patch-Makefile.in
>>> b/www/nghttp2/files/patch-Makefile.in
>>> new file mode 100644
>>> index 000000000000..4bc7bac0f224
>>> --- /dev/null
>>> +++ b/www/nghttp2/files/patch-Makefile.in
>>> @@ -0,0 +1,11 @@
>>> +--- Makefile.in.orig 2022-11-13 06:53:28 UTC
>>> ++++ Makefile.in
>>> +@@ -465,7 +465,7 @@ top_srcdir = @top_srcdir@
>>> + # LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN
>>> ACTION
>>> + # OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
>>> CONNECTION
>>> + # WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
>>> +-SUBDIRS = lib third-party src bpf examples python tests
>>> integration-tests \
>>> ++SUBDIRS = third-party src bpf examples python tests 
>>> integration-tests
>>> \
>>> +     doc contrib script
>>> +
>>> +
>>> diff --git a/www/nghttp2/files/patch-examples-Makefile.in
>>> b/www/nghttp2/files/patch-examples-Makefile.in
>>> new file mode 100644
>>> index 000000000000..5bfb48b8f134
>>> --- /dev/null
>>> +++ b/www/nghttp2/files/patch-examples-Makefile.in
>>> @@ -0,0 +1,66 @@
>>> +% sed -i .orig 's|$[({]top_builddir[})]/lib/libnghttp2.la 
>>> [4]|-lnghttp2|'
>>> */Makefile.in
>>> +
>>> +--- examples/Makefile.in.orig        2022-11-13 06:53:28 UTC
>>> ++++ examples/Makefile.in
>>> +@@ -146,7 +146,7 @@ am__asio_cl_SOURCES_DIST = asio-cl.cc
>>> + @ENABLE_ASIO_LIB_TRUE@@ENABLE_EXAMPLES_TRUE@am_asio_cl_OBJECTS =
>>> asio_cl-asio-cl.$(OBJEXT)
>>> + asio_cl_OBJECTS = $(am_asio_cl_OBJECTS)
>>> + am__DEPENDENCIES_1 =
>>> +-@ENABLE_ASIO_LIB_TRUE@@ENABLE_EXAMPLES_TRUE@am__DEPENDENCIES_2 =
>>> $(top_builddir)/lib/libnghttp2.la [4] \
>>> ++@ENABLE_ASIO_LIB_TRUE@@ENABLE_EXAMPLES_TRUE@am__DEPENDENCIES_2 =
>>> -lnghttp2 \
>>> +
>>> @ENABLE_ASIO_LIB_TRUE@@ENABLE_EXAMPLES_TRUE@  
>>> $(top_builddir)/src/libnghttp2_asio.la [5]
>>> \
>>> +
>>> @ENABLE_ASIO_LIB_TRUE@@ENABLE_EXAMPLES_TRUE@  
>>> $(top_builddir)/third-party/liburl-parser.la [6]
>>> \
>>> + @ENABLE_ASIO_LIB_TRUE@@ENABLE_EXAMPLES_TRUE@        
>>> $(am__DEPENDENCIES_1) \
>>> +@@ -175,14 +175,14 @@ am__client_SOURCES_DIST = client.c
>>> + client_OBJECTS = $(am_client_OBJECTS)
>>> + client_LDADD = $(LDADD)
>>> + @ENABLE_EXAMPLES_TRUE@client_DEPENDENCIES =  \
>>> +-@ENABLE_EXAMPLES_TRUE@      $(top_builddir)/lib/libnghttp2.la [4] \
>>> ++@ENABLE_EXAMPLES_TRUE@      -lnghttp2 \
>>> + @ENABLE_EXAMPLES_TRUE@      
>>> $(top_builddir)/third-party/liburl-parser.la [6]
>>> + am__deflate_SOURCES_DIST = deflate.c
>>> + @ENABLE_EXAMPLES_TRUE@am_deflate_OBJECTS = deflate.$(OBJEXT)
>>> + deflate_OBJECTS = $(am_deflate_OBJECTS)
>>> + deflate_LDADD = $(LDADD)
>>> + @ENABLE_EXAMPLES_TRUE@deflate_DEPENDENCIES =  \
>>> +-@ENABLE_EXAMPLES_TRUE@      $(top_builddir)/lib/libnghttp2.la [4] \
>>> ++@ENABLE_EXAMPLES_TRUE@      -lnghttp2 \
>>> + @ENABLE_EXAMPLES_TRUE@      
>>> $(top_builddir)/third-party/liburl-parser.la [6]
>>> + am__libevent_client_SOURCES_DIST = libevent-client.c
>>> + @ENABLE_EXAMPLES_TRUE@am_libevent_client_OBJECTS =  \
>>> +@@ -190,7 +190,7 @@ am__libevent_client_SOURCES_DIST =
>>> libevent-client.c
>>> + libevent_client_OBJECTS = $(am_libevent_client_OBJECTS)
>>> + libevent_client_LDADD = $(LDADD)
>>> + @ENABLE_EXAMPLES_TRUE@libevent_client_DEPENDENCIES =  \
>>> +-@ENABLE_EXAMPLES_TRUE@      $(top_builddir)/lib/libnghttp2.la [4] \
>>> ++@ENABLE_EXAMPLES_TRUE@      -lnghttp2 \
>>> + @ENABLE_EXAMPLES_TRUE@      
>>> $(top_builddir)/third-party/liburl-parser.la [6]
>>> + am__libevent_server_SOURCES_DIST = libevent-server.c
>>> + @ENABLE_EXAMPLES_TRUE@am_libevent_server_OBJECTS =  \
>>> +@@ -198,7 +198,7 @@ am__libevent_server_SOURCES_DIST =
>>> libevent-server.c
>>> + libevent_server_OBJECTS = $(am_libevent_server_OBJECTS)
>>> + libevent_server_LDADD = $(LDADD)
>>> + @ENABLE_EXAMPLES_TRUE@libevent_server_DEPENDENCIES =  \
>>> +-@ENABLE_EXAMPLES_TRUE@      $(top_builddir)/lib/libnghttp2.la [4] \
>>> ++@ENABLE_EXAMPLES_TRUE@      -lnghttp2 \
>>> + @ENABLE_EXAMPLES_TRUE@      
>>> $(top_builddir)/third-party/liburl-parser.la [6]
>>> + AM_V_P = $(am__v_P_@AM_V@)
>>> + am__v_P_ = $(am__v_P_@AM_DEFAULT_V@)
>>> +@@ -504,7 +504,7 @@ EXTRA_DIST = CMakeLists.txt
>>> + @ENABLE_EXAMPLES_TRUE@      @DEFS@
>>> +
>>> + @ENABLE_EXAMPLES_TRUE@AM_LDFLAGS = @LIBTOOL_LDFLAGS@
>>> +-@ENABLE_EXAMPLES_TRUE@LDADD = $(top_builddir)/lib/libnghttp2.la [4] 
>>> \
>>> ++@ENABLE_EXAMPLES_TRUE@LDADD = -lnghttp2 \
>>> + @ENABLE_EXAMPLES_TRUE@      
>>> $(top_builddir)/third-party/liburl-parser.la [6] \
>>> + @ENABLE_EXAMPLES_TRUE@      @LIBEVENT_OPENSSL_LIBS@ \
>>> + @ENABLE_EXAMPLES_TRUE@      @OPENSSL_LIBS@ \
>>> +@@ -519,7 +519,7 @@ EXTRA_DIST = CMakeLists.txt
>>> + # nghttp2/nghttp2.h) in this package is used rather than installed
>>> + # one.
>>> + @ENABLE_ASIO_LIB_TRUE@@ENABLE_EXAMPLES_TRUE@ASIOCPPFLAGS =
>>> ${AM_CPPFLAGS} ${BOOST_CPPFLAGS}
>>> +-@ENABLE_ASIO_LIB_TRUE@@ENABLE_EXAMPLES_TRUE@ASIOLDADD =
>>> $(top_builddir)/lib/libnghttp2.la [4] \
>>> ++@ENABLE_ASIO_LIB_TRUE@@ENABLE_EXAMPLES_TRUE@ASIOLDADD = -lnghttp2 \
>>> +
>>> @ENABLE_ASIO_LIB_TRUE@@ENABLE_EXAMPLES_TRUE@  
>>> $(top_builddir)/src/libnghttp2_asio.la [5]
>>> @JEMALLOC_LIBS@ \
>>> +
>>> @ENABLE_ASIO_LIB_TRUE@@ENABLE_EXAMPLES_TRUE@  
>>> $(top_builddir)/third-party/liburl-parser.la [6]
>>> \
>>> + @ENABLE_ASIO_LIB_TRUE@@ENABLE_EXAMPLES_TRUE@        @OPENSSL_LIBS@ 
>>> \
>>> diff --git a/www/nghttp2/files/patch-script-fetch-ocsp-response
>>> b/www/nghttp2/files/patch-script-fetch-ocsp-response
>>> new file mode 100644
>>> index 000000000000..e68d0f63ef0e
>>> --- /dev/null
>>> +++ b/www/nghttp2/files/patch-script-fetch-ocsp-response
>>> @@ -0,0 +1,7 @@
>>> +--- script/fetch-ocsp-response.orig  2021-10-19 09:31:47 UTC
>>> ++++ script/fetch-ocsp-response
>>> +@@ -1,4 +1,3 @@
>>> +-#!/usr/bin/env python
>>> + # -*- coding: utf-8 -*-
>>> +
>>> + # nghttp2 - HTTP/2 C Library
>>> diff --git a/www/nghttp2/files/patch-src-Makefile.in
>>> b/www/nghttp2/files/patch-src-Makefile.in
>>> new file mode 100644
>>> index 000000000000..2af9e94c05f9
>>> --- /dev/null
>>> +++ b/www/nghttp2/files/patch-src-Makefile.in
>>> @@ -0,0 +1,83 @@
>>> +% sed -i .orig 's|$[({]top_builddir[})]/lib/libnghttp2.la 
>>> [4]|-lnghttp2|'
>>> */Makefile.in
>>> +
>>> +--- src/Makefile.in.orig     2022-11-13 06:53:29 UTC
>>> ++++ src/Makefile.in
>>> +@@ -318,7 +318,7 @@ am__libnghttpx_a_SOURCES_DIST = util.cc util.h
>>> http2.c
>>> + libnghttpx_a_OBJECTS = $(am_libnghttpx_a_OBJECTS)
>>> + am__DEPENDENCIES_1 =
>>> + @ENABLE_ASIO_LIB_TRUE@libnghttp2_asio_la_DEPENDENCIES =  \
>>> +-@ENABLE_ASIO_LIB_TRUE@      $(top_builddir)/lib/libnghttp2.la [4] \
>>> ++@ENABLE_ASIO_LIB_TRUE@      -lnghttp2 \
>>> + @ENABLE_ASIO_LIB_TRUE@      
>>> $(top_builddir)/third-party/liburl-parser.la [6] \
>>> + @ENABLE_ASIO_LIB_TRUE@      
>>> $(top_builddir)/third-party/libllhttp.la [7] \
>>> + @ENABLE_ASIO_LIB_TRUE@      $(am__DEPENDENCIES_1) \
>>> +@@ -394,7 +394,7 @@ am__deflatehd_SOURCES_DIST = deflatehd.cc
>>> comp_helper.
>>> + @ENABLE_HPACK_TOOLS_TRUE@   $(am__objects_4)
>>> + deflatehd_OBJECTS = $(am_deflatehd_OBJECTS)
>>> + deflatehd_LDADD = $(LDADD)
>>> +-deflatehd_DEPENDENCIES = $(top_builddir)/lib/libnghttp2.la [4] \
>>> ++deflatehd_DEPENDENCIES = -lnghttp2 \
>>> +     $(top_builddir)/third-party/liburl-parser.la [6] \
>>> +     $(top_builddir)/third-party/libllhttp.la [7]
>>> + am__h2load_SOURCES_DIST = util.cc util.h http2.cc http2.h h2load.cc 
>>> \
>>> +@@ -413,7 +413,7 @@ am__h2load_SOURCES_DIST = util.cc util.h 
>>> http2.cc
>>> http
>>> + @ENABLE_APP_TRUE@   $(am__objects_5)
>>> + h2load_OBJECTS = $(am_h2load_OBJECTS)
>>> + h2load_LDADD = $(LDADD)
>>> +-h2load_DEPENDENCIES = $(top_builddir)/lib/libnghttp2.la [4] \
>>> ++h2load_DEPENDENCIES = -lnghttp2 \
>>> +     $(top_builddir)/third-party/liburl-parser.la [6] \
>>> +     $(top_builddir)/third-party/libllhttp.la [7]
>>> + am__inflatehd_SOURCES_DIST = inflatehd.cc comp_helper.c 
>>> comp_helper.h
>>> \
>>> +@@ -422,7 +422,7 @@ am__inflatehd_SOURCES_DIST = inflatehd.cc
>>> comp_helper.
>>> + @ENABLE_HPACK_TOOLS_TRUE@   $(am__objects_4)
>>> + inflatehd_OBJECTS = $(am_inflatehd_OBJECTS)
>>> + inflatehd_LDADD = $(LDADD)
>>> +-inflatehd_DEPENDENCIES = $(top_builddir)/lib/libnghttp2.la [4] \
>>> ++inflatehd_DEPENDENCIES = -lnghttp2 \
>>> +     $(top_builddir)/third-party/liburl-parser.la [6] \
>>> +     $(top_builddir)/third-party/libllhttp.la [7]
>>> + am__nghttp_SOURCES_DIST = util.cc http2.cc timegm.c app_helper.cc \
>>> +@@ -441,7 +441,7 @@ am__objects_7 =
>>> + @ENABLE_APP_TRUE@   $(am__objects_7) tls.$(OBJEXT)
>>> + nghttp_OBJECTS = $(am_nghttp_OBJECTS)
>>> + nghttp_LDADD = $(LDADD)
>>> +-nghttp_DEPENDENCIES = $(top_builddir)/lib/libnghttp2.la [4] \
>>> ++nghttp_DEPENDENCIES = -lnghttp2 \
>>> +     $(top_builddir)/third-party/liburl-parser.la [6] \
>>> +     $(top_builddir)/third-party/libllhttp.la [7]
>>> + am__nghttpd_SOURCES_DIST = util.cc http2.cc timegm.c app_helper.cc 
>>> \
>>> +@@ -453,13 +453,13 @@ am__nghttpd_SOURCES_DIST = util.cc http2.cc
>>> timegm.c a
>>> + @ENABLE_APP_TRUE@   tls.$(OBJEXT) HttpServer.$(OBJEXT)
>>> + nghttpd_OBJECTS = $(am_nghttpd_OBJECTS)
>>> + nghttpd_LDADD = $(LDADD)
>>> +-nghttpd_DEPENDENCIES = $(top_builddir)/lib/libnghttp2.la [4] \
>>> ++nghttpd_DEPENDENCIES = -lnghttp2 \
>>> +     $(top_builddir)/third-party/liburl-parser.la [6] \
>>> +     $(top_builddir)/third-party/libllhttp.la [7]
>>> + am__nghttpx_SOURCES_DIST = shrpx.cc shrpx.h
>>> + @ENABLE_APP_TRUE@am_nghttpx_OBJECTS = nghttpx-shrpx.$(OBJEXT)
>>> + nghttpx_OBJECTS = $(am_nghttpx_OBJECTS)
>>> +-am__DEPENDENCIES_2 = $(top_builddir)/lib/libnghttp2.la [4] \
>>> ++am__DEPENDENCIES_2 = -lnghttp2 \
>>> +     $(top_builddir)/third-party/liburl-parser.la [6] \
>>> +     $(top_builddir)/third-party/libllhttp.la [7]
>>> + @ENABLE_APP_TRUE@nghttpx_DEPENDENCIES = libnghttpx.a \
>>> +@@ -1150,7 +1150,7 @@ AM_CPPFLAGS = \
>>> +     @DEFS@
>>> +
>>> + AM_LDFLAGS = @LIBTOOL_LDFLAGS@
>>> +-LDADD = $(top_builddir)/lib/libnghttp2.la [4] \
>>> ++LDADD = -lnghttp2 \
>>> +     $(top_builddir)/third-party/liburl-parser.la [6] \
>>> +     $(top_builddir)/third-party/libllhttp.la [7] \
>>> +     @JEMALLOC_LIBS@ \
>>> +@@ -1330,7 +1330,7 @@ LDADD = $(top_builddir)/lib/libnghttp2.la [4] 
>>> \
>>> + @ENABLE_ASIO_LIB_TRUE@libnghttp2_asio_la_CPPFLAGS = ${AM_CPPFLAGS}
>>> ${BOOST_CPPFLAGS}
>>> + @ENABLE_ASIO_LIB_TRUE@libnghttp2_asio_la_LDFLAGS = $(AM_LDFLAGS)
>>> -no-undefined -version-info 1:0:0
>>> + @ENABLE_ASIO_LIB_TRUE@libnghttp2_asio_la_LIBADD = \
>>> +-@ENABLE_ASIO_LIB_TRUE@      $(top_builddir)/lib/libnghttp2.la [4] \
>>> ++@ENABLE_ASIO_LIB_TRUE@      -lnghttp2 \
>>> + @ENABLE_ASIO_LIB_TRUE@      
>>> $(top_builddir)/third-party/liburl-parser.la [6] \
>>> + @ENABLE_ASIO_LIB_TRUE@      
>>> $(top_builddir)/third-party/libllhttp.la [7] \
>>> + @ENABLE_ASIO_LIB_TRUE@      @OPENSSL_LIBS@ \
>>> diff --git a/www/nghttp2/files/patch-tests-Makefile.in
>>> b/www/nghttp2/files/patch-tests-Makefile.in
>>> new file mode 100644
>>> index 000000000000..1fe8da5ede92
>>> --- /dev/null
>>> +++ b/www/nghttp2/files/patch-tests-Makefile.in
>>> @@ -0,0 +1,31 @@
>>> +% sed -i .orig 's|$[({]top_builddir[})]/lib/libnghttp2.la 
>>> [4]|-lnghttp2|'
>>> */Makefile.in
>>> +
>>> +--- tests/Makefile.in.orig   2022-11-13 06:53:29 UTC
>>> ++++ tests/Makefile.in
>>> +@@ -141,7 +141,7 @@ failmalloc_OBJECTS = $(am_failmalloc_OBJECTS)
>>> + am__DEPENDENCIES_1 =
>>> + @ENABLE_STATIC_FALSE@@HAVE_CUNIT_TRUE@am__DEPENDENCIES_2 =
>>> ${top_builddir}/lib/.libs/*.o \
>>> + @ENABLE_STATIC_FALSE@@HAVE_CUNIT_TRUE@      $(am__DEPENDENCIES_1)
>>> +-@ENABLE_STATIC_TRUE@@HAVE_CUNIT_TRUE@am__DEPENDENCIES_2 =
>>> ${top_builddir}/lib/libnghttp2.la [4] \
>>> ++@ENABLE_STATIC_TRUE@@HAVE_CUNIT_TRUE@am__DEPENDENCIES_2 = -lnghttp2 
>>> \
>>> + @ENABLE_STATIC_TRUE@@HAVE_CUNIT_TRUE@       $(am__DEPENDENCIES_1)
>>> + @ENABLE_FAILMALLOC_TRUE@@HAVE_CUNIT_TRUE@failmalloc_DEPENDENCIES =
>>> $(am__DEPENDENCIES_2)
>>> + AM_V_lt = $(am__v_lt_@AM_V@)
>>> +@@ -181,7 +181,7 @@ am__objects_1 =
>>> + main_OBJECTS = $(am_main_OBJECTS)
>>> + @ENABLE_STATIC_FALSE@@HAVE_CUNIT_TRUE@main_DEPENDENCIES =
>>> ${top_builddir}/lib/.libs/*.o \
>>> + @ENABLE_STATIC_FALSE@@HAVE_CUNIT_TRUE@      $(am__DEPENDENCIES_1)
>>> +-@ENABLE_STATIC_TRUE@@HAVE_CUNIT_TRUE@main_DEPENDENCIES =
>>> ${top_builddir}/lib/libnghttp2.la [4] \
>>> ++@ENABLE_STATIC_TRUE@@HAVE_CUNIT_TRUE@main_DEPENDENCIES = -lnghttp2 
>>> \
>>> + @ENABLE_STATIC_TRUE@@HAVE_CUNIT_TRUE@       $(am__DEPENDENCIES_1)
>>> + main_LINK = $(LIBTOOL) $(AM_V_lt) --tag=CC $(AM_LIBTOOLFLAGS) \
>>> +     $(LIBTOOLFLAGS) --mode=link $(CCLD) $(AM_CFLAGS) $(CFLAGS) \
>>> +@@ -743,7 +743,7 @@ EXTRA_DIST = CMakeLists.txt
>>> + # files directly because the tests use symbols not included in 
>>> public
>>> API.
>>> + @ENABLE_STATIC_FALSE@@HAVE_CUNIT_TRUE@main_LDADD =
>>> ${top_builddir}/lib/.libs/*.o \
>>> + @ENABLE_STATIC_FALSE@@HAVE_CUNIT_TRUE@      $(am__append_2)
>>> +-@ENABLE_STATIC_TRUE@@HAVE_CUNIT_TRUE@main_LDADD =
>>> ${top_builddir}/lib/libnghttp2.la [4] \
>>> ++@ENABLE_STATIC_TRUE@@HAVE_CUNIT_TRUE@main_LDADD = -lnghttp2 \
>>> + @ENABLE_STATIC_TRUE@@HAVE_CUNIT_TRUE@       $(am__append_2)
>>> + @HAVE_CUNIT_TRUE@main_LDFLAGS = -static
>>> + @ENABLE_FAILMALLOC_TRUE@@HAVE_CUNIT_TRUE@failmalloc_SOURCES =
>>> failmalloc.c failmalloc_test.c failmalloc_test.h \
>>> diff --git a/www/nghttp2/pkg-plist b/www/nghttp2/pkg-plist
>>> index be8f7de0e846..16fccb59dac4 100644
>>> --- a/www/nghttp2/pkg-plist
>>> +++ b/www/nghttp2/pkg-plist
>>> @@ -10,6 +10,10 @@ man/man1/nghttp.1.gz
>>>  man/man1/nghttpd.1.gz
>>>  man/man1/nghttpx.1.gz
>>>  %%PORTDOCS%%%%DOCSDIR%%/README.rst
>>> +%%PORTEXAMPLES%%%%EXAMPLESDIR%%/asio-cl.cc
>>> +%%PORTEXAMPLES%%%%EXAMPLESDIR%%/asio-cl2.cc
>>> +%%PORTEXAMPLES%%%%EXAMPLESDIR%%/asio-sv.cc
>>> +%%PORTEXAMPLES%%%%EXAMPLESDIR%%/asio-sv2.cc
>>>  %%PORTEXAMPLES%%%%EXAMPLESDIR%%/client
>>>  %%PORTEXAMPLES%%%%EXAMPLESDIR%%/client.c
>>>  %%PORTEXAMPLES%%%%EXAMPLESDIR%%/deflate
>> 
>> I'm missing the benefit with this change? More patches, hacks and it
>> also takes longer to build
>> 
>> Best regards,
>> Daniel
>> 



Links:
------
[1] 
https://github.com/$%7BPORTNAME%7D/$%7BPORTNAME%7D/releases/download/v$%7BDISTVERSION%7D/
[2] 
https://github.com/nghttp2/nghttp2/releases/download/v$%7BPORTVERSION%7D/
[3] http://bsd.port.mk
[4] http://libnghttp2.la
[5] http://libnghttp2_asio.la
[6] http://liburl-parser.la
[7] http://libllhttp.la