Randy Kobes wrote:
I believe the last APR::* module that depends on mod_perl.so
is APR::Bucket - this diff included modperl_bucket.c in the
files copiled into APR.so, so that APR::Bucket doesn't need
mod_perl.so:

+1

I'm not sure this is the last one, since we don't test all APR:: modules.

Also I believe the copying part was originally a quick hack, I see no reason why .c files should be compiled twice. So eventually we should replace copy-n-compile with create-right-dependecies-n-compile-once

====================================================
Index: Makefile.PL
===================================================================
RCS file: /home/cvs/modperl-2.0/xs/APR/APR/Makefile.PL,v
retrieving revision 1.20
diff -u -r1.20 Makefile.PL
--- Makefile.PL 16 Jun 2004 03:55:48 -0000      1.20
+++ Makefile.PL 21 Jun 2004 05:24:11 -0000
@@ -58,7 +58,7 @@

 # link the following into APR.so so other APR:: modules can be used
 # outside of httpd
-my @names = map { "modperl_$_" } (qw(error),
+my @names = map { "modperl_$_" } (qw(error bucket),
                                   map { "common_$_" } qw(util log));
 my(@obj, @clean, %src);
 for (@names) {




--
__________________________________________________________________
Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/     mod_perl Guide ---> http://perl.apache.org
mailto:[EMAIL PROTECTED] http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org   http://ticketmaster.com

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



Reply via email to