Bug 178637 - [patch] net-mgmt/collectd5: add non-default option MODBUS
Summary: [patch] net-mgmt/collectd5: add non-default option MODBUS
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Boris Samorodov
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-05-14 17:50 UTC by Boris Samorodov
Modified: 2013-05-15 12:40 UTC (History)
0 users

See Also:


Attachments
file.diff (2.57 KB, patch)
2013-05-14 17:50 UTC, Boris Samorodov
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Boris Samorodov freebsd_committer freebsd_triage 2013-05-14 17:50:00 UTC
Add option MODBUS (off by default).

Fix: Patch attached with submission follows:
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2013-05-14 17:50:18 UTC
Maintainer of net-mgmt/collectd5,

Please note that PR ports/178637 has just been submitted.

If it contains a patch for an upgrade, an enhancement or a bug fix
you agree on, reply to this email stating that you approve the patch
and a committer will take care of it.

The full text of the PR can be found at:
    http://www.freebsd.org/cgi/query-pr.cgi?pr=ports/178637

-- 
Edwin Groothuis via the GNATS Auto Assign Tool
edwin@FreeBSD.org
Comment 2 Edwin Groothuis freebsd_committer freebsd_triage 2013-05-14 17:50:19 UTC
State Changed
From-To: open->feedback

Awaiting maintainers feedback (via the GNATS Auto Assign Tool)
Comment 3 Boris Samorodov freebsd_committer freebsd_triage 2013-05-14 17:57:13 UTC
Responsible Changed
From-To: freebsd-ports-bugs->bsam

Take.
Comment 4 Krzysztof 2013-05-15 09:24:24 UTC
Hello,

On Tue, May 14, 2013 at 04:50:18PM +0000, Edwin Groothuis wrote:
> Maintainer of net-mgmt/collectd5,
> 
> Please note that PR ports/178637 has just been submitted.
> 
> If it contains a patch for an upgrade, an enhancement or a bug fix
> you agree on, reply to this email stating that you approve the patch
> and a committer will take care of it.
> 
> The full text of the PR can be found at:
>     http://www.freebsd.org/cgi/query-pr.cgi?pr=ports/178637
> 
> -- 
> Edwin Groothuis via the GNATS Auto Assign Tool
> edwin@FreeBSD.org
> 
I approve this patch.

Greetings,
-- 
Krzysztof Stryjek
UNIX administrator/Juniper Networks Specialist
email: wtp (at) bsdserwis (dot) com
http://www.linkedin.com/in/KrzysztofStryjek
GPG fingerprint: 8BD7 40CE 8994 0BBE CE6C  91CD 1292 8959 DC61 0E76

In theory, there is no difference between theory and practice.
In practice, there is.
Comment 5 dfilter service freebsd_committer freebsd_triage 2013-05-15 12:38:31 UTC
Author: bsam
Date: Wed May 15 11:38:21 2013
New Revision: 318238
URL: http://svnweb.freebsd.org/changeset/ports/318238

Log:
  . add new option MODBUS (off by default);      [1]
  . add a mandatory dependency upon libinotify;  [2]
  . bump PORTREVISION.                           [2]
  
  while I'm here:
  . pet portlint: remove whitespace before end of line;
  . add a mandatory dependency upon libinotify to the accompanying
    port (net-mgmt/collectd) and bump PORTREVISION.
  
  PR:		ports/178637  [1]
  		ports/178634  [2]
  Submitted by:	bsam (me)
  Approved by:	ports@bsdserwis.com (maintainer)

Added:
  head/net-mgmt/collectd5/files/patch-src_modbus.c   (contents, props changed)
Modified:
  head/net-mgmt/collectd/Makefile
  head/net-mgmt/collectd5/Makefile
  head/net-mgmt/collectd5/pkg-plist

Modified: head/net-mgmt/collectd/Makefile
==============================================================================
--- head/net-mgmt/collectd/Makefile	Wed May 15 11:07:39 2013	(r318237)
+++ head/net-mgmt/collectd/Makefile	Wed May 15 11:38:21 2013	(r318238)
@@ -3,13 +3,15 @@
 
 PORTNAME=	collectd
 PORTVERSION=	4.10.8
-PORTREVISION=	4
+PORTREVISION=	5
 CATEGORIES=	net-mgmt
 MASTER_SITES=	http://collectd.org/files/
 
 MAINTAINER=	ports@bsdserwis.com
 COMMENT=	Systems & network statistics collection daemon
 
+LIB_DEPENDS=	notify:${PORTSDIR}/devel/libnotify
+
 USE_GMAKE=	yes
 GNU_CONFIGURE=	yes
 USE_AUTOTOOLS=	autoconf autoheader automake libltdl

Modified: head/net-mgmt/collectd5/Makefile
==============================================================================
--- head/net-mgmt/collectd5/Makefile	Wed May 15 11:07:39 2013	(r318237)
+++ head/net-mgmt/collectd5/Makefile	Wed May 15 11:38:21 2013	(r318238)
@@ -3,13 +3,15 @@
 
 PORTNAME=	collectd
 PORTVERSION=	5.2.1
-PORTREVISION=	1
+PORTREVISION=	2
 CATEGORIES=	net-mgmt
 MASTER_SITES=	http://collectd.org/files/
 
 MAINTAINER=	ports@bsdserwis.com
 COMMENT=	Systems & network statistics collection daemon
 
+LIB_DEPENDS=	notify:${PORTSDIR}/devel/libnotify
+
 USE_BZIP2=	yes
 USE_GMAKE=	yes
 GNU_CONFIGURE=	yes
@@ -22,7 +24,7 @@ OPTIONS_DEFINE=		CGI BIND DEBUG GCRYPT V
 OPTIONS_GROUP=		INPUT OUTPUT
 OPTIONS_GROUP_OUTPUT=	RRDTOOL RRDCACHED WRITE_GRAPHITE WRITE_HTTP NOTIFYEMAIL
 OPTIONS_GROUP_INPUT=	APACHE APCUPS CURL CURL_JSON CURL_XML DBI DISK GCRYPT \
-			NUTUPS INTERFACE IPMI MBMON MEMCACHED MYSQL NGINX \
+			NUTUPS INTERFACE IPMI MBMON MEMCACHED MODBUS MYSQL NGINX \
 			OPENVPN PDNS PGSQL PING PYTHON ROUTEROS SNMP TABLE \
 			TOKYOTYRANT VARNISH XMMS
 
@@ -46,6 +48,7 @@ INTERFACE_DESC=		Network interfaces (lib
 IPMI_DESC=		IPMI plugin (openipmi)
 MBMON_DESC=		MBMon 
 MEMCACHED_DESC=		Memcached
+MODBUS_DESC=		Modbus support via libmodbus
 MYSQL_DESC=		MySQL
 NOTIFYEMAIL_DESC=	Email notifications (libesmtp)
 NGINX_DESC=		Nginx
@@ -111,8 +114,6 @@ CONFIGURE_ARGS=	--localstatedir=/var \
 		--disable-match_timediff \
 		--disable-match_value \
 		--disable-memcachec \
-		--disable-modbus \
-		--without-libmodbus \
 		--disable-multimeter \
 		--disable-netapp \
 		--without-libnetapp \
@@ -289,6 +290,15 @@ CONFIGURE_ARGS+=--disable-memcached --wi
 PLIST_SUB+=	MEMCACHED="@comment "
 .endif
 
+.if ${PORT_OPTIONS:MMODBUS}
+CONFIGURE_ARGS+=--enable-modbus
+LIB_DEPENDS+=	modbus:${PORTSDIR}/comms/libmodbus
+PLIST_SUB+=	MODBUS=""
+.else
+CONFIGURE_ARGS+=--disable-modbus
+PLIST_SUB+=	MODBUS="@comment "
+.endif
+
 .if ${PORT_OPTIONS:MMYSQL}
 USE_MYSQL=	yes
 CONFIGURE_ARGS+=--enable-mysql

Added: head/net-mgmt/collectd5/files/patch-src_modbus.c
==============================================================================
--- /dev/null	00:00:00 1970	(empty, because file is newly added)
+++ head/net-mgmt/collectd5/files/patch-src_modbus.c	Wed May 15 11:38:21 2013	(r318238)
@@ -0,0 +1,12 @@
+--- src/modbus.c.orig       2013-01-27 17:47:18.246255000 +0400
++++ src/modbus.c    2013-05-14 18:40:05.000000000 +0400
+@@ -25,6 +25,9 @@
+ #include "plugin.h"
+ #include "configfile.h"
+ 
++/* FreeBSD needs this */
++#include <sys/socket.h> 
++
+ #include <netdb.h>
+ 
+ #include <modbus/modbus.h>

Modified: head/net-mgmt/collectd5/pkg-plist
==============================================================================
--- head/net-mgmt/collectd5/pkg-plist	Wed May 15 11:07:39 2013	(r318237)
+++ head/net-mgmt/collectd5/pkg-plist	Wed May 15 11:38:21 2013	(r318238)
@@ -56,6 +56,8 @@ lib/collectd/load.la
 lib/collectd/load.so
 lib/collectd/logfile.la
 lib/collectd/logfile.so
+%%MODBUS%%lib/collectd/modbus.la
+%%MODBUS%%lib/collectd/modbus.so
 %%MBMON%%lib/collectd/mbmon.la
 %%MBMON%%lib/collectd/mbmon.so
 %%MEMCACHED%%lib/collectd/memcached.la
_______________________________________________
svn-ports-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-ports-all
To unsubscribe, send any mail to "svn-ports-all-unsubscribe@freebsd.org"
Comment 6 Boris Samorodov freebsd_committer freebsd_triage 2013-05-15 12:38:45 UTC
State Changed
From-To: feedback->closed

Committed, thanks! ;-)