Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dynamic modules do not compile #17

Closed
badfiles opened this issue Feb 11, 2016 · 11 comments
Closed

Dynamic modules do not compile #17

badfiles opened this issue Feb 11, 2016 · 11 comments

Comments

@badfiles
Copy link

To compile dynamic modules I had to add rule

modules:
    $(MAKE) -f objs/Makefile modules

and issue make modules.
By default dynamic modules do not compile.

@PiotrSikora
Copy link
Contributor

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.

@badfiles
Copy link
Author

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)

@PiotrSikora
Copy link
Contributor

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?

@badfiles
Copy link
Author

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.

@badfiles
Copy link
Author

out.txt

@PiotrSikora
Copy link
Contributor

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?

@badfiles
Copy link
Author

yep

@PiotrSikora
Copy link
Contributor

@RaeesBhatti
Copy link

Has this been fixed?

@PiotrSikora
Copy link
Contributor

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

@RaeesBhatti
Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants