[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Experimental C unit test suite available for hacking

On Thu, May 24, 2018 at 11:28 AM, Yann Ylavic <ylavic.dev@xxxxxxxxx> wrote:
> On Thu, May 24, 2018 at 11:23 AM, Yann Ylavic <ylavic.dev@xxxxxxxxx> wrote:
>> On Thu, May 24, 2018 at 11:06 AM, Micha Lenk <micha@xxxxxxxxx> wrote:
>>> On 05/24/2018 10:41 AM, Yann Ylavic wrote:
>>>> Btw, as Jacob noted, the attached patch seems to work for me (even
>>>> without the above option).
>>> Yes, for me too, except that the linker problem with undefined symbols now
>>> seems to shift to the modules. I had to disable a few modules
>>> (--enable-mods-static=most --disable-apreq --disable-proxy-fcgi
>>> --disable-session-cookie --disable-session-dbd --disable-dav) to finally get
>>> to a successful build.
>> Oh indeed, the correct order seems to be:
>>   $(MPM_LIB) \
>>   $(BUILTIN_LIBS) \
>>   server/libmain.la \
>>   os/$(OS_DIR)/libos.la
> Argh no, it now fails with --enable-mods-shared=...

Looks like the right order to me, however it fails with shared modules
because "server/core.c" (in libmain) uses ap_set_etag() function from
"modules/http/http_etag.c" (in BUILTIN_LIBS's libmod_http).

I think "core" shouldn't depend on a module (even builtin), for
instance ap_set_{last_modified,accept_range,content_length,...} also
used by the core are defined in "server/protocol.c".