[asterisk-bugs] [Asterisk 0014325]: [patch] Config file layout

[Asterisk Bug Tracker]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=14325 
====================================================================== 
Reported By:                DEA
Assigned To:                mvanbaak
====================================================================== 
Project:                    Asterisk
Issue ID:                   14325
Category:                   Channels/chan_skinny
Reproducibility:            always
Severity:                   trivial
Priority:                   normal
Status:                     assigned
Asterisk Version:           SVN 
Regression:                 No 
SVN Branch (only for SVN checkouts, not tarball releases):  trunk 
SVN Revision (number only!): 170794 
Request Review:              
====================================================================== 
Date Submitted:             2009-01-24 18:03 CST
Last Modified:              2009-01-26 14:21 CST
====================================================================== 
Summary:                    [patch] Config file layout
Description: 
Now that chan skinny uses seperate configuration sections for
lines and devices, it is important to have the [lines] section
before the [devices]

I noticed after updating trunk that while my skinny device was working,
line specific values had not been loaded, such as caller-id, line lable
or voicemail box.
====================================================================== 

---------------------------------------------------------------------- 
 (0098785) wedhorn (reporter) - 2009-01-26 14:21
 http://bugs.digium.com/view.php?id=14325#c98785 
---------------------------------------------------------------------- 
DEA, I was away for a few days so didn't get to have a look at this one
before it was committed. The reload stuff (rev:171081) loads the lines
section (line 6681) and then the devices section (line 6691).

I can't see how your patch would have helped which seems to indicate that
there is an issue in the code that we should probably fix. The configuring
of lines and devices is logged into 'messages' which should indicate that
the lines are being loaded first.

As your patch has the effect of obscuring an issue in the code (it is
already meant to do lines section first), could you please provide some
more info so this can be fixed. 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2009-01-26 14:21 wedhorn        Note Added: 0098785                          
======================================================================