<span style="font-family: Arial, Helvetica, sans-serif; font-size: 10pt"><span style="font-family: tahoma,arial,sans-serif; font-size: 10pt;"> <hr width="100%" size="2" align="center" />
<b>From</b>: "Kevin P. Fleming" <kpfleming@digium.com><br />
<b>Sent</b>: Monday, January 31, 2011 5:13 PM<br />
<b>To</b>: asterisk-users@lists.digium.com<br />
<b>Subject</b>: Re: [asterisk-users] res_fax</span><br />
<br />
On 01/31/2011 02:08 PM, Bryant Zimmerman wrote:<br />
> ------------------------------------------------------------------------<br />
> *From*: "Kevin P. Fleming" <kpfleming@digium.com><br />
> *Sent*: Thursday, January 27, 2011 3:08 PM<br />
> *To*: asterisk-users@lists.digium.com<br />
> *Subject*: Re: [asterisk-users] res_fax<br />
><br />
> On 01/27/2011 09:21 AM, Bryant Zimmerman wrote:<br />
>><br />
>> > Kevin<br />
>> ><br />
>> > That is grate. I dove into the code and tried to add it my self I added<br />
>> > a F option but I have not figured out the right spot to force the<br />
>> > exclusion to shut off the T38.<br />
>> ><br />
>> > Where will the patch be posted?<br />
>><br />
>> http://svnview.digium.com/svn/asterisk?view=rev&rev=304342<br />
>><br />
>> Kevin<br />
>><br />
>> I tried everthing I could think of to get the n option to work last<br />
>> night but it would not do a complete shut off of the T.38 option and<br />
>> would not receive a fax. What do you need from me on the debug side so I<br />
>> can help you get it working as expected?<br />
><br />
> Revision 304599 should fix this (and I also changed the option letter<br />
> from 'n' to 'F' since it really means 'force audio').<br />
> _____________________________________________________________________<br />
><br />
> Kevin<br />
><br />
> The 304599 rev does seem to work good. I just finished my testing on it<br />
> and the F option works great.<br />
> I have three more test to do and if they pass it should be good to go.<br />
> When could it get into the releases?<br />
<br />
It's a new feature, so it won't go into any existing release branches; <br />
the first release that will have this addition is Asterisk 1.10.1. Of <br />
course, the patch is quite small as you've seen, so it will be easy for <br />
you to apply it to your installations.<br />
<br />
_____________________________________________________________________<br />
<br />
Kevin<br />
<br />
I just replaced the res_fax.c file with the one from 304599. Would I just keep doing that as I step forward on versions of 1.8.x?<br />
If this is the case how would I get any other critical changes to res_fax.c that may occur after rev 304599?<br />
How would I create a patch that would allow me to apply it to additional release version of asterisk.<br />
Sorry for the simple questions I do most of my dev on windows machines and this process is a still a bit confusing to me.<br />
<br />
Thanks<br />
Bryant<br /></span>