<p> Attention is currently required from: Alexander Traud. </p>
<p>Patch set 3:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #d4ffd4; color: #000000;">Code-Review +1</span></p><p><a href="https://gerrit.asterisk.org/c/asterisk/+/17578">View Change</a></p><p>3 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/0caf25f6_f2441f2f">Patch Set #2, Line 22:</a> <code style="font-family:monospace,monospace">ASTERISK-26991</code></p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><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></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I personally would be fine with changing the wording around that. It could certainly be made less ambiguous.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Not sure about removing the whole make target though. Perhaps in master only. Otherwise it might break current user's build and install process. Perhaps add an 'apidoc' and/or 'doxygen' target(s) in 16, 18, 19, with 'progdocs' still as an alias. Then in master could remove what you want.</p></li></ul></li><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/+/17578?tab=comments">Patch Set #3:</a> </p><p style="white-space: pre-wrap; word-wrap: break-word;">Current patch looks good to me, so giving +1. However, feel free to update the after install text if you want.</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/172ed157_4382887d">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;">Yes. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">It's not a big deal to leave it, and seems like the safer route.</p><p style="white-space: pre-wrap; word-wrap: break-word;">However, I believe items listed in .gitignore are untracked files, so removing it would make 'asterisk-ng-doxygen' show up under "Untracked files" when you issue a "git status" command. Someone would then have to do a git add and commit for it to be uploaded.</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: 3 </div>
<div style="display:none"> Gerrit-Owner: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-Reviewer: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-Reviewer: Friendly Automation </div>
<div style="display:none"> Gerrit-Reviewer: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Gerrit-Attention: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 01 Dec 2021 17:09:36 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-Has-Labels: Yes </div>
<div style="display:none"> Comment-In-Reply-To: Kevin Harwell <kharwell@digium.com> </div>
<div style="display:none"> Comment-In-Reply-To: Alexander Traud <pabstraud@compuserve.com> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>