[Home]

Summary:ASTERISK-03383: res_perl no longer compiles (config_pvt.h) cvs head
Reporter:k3v (k3v)Labels:
Date Opened:2005-01-29 21:14:34.000-0600Date Closed:2005-02-06 20:57:48.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Core/Configuration
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) addons_config_fixes_rev1.diff.txt
Description:res_perl.h includes config.h and config_pvt.h.

In file included from res_perl.h:26,
                from res_perl.c:17:
/usr/src/asterisk/include/asterisk/config_pvt.h:59: error: conflicting types for `ast_internal_load'
/usr/src/asterisk/include/asterisk/config_old.h:40: error: previous declaration of `ast_internal_load'
res_perl.c: In function `_load_module':
res_perl.c:995: warning: assignment from incompatible pointer type

Compiled fine against last week (20050120) cvs head.  CVS log shows a bunch of stuff changing wrt config.
Comments:By: Kevin P. Fleming (kpfleming) 2005-01-30 12:38:03.000-0600

Uploaded patch contains proposed fixes for res_perl, res_sqlite3 and cdr_addon_mysql (all the remaining parts of asterisk-addons that were affected by ASTERISK-3373406).

By: k3v (k3v) 2005-01-30 18:14:59.000-0600

Compiles cleanly.  Tested app_perl and cdr_mysql, which seem to work fine.  Thanks!  I don't have anything cooked up that tests the config engine though.

edited on: 01-30-05 18:34

By: Anthony Minessale (anthm) 2005-01-31 09:22:07.000-0600

I applied the patch on res_sqlite and cdr_mysql
I already had res_perl underway and I committed that too.
Thanks