I added that patch to trunk and merged it into 2.4.x.

I'll make another release candidate soonish. Someone wants to merge the back 
ports idling in STATUS?

Cheers,
Stefan

> Am 16.10.2023 um 12:55 schrieb Joe Orton <jor...@redhat.com>:
> 
> Looks like this broke with the websockets backport. mod_h2 is failing to 
> compile on versions of nghttp2 without 
> NGHTTP2_SETTINGS_ENABLE_CONNECT_PROTOCOL - looks like this was added in 
> nghttp2 v1.34.0 [1] so how about something like this, or is there a 
> better way?
> 
> (configure check for a declaration is messier)
> 
> diff --git a/modules/http2/h2.h b/modules/http2/h2.h
> index cfecb3d9a3..06e7087c04 100644
> --- a/modules/http2/h2.h
> +++ b/modules/http2/h2.h
> @@ -20,6 +20,8 @@
> #include <apr_version.h>
> #include <ap_mmn.h>
> 
> +#include <nghttp2/nghttp2ver.h>
> +
> struct h2_session;
> struct h2_stream;
> 
> @@ -39,7 +41,7 @@ struct h2_stream;
> #define H2_USE_POLLFD_FROM_CONN 0
> #endif
> 
> -#if H2_USE_PIPES
> +#if H2_USE_PIPES && defined(NGHTTP2_VERSION_NUM) && NGHTTP2_VERSION_NUM >= 
> 0x012200
> #define H2_USE_WEBSOCKETS       1
> #else
> #define H2_USE_WEBSOCKETS       0
> 
> 
> 
> [1] https://github.com/nghttp2/nghttp2/milestone/30?closed=1 
> 

Reply via email to