Skip to content

Commit

Permalink
Configure: fixed perl Makefile generation (ticket #334).
Browse files Browse the repository at this point in the history
Dependancy tracking introduced in r5169 were not handled absolute path
names properly.  Absolute names might appear in CORE_DEPS if --with-openssl
or --with-pcre configure arguments are used to build OpenSSL/PCRE
libraries.

Additionally, revert part of r5169 to set NGX_INCS from Makefile
variables.  Makefile variables have $ngx_include_opt in them, which
might result in wrong include paths being used.  As a side effect,
this also restores build with --with-http_perl_module and --without-http
at the same time.


git-svn-id: svn://svn.nginx.org/nginx/trunk@5181 73f98a42-aea0-e011-b76d-00259023448c
  • Loading branch information
mdounin committed Apr 19, 2013
1 parent f204e0b commit 616fa58
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion auto/lib/perl/make
Expand Up @@ -31,7 +31,7 @@ $NGX_OBJS/src/http/modules/perl/Makefile: \\

cd $NGX_OBJS/src/http/modules/perl \\
&& NGX_PM_CFLAGS="\$(NGX_PM_CFLAGS) -g $NGX_CC_OPT" \\
NGX_INCS="\$(CORE_INCS) \$(HTTP_INCS)" \\
NGX_INCS="$CORE_INCS $NGX_OBJS $HTTP_INCS" \\
NGX_DEPS="\$(CORE_DEPS) \$(HTTP_DEPS)" \\
$NGX_PERL Makefile.PL \\
LIB=$NGX_PERL_MODULES \\
Expand Down
2 changes: 1 addition & 1 deletion src/http/modules/perl/Makefile.PL
Expand Up @@ -22,7 +22,7 @@ WriteMakefile(

depend => {
'nginx.c' => join(" ", map {
"../../../../../$_"
m#^/# ? $_ : "../../../../../$_"
} (split(/\s+/, $ENV{NGX_DEPS}),
"src/http/modules/perl/ngx_http_perl_module.h"))
},
Expand Down

0 comments on commit 616fa58

Please sign in to comment.