Change in libosmo-netif[master]: tests: AM_LDFLAGS = -noinstall for all tests

osmith gerrit-no-reply at
Tue Mar 19 10:47:27 UTC 2019

osmith has posted comments on this change. ( )

Change subject: tests: AM_LDFLAGS = -noinstall for all tests

Patch Set 4:

> similar patch for all unit tests in all repositories would be good. thanks

I've started with that, and realized that most other repositories use
noinst_PROGRAMS instead of check_PROGRAMS in test's files.

Reference for both:

I have checked, that noinst_PROGRAMS don't need AM_LDFLAGS = -noinstall.

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.

However, replacing noinst_PROGRAMS with check_PROGRAMS in test's across all repositories would be quite some effort (most repositories have subdirs in tests, and even more files in there). So it does not seem like it is worth it to me. (Other opinions?)

I'm working on patches for the repositories with check_PROGRAMS to use -no-install now.

To view, visit
To unsubscribe, or for help writing mail filters, visit

Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I94ccff42dfba71aaf59bb30ca312db0bac58c27d
Gerrit-Change-Number: 13241
Gerrit-PatchSet: 4
Gerrit-Owner: osmith <osmith at>
Gerrit-Reviewer: Alexander Huemer <alexander.huemer at>
Gerrit-Reviewer: Harald Welte <laforge at>
Gerrit-Reviewer: Jenkins Builder (1000002)
Gerrit-Reviewer: Max <msuraev at>
Gerrit-Reviewer: Pablo Neira Ayuso <pablo at>
Gerrit-Reviewer: dexter <pmaier at>
Gerrit-Reviewer: osmith <osmith at>
Gerrit-Reviewer: steve-m <steve at>
Gerrit-Reviewer: tnt <tnt at>
Gerrit-Comment-Date: Tue, 19 Mar 2019 10:47:27 +0000
Gerrit-HasComments: No
Gerrit-HasLabels: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the gerrit-log mailing list