<p> Attention is currently required from: Sean Bright, N A. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/17362">View Change</a></p><p>1 comment:</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/+/17362?tab=comments">Patch Set #2:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">The commit message and the JIRA issue seem to suggest that these changes are fixing compilation prob […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Regarding the main/http.c patch, it looks like it is actually obsolete.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Looking at the FreeBSD ports commit history I could not find any definitive information, but it was added in the timeframe when FreeBSD project moved to clang.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Early clang had issues with the RAII_VAR construct.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Modern clang has no such issues. So that patch can most probably be dropped. I'm testing to remove it from FreeBSD ports tree.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The channels/chan_dahdi.c is a different thing, I think. I have never used dahdi hardware, but I suspect the FreeBSD dahdi driver is slightly different from the linux one. Maybe the one we have in the tree is also slightly older, and requires different interfacing. But I'm not sure right now about this. I need to do some studying.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/17362">change 17362</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/+/17362"/><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: Ice4f615a9ea4662dd6d83594f0c9989e1b5251c0 </div>
<div style="display:none"> Gerrit-Change-Number: 17362 </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: Sean Bright <sean@seanbright.com> </div>
<div style="display:none"> Gerrit-CC: Guido Falsi <madpilot@freebsd.org> </div>
<div style="display:none"> Gerrit-Attention: Sean Bright <sean@seanbright.com> </div>
<div style="display:none"> Gerrit-Attention: N A <mail@interlinked.x10host.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 16 Nov 2021 19:41:53 +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: Sean Bright <sean@seanbright.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>