<p> Attention is currently required from: Alexander Traud. </p>
<p><a href="https://gerrit.asterisk.org/c/asterisk/+/17578">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/+/17578?tab=comments">Patch Set #3:</a> </p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">So, you want to see the warnings. And your suggestion is to bring back doxygen.log. I have to understand that first. That is the reason for the -1. And that is your suggestion to solve the -1. Correct?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Yes.  The whole point of this review is to get someone to do something about the warnings right?  If the warnings are lost in all the other output from doxygen, no one will see them.  That's the situation today.  If you produce more warnings, they're still going to get lost in the normal output.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">What do you mean with Wiki, update the Wiki about make progdocs and add there that a doxygen.log is generated?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I meant that we could remind folks via the Wiki and asterisk-dev list that if they create or change doxygen comments, they should run "make progdocs" and examine <br>doxygen.log for issues.</p><pre style="font-family: monospace,monospace; white-space: pre-wrap;"> <br>> I fixed this issue in around a dozen projects, and none of them worked with a logfile, some even bigger (and with much more output in Doxygen) then Asterisk. You do not even have to save/pipe the output, just search. </pre><p style="white-space: pre-wrap; word-wrap: break-word;">Search what?</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">a) bring back doxygen.log and place an echo in the Makefile that there is a doxygen.log?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Yes.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">b) now that we have zero warnings (right now), what about error on first warning b1) in Developer Mode b2) in general?</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">This could mess up build scripts that aren't expecting the failure.</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">c) filter the output of Doxygen via the Makefile via your proposed sed and display that.</p></blockquote><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">d) In he Makefile, I could redirect stdout to nowhere and just display stderr.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">The reason I suggested the filter was because the generation process takes so long. If there are no progress messages at all, you won't know what's happening.  If you can find a way to reduce doxygen and dot's verbosity to just a handful of progress messages, that would be better.  In fact, I forgot about filtering dot's output in my sed command.</p><p style="white-space: pre-wrap; word-wrap: break-word;">sed -r -e "/(Reading|Searching|Parsing|Preprocessing|(Generating (docs|code))|Running dot|Patching output)/d"</p><p style="white-space: pre-wrap; word-wrap: break-word;">To summarize...<br>We need to keep doxygen.log with warnings and errors.<br>We need to reduce the console output to higher level progress messages with either the filter or by telling doxygen and dot to not produce so much.</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: George Joseph <gjoseph@digium.com> </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: Tue, 07 Dec 2021 15:43:49 +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: George Joseph <gjoseph@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>