Re: [update] rsyslog 8.33.1

2018-03-11 Thread Daniel Jakots
On Sun, 11 Mar 2018 10:56:14 +0100, Remi Locherer wrote: > > > With that added (unless you think it's wrong), ok danj@ > > > > > > > That is wrong, the elasticsearch package does not provide a library > > so a LIB_DEPENDS dependency would just be stripped ("not needed > > for" message). > >

Re: [update] rsyslog 8.33.1

2018-03-11 Thread Stuart Henderson
On 2018/03/11 10:56, Remi Locherer wrote: > On Sun, Mar 11, 2018 at 09:07:03AM +, Stuart Henderson wrote: > > On 2018/03/10 22:11, Daniel Jakots wrote: > > > On Thu, 8 Mar 2018 22:18:01 +0100, Remi Locherer > > > wrote: > > > > > > > +RUN_DEPENDS-elasticsearch =${PKGNAME-main}:${PKGPATH},

Re: [update] rsyslog 8.33.1

2018-03-11 Thread Remi Locherer
On Sun, Mar 11, 2018 at 09:07:03AM +, Stuart Henderson wrote: > On 2018/03/10 22:11, Daniel Jakots wrote: > > On Thu, 8 Mar 2018 22:18:01 +0100, Remi Locherer > > wrote: > > > > > +RUN_DEPENDS-elasticsearch = ${PKGNAME-main}:${PKGPATH},-main > > > > > + --enable-elastics

Re: [update] rsyslog 8.33.1

2018-03-11 Thread Stuart Henderson
On 2018/03/10 22:11, Daniel Jakots wrote: > On Thu, 8 Mar 2018 22:18:01 +0100, Remi Locherer > wrote: > > > +RUN_DEPENDS-elasticsearch =${PKGNAME-main}:${PKGPATH},-main > > > + --enable-elasticsearch \ > > You added a subpackage for ES but shouldn't we have a ldep on >

Re: [update] rsyslog 8.33.1

2018-03-10 Thread Daniel Jakots
On Thu, 8 Mar 2018 22:18:01 +0100, Remi Locherer wrote: > +RUN_DEPENDS-elasticsearch = ${PKGNAME-main}:${PKGPATH},-main > + --enable-elasticsearch \ You added a subpackage for ES but shouldn't we have a ldep on elasticsearch as it is with mysql/mariadb? With that added (un

Re: [update] rsyslog 8.33.1

2018-03-08 Thread Remi Locherer
On Tue, Mar 06, 2018 at 08:47:25PM -0500, Daniel Jakots wrote: > On Tue, 6 Mar 2018 23:45:50 +0100, Remi Locherer > wrote: > > > > > This updates rsyslogd to the current version and adds two new > > > > modules. > > /usr/ports/sysutils/rsyslog$ make lib-depends-check > > rsyslog-8.33.1(sysutil

Re: [update] rsyslog 8.33.1

2018-03-07 Thread Stuart Henderson
On 2018/03/06 23:45, Remi Locherer wrote: > On Tue, Mar 06, 2018 at 10:43:46PM +0100, Remi Locherer wrote: > > On Thu, Feb 08, 2018 at 03:17:46PM +0100, Remi Locherer wrote: > > > Hi, > > > > > > This updates rsyslogd to the current version and adds two new modules. > > > It needs the updated libr

Re: [update] rsyslog 8.33.1

2018-03-06 Thread Daniel Jakots
On Tue, 6 Mar 2018 23:45:50 +0100, Remi Locherer wrote: > > > This updates rsyslogd to the current version and adds two new > > > modules. /usr/ports/sysutils/rsyslog$ make lib-depends-check rsyslog-8.33.1(sysutils/rsyslog,-main): Missing: crypto.42 (/usr/local/sbin/rsyslogd) (system lib) Miss

Re: [update] rsyslog 8.33.1

2018-03-06 Thread Remi Locherer
On Tue, Mar 06, 2018 at 10:43:46PM +0100, Remi Locherer wrote: > On Thu, Feb 08, 2018 at 03:17:46PM +0100, Remi Locherer wrote: > > Hi, > > > > This updates rsyslogd to the current version and adds two new modules. > > It needs the updated librelp that I sent earlier. > > > > I looked into upstre