Hello!<br><br><div class="gmail_quote">On Wed, Sep 16, 2009 at 1:14 AM, Kevin P. Fleming <span dir="ltr">&lt;<a href="mailto:kpfleming@digium.com" target="_blank">kpfleming@digium.com</a>&gt;</span> wrote:</div><div class="gmail_quote">
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">SVN commits to the DAHDI project wrote:<br>
<br>
&gt; Modified: tools/trunk/dahdi_monitor.c<br>
&gt; URL: <a href="http://svn.asterisk.org/svn-view/dahdi/tools/trunk/dahdi_monitor.c?view=diff&amp;rev=7134&amp;r1=7133&amp;r2=7134" target="_blank">http://svn.asterisk.org/svn-view/dahdi/tools/trunk/dahdi_monitor.c?view=diff&amp;rev=7134&amp;r1=7133&amp;r2=7134</a><br>


&gt; ==============================================================================<br>
&gt; --- tools/trunk/dahdi_monitor.c (original)<br>
&gt; +++ tools/trunk/dahdi_monitor.c Tue Sep 15 12:51:15 2009<br>
&gt; @@ -42,7 +42,11 @@<br>
&gt;  #include &quot;dahdi_tools_version.h&quot;<br>
&gt;  #include &quot;wavformat.h&quot;<br>
&gt;<br>
&gt; +#if defined(__FreeBSD__)<br>
&gt; +#include &lt;sys/soundcard.h&gt;<br>
&gt; +#else<br>
&gt;  #include &lt;linux/soundcard.h&gt;<br>
&gt; +#endif<br>
<br>
Please use configure script tests for this sort of thing, instead of<br>
hardcoded defines in source files.<br></blockquote><div><br></div><div>This has already been fixed.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

&gt; +#if defined(__FreeBSD__)<br>
&gt; +     if ((res = ioctl(fd, DAHDI_LOADZONE, &amp;h))) {<br>
&gt; +#else<br>
&gt;       if ((res = ioctl(fd, DAHDI_LOADZONE, h))) {<br>
&gt; +#endif<br>
&gt;               fprintf(stderr, &quot;ioctl(DAHDI_LOADZONE) failed: %s\n&quot;, strerror(errno));<br>
&gt;               return res;<br>
&gt;       }<br>
<br>
This is not acceptable; the DAHDI_LOADZONE API must be the same between<br>
all supported DAHDI platforms. If FreeBSD cannot support an ioctl that<br>
has a bare &#39;int&#39; argument, then there must be a new version of<br>
DAHDI_LOADZONE added for FreeBSD *and* Linux that accepts an &#39;int *&#39;,<br>
and the old version of DAHDI_LOADZONE must be supported on Linux via the<br>
compatibility mechanism we already use.</blockquote><div><br></div><div>Attached patches should fix it.</div><div><br></div><div>Max</div></div>