Stas Bekman wrote:
Stas Bekman wrote:

cc: Error: modperl_exports.c, line 195: In the initializer for modperl_hack_io_apache_init, "modperl_io_apache_init" is not declared. (undeclared)
const void *modperl_hack_io_apache_init = (const void *)modperl_io_apache_init;



modperl_exports.c starts with: #include "mod_perl.h"

which includes

#include "modperl_io_apache.h"

which declares:

MP_INLINE void modperl_io_apache_init(pTHX);

though it's #ifdef'ed with:

#ifdef PERLIO_LAYERS

so that means that modperl_exports.c shouldn't include those symbols.

I suppose that we simply didn't test static builds with 5.6.x perls, and hence have missed this problem. let me try that first to see if I can reproduce that.


yes, i can confirm that it breaks with 5.6.1 (anywhere), which doesn't have PERLIO_LAYERS defined. Philippe? looks like with should either cut on 5.8.0 for static builds, or starting parsing the C code :( I think we will have to go with the latter, since PERLIO_LAYERS might be not defined with 5.8.x too.

In fact we already somewhat handle that in modperl_io_apache.h



#ifdef PERLIO_LAYERS
[...]
MP_INLINE void modperl_io_apache_init(pTHX);

#else /* #ifdef PERLIO_LAYERS */

#define modperl_io_apache_init(pTHX)

#endif /* #ifdef PERLIO_LAYERS */



but for some reason define doesn't do the trick, when it gets to do:

  const void *modperl_hack_io_apache_init =
     (const void *)modperl_io_apache_init;

should it be some magic VOID thing?

#define modperl_io_apache_init(pTHX) VOID

--
__________________________________________________________________
Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/     mod_perl Guide ---> http://perl.apache.org
mailto:[EMAIL PROTECTED] http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org   http://ticketmaster.com

--
Report problems: http://perl.apache.org/bugs/
Mail list info: http://perl.apache.org/maillist/modperl.html
List etiquette: http://perl.apache.org/maillist/email-etiquette.html

Reply via email to