March 2023 - gerrit-log - lists.osmocom.org

[S] Change in libosmocore[master]: contrib/talloc_count.sh: improve hexadecimal masking
by neels 14 Mar '23

14 Mar '23

neels has submitted this change. ( https://gerrit.osmocom.org/c/libosmocore/+/31873 ) Change subject: contrib/talloc_count.sh: improve hexadecimal masking ...................................................................... contrib/talloc_count.sh: improve hexadecimal masking Also mask hexadecimal without leading 0x. But take care to not match on every letter a-f,A-F in normal words and names. before: 10 map_rua(NffffNfNecN-example-com-RUA-N)[N] 10 map_sccp(NffffNfNecN-example-com-SCCP-N)[N] 17 map_rua(NffffNcN-example-com-RUA-N)[N] 17 map_sccp(NffffNcN-example-com-SCCP-N)[N] 18 map_rua(NffffNeNbN-example-com-RUA-N)[N] 18 map_sccp(NffffNeNbN-example-com-SCCP-N)[N] 82 map_rua(NffffNfNdN-example-com-RUA-N)[N] 82 map_sccp(NffffNfNdN-example-com-SCCP-N)[N] 85 map_rua(NffffNfN-example-com-RUA-N)[N] 85 map_sccp(NffffNfN-example-com-SCCP-N)[N] 224 struct hnbgw_context_map after: 224 map_rua(N-example-com-RUA-N)[N] 224 map_sccp(N-example-com-SCCP-N)[N] 224 struct hnbgw_context_map Change-Id: I1b42ce3e67c7ed2d38d3e5c9cbfa90ba185a07b7 --- M contrib/talloc_count.sh 1 file changed, 33 insertions(+), 0 deletions(-) Approvals: laforge: Looks good to me, approved pespin: Looks good to me, but someone else must approve Jenkins Builder: Verified diff --git a/contrib/talloc_count.sh b/contrib/talloc_count.sh index 3b03585..61aa3f5 100755 --- a/contrib/talloc_count.sh +++ b/contrib/talloc_count.sh @@ -35,6 +35,7 @@ | grep -v '^$' \ | grep -v '^[0-9]\+$' \ | sed 's/0x[0-9a-fA-F]\+/N/g' \ + | sed 's/\<[0-9a-fA-F]\+\>/N/g' \ | sed 's/[0-9]\+/N/g' \ | sort \ > "$mangled" -- To view, visit https://gerrit.osmocom.org/c/libosmocore/+/31873 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmocore Gerrit-Branch: master Gerrit-Change-Id: I1b42ce3e67c7ed2d38d3e5c9cbfa90ba185a07b7 Gerrit-Change-Number: 31873 Gerrit-PatchSet: 2 Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged

1 0

[S] Change in osmo-hlr[master]: tests/*/Makefile.am: move -I to AM_CPPFLAGS
by laforge 14 Mar '23

14 Mar '23

laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-hlr/+/31785 ) Change subject: tests/*/Makefile.am: move -I to AM_CPPFLAGS ...................................................................... tests/*/Makefile.am: move -I to AM_CPPFLAGS Change-Id: I654053e11b0cc824c198f68e4ff0a0fcb295efb0 --- M tests/auc/Makefile.am M tests/db/Makefile.am M tests/gsup/Makefile.am M tests/mslookup/Makefile.am 4 files changed, 16 insertions(+), 4 deletions(-) Approvals: laforge: Looks good to me, approved msuraev: Looks good to me, but someone else must approve Jenkins Builder: Verified diff --git a/tests/auc/Makefile.am b/tests/auc/Makefile.am index b196dbc..9d0f726 100644 --- a/tests/auc/Makefile.am +++ b/tests/auc/Makefile.am @@ -2,12 +2,12 @@ AM_CPPFLAGS = \ $(all_includes) \ + -I$(top_srcdir)/include \ $(NULL) AM_CFLAGS = \ -Wall \ -ggdb3 \ - -I$(top_srcdir)/include \ $(LIBOSMOCORE_CFLAGS) \ $(LIBOSMOGSM_CFLAGS) \ $(NULL) diff --git a/tests/db/Makefile.am b/tests/db/Makefile.am index ece34d3..b07460e 100644 --- a/tests/db/Makefile.am +++ b/tests/db/Makefile.am @@ -1,7 +1,10 @@ -AM_CFLAGS = \ +AM_CPPFLAGS = \ $(all_includes) \ -I$(top_srcdir)/include \ -I$(top_builddir)/include \ + $(NULL) + +AM_CFLAGS = \ -Wall \ -ggdb3 \ $(LIBOSMOCORE_CFLAGS) \ diff --git a/tests/gsup/Makefile.am b/tests/gsup/Makefile.am index 9a10597..04bc51a 100644 --- a/tests/gsup/Makefile.am +++ b/tests/gsup/Makefile.am @@ -1,9 +1,9 @@ AM_CPPFLAGS = \ $(all_includes) \ + -I$(top_srcdir)/include \ $(NULL) AM_CFLAGS = \ - -I$(top_srcdir)/include \ $(LIBOSMOCORE_CFLAGS) \ $(LIBOSMOGSM_CFLAGS) \ $(NULL) diff --git a/tests/mslookup/Makefile.am b/tests/mslookup/Makefile.am index 04778e8..5c53a02 100644 --- a/tests/mslookup/Makefile.am +++ b/tests/mslookup/Makefile.am @@ -1,11 +1,11 @@ AM_CPPFLAGS = \ $(all_includes) \ + -I$(top_srcdir)/include \ $(NULL) AM_CFLAGS = \ -Wall \ -ggdb3 \ - -I$(top_srcdir)/include \ $(LIBOSMOCORE_CFLAGS) \ $(LIBOSMOGSM_CFLAGS) \ $(LIBOSMOABIS_CFLAGS) \ -- To view, visit https://gerrit.osmocom.org/c/osmo-hlr/+/31785 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-hlr Gerrit-Branch: master Gerrit-Change-Id: I654053e11b0cc824c198f68e4ff0a0fcb295efb0 Gerrit-Change-Number: 31785 Gerrit-PatchSet: 2 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: msuraev <msuraev(a)sysmocom.de> Gerrit-CC: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged

1 0

[S] Change in osmo-hlr[master]: tests/auc/Makefile.am: put object files to LDADD
by laforge 14 Mar '23

14 Mar '23

laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-hlr/+/31784 ) Change subject: tests/auc/Makefile.am: put object files to LDADD ...................................................................... tests/auc/Makefile.am: put object files to LDADD Change-Id: Iaefcfe7a8904841a29094fe40eb5850912544b4c --- M tests/auc/Makefile.am 1 file changed, 13 insertions(+), 4 deletions(-) Approvals: laforge: Looks good to me, approved pespin: Looks good to me, but someone else must approve msuraev: Looks good to me, but someone else must approve Jenkins Builder: Verified diff --git a/tests/auc/Makefile.am b/tests/auc/Makefile.am index 5ae133b..b196dbc 100644 --- a/tests/auc/Makefile.am +++ b/tests/auc/Makefile.am @@ -30,8 +30,8 @@ $(NULL) auc_test_LDADD = \ - $(top_srcdir)/src/auc.c \ - $(top_srcdir)/src/logging.c \ + $(top_builddir)/src/auc.o \ + $(top_builddir)/src/logging.o \ $(LIBOSMOCORE_LIBS) \ $(LIBOSMOGSM_LIBS) \ $(NULL) @@ -41,8 +41,8 @@ $(NULL) auc_ts_55_205_test_sets_LDADD = \ - $(top_srcdir)/src/auc.c \ - $(top_srcdir)/src/logging.c \ + $(top_builddir)/src/auc.o \ + $(top_builddir)/src/logging.o \ $(LIBOSMOCORE_LIBS) \ $(LIBOSMOGSM_LIBS) \ $(NULL) -- To view, visit https://gerrit.osmocom.org/c/osmo-hlr/+/31784 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-hlr Gerrit-Branch: master Gerrit-Change-Id: Iaefcfe7a8904841a29094fe40eb5850912544b4c Gerrit-Change-Number: 31784 Gerrit-PatchSet: 2 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: msuraev <msuraev(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged

1 0

[S] Change in osmo-hlr[master]: tests/*/Makefile.am: move -I to AM_CPPFLAGS
by laforge 14 Mar '23

14 Mar '23

Attention is currently required from: fixeria. laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-hlr/+/31785 ) Change subject: tests/*/Makefile.am: move -I to AM_CPPFLAGS ...................................................................... Patch Set 1: Code-Review+2 -- To view, visit https://gerrit.osmocom.org/c/osmo-hlr/+/31785 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-hlr Gerrit-Branch: master Gerrit-Change-Id: I654053e11b0cc824c198f68e4ff0a0fcb295efb0 Gerrit-Change-Number: 31785 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: msuraev <msuraev(a)sysmocom.de> Gerrit-CC: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Tue, 14 Mar 2023 20:11:41 +0000 Gerrit-HasComments: No Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

[S] Change in libosmo-pfcp[master]: build: libosmo-gtlv: move LIBS to libosmo_gtlv_la_LIBADD
by neels 14 Mar '23

14 Mar '23

Attention is currently required from: neels. Hello Jenkins Builder, laforge, fixeria, pespin, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/libosmo-pfcp/+/31762 to look at the new patch set (#2). Change subject: build: libosmo-gtlv: move LIBS to libosmo_gtlv_la_LIBADD ...................................................................... build: libosmo-gtlv: move LIBS to libosmo_gtlv_la_LIBADD Change-Id: I6fd777bed9366c35f5b1453cde1e6f5c1617e338 --- M src/libosmo-gtlv/Makefile.am 1 file changed, 13 insertions(+), 4 deletions(-) git pull ssh://gerrit.osmocom.org:29418/libosmo-pfcp refs/changes/62/31762/2 -- To view, visit https://gerrit.osmocom.org/c/libosmo-pfcp/+/31762 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmo-pfcp Gerrit-Branch: master Gerrit-Change-Id: I6fd777bed9366c35f5b1453cde1e6f5c1617e338 Gerrit-Change-Number: 31762 Gerrit-PatchSet: 2 Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de> Gerrit-MessageType: newpatchset

1 0

[S] Change in libosmo-pfcp[master]: build: drop LIBOSMOVTY from libosmo-pfcp
by neels 14 Mar '23

14 Mar '23

neels has uploaded this change for review. ( https://gerrit.osmocom.org/c/libosmo-pfcp/+/31917 ) Change subject: build: drop LIBOSMOVTY from libosmo-pfcp ...................................................................... build: drop LIBOSMOVTY from libosmo-pfcp libosmo-pfcp does not use libosmovty. Change-Id: I8c772386ee7c6d035aabbc854fadbbcde03f7d33 --- M src/libosmo-pfcp/Makefile.am 1 file changed, 11 insertions(+), 2 deletions(-) git pull ssh://gerrit.osmocom.org:29418/libosmo-pfcp refs/changes/17/31917/1 diff --git a/src/libosmo-pfcp/Makefile.am b/src/libosmo-pfcp/Makefile.am index 5af9bdb..123aff3 100644 --- a/src/libosmo-pfcp/Makefile.am +++ b/src/libosmo-pfcp/Makefile.am @@ -9,13 +9,11 @@ -Wall \ $(LIBOSMOCORE_CFLAGS) \ $(LIBOSMOGSM_CFLAGS) \ - $(LIBOSMOVTY_CFLAGS) \ $(NULL) AM_LDFLAGS = \ $(LIBOSMOCORE_LIBS) \ $(LIBOSMOGSM_LIBS) \ - $(LIBOSMOVTY_LIBS) \ $(NULL) lib_LTLIBRARIES = \ -- To view, visit https://gerrit.osmocom.org/c/libosmo-pfcp/+/31917 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmo-pfcp Gerrit-Branch: master Gerrit-Change-Id: I8c772386ee7c6d035aabbc854fadbbcde03f7d33 Gerrit-Change-Number: 31917 Gerrit-PatchSet: 1 Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de> Gerrit-MessageType: newchange

1 0

[S] Change in libosmo-pfcp[master]: build: fix linking gen__pfcp_ies_auto to use build-tree libosmo-gtlv
by neels 14 Mar '23

14 Mar '23

Attention is currently required from: neels, fixeria, pespin. Hello Jenkins Builder, fixeria, pespin, I'd like you to reexamine a change. Please visit https://gerrit.osmocom.org/c/libosmo-pfcp/+/31741 to look at the new patch set (#4). Change subject: build: fix linking gen__pfcp_ies_auto to use build-tree libosmo-gtlv ...................................................................... build: fix linking gen__pfcp_ies_auto to use build-tree libosmo-gtlv By moving *_LIBS to libosmo_pfcp_la_LIBADD, fix linking of gen__pfcp_ies_auto to prefer the locally built libosmo-gtlv over a system installed one. Related: OS#5942 Change-Id: I2e92238b214d8e79bab1095d6093139bad05bc66 --- M src/libosmo-pfcp/Makefile.am 1 file changed, 20 insertions(+), 5 deletions(-) git pull ssh://gerrit.osmocom.org:29418/libosmo-pfcp refs/changes/41/31741/4 -- To view, visit https://gerrit.osmocom.org/c/libosmo-pfcp/+/31741 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: libosmo-pfcp Gerrit-Branch: master Gerrit-Change-Id: I2e92238b214d8e79bab1095d6093139bad05bc66 Gerrit-Change-Number: 31741 Gerrit-PatchSet: 4 Gerrit-Owner: neels <nhofmeyr(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-CC: laforge <laforge(a)osmocom.org> Gerrit-Attention: neels <nhofmeyr(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Attention: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: newpatchset

1 0

[S] Change in osmo-hlr[master]: tests/auc/Makefile.am: put object files to LDADD
by laforge 14 Mar '23

14 Mar '23

Attention is currently required from: fixeria. laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-hlr/+/31784 ) Change subject: tests/auc/Makefile.am: put object files to LDADD ...................................................................... Patch Set 1: Code-Review+2 (1 comment) Patchset: PS1: ugh. C sourec code files in LDADD? how could that have ever worked. -- To view, visit https://gerrit.osmocom.org/c/osmo-hlr/+/31784 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-hlr Gerrit-Branch: master Gerrit-Change-Id: Iaefcfe7a8904841a29094fe40eb5850912544b4c Gerrit-Change-Number: 31784 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: msuraev <msuraev(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Comment-Date: Tue, 14 Mar 2023 20:11:01 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Gerrit-MessageType: comment

1 0

[S] Change in osmo-ggsn[master]: tests: use -no-install libtool flag to avoid ./lt-* scripts
by laforge 14 Mar '23

14 Mar '23

laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ggsn/+/31835 ) 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: I2ca675e93dc5b34bb08d3b841adc115e93558137 --- M tests/gtp/Makefile.am M tests/lib/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/gtp/Makefile.am b/tests/gtp/Makefile.am index e49d824..ef69260 100644 --- a/tests/gtp/Makefile.am +++ b/tests/gtp/Makefile.am @@ -1,4 +1,5 @@ AM_CFLAGS = -Wall -I$(top_srcdir)/include $(LIBOSMOCORE_CFLAGS) +AM_LDFLAGS = -no-install EXTRA_DIST = \ gtpie_test.ok \ diff --git a/tests/lib/Makefile.am b/tests/lib/Makefile.am index 8f0301c..e807281 100644 --- a/tests/lib/Makefile.am +++ b/tests/lib/Makefile.am @@ -1,4 +1,5 @@ AM_CFLAGS = -Wall -I$(top_srcdir)/include $(LIBOSMOCORE_CFLAGS) +AM_LDFLAGS = -no-install EXTRA_DIST = ippool_test.ok \ ippool_test.err \ -- To view, visit https://gerrit.osmocom.org/c/osmo-ggsn/+/31835 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ggsn Gerrit-Branch: master Gerrit-Change-Id: I2ca675e93dc5b34bb08d3b841adc115e93558137 Gerrit-Change-Number: 31835 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: msuraev <msuraev(a)sysmocom.de> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged

1 0

[S] Change in osmo-ggsn[master]: Do not hard-code -g and -O2 in CFLAGS
by laforge 14 Mar '23

14 Mar '23

laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmo-ggsn/+/31782 ) Change subject: Do not hard-code -g and -O2 in CFLAGS ...................................................................... Do not hard-code -g and -O2 in CFLAGS Let the user decide on the optimization level and debugging info. Change-Id: I4b0b523b7dac4d67413bda37b546964262e5ea0d --- M ggsn/Makefile.am M gtp/Makefile.am M lib/Makefile.am M sgsnemu/Makefile.am M tests/gtp/Makefile.am M tests/lib/Makefile.am 6 files changed, 17 insertions(+), 6 deletions(-) Approvals: Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve laforge: Looks good to me, approved diff --git a/ggsn/Makefile.am b/ggsn/Makefile.am index eea7c6e..d4c82ad 100644 --- a/ggsn/Makefile.am +++ b/ggsn/Makefile.am @@ -2,7 +2,7 @@ AM_LDFLAGS = @EXEC_LDFLAGS@ -AM_CFLAGS = -O2 -D_GNU_SOURCE -fno-builtin -Wall -DSBINDIR='"$(sbindir)"' -ggdb $(LIBOSMOCORE_CFLAGS) $(LIBOSMOCTRL_CFLAGS) $(LIBOSMOVTY_CFLAGS) +AM_CFLAGS = -D_GNU_SOURCE -fno-builtin -Wall -DSBINDIR='"$(sbindir)"' $(LIBOSMOCORE_CFLAGS) $(LIBOSMOCTRL_CFLAGS) $(LIBOSMOVTY_CFLAGS) osmo_ggsn_LDADD = @EXEC_LDADD@ -lgtp -L../gtp ../lib/libmisc.a $(LIBOSMOCORE_LIBS) $(LIBOSMOCTRL_LIBS) $(LIBOSMOVTY_LIBS) diff --git a/gtp/Makefile.am b/gtp/Makefile.am index 8e2aa0b..cb32da7 100644 --- a/gtp/Makefile.am +++ b/gtp/Makefile.am @@ -8,7 +8,7 @@ include_HEADERS = gtp.h gsn.h pdp.h gtpie.h -AM_CFLAGS = -O2 -fno-builtin -Wall -DSBINDIR='"$(sbindir)"' -ggdb $(LIBOSMOCORE_CFLAGS) +AM_CFLAGS = -fno-builtin -Wall -DSBINDIR='"$(sbindir)"' $(LIBOSMOCORE_CFLAGS) libgtp_la_SOURCES = gtp.c gtp.h gsn.c gsn.h gtpie.c gtpie.h pdp.c pdp.h lookupa.c lookupa.h queue.c queue.h libgtp_la_LDFLAGS = -version-info $(LIBVERSION) -no-undefined diff --git a/lib/Makefile.am b/lib/Makefile.am index 5bd9443..9dcd13b 100644 --- a/lib/Makefile.am +++ b/lib/Makefile.am @@ -2,7 +2,7 @@ noinst_HEADERS = gnugetopt.h ippool.h lookup.h syserr.h tun.h in46_addr.h netdev.h gtp-kernel.h netns.h util.h icmpv6.h checksum.h -AM_CFLAGS = -O2 -fno-builtin -Wall -DSBINDIR='"$(sbindir)"' -ggdb $(LIBOSMOCORE_CFLAGS) +AM_CFLAGS = -fno-builtin -Wall -DSBINDIR='"$(sbindir)"' $(LIBOSMOCORE_CFLAGS) libmisc_a_SOURCES = getopt1.c getopt.c ippool.c lookup.c tun.c debug.c in46_addr.c netdev.c netns.c util.c icmpv6.c checksum.c diff --git a/sgsnemu/Makefile.am b/sgsnemu/Makefile.am index 9f10cd2..1f75de0 100644 --- a/sgsnemu/Makefile.am +++ b/sgsnemu/Makefile.am @@ -2,7 +2,7 @@ AM_LDFLAGS = @EXEC_LDFLAGS@ -AM_CFLAGS = -O2 -D_GNU_SOURCE -fno-builtin -Wall -DSBINDIR='"$(sbindir)"' -ggdb $(LIBOSMOCORE_CFLAGS) +AM_CFLAGS = -D_GNU_SOURCE -fno-builtin -Wall -DSBINDIR='"$(sbindir)"' $(LIBOSMOCORE_CFLAGS) sgsnemu_LDADD = @EXEC_LDADD@ -lgtp -L../gtp ../lib/libmisc.a $(LIBOSMOCORE_LIBS) diff --git a/tests/gtp/Makefile.am b/tests/gtp/Makefile.am index 3dce716..e49d824 100644 --- a/tests/gtp/Makefile.am +++ b/tests/gtp/Makefile.am @@ -1,4 +1,4 @@ -AM_CFLAGS = -Wall -I$(top_srcdir)/include $(LIBOSMOCORE_CFLAGS) -g +AM_CFLAGS = -Wall -I$(top_srcdir)/include $(LIBOSMOCORE_CFLAGS) EXTRA_DIST = \ gtpie_test.ok \ diff --git a/tests/lib/Makefile.am b/tests/lib/Makefile.am index 03a5d99..8f0301c 100644 --- a/tests/lib/Makefile.am +++ b/tests/lib/Makefile.am @@ -1,4 +1,4 @@ -AM_CFLAGS = -Wall -I$(top_srcdir)/include $(LIBOSMOCORE_CFLAGS) -g +AM_CFLAGS = -Wall -I$(top_srcdir)/include $(LIBOSMOCORE_CFLAGS) EXTRA_DIST = ippool_test.ok \ ippool_test.err \ -- To view, visit https://gerrit.osmocom.org/c/osmo-ggsn/+/31782 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ggsn Gerrit-Branch: master Gerrit-Change-Id: I4b0b523b7dac4d67413bda37b546964262e5ea0d Gerrit-Change-Number: 31782 Gerrit-PatchSet: 1 Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: laforge <laforge(a)osmocom.org> Gerrit-Reviewer: pespin <pespin(a)sysmocom.de> Gerrit-MessageType: merged

1 0