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

nginx-1.3.9 - ginx_upstream_check_module patch fails #16

Closed
bgsmalls opened this issue Dec 22, 2012 · 2 comments
Closed

nginx-1.3.9 - ginx_upstream_check_module patch fails #16

bgsmalls opened this issue Dec 22, 2012 · 2 comments

Comments

@bgsmalls
Copy link

root@dellserver:/build/nginx-1.3.9$ patch -p1 < ./modules/nginx_upstream_check_module/check_1.2.2+.patch
patching file src/http/modules/ngx_http_upstream_ip_hash_module.c
patching file src/http/modules/ngx_http_upstream_least_conn_module.c
patching file src/http/ngx_http_upstream_round_robin.c
Hunk #7 FAILED at 462.
Hunk #8 succeeded at 564 (offset 4 lines).
1 out of 8 hunks FAILED -- saving rejects to file src/http/ngx_http_upstream_round_robin.c.rej
patching file src/http/ngx_http_upstream_round_robin.h

@yaoweibin
Copy link
Owner

Thanks for your report. Nginx-1.2.6 and nginx-1.3.9 both breaks the old
patch. I have added a new patch for you:
https://github.com/yaoweibin/nginx_upstream_check_module/blob/master/check_1.2.6%2B.patch

I have also updated the README. This patch should be OK with
nginx-1.2.6+ and nginx-1.3.9+.

On 2012-12-23 3:36, bgsmalls wrote:

root@dellserver:/build/nginx-1.3.9$ patch -p1 <
./modules/nginx_upstream_check_module/check_1.2.2+.patch
patching file src/http/modules/ngx_http_upstream_ip_hash_module.c
patching file src/http/modules/ngx_http_upstream_least_conn_module.c
patching file src/http/ngx_http_upstream_round_robin.c
Hunk #7
#7
FAILED at 462.
Hunk #8
#8
succeeded at 564 (offset 4 lines).
1 out of 8 hunks FAILED -- saving rejects to file
src/http/ngx_http_upstream_round_robin.c.rej
patching file src/http/ngx_http_upstream_round_robin.h


Reply to this email directly or view it on GitHub
#16.

Thanks,
-Weibin Yao

@bgsmalls
Copy link
Author

I just tried the check_1.2.6+.patch on nginx 1.3.9.
It works !!!
Thanks a million for the quick response and fix :)

root@dellserver:/build/nginx-1.3.9$ patch -p1 < ./modules/nginx_upstream_check_module/check_1.2.6+.patch
patching file src/http/modules/ngx_http_upstream_ip_hash_module.c
patching file src/http/modules/ngx_http_upstream_least_conn_module.c
patching file src/http/ngx_http_upstream_round_robin.c
patching file src/http/ngx_http_upstream_round_robin.h

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

2 participants