-
Notifications
You must be signed in to change notification settings - Fork 104
error: ngx_devel_kit is required to build ngx_set_misc; please put it before ngx_set_misc. #27
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
Comments
not confirmed, if use standard autoconf+make
and no sense to use redis2-nginx-module with lua-nginx-module, lua-resty-redis is better choice in this case |
Thank you @TrurlMcByte I installed autoconf with sudo yum groupinstall "Development Tools". Written by David J. MacKenzie and Akim Demaille. The make version: The gcc version: Is there anyting wrong? |
looks ok
and got compiled without any errors |
@TrurlMcByte I solved this problem, before I fetch and checkout lastest tag and got this error, now I checkout master branch, it works OK now. Thank you for your help! |
Strange, i'm on master and i got this message too. i assumed that it was just a required library, but maybe that isn't the case? |
@deweydb You definitely need to provide more details for me to help you out. BTW, you're recommended to use the OpenResty bundle to avoid manual configuration mistakes: https://openresty.org/#Download |
|
Looked into openresty, it doesn't really fit my needs, i don't need most of those modules, and it's missing some of the modules i do need. |
@deweydb The error message on the last line is clear: you need the ngx_devel_kit (NDK) module to proceed. And from your ./configure line, I don't see a corresponding Regarding your OpenResty concerns, you can always disable components you don't need by specifying the corresponding |
Ok. thanks, that's what i said in my first comment, i guess i just wasn't very clear about it. sorry. |
@deweydb And you can always add your own additional modules with |
@agentzh seems I am getting this message too but only with 0.30. With 0.29 it was working fine ! This is with Nginx 1.9.12
now if i use 0.29 version it compiles fine, maybe something to do with nginx 1.9.12 and dynamic module changes ?
with 0.29 set-misc module compiles okay on 1.9.12
|
@centminmod I think you need the latest git master of ngx_devel_kit for NGINX 1.9.11+: |
ah i see there's updates in master branch https://github.com/simpl/ngx_devel_kit/commits/master, not ready for building a new tag version for ngx_devel_kit ? :) |
@centminmod Not ready for a formal release. But I've just tagged v0.3.0rc1 if that helps. |
@agentzh thanks rc1 tag will do perfectly ! compiles fine now :)
|
I'm closing this. |
@agentzh seems error is back when using ngx_devel_kit 0.3.0 not rc1 specific order used - tried reverse order and same message - no problems with 0.3.0rc1 though
looks like if ngx_devel_kit is dynamic module, set-misc needs to be set as dynamic module too as this works - by design ? if ngx_devel_kit is dynamic module, all other openresty modules need to be dynamic modules too ?
|
@centminmod wrote
You can compare ngx_devel_kit 0.3.0 and ngx_devel_kit 0.3.0rc1. The change is just switching two C local variable's type from commit c20318c5522be877f644405411a74c0e16e6f660
Author: Xiaochen Wang <[email protected]>
Date: Tue Apr 5 14:05:57 2016 +0800
fix compiler error: comparison between signed and unsigned integer expressions
diff --git a/src/ndk_complex_value.c b/src/ndk_complex_value.c
index bfca2cd..97681ff 100644
--- a/src/ndk_complex_value.c
+++ b/src/ndk_complex_value.c
@@ -50,7 +50,7 @@ ndk_http_complex_value_array_create (ngx_conf_t *cf, char **s, ngx_int_t n)
ngx_int_t
ndk_http_complex_value_array_compile (ngx_conf_t *cf, ngx_array_t *a)
{
- ngx_int_t i;
+ ngx_uint_t i;
ngx_http_complex_value_t *cv;
if (a == NULL || a == NGX_CONF_UNSET_PTR) {
@@ -109,7 +109,7 @@ ndk_conf_set_http_complex_value_array_slot (ngx_conf_t *cf, ngx_command_t *cmd,
ngx_http_complex_value_t *cv;
ngx_array_t **a;
ngx_conf_post_t *post;
- ngx_int_t i, alloc;
+ ngx_uint_t i, alloc;
a = (ngx_array_t **) (p + cmd->offset);
So it's impossible to make a different during the ./configure phase. The error message you're getting already made it clear that you should add ngx_devel_kit before
Yes, of course. If module A depends on module B, and module B is a dynamic module, then module A must also be a dynamic module, for obvious reasons. It's not a design decision; you just cannot do that the other way around for obvious technical reasons.
No. Only those actually depend on ngx_devel_kit (note that some modules just optionally depend on NDK, like ngx_lua). |
OS version: CentOS Linux release 7.1.1503 (Core)
Kernel version: Linux 3.10.0-229.el7.x86_64 #1 SMP Fri Mar 6 11:36:42 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux
Nginx source version: 1.8.0
Build params:
./configure --prefix=/home/redis/nginx
--add-module=lua-nginx-module
--add-module=redis2-nginx-module
--add-module=ngx_devel_kit
--add-module=set-misc-nginx-module
Error Info:
adding module in redis2-nginx-module
adding module in ngx_devel_kit
adding module in set-misc-nginx-module
error: ngx_devel_kit is required to build ngx_set_misc; please put it before ngx_set_misc.
The text was updated successfully, but these errors were encountered: