The current CVS mp2 doesn't build on Win32 (at least) because modperl_sv2pool (referenced in modperl.def) is an unresolved external.
The attached patch fixes this.
NOTE: I don't know if the MP_INLINE bit is needed or not, I'll leave that to the judgement of somebody who knows what it does.
Thanks, Steve, for the catch. The right patch is below, we simply forgot to remove it from the FunctionTable
--- src\modules\perl\modperl_util.c.orig 2004-01-21 08:39:11.724481200 +0000 +++ src\modules\perl\modperl_util.c 2004-01-21 08:48:13.536802200 +0000
also can your diff app produce unix paths? your patch can't be applied without manual typing in of the filenames.
Index: xs/tables/current/ModPerl/FunctionTable.pm
===================================================================
RCS file: /home/cvs/modperl-2.0/xs/tables/current/ModPerl/FunctionTable.pm,v
retrieving revision 1.136
diff -u -r1.136 FunctionTable.pm
--- xs/tables/current/ModPerl/FunctionTable.pm 14 Jan 2004 21:27:41 -0000 1.136
+++ xs/tables/current/ModPerl/FunctionTable.pm 21 Jan 2004 09:07:22 -0000
@@ -4247,20 +4247,6 @@
]
},
{
- 'return_type' => 'apr_pool_t *',
- 'name' => 'modperl_sv2pool',
- 'args' => [
- {
- 'type' => 'PerlInterpreter *',
- 'name' => 'my_perl'
- },
- {
- 'type' => 'SV *',
- 'name' => 'obj'
- }
- ]
- },
- {
'return_type' => 'request_rec *',
'name' => 'modperl_sv2request_rec',
'attr' => [
__________________________________________________________________ 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]
