ngx_http_proxy_connect_module icon indicating copy to clipboard operation
ngx_http_proxy_connect_module copied to clipboard

ngx_murmurhash.c error how to fix?

Open thestarrypage opened this issue 2 years ago • 1 comments

0. Before Your ASK

  1. Try to find an answer by reading a FAQ.

Ⅰ. Issue Description

When running make && makeinstall

src/core/ngx_murmurhash.c: In function ‘ngx_murmur_hash2’:
src/core/ngx_murmurhash.c:37:11: error: this statement may fall through [-Werror=implicit-fallthrough=]
   37 |         h ^= data[2] << 16;
      |         ~~^~~~~~~~~~~~~~~~
src/core/ngx_murmurhash.c:38:5: note: here
   38 |     case 2:
      |     ^~~~
src/core/ngx_murmurhash.c:39:11: error: this statement may fall through [-Werror=implicit-fallthrough=]
   39 |         h ^= data[1] << 8;
      |         ~~^~~~~~~~~~~~~~~

Ⅱ. Describe what happened

No idea what is this error

Ⅲ. Describe what you expected to happen

to be able to build

Ⅳ. How to reproduce it (as minimally and precisely as possible)

Follow exacts same steps

$ wget http://nginx.org/download/nginx-1.9.12.tar.gz $ tar -xzvf nginx-1.9.12.tar.gz $ cd nginx-1.9.12/ $ patch -p1 < /path/to/ngx_http_proxy_connect_module/patch/proxy_connect.patch $ ./configure --add-dynamic-module=/path/to/ngx_http_proxy_connect_module $ make && make install

Ⅴ. Anything else we need to know?

  1. If applicable, add nginx debug log.

Ⅵ. Environment:

  1. Tengine/Nginx/OpenResty version (use sbin/nginx -v):
  2. Which patch do you use?

thestarrypage avatar Sep 24 '23 05:09 thestarrypage

See here: https://github.com/arut/nginx-rtmp-module/issues/1255

Two methods could be used to fix:

  1. https://github.com/arut/nginx-rtmp-module/issues/1255#issuecomment-471004506
  2. use higher version of nginx

chobits avatar Nov 27 '23 02:11 chobits