<p style="white-space: pre-wrap; word-wrap: break-word;">(this is a -0.5)</p><p>Patch set 1:<span style="border-radius: 3px; display: inline-block; margin: 0 2px; padding: 4px;background-color: #ffd4d4;">Code-Review -1</span></p><p><a href="https://gerrit.osmocom.org/11803">View Change</a></p><p>3 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/11803/1//COMMIT_MSG">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.osmocom.org/#/c/11803/1//COMMIT_MSG@18">Patch Set #1, Line 18:</a> <code style="font-family:monospace,monospace">the xstl parser, as well as the LaTeX compiler, so they can find the</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">is that the "TEXINPUTS" bit?</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://gerrit.osmocom.org/#/c/11803/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/11803/1/build/Makefile.asciidoc.inc@60">Patch Set #1, Line 60:</a> <code style="font-family:monospace,monospace"> [ -f $(notdir $<) ] || ln -s $< $(notdir $<)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">if you keep using this '[ ... ] ||' syntax you might run into the danger of me getting used to it and not complaining about it anymore. Isn't 'if [ condition ]; then ... fi' much more readable?</p><p style="white-space: pre-wrap; word-wrap: break-word;">ok, wait, I don't get what this is doing at all. '$<' ... the input file, so the .adoc file? and '$(notdir)'??<br>ah, essentially notdir == basename.</p><p style="white-space: pre-wrap; word-wrap: break-word;">so this creates a symlink where, in the $builddir, pointing at the $srcdir?</p><p style="white-space: pre-wrap; word-wrap: break-word;">this took long enough to warrant a comment:</p><p style="white-space: pre-wrap; word-wrap: break-word;"># Create a symlink at $(builddir)/srcfile.adoc pointing at $(srcdir)/srcfile.adoc.<br># $< is the $(srcdir)/srcfile.adoc, $(notdir) is like basename, and $PWD is the $(builddir).</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://gerrit.osmocom.org/#/c/11803/1/build/Makefile.asciidoc.inc@66">Patch Set #1, Line 66:</a> <code style="font-family:monospace,monospace">        TEXINPUTS="$(OSMO_GSM_MANUALS_DIR)" \</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">add comment: "pass TEXINPUTS to help a2x find [...]" (insert files to find)</p></li></ul></li></ul><p>To view, visit <a href="https://gerrit.osmocom.org/11803">change 11803</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/11803"/><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-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: Ie6b212a6518f0fc29fae610a37ae6c533189278d </div>
<div style="display:none"> Gerrit-Change-Number: 11803 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Pau Espin Pedrol <pespin@sysmocom.de> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 20 Nov 2018 02:14:15 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: Yes </div>