Please review.
On Fri, Jan 25, 2013 at 11:39:39AM -0500, David Hill wrote:
>Anyone able to test and/or OK?
>
>On Tue, Jan 22, 2013 at 04:55:33PM -0500, David Hill wrote:
>>New diff with tips from brad@
>>
>>On Mon, Jan 21, 2013 at 11:16:23AM -0500, Brad Smith wrote:
>>>On Mon, Jan 21, 2013 at 11:02:32AM -0500, David Hill wrote:
>>>> Anyone?
>>>>
>>>> On Thu, Jan 10, 2013 at 02:35:31PM -0500, David Hill wrote:
>>>> >This brings sysutils/rsyslog to 7.2.5.
>>>> >Using this in production with postgresql on multiple machines.
>>>> >
>>>> >Please review and test.
>>>
>>>I looked at this port and your diff briefly...
>>>
>>>- Capitalize MySQL/Postgres in the COMMENTs properly
>>>- Remove @comment's from the PLISTs and add to post-install
>>> @rm ${PREFIX}/lib/rsyslog/*.la
>>>- Remove the --sysconfdir option from CONFIGURE_ARGS
>>>- Check if USE_GROFF is necessary or not
>>>
>>
>>Index: Makefile
>>===================================================================
>>RCS file: /cvs/ports/sysutils/rsyslog/Makefile,v
>>retrieving revision 1.13
>>diff -N -u -p Makefile
>>--- Makefile 6 Oct 2012 09:17:23 -0000 1.13
>>+++ Makefile 22 Jan 2013 21:52:51 -0000
>>@@ -5,22 +5,18 @@ BROKEN-hppa = lack of atomic ops
>> SHARED_ONLY = Yes
>>
>> COMMENT-main = syslog daemon supporting databases, TCP, SSL,
>> RELP
>>-COMMENT-mysql = mysql plugin for rsyslog
>>-COMMENT-pgsql = postgresql plugin for rsyslog
>>+COMMENT-mysql = MySQL plugin for rsyslog
>>+COMMENT-pgsql = Postgres plugin for rsyslog
>>
>> MULTI_PACKAGES = -main -mysql -pgsql
>>
>>-V = 4.6.4
>>+V = 7.2.5
>> DISTNAME = rsyslog-$V
>> PKGNAME-main = rsyslog-$V
>> PKGNAME-mysql = rsyslog-mysql-$V
>> PKGNAME-pgsql = rsyslog-pgsql-$V
>> CATEGORIES = sysutils
>>
>>-REVISION-main = 5
>>-REVISION-mysql = 2
>>-REVISION-pgsql = 2
>>-
>> HOMEPAGE = http://www.rsyslog.com/
>>
>> # GPLv3+
>>@@ -31,13 +27,17 @@ PERMIT_DISTFILES_FTP = Yes
>>
>> MODULES = devel/gettext
>>
>>-WANTLIB-main += c gmp hogweed nettle gnutls pthread p11-kit
>>-WANTLIB-main += relp tasn1 z ${MODGETTEXT_WANTLIB}
>>+WANTLIB-main += c e2fs-uuid ee estr gmp hogweed nettle gnutls
>>json m pthread
>>+WANTLIB-main += p11-kit relp tasn1 z ${MODGETTEXT_WANTLIB}
>> WANTLIB-mysql += crypto m mysqlclient ssl z
>> WANTLIB-pgsql += crypto com_err pq ssl
>>
>>-LIB_DEPENDS-main = security/gnutls \
>>- sysutils/librelp
>>+LIB_DEPENDS-main = devel/json-c \
>>+ devel/libee>=0.4.0 \
>>+ devel/libestr>=0.1.2 \
>>+ security/gnutls \
>>+ sysutils/e2fsprogs \
>>+ sysutils/librelp>=1.0.1
>> LIB_DEPENDS-mysql = databases/mysql
>> RUN_DEPENDS-mysql = ${PKGNAME-main}:${PKGPATH},-main
>> LIB_DEPENDS-pgsql = databases/postgresql
>>@@ -47,10 +47,9 @@ MASTER_SITES =
>>http://www.rsyslog.com/files/download/
>> http://download.rsyslog.com/rsyslog/
>>
>> CONFIGURE_STYLE = autoconf
>>-AUTOCONF_VERSION = 2.65
>>+AUTOCONF_VERSION = 2.67
>> USE_LIBTOOL = Yes
>> LIBTOOL_FLAGS = --tag=disable-static
>>-USE_GROFF = Yes
>> CONFIGURE_ARGS += ${CONFIGURE_SHARED} \
>> --enable-pgsql \
>> --enable-mysql \
>>@@ -60,6 +59,7 @@ CONFIGURE_ARGS += ${CONFIGURE_SHARED} \
>> CONFIGURE_ENV += CPPFLAGS="-I${LOCALBASE}/include"
>>
>> post-install:
>>+ @rm ${PREFIX}/lib/rsyslog/*.la
>> ${INSTALL_DATA_DIR} ${PREFIX}/share/examples/rsyslog
>> ${INSTALL_DATA} ${WRKSRC}/rsyslog.conf \
>> ${PREFIX}/share/examples/rsyslog
>>Index: distinfo
>>===================================================================
>>RCS file: /cvs/ports/sysutils/rsyslog/distinfo,v
>>retrieving revision 1.2
>>diff -N -u -p distinfo
>>--- distinfo 26 Aug 2010 21:28:58 -0000 1.2
>>+++ distinfo 22 Jan 2013 21:52:51 -0000
>>@@ -1,5 +1,2 @@
>>-MD5 (rsyslog-4.6.4.tar.gz) = nNMu7BmEplarh56JtX2Adg==
>>-RMD160 (rsyslog-4.6.4.tar.gz) = qumwkHVsehiXSYTgS8fAhq5T8ck=
>>-SHA1 (rsyslog-4.6.4.tar.gz) = AzM3E7kz3PjhromMYsVicEg7UIc=
>>-SHA256 (rsyslog-4.6.4.tar.gz) = nXEtLKHutWkrlHvDN5dzqLHtGeddgwflhwuEaoOwac8=
>>-SIZE (rsyslog-4.6.4.tar.gz) = 2080038
>>+SHA256 (rsyslog-7.2.5.tar.gz) = jM6MYWEL3k6wncN5KBNiZQzO1MdQl8l1aszov6rJa/s=
>>+SIZE (rsyslog-7.2.5.tar.gz) = 2704775
>>Index: patches/patch-grammar_lexer_l
>>===================================================================
>>RCS file: patches/patch-grammar_lexer_l
>>diff -N -u -p patches/patch-grammar_lexer_l
>>--- /dev/null 22 Jan 2013 14:52:51 -0000
>>+++ patches/patch-grammar_lexer_l 22 Jan 2013 21:52:51 -0000
>>@@ -0,0 +1,15 @@
>>+$OpenBSD$
>>+--- grammar/lexer.l.orig Fri Dec 7 03:58:13 2012
>>++++ grammar/lexer.l Wed Jan 2 12:43:49 2013
>>+@@ -85,11 +85,6 @@ int cnfSetLexFile(char *fname);
>>+
>>+ extern int yydebug;
>>+
>>+-/* somehow, I need these prototype even though the headers are
>>+- * included. I guess that's some autotools magic I don't understand...
>>+- */
>>+-int fileno(FILE *stream);
>>+-
>>+ %}
>>+
>>+ %%
>>Index: patches/patch-plugins_imuxsock_imuxsock_c
>>===================================================================
>>RCS file: patches/patch-plugins_imuxsock_imuxsock_c
>>diff -N -u -p patches/patch-plugins_imuxsock_imuxsock_c
>>--- /dev/null 22 Jan 2013 14:52:51 -0000
>>+++ patches/patch-plugins_imuxsock_imuxsock_c 22 Jan 2013 21:52:51 -0000
>>@@ -0,0 +1,11 @@
>>+$OpenBSD$
>>+--- plugins/imuxsock/imuxsock.c.orig Fri Dec 7 06:23:20 2012
>>++++ plugins/imuxsock/imuxsock.c Wed Jan 2 13:04:41 2013
>>+@@ -38,6 +38,7 @@
>>+ #include <sys/stat.h>
>>+ #include <sys/un.h>
>>+ #include <sys/socket.h>
>>++#include <sys/uio.h>
>>+ #include "dirty.h"
>>+ #include "cfsysline.h"
>>+ #include "unicode-helper.h"
>>Index: patches/patch-rsyslog_conf
>>===================================================================
>>RCS file: patches/patch-rsyslog_conf
>>diff -N -u -p patches/patch-rsyslog_conf
>>--- /dev/null 22 Jan 2013 14:52:51 -0000
>>+++ patches/patch-rsyslog_conf 22 Jan 2013 21:52:51 -0000
>>@@ -0,0 +1,43 @@
>>+$OpenBSD$
>>+--- rsyslog.conf.orig Tue Sep 25 11:33:56 2012
>>++++ rsyslog.conf Thu Jan 10 13:36:04 2013
>>+@@ -13,28 +13,19 @@ $ModLoad imklog # kernel logging (formerly provided
>>+ # Logging much else clutters up the screen.
>>+ #kern.* /dev/console
>>+
>>+-# Log anything (except mail) of level info or higher.
>>+-# Don't log private authentication messages!
>>+-*.info;mail.none;authpriv.none;cron.none -/var/log/messages
>>++*.notice;auth,authpriv,cron,ftp,kern,lpr,mail,user.none -/var/log/messages
>>++kern.debug;syslog,user.info -/var/log/messages
>>++auth.info -/var/log/authlog
>>++authpriv.debug /var/log/secure
>>++cron.info -/var/cron/log
>>++daemon.info -/var/log/daemon
>>++ftp.info -/var/log/xferlog
>>++lpr.debug -/var/log/lpd-errs
>>++mail.info -/var/log/maillog
>>++#uucp.info -/var/log/uucp
>>+
>>+-# The authpriv file has restricted access.
>>+-authpriv.* /var/log/secure
>>+-
>>+-# Log all the mail messages in one place.
>>+-mail.* -/var/log/maillog
>>+-
>>+-
>>+-# Log cron stuff
>>+-cron.* -/var/log/cron
>>+-
>>+ # Everybody gets emergency messages
>>+-*.emerg *
>>+-
>>+-# Save news errors of level crit and higher in a special file.
>>+-uucp,news.crit -/var/log/spooler
>>+-
>>+-# Save boot messages also to boot.log
>>+-local7.* /var/log/boot.log
>>++*.emerg :omusrmsg:*
>>+
>>+ # Remote Logging (we use TCP for reliable delivery)
>>+ # An on-disk queue is created for this action. If the remote host is
>>Index: patches/patch-runtime_msg_c
>>===================================================================
>>RCS file: patches/patch-runtime_msg_c
>>diff -N -u -p patches/patch-runtime_msg_c
>>--- /dev/null 22 Jan 2013 14:52:51 -0000
>>+++ patches/patch-runtime_msg_c 22 Jan 2013 21:52:51 -0000
>>@@ -0,0 +1,11 @@
>>+$OpenBSD$
>>+--- runtime/msg.c.orig Fri Dec 7 06:23:20 2012
>>++++ runtime/msg.c Wed Jan 2 12:44:13 2013
>>+@@ -36,7 +36,6 @@
>>+ #include <assert.h>
>>+ #include <ctype.h>
>>+ #include <sys/socket.h>
>>+-#include <sys/sysinfo.h>
>>+ #include <netdb.h>
>>+ #include <libestr.h>
>>+ #include <json/json.h>
>>Index: patches/patch-runtime_nsd_gtls_c
>>===================================================================
>>RCS file: /cvs/ports/sysutils/rsyslog/patches/patch-runtime_nsd_gtls_c,v
>>retrieving revision 1.1
>>diff -N -u -p patches/patch-runtime_nsd_gtls_c
>>--- patches/patch-runtime_nsd_gtls_c 14 Sep 2011 21:25:39 -0000 1.1
>>+++ /dev/null 22 Jan 2013 20:57:01 -0000
>>@@ -1,38 +0,0 @@
>>-$OpenBSD: patch-runtime_nsd_gtls_c,v 1.1 2011/09/14 21:25:39 naddy Exp $
>>-
>>-GnuTLS >= 2.12 does not depend on libgcrypt initialization any longer
>>-and may in fact not use libgcrypt at all.
>>-
>>---- runtime/nsd_gtls.c.orig Wed Sep 14 03:05:03 2011
>>-+++ runtime/nsd_gtls.c Wed Sep 14 03:06:21 2011
>>-@@ -29,7 +29,9 @@
>>- #include <string.h>
>>- #include <gnutls/gnutls.h>
>>- #include <gnutls/x509.h>
>>-+#if GNUTLS_VERSION_NUMBER <= 0x020b00
>>- #include <gcrypt.h>
>>-+#endif
>>- #include <errno.h>
>>- #include <sys/stat.h>
>>- #include <unistd.h>
>>-@@ -53,7 +55,9 @@
>>- #define CRLFILE "crl.pem"
>>-
>>-
>>-+#if GNUTLS_VERSION_NUMBER <= 0x020b00
>>- GCRY_THREAD_OPTION_PTHREAD_IMPL;
>>-+#endif
>>- MODULE_TYPE_LIB
>>-
>>- /* static data */
>>-@@ -559,8 +563,10 @@ gtlsGlblInit(void)
>>- uchar *cafile;
>>- DEFiRet;
>>-
>>-+#if GNUTLS_VERSION_NUMBER <= 0x020b00
>>- /* gcry_control must be called first, so that the thread system is
>>correctly set up */
>>- gcry_control (GCRYCTL_SET_THREAD_CBS, &gcry_threads_pthread);
>>-+#endif
>>- CHKgnutls(gnutls_global_init());
>>-
>>- /* X509 stuff */
>>Index: patches/patch-runtime_stringbuf_c
>>===================================================================
>>RCS file: /cvs/ports/sysutils/rsyslog/patches/patch-runtime_stringbuf_c,v
>>retrieving revision 1.1
>>diff -N -u -p patches/patch-runtime_stringbuf_c
>>--- patches/patch-runtime_stringbuf_c 25 Jan 2012 11:03:14 -0000 1.1
>>+++ /dev/null 22 Jan 2013 20:57:01 -0000
>>@@ -1,15 +0,0 @@
>>-$OpenBSD: patch-runtime_stringbuf_c,v 1.1 2012/01/25 11:03:14 ajacoutot Exp $
>>-
>>-CVE-2011-4623: http://secunia.com/advisories/47698/
>>-
>>---- runtime/stringbuf.c.orig Wed Jan 25 10:23:06 2012
>>-+++ runtime/stringbuf.c Wed Jan 25 10:23:24 2012
>>-@@ -156,7 +156,7 @@ rsRetVal
>>- rsCStrExtendBuf(cstr_t *pThis, size_t iMinNeeded)
>>- {
>>- uchar *pNewBuf;
>>-- unsigned short iNewSize;
>>-+ size_t iNewSize;
>>- DEFiRet;
>>-
>>- /* first compute the new size needed */
>>Index: patches/patch-tools_Makefile_in
>>===================================================================
>>RCS file: /cvs/ports/sysutils/rsyslog/patches/patch-tools_Makefile_in,v
>>retrieving revision 1.1
>>diff -N -u -p patches/patch-tools_Makefile_in
>>--- patches/patch-tools_Makefile_in 14 Sep 2011 21:25:39 -0000 1.1
>>+++ patches/patch-tools_Makefile_in 22 Jan 2013 21:52:51 -0000
>>@@ -1,10 +1,10 @@
>>-$OpenBSD: patch-tools_Makefile_in,v 1.1 2011/09/14 21:25:39 naddy Exp $
>>---- tools/Makefile.in.orig Tue Sep 13 23:09:17 2011
>>-+++ tools/Makefile.in Tue Sep 13 23:11:24 2011
>>-@@ -141,7 +141,8 @@ CPP = @CPP@
>>- CPPFLAGS = @CPPFLAGS@
>>- CXX = @CXX@
>>+$OpenBSD$
>>+--- tools/Makefile.in.orig Thu Jan 10 13:14:13 2013
>>++++ tools/Makefile.in Thu Jan 10 13:14:53 2013
>>+@@ -175,7 +175,8 @@ CURL_LIBS = @CURL_LIBS@
>> CYGPATH_W = @CYGPATH_W@
>>+ CZMQ_CFLAGS = @CZMQ_CFLAGS@
>>+ CZMQ_LIBS = @CZMQ_LIBS@
>> -DEFS = @DEFS@
>> +DEFS = @DEFS@ -DSYSLOGD_PIDNAME=\"rsyslogd.pid\" \
>> + -DCONFIGFILE=\"$(sysconfdir)/rsyslog.conf\"
>>Index: patches/patch-tools_syslogd_c
>>===================================================================
>>RCS file: /cvs/ports/sysutils/rsyslog/patches/patch-tools_syslogd_c,v
>>retrieving revision 1.2
>>diff -N -u -p patches/patch-tools_syslogd_c
>>--- patches/patch-tools_syslogd_c 5 Sep 2011 18:24:08 -0000 1.2
>>+++ patches/patch-tools_syslogd_c 22 Jan 2013 21:52:51 -0000
>>@@ -1,17 +1,10 @@
>>-$OpenBSD: patch-tools_syslogd_c,v 1.2 2011/09/05 18:24:08 jasper Exp $
>>-
>>-Last chunk:
>>-Security fix for CVE-2011-3200,
>>-rsyslog: parseLegacySyslogMsg off-by-two buffer overflow
>>-
>>-Patch from upstream via redhat:
>>https://bugzilla.redhat.com/show_bug.cgi?id=727644
>>-
>>---- tools/syslogd.c.orig Thu Aug 5 11:33:54 2010
>>-+++ tools/syslogd.c Mon Sep 5 20:22:07 2011
>>-@@ -173,9 +173,8 @@ static rsRetVal GlobalClassExit(void);
>>- #endif
>>- #endif
>>+$OpenBSD$
>>+--- tools/syslogd.c.orig Fri Dec 7 06:23:21 2012
>>++++ tools/syslogd.c Thu Jan 10 12:58:56 2013
>>+@@ -147,9 +147,8 @@ static rsRetVal GlobalClassExit(void);
>>+ static rsRetVal queryLocalHostname(void);
>>
>>+
>> -#ifndef _PATH_LOGCONF
>> -#define _PATH_LOGCONF "/etc/rsyslog.conf"
>> -#endif
>>@@ -20,12 +13,3 @@ Patch from upstream via redhat: https://bugzilla.redha
>>
>> #ifndef _PATH_MODDIR
>> # if defined(__FreeBSD__)
>>-@@ -1291,7 +1290,7 @@ int parseLegacySyslogMsg(msg_t *pMsg, int flags)
>>- * outputs so that only 32 characters max are used by default.
>>- */
>>- i = 0;
>>-- while(lenMsg > 0 && *p2parse != ':' && *p2parse != ' ' && i <
>>CONF_TAG_MAXSIZE) {
>>-+ while(lenMsg > 0 && *p2parse != ':' && *p2parse != ' ' && i <
>>CONF_TAG_MAXSIZE - 2) {
>>- bufParseTAG[i++] = *p2parse++;
>>- --lenMsg;
>>- }
>>Index: pkg/PLIST-main
>>===================================================================
>>RCS file: /cvs/ports/sysutils/rsyslog/pkg/PLIST-main,v
>>retrieving revision 1.1.1.1
>>diff -N -u -p pkg/PLIST-main
>>--- pkg/PLIST-main 29 Jul 2010 21:40:21 -0000 1.1.1.1
>>+++ pkg/PLIST-main 22 Jan 2013 21:52:51 -0000
>>@@ -1,40 +1,23 @@
>> @comment $OpenBSD: PLIST-main,v 1.1.1.1 2010/07/29 21:40:21 steven Exp $
>> lib/rsyslog/
>>-@comment lib/rsyslog/imklog.la
>> lib/rsyslog/imklog.so
>>-@comment lib/rsyslog/immark.la
>> lib/rsyslog/immark.so
>>-@comment lib/rsyslog/imrelp.la
>> lib/rsyslog/imrelp.so
>>-@comment lib/rsyslog/imtcp.la
>> lib/rsyslog/imtcp.so
>>-@comment lib/rsyslog/imudp.la
>> lib/rsyslog/imudp.so
>>-@comment lib/rsyslog/imuxsock.la
>> lib/rsyslog/imuxsock.so
>>-@comment lib/rsyslog/lmnet.la
>> lib/rsyslog/lmnet.so
>>-@comment lib/rsyslog/lmnetstrms.la
>> lib/rsyslog/lmnetstrms.so
>>-@comment lib/rsyslog/lmnsd_gtls.la
>> lib/rsyslog/lmnsd_gtls.so
>>-@comment lib/rsyslog/lmnsd_ptcp.la
>> lib/rsyslog/lmnsd_ptcp.so
>>-@comment lib/rsyslog/lmregexp.la
>> lib/rsyslog/lmregexp.so
>>-@comment lib/rsyslog/lmstrmsrv.la
>> lib/rsyslog/lmstrmsrv.so
>>-@comment lib/rsyslog/lmtcpclt.la
>> lib/rsyslog/lmtcpclt.so
>>-@comment lib/rsyslog/lmtcpsrv.la
>> lib/rsyslog/lmtcpsrv.so
>>-@comment lib/rsyslog/lmzlibw.la
>> lib/rsyslog/lmzlibw.so
>>-@comment lib/rsyslog/ommail.la
>> lib/rsyslog/ommail.so
>>-@comment lib/rsyslog/omrelp.la
>> lib/rsyslog/omrelp.so
>>-@comment lib/rsyslog/omtesting.la
>>+lib/rsyslog/omruleset.so
>> lib/rsyslog/omtesting.so
>> @man man/man5/rsyslog.conf.5
>> @man man/man8/rsyslogd.8
>>Index: pkg/PLIST-mysql
>>===================================================================
>>RCS file: /cvs/ports/sysutils/rsyslog/pkg/PLIST-mysql,v
>>retrieving revision 1.1.1.1
>>diff -N -u -p pkg/PLIST-mysql
>>--- pkg/PLIST-mysql 29 Jul 2010 21:40:21 -0000 1.1.1.1
>>+++ pkg/PLIST-mysql 22 Jan 2013 21:52:51 -0000
>>@@ -1,4 +1,3 @@
>> @comment $OpenBSD: PLIST-mysql,v 1.1.1.1 2010/07/29 21:40:21 steven Exp $
>>-@comment lib/rsyslog/ommysql.la
>> lib/rsyslog/ommysql.so
>> share/examples/rsyslog/createDB_mysql.sql
>>Index: pkg/PLIST-pgsql
>>===================================================================
>>RCS file: /cvs/ports/sysutils/rsyslog/pkg/PLIST-pgsql,v
>>retrieving revision 1.1.1.1
>>diff -N -u -p pkg/PLIST-pgsql
>>--- pkg/PLIST-pgsql 29 Jul 2010 21:40:21 -0000 1.1.1.1
>>+++ pkg/PLIST-pgsql 22 Jan 2013 21:52:51 -0000
>>@@ -1,4 +1,3 @@
>> @comment $OpenBSD: PLIST-pgsql,v 1.1.1.1 2010/07/29 21:40:21 steven Exp $
>>-@comment lib/rsyslog/ompgsql.la
>> lib/rsyslog/ompgsql.so
>> share/examples/rsyslog/createDB_pgsql.sql
>>
>