<p> Attention is currently required from: Mark Murawski. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/19939">View Change</a></p><p>11 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">Patchset:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/19939?tab=comments">Patch Set #2:</a> </p><p style="white-space: pre-wrap; word-wrap: break-word;">Applied fixes as per the code review from K.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File contrib/scripts/install_prereq:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/19939/comment/e12feadf_47f71740">Patch Set #2, Line 7:</a> <code style="font-family:monospace,monospace">installing</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Ack</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Fix for 80 column terminals. ✅ 2023-03-06 19:29:54</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/19939/comment/f322e966_7605e238">Patch Set #2, Line 9:</a> <code style="font-family:monospace,monospace"># - DragonFly</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Ack</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Alphabetically sort this list ✅ 2023-03-06 18:55:12</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/19939/comment/dd7dbd33_d3e73ea2">Patch Set #2, Line 25:</a> <code style="font-family:monospace,monospace"># Function name: handle_debian</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Ack</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Apologies. This is my styley. I have removed them now.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Remove unnecessary 'x'.  Straight string comparison is sufficient here. ✅ 2023-03-06 18:56:14</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/19939/comment/64acf430_99dcb060">Patch Set #2, Line 26:</a> <code style="font-family:monospace,monospace"># Date: 01/11/22</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Ack</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I don't think it's necessary to date every function here.  This is not a library/api file.  Same goes for all the others. ✅ 2023-03-06 19:02:49</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/19939/comment/396ce2da_21541717">Patch Set #2, Line 30:</a> <code style="font-family:monospace,monospace">    if ! [ -x "$(command -v aptitude)" ] ; then</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Ack</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">All indenting needs to be tabs ✅ 2023-03-06 18:57:53</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/19939/comment/b4307ab9_d695d9dc">Patch Set #2, Line 41:</a> <code style="font-family:monospace,monospace"> </code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Ack</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Remove unnecessary 'x'.  Straight string comparison is sufficient here. ✅ 2023-03-06 18:56:14</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/19939/comment/9d7d5e35_7dfd8496">Patch Set #2, Line 48:</a> <code style="font-family:monospace,monospace">    # We're done!</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Ack</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Don't need to write a 'done' comment for every return unless there's something special going on. ✅ 2023-03-06 19:03:49</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/19939/comment/8c15a094_0a1fd369">Patch Set #2, Line 60:</a> <code style="font-family:monospace,monospace">    if [ x"$extra_packs" != "x" ] ; then</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Ack</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Remove unnecessary 'x'.  Straight string comparison is sufficient here. ✅ 2023-03-06 18:56:14</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/19939/comment/5837f102_5de90a4b">Patch Set #2, Line 79:</a> <code style="font-family:monospace,monospace">    if [ x"$extra_packs" != "x" ] ; then</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Ack</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Remove unnecessary 'x'.  Straight string comparison is sufficient here. ✅ 2023-03-06 18:56:14</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.asterisk.org/c/asterisk/+/19939/comment/a15b8401_466cef72">Patch Set #2, Line 303:</a> <code style="font-family:monospace,monospace">        </code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Ack</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Fix coding style; if [ condition ]; then;   dostuff; fi ✅ 2023-03-06 19:09:30</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/19939">change 19939</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/+/19939"/><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: I4bd53ae429e113a76d13a23b48da714a8eefad26 </div>
<div style="display:none"> Gerrit-Change-Number: 19939 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Martin McCarthy <martin.c.mccarthy@outlook.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: Mark Murawski <markm@intellasoft.net> </div>
<div style="display:none"> Gerrit-Attention: Mark Murawski <markm@intellasoft.net> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 06 Mar 2023 19:30:51 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Comment-In-Reply-To: Mark Murawski <markm@intellasoft.net> </div>
<div style="display:none"> Comment-In-Reply-To: Martin McCarthy <martin.c.mccarthy@outlook.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>