Dynamic modules do not compile about ngx_brotli HOT 11 CLOSED

They are built by default, the line you've added is part of generated Makefile in nginx-1.9.11, see: http://hg.nginx.org/nginx/diff/85dea406e18f/auto/install.

from ngx_brotli.

This is the result I get

cp  objs/ngx_http_brotli_static_module.so objs/ngx_http_brotli_filter_module.so '/usr/nginx/modules'
cp: cannot stat ‘objs/ngx_http_brotli_static_module.so’: No such file or directory
cp: cannot stat ‘objs/ngx_http_brotli_filter_module.so’: No such file or directory
make[1]: *** [install] Error 1
make[1]: Leaving directory `/src/nginx-1.9.11'

objs/addon/src is also empty,
so I have to manually compile modules.
My config

./configure --prefix=/usr/nginx   \
        --add-dynamic-module=ngx_brotli/ \
        --with-file-aio \
        --with-http_ssl_module \
        --with-http_realip_module \
        --with-http_gunzip_module \
        --with-http_gzip_static_module \
        --with-threads \
        --with-http_v2_module 

(but with add-module everything works great)

from ngx_brotli.

Weird... Are you using stock nginx-1.9.11 or patched version? Also, could you post complete output of ./configure --add-dynamic-module=ngx_brotli, make and make install?

from ngx_brotli.

Last time I tested with rev ba3c2ca21aa5,
having removed this code from module config
CFLAGS="$CFLAGS -Wno-write-strings"
I make with checkinstall, the output you requested will be in a while.

from ngx_brotli.

out.txt

from ngx_brotli.

Thanks! That was helpful. Do you have a name that I should add to the commit message or is "Reported by badfiles on GitHub" fine with you?

from ngx_brotli.

yep

from ngx_brotli.

http://mailman.nginx.org/pipermail/nginx-devel/2016-February/007909.html

from ngx_brotli.

Has this been fixed?

from ngx_brotli.

I believe it was fixed in http://hg.nginx.org/nginx/rev/cfc3cfa434ec (nginx-1.9.12), why do you ask?

from ngx_brotli.

I am going to compile nginx with this module as dynamic. So, just checking if there are any issues I should be aware of.

from ngx_brotli.