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

新版编译错误 #288

Closed
zzjin opened this issue Aug 1, 2013 · 6 comments
Closed

新版编译错误 #288

zzjin opened this issue Aug 1, 2013 · 6 comments

Comments

@zzjin
Copy link
Contributor

zzjin commented Aug 1, 2013

新版1.5.0按照1.4.5的相同配置,configure能成功,编译的时候提示错误:

src/http/ngx_http_request_body.c:517:32: error: variable ‘clcf’ set but not used [-Werror=unused-but-set-variable]

编译参数是:

./configure  --conf-path=/etc/nginx/nginx.conf --error-log-path=/var/log/nginx/error.log --http-client-body-temp-path=/var/lib/nginx/body --http-fastcgi-temp-path=/var/lib/nginx/fastcgi --http-log-path=/var/log/nginx/access.log --http-proxy-temp-path=/var/lib/nginx/proxy --lock-path=/var/lock/nginx.lock --pid-path=/var/run/nginx.pid --with-debug --with-http_dav_module --with-http_flv_module --with-http_geoip_module --with-http_gzip_static_module --with-http_realip_module --with-http_stub_status_module --with-http_ssl_module --with-http_sub_module --with-ipv6 --with-mail --with-mail_ssl_module

机器是debian7.0 i386

是ngx_http_read_non_buffered_client_request_body这个新增的模块的参数问题?

@zhuzhaoyuan
Copy link
Member

多谢反馈!
你能针对这个问题提个pull request吗?——应该删除掉这个多余的这个clcf就可以了(删掉517-519行)。
这样我们在contributors里就能看到你的贡献 :)

zzjin added a commit to zzjin/tengine that referenced this issue Aug 1, 2013
@yaoweibin
Copy link
Member

多谢贡献,已经merge

@icedfish
Copy link
Contributor

icedfish commented Aug 3, 2013

@zhuzhaoyuan 前天我提的也是一模一样这个问题,出手晚了>_< 你们是会重新发布一个小版本1.5.1之么? 还是让碰到问题的人直接clone git的master来用呢?

@yaoweibin
Copy link
Member

会发布1.5.1,谢谢关注

@zhuzhaoyuan
Copy link
Member

@icedfish 哈哈,我也想跟你说有人比你手更快,哈哈。不过看到你的DSO配置sample已经commit了。赞!

@icedfish
Copy link
Contributor

icedfish commented Aug 3, 2013

@zhuzhaoyuan 我努力在使用者角度给你们帮帮忙 : ) 加油~

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

4 participants