Apologies for bringing up an old thread but I had hoped it would have continued on it&#39;s own and I take issue with Xorcom&#39;s code getting included and causing issue after issue.<br><br>Kevin said &quot;I believe that <span class="nfakPe">redfone</span> did submit their driver for inclusion into<br>

Zaptel, and that it was rejected for the reasons you mentioned (coding<br>
guidelines and code quality). We do not accept code that does not<br>
conform to at least reasonable standards of quality, because doing so<br>
would allow the code base to deteriorate and increase everyone else&#39;s<br>
support burden.&quot;<br><br>Joeseph has a different story.&nbsp; I would like to know what the &quot;REAL&quot; reason is for not including his code.<br><br>I would also like to know why xpp (Xorcom) stuff has it&#39;s own Zaptel release bug fixes (and I assume DAHDI as well) because it is released buggy and won&#39;t build without hacking source code.&nbsp; By default, I do make menuselect and remove xpp now because I don&#39;t feel like fixing buggy Xorcom code.<br>
<br>I only mention this because of Kevin&#39;s statements about not meeting coding guidlines.&nbsp; I would assume part of coding guidelines is the ability to build from source provided dependencies are met,&nbsp; without crapping out.<br>
<br>Kevin Flemming &quot;We do not accept code that does not<br>

conform to at least reasonable standards of quality, because doing so<br>

would allow the code base to deteriorate and increase everyone else&#39;s<br>

support burden.&quot;<br><br>-- <br>Thanks,<br>Steve Totaro <br>+18887771888 (Toll Free)<br>+12409381212 (Cell)<br>+12024369784 (Skype)<br><br><div class="gmail_quote">On Fri, Oct 31, 2008 at 6:46 PM, Joseph Benden <span dir="ltr">&lt;<a href="mailto:joe@thrallingpenguin.com">joe@thrallingpenguin.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">Hello all,<br>
<br>
I am the author and maintainer of TDMoE-Multiframe.<br>
<br>
[please do not read the following as an attack, but rather simple<br>
comments.]<br>
<br>
The code did not have any major coding or quality issues, but rather<br>
had licensing issues because I had an old license on file and Digium<br>
switching to a newer license (which I was unaware of the change), I<br>
submitted the new license (which was approved) and now the code sits<br>
and rots in the tracker, along with the new DAHDI implementation of<br>
the driver.<br>
<br>
This is not unusual around this part of the open source world. I&#39;ve<br>
had other patches and contributions that have experienced the same fate.<br>
<br>
I do deal (program, patch, etc.) with other open source software and<br>
do not have this much trouble in having the maintainers accept the<br>
code. I assume their projects are just as widespread (proftpd, clamav,<br>
freebsd), so using the number of outstanding patches as a reason for<br>
not accepting them is not convincing to me.<br>
<br>
Because of the difficulties in having Asterisk accept the Solaris<br>
community (read as: accept high availability patches and Solaris/<br>
OpenSolaris specific patches), I had to fork the project which then<br>
became SolarisVoip.com<br>
<br>
I am trying to work towards updating SolarisVoip.com to 1.6 (including<br>
the DAHDI drivers), but am very hesitant. Would Digium finally agree<br>
to working with me in possibly having my yet-to-be patches for<br>
Solaris? How about a branch/module for the Solaris specific DAHDI<br>
drivers? I tried in the past at working with Digium, but was always<br>
pushed to the side and forgotten (even though some within Digium<br>
seemed happy to work with me.)<br>
<br>
Comments, suggestions, or ideas welcomed!<br>
<br>
Best Regards,<br>
Joseph Benden<br>
<br>
 &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;.--.<br>
 &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; |o_o |<br>
 &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; |:_/ |<br>
 &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;// &nbsp; \ \<br>
 &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; (| &nbsp; &nbsp; | )<br>
 &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; /&#39;\_ &nbsp; _/`\<br>
 &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; \___)=(___/<br>
<a href="http://www.ThrallingPenguin.com/" target="_blank">http://www.ThrallingPenguin.com/</a><br>
--------------------------------<br>
 &nbsp;We design, develop, and extend<br>
 &nbsp;software technologies for the<br>
 &nbsp; &nbsp; most demanding business<br>
 &nbsp; &nbsp;applications, as well as<br>
 &nbsp; &nbsp; &nbsp;offer VoIP Consulting<br>
 &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;services.<br>
<div><div></div><div class="Wj3C7c"><br>
<br>
<br>
On Oct 31, 2008, at 6:21 PM, Kevin P. Fleming wrote:<br>
<br>
&gt; Kevin P. Fleming wrote:<br>
&gt;&gt; Bill Michaelson wrote:<br>
&gt;&gt;<br>
&gt;&gt;&gt; ztd-ethmf.c<br>
&gt;&gt;<br>
&gt;&gt; As I said, that driver has been part of Zaptel, so could not have<br>
&gt;&gt; been<br>
&gt;&gt; removed.<br>
&gt;<br>
&gt; Correction: it was in Zaptel SVN for a very short time, it had been<br>
&gt; committed without proper code review. It was removed quickly<br>
&gt; thereafter,<br>
&gt; and was never part of an Zaptel release.<br>
&gt;<br>
&gt; --<br>
&gt; Kevin P. Fleming<br>
&gt; Director of Software Technologies<br>
&gt; Digium, Inc. - &quot;The Genuine Asterisk Experience&quot; (TM)<br>
&gt;<br>
&gt; _______________________________________________<br>
&gt; --Bandwidth and Colocation Provided by <a href="http://www.api-digital.com--" target="_blank">http://www.api-digital.com--</a><br>
&gt;<br>
&gt; asterisk-biz mailing list<br>
&gt; To UNSUBSCRIBE or update options visit:<br>
&gt; &nbsp; <a href="http://lists.digium.com/mailman/listinfo/asterisk-biz" target="_blank">http://lists.digium.com/mailman/listinfo/asterisk-biz</a><br>
&gt;<br>
<br>
<br>
_______________________________________________<br>
--Bandwidth and Colocation Provided by <a href="http://www.api-digital.com--" target="_blank">http://www.api-digital.com--</a><br>
<br>
asterisk-biz mailing list<br>
To UNSUBSCRIBE or update options visit:<br>
 &nbsp; <a href="http://lists.digium.com/mailman/listinfo/asterisk-biz" target="_blank">http://lists.digium.com/mailman/listinfo/asterisk-biz</a><br>
</div></div></blockquote></div><br><br clear="all"><br>