<p><a href="https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/15031">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/15031/1/build/Makefile.asciidoc.inc">File build/Makefile.asciidoc.inc:</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.osmocom.org/#/c/15031/1/build/Makefile.asciidoc.inc@79">Patch Set #1, Line 79:</a> <code style="font-family:monospace,monospace">       if [ -n "$$ASCIIDOC_WARNINGS_CHECK" ]; then \</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;">I guess the ratioanle is that "make check" currently only relates to whether our unit tests pass.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Before this patch, ./configure --enable-manuals and "make check" would always check the manuals too. That is due to how automake works, it finds the "check" rule and runs it in the subdirectory. With the patch, this behavior is overwritten and now the checks only run with "ASCIIDOC_WARNINGS_CHECK=1 make check".</p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">I'm sure that some years down the road asciidoc has more warnings than today, and you wouldn't be able to pass a "make check" anymore, which in turn makes people believe that the software is somehow broken.</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">Yes, the warnings changing being asciidoc versions is the reason why I created the patch. Not only some years down the road, but also in OBS where we have various distros and asciidoc versions, some may print additional warnings that let the documentation check fail.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The new logic is that we don't check for the documentation warnings in OBS or other regular builds, just in jenkins where we have one specific asciidoc version.</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/15031">change 15031</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://gerrit.osmocom.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.osmocom.org/c/osmo-gsm-manuals/+/15031"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: osmo-gsm-manuals </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-Change-Id: Iac993a0d1b17205397a1f1ef1a7bd3f9df739e36 </div>
<div style="display:none"> Gerrit-Change-Number: 15031 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder </div>
<div style="display:none"> Gerrit-Reviewer: laforge <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 21 Aug 2019 11:32:30 +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: pespin <pespin@sysmocom.de> </div>
<div style="display:none"> Comment-In-Reply-To: laforge <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-MessageType: comment </div>