[Home]

Summary:ASTERISK-15000: Phone init problem solved with backport.
Reporter:BrettS (ughnz)Labels:
Date Opened:2009-10-16 20:45:40Date Closed:2009-11-23 16:01:01.000-0600
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Addons/chan_mobile
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) chan_mobile.zip
Description:Back ported trunk of addons with no major issues and it is working well.

Would it be worth supplying a diff for others that may have the same problems with the 1.6.2 addons? Is it even permitted to supply backport diffs? Could I even supply the modified source code?

Thanks Brett.
Comments:By: Tyler Oderkirk (tylero) 2009-10-22 01:29:01

Brett,

The GPL under which Matthew et. al. have licensed chan_mobile.c explicitly grants you the right to distribute modified versions of the code.

I don't see anything barring such action in the "Issue Tracker Guidelines" [0]  either, not even in the "Contributor License Agreement" section.

I think I'm having the same trouble that you reported in bug 0015549 [1]...

Would you mind uploading your backport patch or the "final product"?

Thanks!

-Tyler

[0] http://www.asterisk.org/developers/bug-guidelines
[1] https://issues.asterisk.org/view.php?id=15549

By: BrettS (ughnz) 2009-10-23 18:08:24

Hi,

I have attached the modified chan_mobile.c

To build I downloaded asterisk-addons-1.6.2.0-rc1 and replaced the chan_mobile.c and did the usual ./configure make make install

Hope it works for you.

Cheers Brett.

By: Digium Subversion (svnbot) 2009-11-23 16:01:00.000-0600

Repository: asterisk-addons
Revision: 1060

U   branches/1.6.2/channels/chan_mobile.c

------------------------------------------------------------------------
r1060 | mnicholson | 2009-11-23 16:00:59 -0600 (Mon, 23 Nov 2009) | 9 lines

Backport of phone init code from trunk.

(closes issue ASTERISK-14577)
Reported by: ughnz

(closes issue ASTERISK-15000)
Reported by: ughnz


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

http://svn.digium.com/view/asterisk-addons?view=rev&revision=1060