<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">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;">Patch Set 1: Code-Review-1</p><p style="white-space: pre-wrap; word-wrap: break-word;">Hi Guido.  I see you submitted a pull request to pjproject for this.  That's great!<br>https://github.com/pjsip/pjproject/pull/2426</p><p style="white-space: pre-wrap; word-wrap: break-word;">I'm marking these reviews -1 temporarily until they've accepted the pull request.<br>When they do, can you replace the patch file with "git show" for that commit so we have the commit message and hash from the pjproject repo?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Unluckily the patch present in the pull request is against pjproject 2.10 sources, the patch required for pjproject 2.9 bundled in asterisk is slightly different.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Anyway I can grab that patch and manually adapt it, keeping the headers. If this is acceptable.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;"><br>In the mean time, we'll let them know that you're an Asterisk contributor which should speed up the process.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Thanks. I'm also going to include these patches in the FreeBSD asterisk ans pjsip ports. These are used by many FreeBSD users.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">How about if you just copy the text header part of the "git show" that has the commit, date, commit message etc. into the current patch file.  That way we'll at least know it made it into the pjproject repo.<br></p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/14401">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/14401">change 14401</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/+/14401"/><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-Change-Id: Ie1b0dc365dafe4a84c5248097fe8d73804043c22 </div>
<div style="display:none"> Gerrit-Change-Number: 14401 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Guido Falsi <madpilot@freebsd.org> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: George Joseph <gjoseph@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 08 May 2020 16:05:26 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-Has-Labels: No </div>
<div style="display:none"> Gerrit-MessageType: comment </div>