Hi all,

If you've been following bnicholes rework of AAA yet again, its looking good, but thinks have moved and mod_perl2 no longer compiles against the httpd-trunk.

I believe I can fix this one, but I've got a few questions.

First, I need to add the missing include to modperl_constants.[ch] or modperl_const.h -- I picked modperl_constants.c because AFAIK, they are not referenced outside that file. (diff at end)

My current problem is how do I tweak the generation of
xs/maps/apache2_functions.map
xs/tables/current/Apache2/FunctionTable.pm

to be conditional on the function list based on httpd version ?

Obviously the function has to be there for pre 2.3.0 and not for 2.3.0+

On another note, 'Order', 'Allow', 'Deny' are disappearing too and mod_auth_comat.h was added for back compat.

I'll fix the line wrap before a commit.

modperl/trunk/lib/ModPerl 18>svn diff Code.pm
\Index: Code.pm
===================================================================
--- Code.pm     (revision 406886)
+++ Code.pm     (working copy)
@@ -791,7 +791,10 @@
     my ($self, $h_fh, $c_fh) = @_;

     require Apache2::ConstantsTable;
-
+
+ print $c_fh qq{\#if AP_SERVER_MAJORVERSION_NUMBER>2 || AP_SERVER_MINORVERSION_NUMBER>=3\n};
+    print $c_fh qq{\#include "mod_auth.h"\n};
+    print $c_fh qq{\#endif /* ! 2.3.x+ */\n\n};
     print $c_fh qq{\#include "modperl_const.h"\n};
     print $h_fh "#define MP_ENOCONST -3\n\n";



--
------------------------------------------------------------------------
Philip M. Gollucci ([EMAIL PROTECTED]) 323.219.4708
Consultant / http://p6m7g8.net/Resume/resume.shtml
Senior Software Engineer - TicketMaster - http://ticketmaster.com
1024D/A79997FA F357 0FDD 2301 6296 690F  6A47 D55A 7172 A799 97F

"It takes a minute to have a crush on someone, an hour to like someone,
and a day to love someone, but it takes a lifetime to forget someone..."

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

Reply via email to