Hi Jan, Thank you so much for all your work on this!
I've finally got round to having a look at it (sorry it's been so long!), and have run into some teething problems here on Windows... The first problem was "Warning (mostly harmless): No library found for -laprutil-1" appearing from Makefile.PL between ModPerl::WrapXS and APR (presumably it applies to the latter), which I've ignored for now but will probably come back to bite me later. No other such issues are reported, and all the libraries (apr-1.lib, aprutil-1.lib, libapr-1.lib, libapriconv-1.lib, libaprutil-1.lib, libhttpd.lib, mod_dav.lib and xml.lib) are together in C:\Apache24\lib, so I'm not sure what the problem is there yet. Next up, modperl_util.c contained various declarations scatted amongst code which VC++ doesn't support when compiling C (rather than C++). I've fixed that in r1502045. Next, modperl_apache_compat.c complains that we're *defining* the missing httpd function "ap_get_server_version", but we've declared it the same way as httpd would do -- that is, marked "dllexport" when compiled in httpd and otherwise marked "dllimport" to say that we're third-party code importing it from httpd. We're third-party code, of course, but *defining* a function marked "dllimport" isn't allowed. Removing dllexport/dllimport from the declaration (see attached patch) fixes this for me, but I don't know if that's the right thing on other OSes? The build now progress to APR::Brigade, but falls over complaining that modperl_error.c references the symbol "perl_module", but that isn't defined anywhere. I've run out of time tonight but will come back to this very soon. If you can shed any light on the remaining problems so far that might assist me when I look again then please let me know. Steve On 9 July 2013 11:16, Jan Kaluža <jkal...@redhat.com> wrote: > Hi, > > I just want to say that from my point of view, it should be possible to > merge httpd24 branch with trunk now. Maybe it's time to actually do the > merge and give that code some more testing. > > Before future release, we have to coordinate with Apache-Test to release > also new Apache-Test which contains fixes needed to run mod_perl with > httpd24. > > I'm going to update mod_perl to the HEAD of httpd24 branch in Fedora soon > too to give it more testing. > > Regards, > Jan Kaluza > > ------------------------------**------------------------------**--------- > To unsubscribe, e-mail: > dev-unsubscribe@perl.apache.**org<dev-unsubscr...@perl.apache.org> > For additional commands, e-mail: dev-h...@perl.apache.org > >
nodllimport.patch
Description: Binary data
--------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@perl.apache.org For additional commands, e-mail: dev-h...@perl.apache.org
