Hi Stefan,

Yes, *only* in main.c which is the only problematic file.

The warning directly below causes me to suspect this is why we are seeing the problem in this one file alone.

#ifndef WIN32
APLOG_USE_MODULE(core);
#endif

/* WARNING: Win32 binds http_main.c dynamically to the server. Please place
 *          extern functions and global data in another appropriate module.
 *
 * Most significant main() global data can be found in http_config.c
 */

Cheers,
Gregg


Stefan Fritsch wrote:
Hi Gregg,

On Tuesday 08 June 2010, Gregg L. Smith wrote:
Sorry to say this still will not build and just changes the error
message.

I however would like to see 2.3.6 out this week as well as
everyone  else. If worse comes to worse, #ifndef WIN32 around this
one line does allow apache to compile and run, and a quick testing
it seems to not be causing problems with normal logging. It may
not have the desired new functionality you are looking for but as
AFAIK this is still going to be under the alpha banner, so, no big
deal.

Thanks for testing.

Do you mean it works if you remove - the one line only from main.c
- APLOG_USE_MODULE(core) in all files where it appears
- APLOG_USE_MODULE(...) in all files

But you are right, not having those lines does not cause any problems. And I agree that we should just disable it if it threatens to delay the alpha release.

If it only affects the core module, a possible fix may be to exploit the fact that core_module always seems to have the module_index 0. Though I would have to look at this in more detail.

Cheers,
Stefan


Reply via email to