<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sat, Apr 5, 2014 at 4:46 PM, Paul Belanger <span dir="ltr"><<a href="mailto:paul.belanger@polybeacon.com" target="_blank">paul.belanger@polybeacon.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">On Sat, Apr 5, 2014 at 2:58 AM, Olle E. Johansson <<a href="mailto:oej@edvina.net">oej@edvina.net</a>> wrote:<br>


><br>
> On 04 Apr 2014, at 20:32, SVN commits to the Digium repositories <<a href="mailto:svn-commits@lists.digium.com">svn-commits@lists.digium.com</a>> wrote:<br>
><br>
>> -             case 'I':<br>
>> -                     ast_set_flag(&ast_options, AST_OPT_FLAG_INTERNAL_TIMING);<br>
>> -                     break;<br>
><br>
> Just checking... I would rather add a NOTICE log here that "i" is not needed any more. Please make sure that configurations starting with "- i" will not suddenly fail.<br>
><br>
</div>I agree with Olle here, this seems to be a massive change mid-release.<br>
 Removing a command-line option is certainly going to break some<br>
peoples boxes.  Why not a deprecated warning and then removal from<br>
trunk to give people time to react?<br></blockquote><div><br></div><div>FWIW, specifying this command line option or asterisk.conf option, even after it has been removed, should be fine.  It will just be ignored and no new warnings will be generated, AFAICT.</div>

<div><br></div><div>On the surface, this looks like a change that shouldn't be made in a release branch.  However, this really is an option that should have never existed.  It's never the right thing to turn it off.  The change to make it the only way it works is really the right thing to do.  It was equivalent to an option called "make_things_work_properly=yes".</div>

<div><br></div><div>-- </div><div>Russell Bryant</div></div></div></div>