[Home]

Summary:ASTERISK-01417: [patch] Zonedata update for Australia data
Reporter:duane (duane)Labels:
Date Opened:2004-04-15 22:08:22Date Closed:2008-06-07 10:41:43
Priority:MinorRegression?No
Status:Closed/CompleteComponents:Core/General
Versions:Frequency of
Occurrence
Related
Issues:
Environment:Attachments:( 0) aussiezones.diff
Description:The normal zone settings for australia sound a little off, this isn't by me, was posted to the mailing list earlier in the week by ben at team dot dcsi dot net dot au
Comments:By: Brian West (bkw918) 2004-04-15 22:40:27

That person that posted it will HAVE to disclaim this.  before it can go.. can you please contact them and have them fill out the info?

By: ben_au (ben_au) 2004-04-16 00:20:04

I've faxed in my disclaimer in!

There has actually been a slight change to the indications. My patch is right. bug_id=0001396

The change was to the dialtone and ringtones, where you see 400+425 it should read 413+438.

By: duane (duane) 2004-04-16 00:26:06

I saw the other bug patch, but it didn't address the incorrect sounding dial tone  in the zaptel driver which is why I filed this one...

By: ben_au (ben_au) 2004-04-16 00:35:04

Yeah, my patch is for indications.conf.

The dialtone should be 413+438, the patch file here is using 400+425 -- which is what I originally thought it was.

By: duane (duane) 2004-04-16 00:36:59

I've updated the patch and uploaded it but I can't work out how to remove the old patch...

By: ben_au (ben_au) 2004-04-16 00:57:49

That was quick! I don't think you can remove a file yourself unless you're a bug marshall like bkw -- he can fix it!

By: ben_au (ben_au) 2004-04-16 04:16:00

Doh! One more thing... this is the last change -- promise*!

The info tone is 400, do you want to change that to 425

I've sent up the updated patch for 0001396 (indications.conf)

* - Not a real promise.

By: duane (duane) 2004-04-16 10:43:07

Patches don't take long to build if I have a second free to do it, uploaded a new patch with your suggested change as well, if everything is fine I think oej is happy to send this to CVS...

By: twisted (twisted) 2004-05-02 01:15:43

ok... for some reason that patch wouldn't apply.. I fixed that, but, These are NOT to spec per ITU docs, is there a reason for this?

edited on: 05-02-04 10:52

By: ben_au (ben_au) 2004-05-11 05:23:50

I don't know who the ITU think they are! The tone definitions were taken from Standard Australia and the ACA.

By: Olle Johansson (oej) 2004-05-14 08:33:33

Seems like we have an agreement that this patch is the way to go. Ready for CVS

By: Mark Spencer (markster) 2004-05-25 01:39:10

Fixed in CVS

By: Digium Subversion (svnbot) 2008-06-07 10:41:43

Repository: dahdi
Revision: 398

U   trunk/zonedata.c

------------------------------------------------------------------------
r398 | markster | 2008-06-07 10:41:42 -0500 (Sat, 07 Jun 2008) | 2 lines

Repair australian tone zones (bug ASTERISK-1417)

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

http://svn.digium.com/view/dahdi?view=rev&revision=398