Summary: | ASTERISK-15649: Template examples in documentation imply well defined overriding semantics, but this is not true | ||
Reporter: | David Woolley (davidw) | Labels: | |
Date Opened: | 2010-02-17 13:12:24.000-0600 | Date Closed: | 2010-03-02 14:46:53.000-0600 |
Priority: | Trivial | Regression? | No |
Status: | Closed/Complete | Components: | Documentation |
Versions: | Frequency of Occurrence | ||
Related Issues: | |||
Environment: | Attachments: | ||
Description: | tex/configuration.tex includes an example of template use that only makes sense if there are guaranteed semantics for overriding of template definitions by the definitions in the template user (specifically host is overridden in a sip.conf example). However, ASTERISK-14388 was rejected on the grounds that no such semantics exist. The documentation needs to be changed to remove these overriding examples and to give positive warnings that there are no guaranteed overriding semantics. ****** ADDITIONAL INFORMATION ****** This is an an abstract of configurations.tex illustrating its assumption that overriding works. [bar] permit=192.168.1.2 host=jkl deny=192.168.1.1 [baz](foo,bar) permit=192.168.3.1 host=bnm This only makes sense if host=bnm overrides host=jkl. The big difference in the code is whether modules use ast_variable_retrieve (overriding fails) or loop over ast_variable_browse, switching on each variable name. app_queue.c, for example, uses both methods, for different parameters. | ||
Comments: | By: Digium Subversion (svnbot) 2010-03-02 14:36:11.000-0600 Repository: asterisk Revision: 250037 U trunk/doc/tex/configuration.tex ------------------------------------------------------------------------ r250037 | lmadsen | 2010-03-02 14:36:11 -0600 (Tue, 02 Mar 2010) | 4 lines Update documentation to not imply we support overriding options. (closes issue ASTERISK-15649) Reported by: davidw ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=250037 By: Digium Subversion (svnbot) 2010-03-02 14:36:59.000-0600 Repository: asterisk Revision: 250038 _U branches/1.6.2/ U branches/1.6.2/doc/tex/configuration.tex ------------------------------------------------------------------------ r250038 | lmadsen | 2010-03-02 14:36:59 -0600 (Tue, 02 Mar 2010) | 12 lines Merged revisions 250037 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ........ r250037 | lmadsen | 2010-03-02 15:36:10 -0500 (Tue, 02 Mar 2010) | 4 lines Update documentation to not imply we support overriding options. (closes issue ASTERISK-15649) Reported by: davidw ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=250038 By: Digium Subversion (svnbot) 2010-03-02 14:38:23.000-0600 Repository: asterisk Revision: 250039 _U branches/1.6.1/ U branches/1.6.1/doc/tex/configuration.tex ------------------------------------------------------------------------ r250039 | lmadsen | 2010-03-02 14:38:22 -0600 (Tue, 02 Mar 2010) | 12 lines Merged revisions 250037 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ........ r250037 | lmadsen | 2010-03-02 15:36:10 -0500 (Tue, 02 Mar 2010) | 4 lines Update documentation to not imply we support overriding options. (closes issue ASTERISK-15649) Reported by: davidw ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=250039 By: Digium Subversion (svnbot) 2010-03-02 14:42:42.000-0600 Repository: asterisk Revision: 250040 U branches/1.6.0/doc/tex/configuration.tex ------------------------------------------------------------------------ r250040 | lmadsen | 2010-03-02 14:42:42 -0600 (Tue, 02 Mar 2010) | 12 lines Merged revisions 250037 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ........ r250037 | lmadsen | 2010-03-02 15:36:10 -0500 (Tue, 02 Mar 2010) | 4 lines Update documentation to not imply we support overriding options. (closes issue ASTERISK-15649) Reported by: davidw ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=250040 By: Digium Subversion (svnbot) 2010-03-02 14:45:38.000-0600 Repository: asterisk Revision: 250041 U branches/1.4/doc/configuration.txt ------------------------------------------------------------------------ r250041 | lmadsen | 2010-03-02 14:45:37 -0600 (Tue, 02 Mar 2010) | 4 lines Update documentation to not imply we support overriding options. (issue ASTERISK-15649) Reported by: davidw ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=250041 By: Digium Subversion (svnbot) 2010-03-02 14:46:53.000-0600 Repository: asterisk Revision: 250042 _U trunk/ ------------------------------------------------------------------------ r250042 | lmadsen | 2010-03-02 14:46:52 -0600 (Tue, 02 Mar 2010) | 12 lines Recorded merge of revisions 250041 via svnmerge from https://origsvn.digium.com/svn/asterisk/branches/1.4 ........ r250041 | lmadsen | 2010-03-02 15:45:37 -0500 (Tue, 02 Mar 2010) | 4 lines Update documentation to not imply we support overriding options. (issue ASTERISK-15649) Reported by: davidw ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=250042 |