<p>Kevin Harwell <strong>submitted</strong> this change.</p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/18616">View Change</a></p><div style="white-space:pre-wrap">Approvals:
  Joshua Colp: Looks good to me, but someone else must approve
  Kevin Harwell: Looks good to me, approved; Approved for Submit

</div><pre style="font-family: monospace,monospace; white-space: pre-wrap;">asterisk.c: Fix incompatibility warnings for remote console.<br><br>A previous review fixing ASTERISK_22246 and ASTERISK_26582<br>got a couple of the options mixed up as to whether or not<br>they are compatible with the remote console. This fixes<br>those to the best of my knowledge.<br><br>ASTERISK-30097 #close<br><br>Change-Id: Id54166991aa79f04fb02699cc499bedda854253b<br>---<br>M main/asterisk.c<br>1 file changed, 5 insertions(+), 3 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/main/asterisk.c b/main/asterisk.c</span><br><span>index b965a4d..9d1561e 100644</span><br><span>--- a/main/asterisk.c</span><br><span>+++ b/main/asterisk.c</span><br><span>@@ -3725,13 +3725,18 @@</span><br><span>                     switch (c) {</span><br><span>                         /* okay to run with remote console */</span><br><span>                        case 'B': /* force black background */</span><br><span style="color: hsl(120, 100%, 40%);">+                        case 'C': /* set config path */</span><br><span>                      case 'd': /* debug */</span><br><span>                        case 'h': /* help */</span><br><span>                         case 'I': /* obsolete timing option: warning already thrown if used */</span><br><span>                       case 'L': /* max load */</span><br><span>                     case 'M': /* max calls */</span><br><span style="color: hsl(120, 100%, 40%);">+                     case 'm': /* mute */</span><br><span style="color: hsl(120, 100%, 40%);">+                  /*! \note The q option is never used anywhere, only defined */</span><br><span style="color: hsl(120, 100%, 40%);">+                        case 'q': /* quiet */</span><br><span>                        case 'R': /* reconnect */</span><br><span>                    case 'r': /* remote */</span><br><span style="color: hsl(120, 100%, 40%);">+                        /*! \note Can ONLY be used with remote console */</span><br><span>                    case 's': /* set socket path */</span><br><span>                      case 'V': /* version */</span><br><span>                      case 'v': /* verbose */</span><br><span>@@ -3741,7 +3746,6 @@</span><br><span>                              break;</span><br><span>                       /* can only be run when Asterisk is starting */</span><br><span>                      case 'X': /* enables #exec for asterisk.conf only. */</span><br><span style="color: hsl(0, 100%, 40%);">-                   case 'C': /* set config path */</span><br><span>                      case 'c': /* foreground console */</span><br><span>                   case 'e': /* minimum memory free */</span><br><span>                  case 'F': /* always fork */</span><br><span>@@ -3749,10 +3753,8 @@</span><br><span>                         case 'G': /* run group */</span><br><span>                    case 'g': /* dump core */</span><br><span>                    case 'i': /* init keys */</span><br><span style="color: hsl(0, 100%, 40%);">-                       case 'm': /* mute */</span><br><span>                         case 'n': /* no color */</span><br><span>                     case 'p': /* high priority */</span><br><span style="color: hsl(0, 100%, 40%);">-                   case 'q': /* quiet */</span><br><span>                        case 'T': /* timestamp */</span><br><span>                    case 't': /* cache record files */</span><br><span>                   case 'U': /* run user */</span><br><span></span><br></pre><div style="white-space:pre-wrap"></div><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/18616">change 18616</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.asterisk.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://gerrit.asterisk.org/c/asterisk/+/18616"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Id54166991aa79f04fb02699cc499bedda854253b </div>
<div style="display:none"> Gerrit-Change-Number: 18616 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: Joshua Colp <jcolp@sangoma.com> </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-MessageType: merged </div>