Attention is currently required from: laforge, pespin.
msuraev has posted comments on this change. (
https://gerrit.osmocom.org/c/libosmocore/+/30301 )
Change subject: osmo-stat-dummy: add rate counters and statsd tester
......................................................................
Patch Set 24:
(1 comment)
File utils/osmo-stat-dummy/Makefile.am:
https://gerrit.osmocom.org/c/libosmocore/+/30301/comment/1b42d89f_f047ac30
PS23, Line 6: osmo_stat_dummy_CFLAGS = -I../../.. $(TALLOC_CFLAGS) $(LIBOSMOCORE_CFLAGS)
$(LIBOSMOVTY_CFLAGS) $(LIBOSMOCTRL_CFLAGS)
$(top_builddir) instead of ../../.. […]
Because
it's located in it's own subfolder (as you suggested) so it needs to go one level
higher to reach for the libraries to link against.
However, if you check gcc invocation right before failure it has several
"-I../.." which is wrong - and I have no idea where this comes from.
--
To view, visit
https://gerrit.osmocom.org/c/libosmocore/+/30301
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: Ibd8b17aa3ba9ceb527c6231310f01d736fb542a7
Gerrit-Change-Number: 30301
Gerrit-PatchSet: 24
Gerrit-Owner: msuraev <msuraev(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-CC: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Sat, 17 Dec 2022 13:49:27 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: comment