<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Dec 7, 2016 at 1:42 PM, Corey Farrell <span dir="ltr"><<a href="mailto:git@cfware.com" target="_blank">git@cfware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Overall I like it.  I think leaving pbx_ael out of this is best as it<br>
might not be straight forward.  Anyone who wants pbx_ael to provide<br>
information about config sources can create a follow-up patch once the<br>
core functionality is implemented.<br>
<br>
Some thoughts:<br>
* It would be nice if the registrar ("pbx_config:") were omitted when<br>
config file/line are available.<br>
* I assume we can strip 'ast_config_AST_CONFIG_DIR' from included<br>
config filenames?<br>
<br>
I suspect both of these things could be done in follow-up patches, I<br>
just want to consider limited terminal width and try to avoid unneeded<br>
wrapping.<br></blockquote><div><br></div><div>Omitting registrar when config file is available is certainly doable. It<br></div><div>might be worth holding out for additional opinions on that one, though<br></div><div>I can't imagine people actually *needing* to know the registrar when<br></div><div>they have the config file under normal circumstances.<br><br></div><div>Stripping the config directory shouldn't be too hard. Might just end up<br></div><div>stripping all the path info in general and just leaving the file name...<br></div><div>if people need to know the full path they can simply watch the verbose<br></div><div>output at startup.<br></div><div>-- <br></div></div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div><b>Jonathan R. Rose<br></b><span><span>Senior Systems Engineer<br><br></span></span></div>Emergency CallWorks<br></div>Motorola Solutions<br><br></div><div>email: <a href="mailto:jonathan.rose@motorolasolutions.com" target="_blank">jonathan.rose@motorolasolutions.com</a><br></div></div></div></div>
</div></div>