This patch will make sure that additional library paths are included in the build flags so that mod_perl will use the same versions of libraries that APR does.

For example if APR were configured thusly:

   configure --with-expat=/mydir/expat

when running:

   apu-config --link-ld --ldflags --libs

--ldflags will make sure -L/mydir/expat is included in the build flags.

Otherwise, it is not included and mod_perl will silently use a different version of the library or users need to use the LIBS MakeMaker parameter, which was only possible as of r386681.

--
Mike


Index: mod_perl-2.0-r386681/lib/Apache2/Build.pm
===================================================================
--- mod_perl-2.0-r386681/lib/Apache2/Build.pm   (revision 386681)
+++ mod_perl-2.0-r386681/lib/Apache2/Build.pm   (working copy)
@@ -1117,7 +1117,7 @@
     # first use apu_config_path and then apr_config_path in order to
     # resolve the symbols right during linking
     for ($self->apu_config_path, $self->apr_config_path) {
-        if (my $link = $_ && -x $_ && qx{$_ --link-ld --libs}) {
+ if (my $link = $_ && -x $_ && qx{$_ --link-ld --ldflags --libs}) {
             chomp $link;

# Change '/path/to/libanything.la' to '-L/path/to -lanything'
Index: mod_perl-2.0-r386681/lib/Apache2/Build.pm
===================================================================
--- mod_perl-2.0-r386681/lib/Apache2/Build.pm   (revision 386681)
+++ mod_perl-2.0-r386681/lib/Apache2/Build.pm   (working copy)
@@ -1117,7 +1117,7 @@
     # first use apu_config_path and then apr_config_path in order to
     # resolve the symbols right during linking
     for ($self->apu_config_path, $self->apr_config_path) {
-        if (my $link = $_ && -x $_ && qx{$_ --link-ld --libs}) {
+        if (my $link = $_ && -x $_ && qx{$_ --link-ld --ldflags --libs}) {
             chomp $link;

             # Change '/path/to/libanything.la' to '-L/path/to -lanything'


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to