Change in ...osmo-hlr[master]: build: fix mess with 'db_test_SOURCES' and 'db_test_LDADD'
Vadim Yanitskiy has submitted this change and it was merged. ( https://gerrit.osmocom.org/c/osmo-hlr/+/14964 ) Change subject: build: fix mess with 'db_test_SOURCES' and 'db_test_LDADD' .. build: fix mess with 'db_test_SOURCES' and 'db_test_LDADD' Somehow both 'db_test_SOURCES' and 'db_test_LDADD' ended up in 'src/Makefile.am'. This causes automake / autoconf to complain. Let's get rid of both useless declarations. Furthermore, the actual 'db_test_LDADD' in 'tests/Makefile.am' contained references to the source files from '$(top_srcdir)'. Most likely, the original intention was to depend on the object files in '$(top_builddir)'. Let's also fix this. Change-Id: Ib2e436ed91d9b7551dc5b205329d468c2b0ced04 --- M src/Makefile.am M tests/db/Makefile.am 2 files changed, 4 insertions(+), 19 deletions(-) Approvals: laforge: Looks good to me, but someone else must approve neels: Looks good to me, approved Jenkins Builder: Verified diff --git a/src/Makefile.am b/src/Makefile.am index 7ee019f..131b44f 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -87,21 +87,6 @@ $(SQLITE3_LIBS) \ $(NULL) -db_test_SOURCES = \ - auc.c \ - db.c \ - db_auc.c \ - db_test.c \ - logging.c \ - rand_fake.c \ - $(NULL) - -db_test_LDADD = \ - $(LIBOSMOCORE_LIBS) \ - $(LIBOSMOGSM_LIBS) \ - $(SQLITE3_LIBS) \ - $(NULL) - osmo_euse_demo_SOURCES = \ osmo-euse-demo.c \ $(NULL) diff --git a/tests/db/Makefile.am b/tests/db/Makefile.am index afda5be..fa925f8 100644 --- a/tests/db/Makefile.am +++ b/tests/db/Makefile.am @@ -26,10 +26,10 @@ $(NULL) db_test_LDADD = \ - $(top_srcdir)/src/db.c \ - $(top_srcdir)/src/db_hlr.c \ - $(top_srcdir)/src/db_auc.c \ - $(top_srcdir)/src/logging.c \ + $(top_builddir)/src/logging.o \ + $(top_builddir)/src/db_auc.o \ + $(top_builddir)/src/db_hlr.o \ + $(top_builddir)/src/db.o \ $(LIBOSMOCORE_LIBS) \ $(LIBOSMOGSM_LIBS) \ $(LIBOSMOABIS_LIBS) \ -- To view, visit https://gerrit.osmocom.org/c/osmo-hlr/+/14964 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-hlr Gerrit-Branch: master Gerrit-Change-Id: Ib2e436ed91d9b7551dc5b205329d468c2b0ced04 Gerrit-Change-Number: 14964 Gerrit-PatchSet: 2 Gerrit-Owner: Vadim Yanitskiy Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: Vadim Yanitskiy Gerrit-Reviewer: laforge Gerrit-Reviewer: neels Gerrit-Reviewer: pespin Gerrit-MessageType: merged
Change in ...osmo-hlr[master]: build: fix mess with 'db_test_SOURCES' and 'db_test_LDADD'
neels has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-hlr/+/14964 ) Change subject: build: fix mess with 'db_test_SOURCES' and 'db_test_LDADD' .. Patch Set 1: Code-Review+2 very interesting -- To view, visit https://gerrit.osmocom.org/c/osmo-hlr/+/14964 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-hlr Gerrit-Branch: master Gerrit-Change-Id: Ib2e436ed91d9b7551dc5b205329d468c2b0ced04 Gerrit-Change-Number: 14964 Gerrit-PatchSet: 1 Gerrit-Owner: Vadim Yanitskiy Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge Gerrit-Reviewer: neels Gerrit-Reviewer: pespin Gerrit-Comment-Date: Tue, 30 Jul 2019 17:01:12 + Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
Change in ...osmo-hlr[master]: build: fix mess with 'db_test_SOURCES' and 'db_test_LDADD'
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-hlr/+/14964 ) Change subject: build: fix mess with 'db_test_SOURCES' and 'db_test_LDADD' .. Patch Set 1: Code-Review+1 -- To view, visit https://gerrit.osmocom.org/c/osmo-hlr/+/14964 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-hlr Gerrit-Branch: master Gerrit-Change-Id: Ib2e436ed91d9b7551dc5b205329d468c2b0ced04 Gerrit-Change-Number: 14964 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge Gerrit-Comment-Date: Fri, 26 Jul 2019 12:13:22 + Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment
Change in ...osmo-hlr[master]: build: fix mess with 'db_test_SOURCES' and 'db_test_LDADD'
fixeria has uploaded this change for review. ( https://gerrit.osmocom.org/c/osmo-hlr/+/14964 Change subject: build: fix mess with 'db_test_SOURCES' and 'db_test_LDADD' .. build: fix mess with 'db_test_SOURCES' and 'db_test_LDADD' Somehow both 'db_test_SOURCES' and 'db_test_LDADD' ended up in 'src/Makefile.am'. This causes automake / autoconf to complain. Let's get rid of both useless declarations. Furthermore, the actual 'db_test_LDADD' in 'tests/Makefile.am' contained references to the source files from '$(top_srcdir)'. Most likely, the original intention was to depend on the object files in '$(top_builddir)'. Let's also fix this. Change-Id: Ib2e436ed91d9b7551dc5b205329d468c2b0ced04 --- M src/Makefile.am M tests/db/Makefile.am 2 files changed, 4 insertions(+), 19 deletions(-) git pull ssh://gerrit.osmocom.org:29418/osmo-hlr refs/changes/64/14964/1 diff --git a/src/Makefile.am b/src/Makefile.am index 7ee019f..131b44f 100644 --- a/src/Makefile.am +++ b/src/Makefile.am @@ -87,21 +87,6 @@ $(SQLITE3_LIBS) \ $(NULL) -db_test_SOURCES = \ - auc.c \ - db.c \ - db_auc.c \ - db_test.c \ - logging.c \ - rand_fake.c \ - $(NULL) - -db_test_LDADD = \ - $(LIBOSMOCORE_LIBS) \ - $(LIBOSMOGSM_LIBS) \ - $(SQLITE3_LIBS) \ - $(NULL) - osmo_euse_demo_SOURCES = \ osmo-euse-demo.c \ $(NULL) diff --git a/tests/db/Makefile.am b/tests/db/Makefile.am index afda5be..fa925f8 100644 --- a/tests/db/Makefile.am +++ b/tests/db/Makefile.am @@ -26,10 +26,10 @@ $(NULL) db_test_LDADD = \ - $(top_srcdir)/src/db.c \ - $(top_srcdir)/src/db_hlr.c \ - $(top_srcdir)/src/db_auc.c \ - $(top_srcdir)/src/logging.c \ + $(top_builddir)/src/logging.o \ + $(top_builddir)/src/db_auc.o \ + $(top_builddir)/src/db_hlr.o \ + $(top_builddir)/src/db.o \ $(LIBOSMOCORE_LIBS) \ $(LIBOSMOGSM_LIBS) \ $(LIBOSMOABIS_LIBS) \ -- To view, visit https://gerrit.osmocom.org/c/osmo-hlr/+/14964 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-hlr Gerrit-Branch: master Gerrit-Change-Id: Ib2e436ed91d9b7551dc5b205329d468c2b0ced04 Gerrit-Change-Number: 14964 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria Gerrit-MessageType: newchange