Summary:ASTERISK-11525: [patch] chan_vpb doesn't compile against latest stable release of Voicetronix drivers (4.2.24)
Reporter:Mark Mickan (mmickan)Labels:
Date Opened:2008-02-27 00:02:05.000-0600Date Closed:2008-02-27 02:19:02.000-0600
Versions:Frequency of
Environment:Attachments:( 0) chan_vpb.cc.diff
( 1) module.h.diff
( 2) vpb.conf.sample
Description:There have been some API changes in the vpb-driver package, which the existing chan_vpb doesn't compile against.  Attached patch chan_vpb.cc.diff fixes this so it now compiles.  Tested here and works for us, along with a couple of clients.

Also attached is a very minimal patch for include/asterisk/module.h specifically for C++ modules (of which chan_vpb appears to be the only one), which moves AST_MODULE to a different position in the ast_module_info structure to match the structure definition.

To go with the updated chan_vpb is a much nicer looking and more complete vpb.conf.sample, which can replace the existing one.  This one assumes 0 cards rather than 1 card, so that systems with no Voicetronix cards won't segfault Asterisk on startup by default.


This issue was submitted previously as bug 10955, but was rejected due to problems with licensing... couldn't work out how to re-open that bug, so have opened this one in its place.  Patches have been updated since then too.  Licensing issues should be sorted - I've got a disclaimer on file and it's been accepted.
Comments:By: Digium Subversion (svnbot) 2008-02-27 02:19:02.000-0600

Repository: asterisk
Revision: 104502

U   trunk/channels/chan_vpb.cc
U   trunk/configs/vpb.conf.sample
U   trunk/include/asterisk/module.h

r104502 | tilghman | 2008-02-27 02:19:01 -0600 (Wed, 27 Feb 2008) | 8 lines

Bring Voicetronix driver up to date with current drivers
(closes issue ASTERISK-11525)
Reported by: mmickan
      chan_vpb.cc.diff uploaded by mmickan (license 400)
      module.h.diff uploaded by mmickan (license 400)
      vpb.conf.sample uploaded by mmickan (license 400)