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.

Sign up for GitHub

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

Jump to bottom

Compilation issue with --with-http_perl_module #2

Closed
mtourne opened this issue Oct 19, 2010 · 1 comment
Closed

Compilation issue with --with-http_perl_module #2

mtourne opened this issue Oct 19, 2010 · 1 comment

Comments

@mtourne
Copy link
Collaborator

mtourne commented Oct 19, 2010

When compiling with --with-http_perl_module, ngx_http_upstream_check_handler.h included in ngx_http_upstream_round_robin.h cannot be found, because the $ngx_addon_dir is not included by src/http/modules/perl/Makefile.PL

Adding this to ngx_http_upstream_round_robin.c instead ngx_http_upstream_round_robin.h seems to fix the problem :

if (NGX_UPSTREAM_CHECK_MODULE)

include "ngx_http_upstream_check_handler.h"

endif

And adding #include "ngx_http_upstream_check_handler.h" in ngx_http_upstream_check_handler.c
@yaoweibin
Copy link
Owner

Thanks for your bug report. I have fix the problem in the new revision. I'm sorry I have not tested the perl module ever before.

@anprevosto anprevosto mentioned this issue Nov 18, 2014
Open
@fankeke fankeke mentioned this issue Sep 30, 2016
Open
@carlvine500 carlvine500 mentioned this issue Dec 21, 2016
Closed
@yangdailin yangdailin mentioned this issue Jan 19, 2017
Open
@mushroom11s mushroom11s mentioned this issue Oct 17, 2018
Closed
rq0net pushed a commit to rq0net/nginx_upstream_check_module that referenced this issue Dec 9, 2020
@2Fast2BCn
Merge pull request yaoweibin#2 from detailyang/master
6ecb5e0 
expose an api ngx_http_upstream_check_upstream_down for check upstream status
ngarratt pushed a commit to ngarratt/nginx_upstream_check_module that referenced this issue Jul 5, 2023
@lneto
Merge pull request yaoweibin#2 from zenedge/lua_656_https
b085ca1 
LUA-656 added support for ssl (e.g., HTTPS)
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@yaoweibin @mtourne