[asterisk-bugs] [Asterisk 0013786]: [patch] DAHDI_CHECK_HOOKSTATE automatically defined when chan_dahdi is built with zaptel support

[Asterisk Bug Tracker]

The following issue has been RESOLVED. 
====================================================================== 
http://bugs.digium.com/view.php?id=13786 
====================================================================== 
Reported By:                tzafrir
Assigned To:                jpeeler
====================================================================== 
Project:                    Asterisk
Issue ID:                   13786
Category:                   Channels/chan_dahdi
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     resolved
Asterisk Version:           SVN 
SVN Branch (only for SVN checkouts, not tarball releases):  1.4  
SVN Revision (number only!): 151905 
Disclaimer on File?:        N/A 
Request Review:              
Resolution:                 fixed
Fixed in Version:           
====================================================================== 
Date Submitted:             2008-10-26 06:15 CDT
Last Modified:              2008-11-21 18:04 CST
====================================================================== 
Summary:                    [patch] DAHDI_CHECK_HOOKSTATE automatically defined
when chan_dahdi is built with zaptel support
Description: 
chan_dahdi.c has the following compile time option:

/*! \note
 * Define if you want to check the hook state for an FXO (FXS signalled)
interface
 * before dialing on it.  Certain FXO interfaces always think they're out
of
 * service with this method however.
 */
/* #define DAHDI_CHECK_HOOKSTATE */

Sadly, DAHDI_CHECK_HOOKSTATE is defined unconditionally in
include/asterisk/dahdi_compat.h in case you built it with zaptel support:

#define DAHDI_CHECK_HOOKSTATE ZT_CHECK_HOOKSTATE

Thus it should be made conditioned on the definition of
ZT_CHECK_HOOKSTATE, just in case someone had it in a build script.

At first glance it seems that the same applies to:

#ifdef DAHDI_EVENT_EC_DISABLED
#ifdef DAHDI_EVENT_RINGBEGIN
#ifdef DAHDI_GETCONFMUTE
#ifdef DAHDI_PRI
#ifdef DAHDI_TONEDETECT

======================================================================
Relationships       ID      Summary
----------------------------------------------------------------------
has duplicate       0013927 Unable to Dial() through FXO port until...
====================================================================== 

---------------------------------------------------------------------- 
 (0095313) svnbot (reporter) - 2008-11-21 18:04
 http://bugs.digium.com/view.php?id=13786#c95313 
---------------------------------------------------------------------- 
Repository: asterisk
Revision: 158629

U   branches/1.4/channels/chan_dahdi.c
U   branches/1.4/include/asterisk/dahdi_compat.h

------------------------------------------------------------------------
r158629 | jpeeler | 2008-11-21 18:04:44 -0600 (Fri, 21 Nov 2008) | 5 lines

(closes issue http://bugs.digium.com/view.php?id=13786)
Reported by: tzafrir

When compiling against Zaptel dahdi_compat will now only define all the
DAHDI defines if the Zaptel define is present. Also, there is no such thing
as DAHDI_PRI.

------------------------------------------------------------------------

http://svn.digium.com/view/asterisk?view=rev&revision=158629 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2008-11-21 18:04 svnbot         Checkin                                      
2008-11-21 18:04 svnbot         Note Added: 0095313                          
2008-11-21 18:04 svnbot         Status                   assigned => resolved
2008-11-21 18:04 svnbot         Resolution               open => fixed       
======================================================================