<blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><p style="white-space: pre-wrap; word-wrap: break-word;">similar patch for all unit tests in all repositories would be good. thanks</p></blockquote><p style="white-space: pre-wrap; word-wrap: break-word;">I've started with that, and realized that most other repositories use<br>noinst_PROGRAMS instead of check_PROGRAMS in test's Makefile.am files.</p><p style="white-space: pre-wrap; word-wrap: break-word;">Reference for both:<br>https://www.gnu.org/software/automake/manual/automake.html#index-noinst_005f</p><p style="white-space: pre-wrap; word-wrap: break-word;">I have checked, that noinst_PROGRAMS don't need AM_LDFLAGS = -noinstall.</p><p style="white-space: pre-wrap; word-wrap: break-word;">The benefit of check_PROGRAMS is, that they only get built when "make check" gets executed. So in theory, using check_PROGRAMS together with -no-install seems to be the better choice.</p><p style="white-space: pre-wrap; word-wrap: break-word;">However, replacing noinst_PROGRAMS with check_PROGRAMS in test's Makefile.am across all repositories would be quite some effort (most repositories have subdirs in tests, and even more Makefile.am files in there). So it does not seem like it is worth it to me. (Other opinions?)</p><p style="white-space: pre-wrap; word-wrap: break-word;">I'm working on patches for the repositories with check_PROGRAMS to use -no-install now.</p><p><a href="https://gerrit.osmocom.org/13241">View Change</a></p><ul style="list-style: none; padding: 0;"></ul><p>To view, visit <a href="https://gerrit.osmocom.org/13241">change 13241</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/13241"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: libosmo-netif </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I94ccff42dfba71aaf59bb30ca312db0bac58c27d </div>
<div style="display:none"> Gerrit-Change-Number: 13241 </div>
<div style="display:none"> Gerrit-PatchSet: 4 </div>
<div style="display:none"> Gerrit-Owner: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Alexander Huemer <alexander.huemer@xx.vu> </div>
<div style="display:none"> Gerrit-Reviewer: Harald Welte <laforge@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: Jenkins Builder (1000002) </div>
<div style="display:none"> Gerrit-Reviewer: Max <msuraev@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: Pablo Neira Ayuso <pablo@gnumonks.org> </div>
<div style="display:none"> Gerrit-Reviewer: dexter <pmaier@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: osmith <osmith@sysmocom.de> </div>
<div style="display:none"> Gerrit-Reviewer: steve-m <steve@steve-m.de> </div>
<div style="display:none"> Gerrit-Reviewer: tnt <tnt@246tNt.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 19 Mar 2019 10:47:27 +0000 </div>
<div style="display:none"> Gerrit-HasComments: No </div>
<div style="display:none"> Gerrit-HasLabels: No </div>