On Thu, 29 Apr 2021 19:01:00 +0100, Stuart Henderson wrote: > even though it seems likely that the ABI is compatible, I think it may > be beneficial to bump the library version of this compared to what > sysutils/eventlog has. I'd probably go with a major bump to make sure > things are clean. > > At least the static library conflicts with sysutils/eventlog so it needs > "@conflict eventlog-*". Also "@pkgpath sysutils/eventlog" so that pkg_add -u > is able to update an existing installed syslog-ng otherwise it will just > error out with a conflict. > > (Not related to the diff, it picks up libuuid from e2fsprogs if present > at build time, though it seems likely that the newer version you're looking > at doesn't do this so maybe ignore that!)
Thanks for the feedback. With the following diff I can update from syslog-ng-3.12p7 to syslog-ng-3.12p8 without problems. OK? - todd Index: sysutils/syslog-ng/Makefile =================================================================== RCS file: /cvs/ports/sysutils/syslog-ng/Makefile,v retrieving revision 1.68 diff -u -p -u -r1.68 Makefile --- sysutils/syslog-ng/Makefile 30 Dec 2019 22:45:11 -0000 1.68 +++ sysutils/syslog-ng/Makefile 30 Apr 2021 16:50:31 -0000 @@ -6,7 +6,7 @@ V = 3.12.1 REL = 3.12 DISTNAME = syslog-ng-$V CATEGORIES = sysutils -REVISION = 7 +REVISION = 8 MAINTAINER = Steven Mestdagh <ste...@openbsd.org> @@ -15,13 +15,15 @@ HOMEPAGE = https://syslog-ng.com/ # GPL v2 PERMIT_PACKAGE = Yes -WANTLIB += c crypto curl dbi evtlog glib-2.0 gmodule-2.0 gthread-2.0 +WANTLIB += c crypto curl dbi glib-2.0 gmodule-2.0 gthread-2.0 WANTLIB += iconv intl ivykis json-c m maxminddb nghttp2 pcre pthread WANTLIB += ssl z MASTER_SITES = http://github.com/balabit/syslog-ng/releases/download/${DISTNAME}/ -SHARED_LIBS = syslog-ng 5.0 \ +SHARED_LIBS = evtlog 2.0 \ + evtlog-${REL} 2.0 \ + syslog-ng 5.0 \ syslog-ng-${REL} 5.0 # needs a newer compiler due to libmaxminddb header @@ -31,7 +33,6 @@ COMPILER_LANGS = c LIB_DEPENDS = devel/glib2 \ devel/json-c \ devel/libivykis \ - sysutils/eventlog \ databases/libdbi>=0.9.0 \ net/curl \ net/libmaxminddb @@ -62,9 +63,7 @@ CONFIGURE_ENV += OPENSSL_CFLAGS="-I/usr/ LDFLAGS="-L${LOCALBASE}/lib" # json-c only installs libjson-c.pc; syslog-ng checks for libjson.pc CONFIGURE_ENV += JSON_C_CFLAGS="`pkg-config json-c --cflags`" \ - JSON_C_LIBS="`pkg-config json-c --libs`" \ - EVTLOG_LIBS="-L${LOCALBASE}/lib -levtlog" \ - EVTLOG_CFLAGS="-I${LOCALBASE}/include/eventlog" + JSON_C_LIBS="`pkg-config json-c --libs`" USE_GMAKE = Yes Index: sysutils/syslog-ng/patches/patch-Makefile_in =================================================================== RCS file: /cvs/ports/sysutils/syslog-ng/patches/patch-Makefile_in,v retrieving revision 1.11 diff -u -p -u -r1.11 patch-Makefile_in --- sysutils/syslog-ng/patches/patch-Makefile_in 10 Nov 2017 21:58:26 -0000 1.11 +++ sysutils/syslog-ng/patches/patch-Makefile_in 29 Apr 2021 17:11:47 -0000 @@ -2,20 +2,10 @@ $OpenBSD: patch-Makefile_in,v 1.11 2017/ - temporary hack until libtool is fixed. - work around different getent API, adapted from https://github.com/balabit/syslog-ng/commit/8b33cbe143b00d6af72cb5145fecd67e6158a9dd -- avoid rebuilding libeventlog, already in devel/eventlog Index: Makefile.in --- Makefile.in.orig +++ Makefile.in -@@ -3572,7 +3572,7 @@ noinst_LIBRARIES = - noinst_DATA = - noinst_LTLIBRARIES = modules/dbparser/libsyslog-ng-patterndb.la \ - modules/diskq/libsyslog-ng-disk-buffer.la --lib_LTLIBRARIES = lib/eventlog/src/libevtlog.la lib/libsyslog-ng.la -+lib_LTLIBRARIES = lib/libsyslog-ng.la - - # this is intentionally formatted so conflicts are less likely to arise. one name in every line. - pkginclude_HEADERS = syslog-ng-config.h lib/eventlog/src/evtmaps.h \ @@ -4183,7 +4183,9 @@ compatinclude_HEADERS = \ lib/compat/string.h \ lib/compat/time.h \ @@ -37,15 +27,6 @@ Index: Makefile.in lib_compat_tests_TESTS = \ lib/compat/tests/test_strtok_r -@@ -4327,7 +4330,7 @@ lib_libsyslog_ng_la_LDFLAGS = -no-undefined -release $ - -version-info ${LSNG_CURRENT}:${LSNG_REVISION}:${LSNG_AGE} - - lib_test_subdirs = lib_filter lib_logproto lib_parser lib_rewrite lib_template lib_stats lib_control --lib_libsyslog_ng_la_DEPENDENCIES = lib/eventlog/src/libevtlog.la \ -+lib_libsyslog_ng_la_DEPENDENCIES = \ - $(am__append_8) - - # this is intentionally formatted so conflicts are less likely to arise. one name in every line. @@ -7089,7 +7092,7 @@ lib/str-repr/lib_libsyslog_ng_la-encode.lo: \ lib/str-repr/$(am__dirstamp) \ lib/str-repr/$(DEPDIR)/$(am__dirstamp) Index: sysutils/syslog-ng/pkg/PLIST =================================================================== RCS file: /cvs/ports/sysutils/syslog-ng/pkg/PLIST,v retrieving revision 1.24 diff -u -p -u -r1.24 PLIST --- sysutils/syslog-ng/pkg/PLIST 9 Dec 2019 14:45:16 -0000 1.24 +++ sysutils/syslog-ng/pkg/PLIST 30 Apr 2021 16:45:08 -0000 @@ -1,4 +1,6 @@ @comment $OpenBSD: PLIST,v 1.24 2019/12/09 14:45:16 fcambus Exp $ +@conflict eventlog-* +@pkgpath sysutils/eventlog @rcscript ${RCDIR}/syslog_ng @bin bin/dqtool @bin bin/loggen @@ -225,9 +227,13 @@ include/syslog-ng/value-pairs/evttag.h include/syslog-ng/value-pairs/transforms.h include/syslog-ng/value-pairs/value-pairs.h include/syslog-ng/versioning.h +@lib lib/libevtlog-3.12.so.${LIBevtlog-3.12_VERSION} +@static-lib lib/libevtlog.a +lib/libevtlog.la +@lib lib/libevtlog.so.${LIBevtlog_VERSION} @lib lib/libsyslog-ng-3.12.so.${LIBsyslog-ng-3.12_VERSION} -lib/libsyslog-ng-native-connector.a -lib/libsyslog-ng.a +@static-lib lib/libsyslog-ng-native-connector.a +@static-lib lib/libsyslog-ng.a lib/libsyslog-ng.la @lib lib/libsyslog-ng.so.${LIBsyslog-ng_VERSION} lib/pkgconfig/syslog-ng-add-contextual-data.pc @@ -235,37 +241,37 @@ lib/pkgconfig/syslog-ng-native-connector lib/pkgconfig/syslog-ng-test.pc lib/pkgconfig/syslog-ng.pc lib/syslog-ng/ -lib/syslog-ng/libadd-contextual-data.so -lib/syslog-ng/libaffile.so -lib/syslog-ng/libafprog.so -lib/syslog-ng/libafsocket.so -lib/syslog-ng/libafsql.so -lib/syslog-ng/libafuser.so -lib/syslog-ng/libbasicfuncs.so -lib/syslog-ng/libcef.so -lib/syslog-ng/libconfgen.so -lib/syslog-ng/libcryptofuncs.so -lib/syslog-ng/libcsvparser.so -lib/syslog-ng/libdate.so -lib/syslog-ng/libdbparser.so -lib/syslog-ng/libdisk-buffer.so -lib/syslog-ng/libgeoip2-plugin.so -lib/syslog-ng/libgraphite.so -lib/syslog-ng/libhttp.so -lib/syslog-ng/libjson-plugin.so -lib/syslog-ng/libkvformat.so -lib/syslog-ng/liblinux-kmsg-format.so -lib/syslog-ng/libmap-value-pairs.so -lib/syslog-ng/libpseudofile.so -lib/syslog-ng/libsnmptrapd-parser.so -lib/syslog-ng/libstardate.so -lib/syslog-ng/libsyslogformat.so -lib/syslog-ng/libsystem-source.so -lib/syslog-ng/libtags-parser.so +@so lib/syslog-ng/libadd-contextual-data.so +@so lib/syslog-ng/libaffile.so +@so lib/syslog-ng/libafprog.so +@so lib/syslog-ng/libafsocket.so +@so lib/syslog-ng/libafsql.so +@so lib/syslog-ng/libafuser.so +@so lib/syslog-ng/libbasicfuncs.so +@so lib/syslog-ng/libcef.so +@so lib/syslog-ng/libconfgen.so +@so lib/syslog-ng/libcryptofuncs.so +@so lib/syslog-ng/libcsvparser.so +@so lib/syslog-ng/libdate.so +@so lib/syslog-ng/libdbparser.so +@so lib/syslog-ng/libdisk-buffer.so +@so lib/syslog-ng/libgeoip2-plugin.so +@so lib/syslog-ng/libgraphite.so +@so lib/syslog-ng/libhttp.so +@so lib/syslog-ng/libjson-plugin.so +@so lib/syslog-ng/libkvformat.so +@so lib/syslog-ng/liblinux-kmsg-format.so +@so lib/syslog-ng/libmap-value-pairs.so +@so lib/syslog-ng/libpseudofile.so +@so lib/syslog-ng/libsnmptrapd-parser.so +@so lib/syslog-ng/libstardate.so +@so lib/syslog-ng/libsyslogformat.so +@so lib/syslog-ng/libsystem-source.so +@so lib/syslog-ng/libtags-parser.so lib/syslog-ng/libtest/ -lib/syslog-ng/libtest/libsyslog-ng-test.a -lib/syslog-ng/libtfgetent.so -lib/syslog-ng/libxml.so +@static-lib lib/syslog-ng/libtest/libsyslog-ng-test.a +@so lib/syslog-ng/libtfgetent.so +@so lib/syslog-ng/libxml.so @man man/man1/dqtool.1 @man man/man1/loggen.1 @man man/man1/pdbtool.1