perl-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Steve Hay <steve.m....@googlemail.com>
Subject Re: [httpd24 branch] merge with trunk?
Date Wed, 10 Jul 2013 23:17:52 GMT
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 <jkaluza@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-unsubscribe@perl.apache.org>
> For additional commands, e-mail: dev-help@perl.apache.org
>
>

Mime
View raw message