<p>Evandro César Arruda <strong>posted comments</strong> on this change.</p><p><a href="https://gerrit.asterisk.org/8767">View Change</a></p><p>Patch set 1:</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">Sorry, I am just seeing the comments from the other reviews (the<br>reviews were not properly linked).</p><p style="white-space: pre-wrap; word-wrap: break-word;">I still am not sure why this change is necessary as the define is<br>there, even in mariadb.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">In this case you are wrong...</p><p style="white-space: pre-wrap; word-wrap: break-word;">/usr/include/mysql# cat mysql.h | grep  \_version | grep include<br>#include "mariadb_version.h"<br>/usr/include/mysql# cat mariadb_version.h | grep 3306<br>#define MARIADB_PORT                  3306</p><p style="white-space: pre-wrap; word-wrap: break-word;">MariaDB stopped use of MYSQL_PORT declaration, or leave patch just checking MYSQL_PORT or create some check about MYSQL_PORT and MARIADB_PORT and set correct MYSQL_PORT value. What you think about it?</p><ul style="list-style: none; padding-left: 20px;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/8767">change 8767</a>. To unsubscribe, 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/8767"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 13 </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ieee56c836fc2e8bd021c456145bba04c6068bb77 </div>
<div style="display:none"> Gerrit-Change-Number: 8767 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Evandro César Arruda <ecarruda@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Evandro César Arruda <ecarruda@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins2 </div>
<div style="display:none"> Gerrit-Reviewer: Sean Bright <sean.bright@gmail.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 11 Apr 2018 18:39:53 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>