fixeria has submitted this change. ( https://gerrit.osmocom.org/c/osmo-mgw/+/31836 )
Change subject: tests: use -no-install libtool flag to avoid ./lt-* scripts
......................................................................
tests: use -no-install libtool flag to avoid ./lt-* scripts
This option should be used for any executables which are used only
for testing, or for generating other files and are consequently never
installed. By specifying this option, we are telling Libtool that
the executable it links will only ever be executed from where it is
built in the build tree. Libtool is usually able to considerably
speed up the link process for such executables.
Change-Id: I4bc03bdc1f2de2574558e2ad753e116486993a7f
---
M tests/mgcp/Makefile.am
M tests/mgcp_client/Makefile.am
2 files changed, 18 insertions(+), 0 deletions(-)
Approvals:
Jenkins Builder: Verified
msuraev: Looks good to me, but someone else must approve
pespin: Looks good to me, approved
diff --git a/tests/mgcp/Makefile.am b/tests/mgcp/Makefile.am
index af6201a..73ad1cf 100644
--- a/tests/mgcp/Makefile.am
+++ b/tests/mgcp/Makefile.am
@@ -18,6 +18,7 @@
AM_LDFLAGS = \
$(COVERAGE_LDFLAGS) \
+ -no-install \
$(NULL)
EXTRA_DIST = \
diff --git a/tests/mgcp_client/Makefile.am b/tests/mgcp_client/Makefile.am
index 3895d02..f6f1838 100644
--- a/tests/mgcp_client/Makefile.am
+++ b/tests/mgcp_client/Makefile.am
@@ -15,6 +15,7 @@
AM_LDFLAGS = \
$(COVERAGE_LDFLAGS) \
+ -no-install \
$(NULL)
EXTRA_DIST = \
--
To view, visit https://gerrit.osmocom.org/c/osmo-mgw/+/31836
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-mgw
Gerrit-Branch: master
Gerrit-Change-Id: I4bc03bdc1f2de2574558e2ad753e116486993a7f
Gerrit-Change-Number: 31836
Gerrit-PatchSet: 3
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: msuraev <msuraev(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: merged
Attention is currently required from: laforge.
fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-smlc/+/29522 )
Change subject: configure.ac: set -std=gnu11
......................................................................
Patch Set 2: Code-Review+2
(1 comment)
Commit Message:
https://gerrit.osmocom.org/c/osmo-smlc/+/29522/comment/276540c2_ff27a03a
PS1, Line 14: https://lists.osmocom.org/pipermail/openbsc/2021-January/013360.html
> Minor: I think it makes sense to link to hyperkitty instead of the legacy archive (https://lists. […]
These new URLs look unreadable. If you don't mind, I would prefer to keep the old URLs here. The related discussion was before we moved to hyperkitty.
--
To view, visit https://gerrit.osmocom.org/c/osmo-smlc/+/29522
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-smlc
Gerrit-Branch: master
Gerrit-Change-Id: I76ed877d160f98e3627293b0539bec5fac1e8469
Gerrit-Change-Number: 29522
Gerrit-PatchSet: 2
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Comment-Date: Mon, 13 Mar 2023 10:50:44 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: laforge <laforge(a)osmocom.org>
Gerrit-MessageType: comment