<p> Attention is currently required from: Kevin Harwell. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/17578">View Change</a></p><p>2 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="null">Commit Message:</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/+/17578/comment/ab7d6b65_dc11849e">Patch Set #2, Line 22:</a> <code style="font-family:monospace,monospace">ASTERISK-26991</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Is this fixing this issue, thus should close it? I believe the auto-close script just looks for ASTE […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">To answer your question: Yes, closing is intended.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Long answer:</p><p style="white-space: pre-wrap; word-wrap: break-word;">ASTERISK-20259 did not fix all issues. Or more precisely, did a lot but was not finished, unfortunately. Now, with this change, I consider ASTERISK-20259 fixed, at least syntactically. Consistent style over all Doxygen comments and a birds-eye-view (semantically) is another story. Anyway, ASTERISK-26991 was about finding an alternative for the developer to get the Doxygen documentation. Now that ASTERISK-20259 is considered fixed, I think, the developer has a working alternative, especially because it is advertised after ‘make install’. And that closes ASTERISK-26991.</p><p style="white-space: pre-wrap; word-wrap: break-word;">If I am not totally mistaken, the server http://doxygen.asterisk.org is not up anymore (since Sep. 2018) without replacement. So the problem about search-engine indexing is fixed as well. Finally, after going through the Doxygen comments, that documentation look (for me) more like API documentation. Therefore it is not targeted to normal users but API users, developers. And I consider those should be able to build the API documentation locally if they really need it.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Looking at what I said now, I am not sure about the advertisement after ‘make install’ anymore. Is that advertisement helpful? It might confuse normal users, not being API developers. Perhaps the wording should be changed from ‘program doc’ to ‘API doc’ and tell the user that the ‘usage doc’ was already built because of the XML documentation. What do you think? Perhaps even the whole make target should be removed and renamed to ‘apidoc’ or just ‘doxygen’ to be inline with the xml-doc which is a usage-doc actually.</p><p style="white-space: pre-wrap; word-wrap: break-word;">By the way, https://doxygen.phreaknet.org has a current online copy.</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="null">File doc/.gitignore:</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/+/17578/comment/f86886ca_60062445">Patch Set #2, Line 5:</a> <code style="font-family:monospace,monospace">asterisk-ng-doxygen</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">Any reason to not just remove this?</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">Yes. The developer might still have it in its local copy, and then it would be uploaded after that change. I am not sure how to handle gitignore in Asterisk, being cumulative and avoiding any mistakes from past/future or list just current possible files. I opted for the former. If you like to have the latter, please, just say so. It was more like flipping-a-coin decision, keep it or drop it. I am fine with either way but had to make a decision. Therefore, I went for the more conservative approach in case of gitignore. With 'make distclean’, I went for the ‘opposite’.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.asterisk.org/c/asterisk/+/17578">change 17578</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/+/17578"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: asterisk </div>
<div style="display:none"> Gerrit-Branch: 16 </div>
<div style="display:none"> Gerrit-Change-Id: I4129092a199d5e24c319a09cd088614b121015af </div>
<div style="display:none"> Gerrit-Change-Number: 17578 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-CC: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Attention: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 01 Dec 2021 09:03:45 +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: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>